#metabrainz

/

      • d4rkie has quit
      • d4rkie joined the channel
      • d4rkie has quit
      • d4rkie joined the channel
      • vardhan joined the channel
      • vardhan_ joined the channel
      • Clint has quit
      • Clint joined the channel
      • reosarevok[m]
        akshaaatt: forwarded you an email about logging in issues in the app, in case you can take a look
      • d4rkie has quit
      • d4rkie joined the channel
      • d4rkie has quit
      • lucifer[m]
        reosarevok: maybe also send it to jasje
      • reosarevok[m]
        Does he have a meb address? I don't have it I think
      • lucifer[m]
        i don't think he does and i don't remember his email off the top of my head.
      • on a side note, mayhem do you think we could jasje a @meb address if he's interested?
      • mayhem[m]
        totally.
      • reosarevok[m]
        Ok, jasje, if you want me to send you mobile support stuff please PM me to which mail (or yes, demand a meb one)
      • jasje[m]
        If i can get an meb mail, it would noicee
      • mayhem[m]
        oki
      • jasje[m]
        Keeps things sorted for the most part
      • reosarevok[m]: Send it to 07jasjeet@gmail.com for now
      • mayhem[m]
        jasje@ jasjeet@ ?
      • jasje[m]
        Yups
      • mayhem[m]
        which of the two?
      • jasje[m]
        Jasjeet
      • mayhem[m]
        Last name Singh?
      • jasje[m]
        Middle name but works
      • mayhem[m]
        what is the last then?
      • reosarevok[m]
        Maybe it's a secret!
      • jasje[m]
        Yes secret :P
      • mayhem[m]
        ok, we got with Singh. :)
      • jasje[m]
        <jasje[m]> "Send it to 07jasjeet@gmail.com..." <- Send it to jasjeet@metabrainz.org 😜
      • mayhem[m]
        jasjeet@ is now a thing!
      • d4rkie joined the channel
      • d4rkie has quit
      • d4rkie joined the channel
      • mayhem[m] uploaded an image: (65KiB) < https://matrix.chatbrainz.org/_matrix/media/v3/download/chatbrainz.org/GmrxFpLOrCKrPkAaWjSjZCgb/image.png >
      • BrainzGit
        [bookbrainz-site] 14vedansh2001 opened pull request #1158 (03master…refactor/async-await-utils): BB-839: Refactor promises in src/common/helpers/utils.ts to use async/await https://github.com/metabrainz/bookbrainz-site/p...
      • lucifer[m]
        mayhem[m]: is it without or without search enabled?
      • mayhem[m]
        Not sure, I just ran that query
      • lucifer[m]
        ah okay, i mistook it for deepseek.
      • fwiw, if you search it on chatgpt now. it searches the web finds the correct mbid too.
      • lucifer[m] uploaded an image: (53KiB) < https://matrix.chatbrainz.org/_matrix/media/v3/download/chatbrainz.org/tUUGlGkMbwMbtSDzwMIPyFdv/image.png >
      • lucifer[m]: spoke too soon, it retrieved it from its dataset too.
      • mayhem[m]
        oh, openai now has our data? how interesting.
      • Kladky joined the channel
      • reosarevok[m]
        Can we sue yet? :p
      • mayhem[m]
        get in line.
      • lucifer[m]
        might be worth reaching out to them and asking them to support us because they are using our data?
      • fettuccinae[m]
        mayhem: can we add some ui element to the notifications(something like how many notification are marked as sent, how many are being sent etc) in the admin page for metabrainz and add a js library for sending notifs for future implementation in bookbrainz and increase the size of notification project to 350 hours .
      • lucifer[m]
        They are paying some news outlets a lot to use their data. (And being sued by the rest but that's a different matter)
      • mayhem[m]
        fettuccinae[m]: i'm not sure those statistics are really useful, so no need to implement that. if you're trying to pad the project out to 350 hours, then do more integration with existing projects, I would say.
      • fettuccinae[m]
        mayhem[m]: so integration into musicbrainz, bookbrainz, listenbrainz, critiquebrainz, picard, caa, eaa.
      • mayhem[m]
        well not all of those.
      • MB, BB, LB for sure.
      • fettuccinae[m]
        mayhem[m]: for a 350hr project? Thanks for clarifying.
      • s/./😃/
      • mayhem[m]
        we'll need to look at your proposal in detail to see if it fits, but that seems about right.
      • zas: can you please take a look at the "request for rate limit..." email to support@? that would be all you who had to make that work...
      • aerozol: monkey ansh https://github.com/openui/open-ui
      • zas[m]
        mayhem: reosarevok answered already, suggesting to use a mirror (which is imho the best option in this case). The user didn't answer yet
      • mayhem[m]
        ah, I didn't see the reply. maybe look at spam again.
      • BrainzGit
        [listenbrainz-server] 14MonkeyDo opened pull request #3222 (03master…monkey/cleanup-brainzplayer): LB-1659: Fix stale closures in BrainzPlayer metadata https://github.com/metabrainz/listenbrainz-serv...
      • monkey[m]
        <mayhem[m]> "aerozol: monkey ansh https://..." <- Context? Is this just for info?
      • mayhem[m]
        yes, FYI. I thought it was an interesting idea
      • monkey[m]
        I mean, sounds like regular W3C to me, apart form the large scope of it I guess
      • vardhan__ joined the channel
      • vardhan_ has quit
      • ansh: Speaking of LB#2940 , did you have anything else you wanted changed apart from the tooltip ?
      • I think I will keep the tooltip issue for another time, I don't think it is a new issue and it will take some more thinking to resolve it. I think more than anything the library is probably not made to work with touch devices, or there is something odd going on there.
      • BrainzBot
        Add Music Player UI for Mobile Devices: https://github.com/metabrainz/listenbrainz-serv...
      • monkey[m]
        (since we use it on hover over the scrollbar, not sure what that means for touchscreen use)
      • ansh[m]
        monkey[m]: Nope, Apart from the tooltip issue, I think it's good to go. We can probably take it up in a separate follow up PR.
      • monkey[m]
        Yep. Maybe deactivate it entirely on mobile screens? Or at least revisit how it is used
      • ansh[m]
        Cool! Merging the PR
      • BrainzGit
        [listenbrainz-server] 14anshg1214 merged pull request #2940 (03master…add-music-player-ui): Add Music Player UI for Mobile Devices https://github.com/metabrainz/listenbrainz-serv...
      • monkey[m]
        🎉
      • I'm going to put it on beta now
      • Actually, I'll do a quick final review of the BP closure PR and put that out too
      • BrainzGit
        [listenbrainz-server] 14anshg1214 merged pull request #3219 (03master…link-to-source-code): LB-1760: Update broken link in Music Services page https://github.com/metabrainz/listenbrainz-serv...
      • [listenbrainz-server] 14MonkeyDo merged pull request #3222 (03master…monkey/cleanup-brainzplayer): LB-1659: Fix stale closures in BrainzPlayer metadata https://github.com/metabrainz/listenbrainz-serv...
      • monkey[m]
        Updating LB beta
      • BrainzGit
        [listenbrainz-server] 14MonkeyDo opened pull request #3223 (03master…fix-cover-art-fallback): Fix fallback to release-group cover art https://github.com/metabrainz/listenbrainz-serv...
      • [listenbrainz-server] 14MonkeyDo opened pull request #3224 (03master…refactor-BPUI): Use BrainzPlayer context directly in BrainzPlayerUI https://github.com/metabrainz/listenbrainz-serv...
      • Kladky has quit
      • Kladky joined the channel
      • Kladky has quit
      • Kladky joined the channel
      • Kladky has quit
      • Kladky joined the channel
      • zas[m]
        bitmap: we still get OOM kills on selda, see alerts
      • BrainzGit
        [listenbrainz-server] 14MonkeyDo merged pull request #3182 (03master…dependabot/npm_and_yarn/dompurify-3.2.4): Bump dompurify from 2.5.8 to 3.2.4 https://github.com/metabrainz/listenbrainz-serv...
      • bitmap[m]
        <zas[m]> "bitmap: we still get OOM kills..." <- I still suspect it's from serving cover art through plackup, so I disabled the endpoint for now, we'll see if it stops or not
      • the memory usage graph for musicbrainz-website-test on selda doesn't seem to approach the 8g memory limit on the container though, which I'm confused by
      • kellnerd[m] has quit
      • mayhem[m] is off to see https://listenbrainz.org/artist/8229a8f1-b315-4fae-af57-b3eb71efdaf4/
      • GautamShorewala[ has quit
      • zas[m]
        bitmap: according to kern.log that was 8f12926c4a65 container which is musicbrainz-website-prod
      • 2025-03-12T17:05:31.125319+00:00 selda kernel:
      • oom-kill:constraint=CONSTRAINT_MEMCG,nodemask=(null),cpuset=docker-8f12926c4a65e5359e63f2fbbed330cfa68b3e7c59b03ad43f152545b4e7ae93.scope,mems_allowed=0,oom_memcg=/system.slice/docker-8f12926c4a65e5359e63f2fbbed330cfa68b3e7c59b03ad43f152545b4e7ae93.scope,task_memcg=/system.slice/docker-8f12926c4a65e5359e63f2fbbed330cfa68b3e7c59b03ad43f152545b4e7ae93.scope,task=/home/musicbrai,pid=1243257,uid=1000
      • 2025-03-12T17:05:31.125320+00:00 selda kernel: Memory cgroup out of memory: Killed process 1243257 (/home/musicbrai) total-vm:2982904kB, anon-rss:2905052kB, file-rss:17920kB, shmem-rss:0kB, UID:1000 pgtables:5828kB oom_score_adj:0
      • bitmap[m]
        ah right, I assumed it was the test one again... let me try something else then
      • the plackup workers will be recycled faster now so they shouldn't accumulate as much memory, if this helps I'll commit the change
      • _flaky[m] has quit
      • we might have too many workers too, I'll try reducing them until it has a visible effect on response time
      • minimal joined the channel
      • I lowered the number of website workers on selda from 45 to 10 without any issue so far. (500 hits/minute) * 500 ms = ~4 so it could probably be reduced even further as that calculation was already conservative...