#metabrainz

/

      • ruaok
        we're still tiny in the grand scale of things
      • 2020-02-21 05240, 2020

      • ruaok
        MediumData
      • 2020-02-21 05241, 2020

      • ruaok
        that's us.
      • 2020-02-21 05206, 2020

      • ruaok
        hey, mail!!
      • 2020-02-21 05223, 2020

      • iliekcomputers pats brainzbot on the back
      • 2020-02-21 05225, 2020

      • iliekcomputers
        good bot
      • 2020-02-21 05234, 2020

      • ruaok
        woof.
      • 2020-02-21 05250, 2020

      • ruaok
        er. <bleep>
      • 2020-02-21 05207, 2020

      • shivam-kapila
        iliekcomputers: Finally pushed the test. The rebase was stuck as I had some commit in the branch locally that was causing conflict. Sorry for the fuss
      • 2020-02-21 05221, 2020

      • iliekcomputers
        shivam-kapila: already reviewed
      • 2020-02-21 05241, 2020

      • iliekcomputers
        lol
      • 2020-02-21 05244, 2020

      • shivam-kapila
        wow!
      • 2020-02-21 05202, 2020

      • iliekcomputers
        ruaok: did you see the warnings in sentry 🤣
      • 2020-02-21 05203, 2020

      • iliekcomputers
        ouch
      • 2020-02-21 05254, 2020

      • iliekcomputers
        well, at least it works
      • 2020-02-21 05200, 2020

      • iliekcomputers
        for some definition of works
      • 2020-02-21 05205, 2020

      • ruaok
        I didn't look. what happened?
      • 2020-02-21 05218, 2020

      • shivam-kapila
        iliekcomputers: How can I test whether the listens are deleted or not. Any reference in already existing tests?
      • 2020-02-21 05234, 2020

      • iliekcomputers
        there's a log for when a file doesn't exist in hdfs and we try to load it. we don't have any listens for the years of 2005 - 2007 i think, and we try to load it all.
      • 2020-02-21 05249, 2020

      • iliekcomputers
        so a bunch of "file doesn't exist" spam in sentry
      • 2020-02-21 05254, 2020

      • iliekcomputers
        shivam-kapila: hey
      • 2020-02-21 05256, 2020

      • iliekcomputers
        yes!
      • 2020-02-21 05211, 2020

      • ruaok
        joy
      • 2020-02-21 05228, 2020

      • iliekcomputers
      • 2020-02-21 05241, 2020

      • iliekcomputers
        the tests in this file use `send_data` to submit listens
      • 2020-02-21 05213, 2020

      • iliekcomputers
        there is prior art to check that the listens exist in this test: https://github.com/metabrainz/listenbrainz-server…
      • 2020-02-21 05226, 2020

      • iliekcomputers
        after this, you'd probably submit a POST to the endpoint you added
      • 2020-02-21 05248, 2020

      • iliekcomputers
        and then call the get_listens endpoint again, which should this time not return any listens
      • 2020-02-21 05252, 2020

      • iliekcomputers
        does that make sense?
      • 2020-02-21 05216, 2020

      • shivam-kapila
        yeah. thanks a lot :)
      • 2020-02-21 05242, 2020

      • iliekcomputers
        shivam-kapila: also, if you're adding something that is user-facing, it is very helpful if you post a video / gif / screenshot showing what you've changed
      • 2020-02-21 05218, 2020

      • shivam-kapila
        Sure. Locked in mind from now.
      • 2020-02-21 05201, 2020

      • shivam-kapila
        The check that listens were deleted successfully are to be added to both tesh.sh and integration-test.sh. Right?
      • 2020-02-21 05226, 2020

      • iliekcomputers
        i don't think they'll work in test.sh
      • 2020-02-21 05233, 2020

      • iliekcomputers
        let's add an integration test
      • 2020-02-21 05200, 2020

      • iliekcomputers
      • 2020-02-21 05206, 2020

      • shivam-kapila
        yeah I was also confused for the same reason.
      • 2020-02-21 05208, 2020

      • iliekcomputers
        this seems the relevant file
      • 2020-02-21 05234, 2020

      • shivam-kapila
        Okay thanks
      • 2020-02-21 05233, 2020

      • Pac23 has quit
      • 2020-02-21 05224, 2020

      • Pac23 joined the channel
      • 2020-02-21 05221, 2020

      • iliekcomputers
        shivam-kapila: did i ever follow up with you about the figma you made?
      • 2020-02-21 05211, 2020

      • supersandro2000 has quit
      • 2020-02-21 05224, 2020

      • supersandro2000 joined the channel
      • 2020-02-21 05204, 2020

      • shivam-kapila
        My fault here. I didnt discuss my status on it. Actually I am trying to remake the design using image based card view design. So as ruaok pointed out that there can be issues with loading so many images. So I am trying to optimise that. Also for the graph plotting I found http://recharts.org. But they dont have quite good docs so trying tweaking through that. I will finish that just after my exams.
      • 2020-02-21 05213, 2020

      • iliekcomputers
        question
      • 2020-02-21 05224, 2020

      • iliekcomputers
        can we do them one by one
      • 2020-02-21 05258, 2020

      • shivam-kapila
        You mean image loading?
      • 2020-02-21 05215, 2020

      • iliekcomputers
        like create a graph each in different react components, each on it's own page and then later add a dashboard
      • 2020-02-21 05239, 2020

      • shivam-kapila
        I guess yes.
      • 2020-02-21 05205, 2020

      • shivam-kapila
        Will try to separate the graph here and import it into the dashboard
      • 2020-02-21 05214, 2020

      • iliekcomputers
        recharts looks pretty nice
      • 2020-02-21 05216, 2020

      • iliekcomputers
        so
      • 2020-02-21 05232, 2020

      • iliekcomputers
      • 2020-02-21 05239, 2020

      • iliekcomputers
        could we migrate this to recharts?
      • 2020-02-21 05212, 2020

      • iliekcomputers
        (or just react in general)
      • 2020-02-21 05230, 2020

      • shivam-kapila
        yeah sure. I will work on it too. Recharts has barcharts too so migrating this page wont be difficult
      • 2020-02-21 05238, 2020

      • iliekcomputers
        shivam-kapila: ok, cool.
      • 2020-02-21 05239, 2020

      • iliekcomputers
        so
      • 2020-02-21 05248, 2020

      • iliekcomputers
        here's what i think is the best way forward
      • 2020-02-21 05253, 2020

      • iliekcomputers
        assume you have data in the following format: [{artist_name: 'Kanye West', count: 10}, {'artist_name': 'Frank Ocean', count: 2}]
      • 2020-02-21 05206, 2020

      • iliekcomputers
        migrate the page I just linked to react
      • 2020-02-21 05217, 2020

      • iliekcomputers
        add tests (once ishaanshah[m]'s PR is merged)
      • 2020-02-21 05248, 2020

      • iliekcomputers
        once we have that, we'll know more about how usable recharts actually is.
      • 2020-02-21 05258, 2020

      • iliekcomputers
        then we can look into more graphs
      • 2020-02-21 05203, 2020

      • iliekcomputers
        does that make sense?
      • 2020-02-21 05253, 2020

      • shivam-kapila
        Got it. Thanks for the work plan. Will start this just after my exams.
      • 2020-02-21 05210, 2020

      • iliekcomputers
        sure.
      • 2020-02-21 05247, 2020

      • iliekcomputers
        thanks
      • 2020-02-21 05217, 2020

      • shivam-kapila
        A lil doubt here. For testing the invalid auth token. Is it okay if I append some unnecessary characters to the auth token?
      • 2020-02-21 05243, 2020

      • iliekcomputers
        anything invalid would work
      • 2020-02-21 05253, 2020

      • iliekcomputers
        obvious string is 'invalid-auth-token'
      • 2020-02-21 05213, 2020

      • iliekcomputers
        so that the reader understands what the test is doing
      • 2020-02-21 05231, 2020

      • shivam-kapila
        Makes sense.