#metabrainz

/

      • milkii has quit
      • 2023-07-17 19819, 2023

      • milkii joined the channel
      • 2023-07-17 19840, 2023

      • aerozol
        akshaaatt: the figma mockups for ‘entity pages’ (artist, album, recording) are here: https://www.figma.com/file/YRbCOtFHBez8XmMdCKbGta…
      • 2023-07-17 19807, 2023

      • aerozol
        But they are not signed off, monkey and I are still brainstorming them. Feedback/comments on the figma very welcome!
      • 2023-07-17 19845, 2023

      • milkii has quit
      • 2023-07-17 19812, 2023

      • milkii joined the channel
      • 2023-07-17 19847, 2023

      • d4rkie has quit
      • 2023-07-17 19842, 2023

      • d4rkie joined the channel
      • 2023-07-17 19817, 2023

      • mayhem
        moooin! everyone!
      • 2023-07-17 19844, 2023

      • mayhem
        today is MusicBrainz' birthday -- at least what is says on the birth-certificate. whoo hoo happy birthday MusicBrainz!
      • 2023-07-17 19834, 2023

      • mayhem
        gah, taylor swift barfed all over my fresh releases. ick.
      • 2023-07-17 19840, 2023

      • texke has quit
      • 2023-07-17 19833, 2023

      • texke joined the channel
      • 2023-07-17 19803, 2023

      • lucifer
      • 2023-07-17 19824, 2023

      • mayhem
        I will get to reviews, this afternoon, promise
      • 2023-07-17 19820, 2023

      • texke has quit
      • 2023-07-17 19810, 2023

      • texke joined the channel
      • 2023-07-17 19803, 2023

      • mayhem
        lucifer: done. what other PRs should I look at ?
      • 2023-07-17 19822, 2023

      • lucifer
        mayhem: meb react PR
      • 2023-07-17 19845, 2023

      • mayhem
        ah, still marked as draft, so it was not on my radar. will look now.
      • 2023-07-17 19824, 2023

      • mayhem
        done.
      • 2023-07-17 19802, 2023

      • mayhem
        lucifer: I see that periodic jams were not generated today -- but they should've no? did the cron container get updated?
      • 2023-07-17 19812, 2023

      • lucifer
        mayhem: will take a look
      • 2023-07-17 19824, 2023

      • mayhem
        thanks. now the other task that is blocking me from pushing lb-radio for community feedback period is that the tag query is on test and if someone updates test then lb-radio will break. how much work would a PR for that feature be?
      • 2023-07-17 19821, 2023

      • lucifer
        mayhem: can do that by tomorrow. just need to make the tags dataset refreshable.
      • 2023-07-17 19837, 2023

      • BrainzGit
        [listenbrainz-server] 14amCap1712 merged pull request #2525 (03master…flask-upgrade): Upgrade Flask and related dependencies https://github.com/metabrainz/listenbrainz-server…
      • 2023-07-17 19854, 2023

      • mayhem
        great, thanks!
      • 2023-07-17 19802, 2023

      • BrainzGit
        [listenbrainz-server] 14amCap1712 merged pull request #2524 (03master…dependabot/npm_and_yarn/semver-5.7.2): Bump semver from 5.7.1 to 5.7.2 https://github.com/metabrainz/listenbrainz-server…
      • 2023-07-17 19844, 2023

      • BrainzGit
        [listenbrainz-server] 14dependabot[bot] closed pull request #2519 (03master…dependabot/npm_and_yarn/stylelint-15.10.1): Bump stylelint from 14.13.0 to 15.10.1 https://github.com/metabrainz/listenbrainz-server…
      • 2023-07-17 19820, 2023

      • BrainzGit
        [listenbrainz-server] 14amCap1712 closed pull request #2466 (03master…dependabot/pip/flask-2.3.2): Bump flask from 2.1.3 to 2.3.2 https://github.com/metabrainz/listenbrainz-server…
      • 2023-07-17 19824, 2023

      • lucifer
        vscode_: hi! riksucks is currently unwell so he is unable to review your PR currently. I took a quick look and its looks mostly fine to me. I would suggest working on adding some tests for the AppleMusicPlayer component. you can look at the existing players for ideas on what tests need to be added.
      • 2023-07-17 19822, 2023

      • lucifer
        also, I will put up the PR for testing on ListenBrainz Beta/Test so that others can take a look at how it works and suggest improvements to the workflow.
      • 2023-07-17 19851, 2023

      • mayhem
        Happy 23rd birthday MusicBrainz!
      • 2023-07-17 19859, 2023

      • mayhem
        Check out where the name came from: https://blog.metabrainz.org/2023/07/17/happy-23rd…
      • 2023-07-17 19817, 2023

      • lucifer
        🎉🎉
      • 2023-07-17 19828, 2023

      • mayhem
        aerozol: not sure if this fluff piece is working attempting to make noise about, but I'll post it to LI.
      • 2023-07-17 19836, 2023

      • lucifer
        mayhem: add some corp speak to it and it'll be a linkedin hit
      • 2023-07-17 19848, 2023

      • mayhem
        urgh.
      • 2023-07-17 19802, 2023

      • mayhem
        Have you been unaware of the advent of the less-than-institutional name MusicBrainz? Gain many synergy points for your next ROI phase by reading this amazing article that will drastically improve many of your KPIs! https://blog.metabrainz.org/2023/07/17/happy-23rd…
      • 2023-07-17 19807, 2023

      • mayhem
        or more?
      • 2023-07-17 19815, 2023

      • lucifer
      • 2023-07-17 19821, 2023

      • lucifer
        chatgpt suggests ^
      • 2023-07-17 19854, 2023

      • mayhem
        🤣
      • 2023-07-17 19849, 2023

      • lucifer
        mayhem: with your version, i think some consultancies might reach out asking for a quote 🤣
      • 2023-07-17 19839, 2023

      • yvanzo
        Happy Birthday! 🥳
      • 2023-07-17 19836, 2023

      • lucifer
        param: hi! can you try working on the patron page based off this PR branch? https://github.com/metabrainz/metabrainz.org/pull…
      • 2023-07-17 19801, 2023

      • akshaaatt
        I could see mayhem registering the name
      • 2023-07-17 19805, 2023

      • lucifer
        if something doesn't work for you, lmk and i'll fix it in the setup. if the page works fine, i'll merge and deploy.
      • 2023-07-17 19819, 2023

      • mayhem
        akshaaatt: that was a fun party too. :)
      • 2023-07-17 19829, 2023

      • akshaaatt
        ⚡️⚡️
      • 2023-07-17 19858, 2023

      • mayhem
        lucifer: we should probably loop aerozol in before releasing the patron page -- he has some ideas on this front.
      • 2023-07-17 19828, 2023

      • lucifer
        mayhem: yes makes sense, by merge and deploy i meant the just adding react support to MeB PR.
      • 2023-07-17 19816, 2023

      • jasje
        any function to put in a key:value pair into jsonb in postgres?
      • 2023-07-17 19817, 2023

      • mayhem
        ah got it. thanks. just trying to not have to deal with the wrath of aerozol . :)
      • 2023-07-17 19825, 2023

      • jasje
        jsonb dict**
      • 2023-07-17 19840, 2023

      • lucifer
        jasje: json.dumps({key: value})
      • 2023-07-17 19815, 2023

      • lucifer
        or inside the SQL query, jsonb_build_object(:key, :value)
      • 2023-07-17 19834, 2023

      • jasje
        yea i need the one in sql query
      • 2023-07-17 19856, 2023

      • jasje
        i already have a key:value pair inside
      • 2023-07-17 19803, 2023

      • jasje
        i don't want that to be lost
      • 2023-07-17 19817, 2023

      • lucifer
        can you share the code/sql query, hard to tell without that
      • 2023-07-17 19830, 2023

      • jasje
        well since its come to that
      • 2023-07-17 19850, 2023

      • jasje
        i was trying to add similar users into recommendation.similar_user
      • 2023-07-17 19805, 2023

      • jasje
        it has a column similar_users
      • 2023-07-17 19814, 2023

      • jasje
        which is of type jsonb
      • 2023-07-17 19820, 2023

      • jasje
        and has data as follows
      • 2023-07-17 19837, 2023

      • jasje
        {1:[0.75,0.01]}
      • 2023-07-17 19852, 2023

      • jasje
        {user_id:[local_sim, global_sim]}
      • 2023-07-17 19857, 2023

      • jasje
        is that right?
      • 2023-07-17 19819, 2023

      • lucifer
        let me check the code
      • 2023-07-17 19854, 2023

      • lucifer
        yup that's right
      • 2023-07-17 19809, 2023

      • jasje
      • 2023-07-17 19821, 2023

      • jasje
        so to add another similar user
      • 2023-07-17 19825, 2023

      • jasje
        i just do
      • 2023-07-17 19852, 2023

      • jasje
        {1:[0.75, 0.01], 2:[local, global]}
      • 2023-07-17 19853, 2023

      • lucifer
        you mean you want to add a similar user for tests?
      • 2023-07-17 19859, 2023

      • jasje
        yessir
      • 2023-07-17 19812, 2023

      • lucifer
        yeah modify that `data = ` line
      • 2023-07-17 19838, 2023

      • jasje
        i want to use ON CONFLICT
      • 2023-07-17 19858, 2023

      • lucifer
        no need for tests
      • 2023-07-17 19835, 2023

      • jasje
        okeys
      • 2023-07-17 19849, 2023

      • jasje
        so this function
      • 2023-07-17 19820, 2023

      • jasje
        ill just save data globally and append to it when called?
      • 2023-07-17 19840, 2023

      • jasje
        like self.data
      • 2023-07-17 19812, 2023

      • jasje
        because without memory this function won't be resuable
      • 2023-07-17 19836, 2023

      • jasje
        I came up with something like this https://www.irccloud.com/pastebin/9LdpQUbf/
      • 2023-07-17 19859, 2023

      • jasje
        lucifer^
      • 2023-07-17 19815, 2023

      • lucifer
        jasje: i am not sure what you are trying to do. can you put an updated test on github that i can look at?
      • 2023-07-17 19821, 2023

      • mayhem
        3 albums discovered and loved because of lb-radio. #happy
      • 2023-07-17 19849, 2023

      • lucifer
        just put a test that shows what you want to test, i'll help in figuring how to implement the necessary/missing details.
      • 2023-07-17 19807, 2023

      • jasje
        lucifer: alright ill push it
      • 2023-07-17 19828, 2023

      • jasje
        give me some mins
      • 2023-07-17 19854, 2023

      • lucifer
        sure
      • 2023-07-17 19800, 2023

      • mayhem
        lucifer: on this endpoint, which we need to improve for Arsh, would it be possible to return which release the recording is from? https://datasets.listenbrainz.org/popular-recordi…
      • 2023-07-17 19819, 2023

      • lucifer
        mayhem: the canonical release?
      • 2023-07-17 19836, 2023

      • mayhem
        yyes, sure.
      • 2023-07-17 19802, 2023

      • mayhem
        yea, any would be fine, but canonical would be easiest, I suppose.
      • 2023-07-17 19851, 2023

      • lucifer
        yeah i mean you need to find a way to pick from one of the all available, i guess we'd end with similar criteria and at canonical anyways.
      • 2023-07-17 19801, 2023

      • mayhem
        yep
      • 2023-07-17 19830, 2023

      • jasje
        lucifer: code pushed
      • 2023-07-17 19826, 2023

      • lucifer
      • 2023-07-17 19840, 2023

      • lucifer
        jasje: these 3 lines should go in the start of your test function
      • 2023-07-17 19811, 2023

      • jasje
        Alright Ill shift them
      • 2023-07-17 19823, 2023

      • lucifer
        as for the method i see what you are trying to do now.
      • 2023-07-17 19858, 2023

      • lucifer
        SET similar_users = EXCLUDED.similar_users || {json.dumps(self.similar_user_data)}
      • 2023-07-17 19803, 2023

      • lucifer
        and it should work
      • 2023-07-17 19828, 2023

      • jasje
        trying
      • 2023-07-17 19832, 2023

      • lucifer
        there are other issues with that method but let's come to fixing those later
      • 2023-07-17 19816, 2023

      • jasje
        ambigious column user_id
      • 2023-07-17 19850, 2023

      • jasje
        welp excluded to the rescue
      • 2023-07-17 19809, 2023

      • reosarevok
        <BANG>
      • 2023-07-17 19822, 2023

      • mayhem
        Happy Birthday to MusicBrainz!
      • 2023-07-17 19839, 2023

      • reosarevok
        Hi and welcome to Birthday Monday
      • 2023-07-17 19814, 2023

      • ShivamAwasthi
        🎊
      • 2023-07-17 19814, 2023

      • reosarevok
        I have two mailed-in reviews this time:
      • 2023-07-17 19823, 2023

      • reosarevok
        monkey says:
      • 2023-07-17 19824, 2023

      • reosarevok
        '''
      • 2023-07-17 19829, 2023

      • reosarevok
        Hello everyone,
      • 2023-07-17 19837, 2023

      • reosarevok
        Last Tuesday a very close friend of mine passed away suddenly.
      • 2023-07-17 19840, 2023

      • reosarevok
        I’ve been grieving and dealing with the surprising amount of logistics that comes with death.
      • 2023-07-17 19843, 2023

      • reosarevok
        As a consequence I only really worked on Monday and have been 98% AFK since.
      • 2023-07-17 19844, 2023

      • reosarevok
        That being said, I did review GSOC projects and filled in the midterm evaluations.
      • 2023-07-17 19846, 2023

      • reosarevok
        That’s it for me.
      • 2023-07-17 19847, 2023

      • reosarevok
        '''
      • 2023-07-17 19850, 2023

      • reosarevok
        hugs, monkey
      • 2023-07-17 19807, 2023

      • mayhem
        that death has affected a lot of people here in BCN
      • 2023-07-17 19855, 2023

      • ShivamAwasthi
        really sorry to hear that
      • 2023-07-17 19819, 2023

      • reosarevok
        The other mail is by VishalFTW (is that vscode_?)
      • 2023-07-17 19824, 2023

      • mayhem
        (I personally didn't really know this person, but most of my friends did)
      • 2023-07-17 19837, 2023

      • reosarevok
        Who says:
      • 2023-07-17 19839, 2023

      • reosarevok
        '''
      • 2023-07-17 19844, 2023

      • reosarevok
        Hi, hope you are doing well!
      • 2023-07-17 19848, 2023

      • reosarevok
        Due to some unforeseen circumstances I have to share my weekly details through, Please let me know if there are any issues.
      • 2023-07-17 19850, 2023

      • lucifer
        reosarevok: its pixelpenguin
      • 2023-07-17 19850, 2023

      • reosarevok
        I have improved datetime support in query ui, link for the commit - https://github.com/metabrainz/data-set-hoster/pul…
      • 2023-07-17 19853, 2023

      • reosarevok
        I have improved formatting of free fall results, link for the commit - https://github.com/metabrainz/data-set-hoster/pul…
      • 2023-07-17 19859, 2023

      • reosarevok
        Kind regards,
      • 2023-07-17 19804, 2023

      • reosarevok
        '''
      • 2023-07-17 19806, 2023

      • reosarevok
        Oh!
      • 2023-07-17 19810, 2023

      • reosarevok
        Ok, thanks lucifer
      • 2023-07-17 19828, 2023

      • reosarevok
        That's it for mailed reviews
      • 2023-07-17 19834, 2023

      • reosarevok
        mayhem: wanna start us?
      • 2023-07-17 19841, 2023

      • reosarevok
        Other people today: reosarevok, bitmap, zas, yvanzo, lucifer, akshaaatt, atj, ApeKattQuest, riksucks, kellnerd, Pratha-Fish, ShivamAwasthi, vscode, arsh, jasje
      • 2023-07-17 19843, 2023

      • mayhem
        sure.
      • 2023-07-17 19848, 2023

      • yvanzo
        hugs to monkey
      • 2023-07-17 19806, 2023

      • mayhem
        I posted a blog post explaining where the MusicBrainz name came from today, since its MB's birthday.
      • 2023-07-17 19835, 2023

      • lucifer
        monkey: oh that's sad to hear :(. 🫂
      • 2023-07-17 19836, 2023

      • mayhem
        last week was kinda slow -- lots of visitors in town and generally hot summer weather in BCN made it difficult to concentrate most of the week.
      • 2023-07-17 19813, 2023

      • mayhem
        but then I worked on lb-radio a bit, reviewed some PRs, accounting/emails and the usual background sort of stuff. hoping to really get back into things this week.
      • 2023-07-17 19818, 2023

      • mayhem
        yvanzo: go!