#metabrainz

/

      • shivam-kapila
        just thinking out loud in the above logic
      • 2020-07-27 20920, 2020

      • shivam-kapila
        > I'm just cautious about introducing code that might break future versions of YT player
      • 2020-07-27 20920, 2020

      • shivam-kapila
        Agreed
      • 2020-07-27 20945, 2020

      • Mr_Monkey
        There should be a safe way to do it though, so I'll look into it.
      • 2020-07-27 20947, 2020

      • white_shadow joined the channel
      • 2020-07-27 20956, 2020

      • shivam-kapila
        Thank you :)
      • 2020-07-27 20902, 2020

      • _lucifer
        ruaok: another license related question (i am really bad at this licensing thing). Someone has asked at the android repo which license is the code available? so i rpelied GPLv3 but once the legacy code is gone we are open to a less restrictive license. That is possible right?
      • 2020-07-27 20941, 2020

      • _lucifer wonders if someone could suggest him a source to learn more about how the licensing stuff
      • 2020-07-27 20943, 2020

      • _lucifer
        works
      • 2020-07-27 20942, 2020

      • Clint
      • 2020-07-27 20910, 2020

      • white_snack joined the channel
      • 2020-07-27 20943, 2020

      • ruaok
        If you can be sure that all old code is gone and all current contributors agree to the new license then yes.
      • 2020-07-27 20948, 2020

      • _lucifer
        okay, and specific parts of the app which are completely written from scratch can they labeled under a different license
      • 2020-07-27 20904, 2020

      • _lucifer
        i ask because vlc does a similiar thing
      • 2020-07-27 20936, 2020

      • shivam-kapila
        _lucifer: Is VLC plugin releasefd
      • 2020-07-27 20957, 2020

      • _lucifer
        shivam-kapila: not merged yet :(. the maintainers haven't had the time
      • 2020-07-27 20905, 2020

      • shivam-kapila
        oh
      • 2020-07-27 20910, 2020

      • _lucifer
        probably post-gsoc they'll be able to merge it
      • 2020-07-27 20919, 2020

      • shivam-kapila
        Hmm
      • 2020-07-27 20931, 2020

      • _lucifer
        but the drop in replacement is ready
      • 2020-07-27 20949, 2020

      • _lucifer
        its linked on the lb website as well
      • 2020-07-27 20901, 2020

      • shivam-kapila
        Noice
      • 2020-07-27 20907, 2020

      • white_shadow has quit
      • 2020-07-27 20949, 2020

      • white_snack
        we should move the app completely to kotlin
      • 2020-07-27 20915, 2020

      • _lucifer
        white_snack: yes, we can do that later because just doing it for the sake of refactoring can wait.
      • 2020-07-27 20942, 2020

      • _lucifer
        once the app is in good shape we'll do that
      • 2020-07-27 20958, 2020

      • _lucifer
        but you can write the new things in kotlin if you want to
      • 2020-07-27 20907, 2020

      • _lucifer
        (i personally prefer java though)
      • 2020-07-27 20941, 2020

      • white_snack
        yes that would be good as it will reduce our effort later on.
      • 2020-07-27 20925, 2020

      • c1e0 has quit
      • 2020-07-27 20918, 2020

      • MFCR_ColbyRay has quit
      • 2020-07-27 20951, 2020

      • BrainzGit
        [bookbrainz-site] snyk-bot opened pull request #476 (master…snyk-fix-470d915a45517883d773e927db238912): [Snyk] Fix for 2 vulnerabilities https://github.com/bookbrainz/bookbrainz-site/pul…
      • 2020-07-27 20906, 2020

      • travis-ci joined the channel
      • 2020-07-27 20906, 2020

      • travis-ci
        Project bookbrainz-site build #3242: errored in 1 min 41 sec: https://travis-ci.org/bookbrainz/bookbrainz-site/…
      • 2020-07-27 20906, 2020

      • travis-ci has left the channel
      • 2020-07-27 20959, 2020

      • white_snack has quit
      • 2020-07-27 20910, 2020

      • Darkloke has quit
      • 2020-07-27 20931, 2020

      • slriv has quit
      • 2020-07-27 20924, 2020

      • travis-ci joined the channel
      • 2020-07-27 20924, 2020

      • travis-ci
        Project bookbrainz-site build #3244: passed in 3 min 35 sec: https://travis-ci.org/bookbrainz/bookbrainz-site/…
      • 2020-07-27 20924, 2020

      • travis-ci has left the channel
      • 2020-07-27 20915, 2020

      • Mr_Monkey
        CatQuest: The search page issue (unable to delete last letter) is fixed and published.
      • 2020-07-27 20920, 2020

      • Mr_Monkey
        Thanks again for reporting
      • 2020-07-27 20913, 2020

      • CatQuest
        :D
      • 2020-07-27 20910, 2020

      • CatQuest
        oops it seems there is an error wen merging with annotations. https://beta.bookbrainz.org/author/4fefd44b-6d3a-… had an annotation before merging but now it does not
      • 2020-07-27 20934, 2020

      • Mr_Monkey
        *sad_trombone.wav*
      • 2020-07-27 20949, 2020

      • CatQuest
        uh
      • 2020-07-27 20931, 2020

      • BrainzGit
        [listenbrainz-server] ishaanshah opened pull request #988 (master…daily_activity_frontend): [WIP] LB-633: Add "Daily Activity" statistics (Frontend) https://github.com/metabrainz/listenbrainz-server…
      • 2020-07-27 20932, 2020

      • BrainzBot
        LB-633: Add "Listening Activity" statistics (API) https://tickets.metabrainz.org/browse/LB-633
      • 2020-07-27 20907, 2020

      • Mr_Monkey
        Yeah, I added annotations after the merge tool and must have forgotten to add them to the list of fields to merge.
      • 2020-07-27 20907, 2020

      • Mr_Monkey
        Do I just merge all the annotations with a separator in-between? Or display them to choose one?
      • 2020-07-27 20902, 2020

      • kori joined the channel
      • 2020-07-27 20958, 2020

      • BrainzGit
        [bookbrainz-site] prabalsingh24 opened pull request #477 (UserCollection…test-create/edit-collection): Test create/edit collection https://github.com/bookbrainz/bookbrainz-site/pul…
      • 2020-07-27 20930, 2020

      • Rotab joined the channel
      • 2020-07-27 20944, 2020

      • Mr_Monkey
        iliekcomputers, ruaok : The LB API docs say about the `/user/(user_name)/listens` endpoint that “Listens are always returned in descending timestamp order.”
      • 2020-07-27 20944, 2020

      • Mr_Monkey
        That's not what I experience: when I use minTs, the order is reversed: https://api.listenbrainz.org/1/user/mr_monkey/lis…
      • 2020-07-27 20944, 2020

      • Mr_Monkey
        Which one is correct?
      • 2020-07-27 20900, 2020

      • iliekcomputers
        I think this behaviour might have changed with timescale migration.
      • 2020-07-27 20950, 2020

      • iliekcomputers
        ruaok will probably know more. If we guaranteed that order earlier, we should fix this up, it could be breaking clients.
      • 2020-07-27 20943, 2020

      • alastairp
        I'm thinking back to when we initially wrote this spec, I think we intended for it to always be in descending order
      • 2020-07-27 20901, 2020

      • alastairp
        in which case that's a bug, though who knows if any clients are relying on it
      • 2020-07-27 20927, 2020

      • alastairp
        are there any stats on who is using the API for GET requests? that could be interesting
      • 2020-07-27 20928, 2020

      • Mr_Monkey
        Not sure when it was introduced. I realized it was reversed when I was working on PR 943, written before the timescale migration.
      • 2020-07-27 20936, 2020

      • Mr_Monkey
        Didn't realize the docs said otherwise until now
      • 2020-07-27 20952, 2020

      • alastairp
        I wonder if it was reversed in the migration to influx ;)
      • 2020-07-27 20920, 2020

      • CatQuest
        [15:59] <Mr_Monkey> Do I just merge all the annotations with a separator in-between? Or display them to choose one?
      • 2020-07-27 20920, 2020

      • CatQuest
        yea that's what makes sense (and what is on mb anyway)
      • 2020-07-27 20926, 2020

      • MajorLurker joined the channel
      • 2020-07-27 20947, 2020

      • MajorLurker has quit
      • 2020-07-27 20940, 2020

      • shivam-kapila
        Mr_Monkey: the logic has changed
      • 2020-07-27 20942, 2020

      • shivam-kapila
      • 2020-07-27 20942, 2020

      • ruaok
        then lets change it back. and add a test for it.
      • 2020-07-27 20918, 2020

      • ruaok
        I'll make a PR and then maybe we can merge pristine___'s PR at the same time and hope for a release soon
      • 2020-07-27 20954, 2020

      • shivam-kapila
        I can make one tomorrow.
      • 2020-07-27 20905, 2020

      • ruaok
        I'll make one now.
      • 2020-07-27 20903, 2020

      • shivam-kapila
        thanks
      • 2020-07-27 20931, 2020

      • slriv joined the channel
      • 2020-07-27 20931, 2020

      • alastairp
        ruaok: do you mean to make it the same as the documentation?
      • 2020-07-27 20959, 2020

      • alastairp
        I have a niggling thought that we did a reverse() in the view if necessary
      • 2020-07-27 20959, 2020

      • ruaok
        no, I thought it would be best to do something entirely different. I'm thinking of returning them in random order...
      • 2020-07-27 20946, 2020

      • alastairp
        ok
      • 2020-07-27 20913, 2020

      • ruaok
        what I don't understand is how we would've changed the test to match what was being returned. that is odd to me. but yes, I think there was a reverse in there.
      • 2020-07-27 20946, 2020

      • ruaok
        right now I am just trying my local installation to actually return me anything I just imported.
      • 2020-07-27 20915, 2020

      • white_shadow joined the channel
      • 2020-07-27 20933, 2020

      • yvanzo
        hi slriv, neither the errors you mentioned nor the jump of 18G is a known thing, it is most likely environmental.
      • 2020-07-27 20958, 2020

      • white_shadow
        Hi _lucifer
      • 2020-07-27 20902, 2020

      • white_shadow
        What's next?
      • 2020-07-27 20917, 2020

      • _lucifer
        white_shadow: i suggest to use the app a bit and get a better feel of how it work
      • 2020-07-27 20932, 2020

      • _lucifer
        i saw that you used the okhttp cache thats good
      • 2020-07-27 20906, 2020

      • _lucifer
        what things will it cache?
      • 2020-07-27 20941, 2020

      • _lucifer
        and do we need an external room db cache or the okhttp one will suffice
      • 2020-07-27 20951, 2020

      • white_shadow
        no need of room db
      • 2020-07-27 20949, 2020

      • _lucifer
        good then, it would come with a lot of complexity which we can avoid now
      • 2020-07-27 20934, 2020

      • white_shadow
        yes
      • 2020-07-27 20915, 2020

      • ruaok
        heh, found a bug in the process, lol
      • 2020-07-27 20943, 2020

      • TOPIC: MetaBrainz Community and Development channel | MusicBrainz non-development: #musicbrainz | GSoC students start here: https://bit.ly/2NUaxgb | Channel is logged; see https://musicbrainz.org/doc/IRC for details | Meeting agenda: Reviews
      • 2020-07-27 20957, 2020

      • ishaanshah
        iliekcomputers: Hi!
      • 2020-07-27 20932, 2020

      • iliekcomputers
        ishaanshah: hey
      • 2020-07-27 20940, 2020

      • white_snack joined the channel
      • 2020-07-27 20957, 2020

      • ishaanshah
        How are you?
      • 2020-07-27 20909, 2020

      • iliekcomputers
        good, good.
      • 2020-07-27 20929, 2020

      • iliekcomputers
        saw the daily activity pr
      • 2020-07-27 20932, 2020

      • iliekcomputers
        exciting!
      • 2020-07-27 20918, 2020

      • ishaanshah
        Its nearly complete
      • 2020-07-27 20918, 2020

      • ishaanshah
        Tests are remaining
      • 2020-07-27 20944, 2020

      • ishaanshah
        and some improvements to listening_activity
      • 2020-07-27 20956, 2020

      • iliekcomputers
        nice
      • 2020-07-27 20903, 2020

      • ishaanshah
        Bring the header inside the card, maintain aspect ratio etc.
      • 2020-07-27 20923, 2020

      • iliekcomputers
        yeah, the header inside the card makes sense. I was gonna point that out.
      • 2020-07-27 20938, 2020

      • prabal
        Mr_Monkey: hi
      • 2020-07-27 20943, 2020

      • Mr_Monkey
        Hiya!
      • 2020-07-27 20907, 2020

      • white_shadow has quit
      • 2020-07-27 20909, 2020

      • Mr_Monkey
        Sorry, give me two minutes to look at your PR
      • 2020-07-27 20916, 2020

      • ishaanshah
        We should get the artist map done till friday ig
      • 2020-07-27 20927, 2020

      • ishaanshah
        So 2 graphs in a week :D
      • 2020-07-27 20929, 2020

      • iliekcomputers
        woo!
      • 2020-07-27 20904, 2020

      • ishaanshah
        I will complete the tests by tomorrow afternoon and start working on artist origin
      • 2020-07-27 20908, 2020

      • iliekcomputers
        the endpoint is slow rn, because of the many GET requests.
      • 2020-07-27 20926, 2020

      • iliekcomputers
        but once we can POST to the dataset hoster, it should be good.
      • 2020-07-27 20930, 2020

      • Mr_Monkey
        prabal: So how's the testing been going?
      • 2020-07-27 20934, 2020

      • iliekcomputers
        ishaanshah: sounds good!
      • 2020-07-27 20953, 2020

      • ishaanshah
        Should I push directly to your branch or create another PR?
      • 2020-07-27 20904, 2020

      • iliekcomputers
        i'd say just push to mine
      • 2020-07-27 20913, 2020

      • iliekcomputers
        i have left a bunch of comments there.
      • 2020-07-27 20916, 2020

      • prabal
        Mr_Monkey: i copied your trick of logging in the agent
      • 2020-07-27 20927, 2020

      • prabal
        From your merge route tests
      • 2020-07-27 20944, 2020

      • ishaanshah
        Cool, will do
      • 2020-07-27 20946, 2020

      • iliekcomputers
        oh also, i used a dictionary for the alpha 2 to 3 mapping, we should use pycountry instead
      • 2020-07-27 20949, 2020

      • Mr_Monkey
        Good :)
      • 2020-07-27 20957, 2020

      • Mr_Monkey
        Certainly useful, that one
      • 2020-07-27 20920, 2020

      • ishaanshah
        Ohk, I will add it to requirments
      • 2020-07-27 20928, 2020

      • prabal
        Cann we make two agents?
      • 2020-07-27 20947, 2020

      • iliekcomputers
        👍🏽
      • 2020-07-27 20953, 2020

      • iliekcomputers
        i don't have anything else.
      • 2020-07-27 20902, 2020

      • Mr_Monkey
        To have two users you mean?
      • 2020-07-27 20912, 2020

      • prabal
      • 2020-07-27 20951, 2020

      • Mr_Monkey
        Right, and an unauthed user won't be able to access the route at all, I see
      • 2020-07-27 20959, 2020

      • ishaanshah
        Cool, have a nice day :)
      • 2020-07-27 20923, 2020

      • prabal
        But this newAgent isn't working properly
      • 2020-07-27 20955, 2020

      • prabal
        I guess I'll have to do agent.get('/logout') first right?
      • 2020-07-27 20902, 2020

      • iliekcomputers
        ishaanshah: you too!
      • 2020-07-27 20929, 2020

      • Mr_Monkey
        Hmm. What doy ou mean by not working? what is it doing instead and what do you expect instead?
      • 2020-07-27 20916, 2020

      • prabal
        I expect newAgent to be logged in with different user than agent
      • 2020-07-27 20909, 2020

      • Mr_Monkey
        Right, it doesn't do that automatically.
      • 2020-07-27 20945, 2020

      • Mr_Monkey
        It's an ugly hack at the moment, where test user 123456 logs in.
      • 2020-07-27 20956, 2020

      • Mr_Monkey
        Let me look at that code again
      • 2020-07-27 20906, 2020

      • prabal
        Yes now i want to login with different user
      • 2020-07-27 20901, 2020

      • Mr_Monkey
        OK, that's something for me to look at and find a way to log in another user.
      • 2020-07-27 20927, 2020

      • Mr_Monkey
        I suppose in the meantime you'll be blocked on some routes but not others, correct?
      • 2020-07-27 20902, 2020

      • Mr_Monkey
        Would it be useful if we set up together a list of the tests you should write?