#metabrainz

/

      • texke has quit
      • d4rk-ph0enix has quit
      • d4rk-ph0enix joined the channel
      • vardhan_ joined the channel
      • d4rk-ph0enix has quit
      • d4rk-ph0enix joined the channel
      • minimal has quit
      • snobdiggy292 joined the channel
      • texke joined the channel
      • d4rk-ph0enix has quit
      • d4rk-ph0enix joined the channel
      • Kladky joined the channel
      • Kladky has quit
      • Kladky joined the channel
      • Kladky has quit
      • Kladky joined the channel
      • snobdiggy292 has quit
      • lucifer[m]
        mayhem: daily jams working fine in general, failing for you because 429 ratelimit.
      • mayhem[m]
        that seems... fitting. #fuckyouinparticular
      • lucifer[m]
        i can convert it to a spark job to avoid this or add retries in general to troi.
      • i think you fixed some other issue the last time because the api calls for fetching recommendations are in liblistenbrainz which hasn't been updated in a year/
      • mayhem[m]
        oh, liblistenbrainz -- yes, that would be it.
      • because we've now finished adding 429 to all aspects of troi, but liblistenbrainz. let me go do that.
      • lucifer[m]
        i would suggest to create a wrapper function in troi/liblistenbrainz and implement "Sessions and HTTPAdapter With a Backoff Strategy" https://www.zenrows.com/blog/python-requests-re...
      • mayhem[m]
      • BrainzBot
        LB-1725: Improve rate limit handling
      • BrainzGit
        [musicbrainz-server] 14reosarevok opened pull request #3448 (03master…MBS-13902): MBS-13902: Add test for persistent (did:plc) Bluesky links https://github.com/metabrainz/musicbrainz-serve...
      • mayhem[m]
        monkey: in the issue to try and make the donation page more readable, simon asked to increase the font size from 14 to 16px. sadly we get:
      • mayhem[m] uploaded an image: (95KiB) < https://matrix.chatbrainz.org/_matrix/media/v3/download/chatbrainz.org/sdrtYlIaVCgpiWeipsoOmyOr/image.png >
      • see navbar. any ideas/tips on how to solve this?
      • monkey[m]
        The easy option is to add font-size: 14px; to the .navbar css class:
      • monkey[m] uploaded an image: (70KiB) < https://matrix.chatbrainz.org/_matrix/media/v3/download/chatbrainz.org/ucYEvuqRXCoBDbMJbHTsXtiZ/image.png >
      • I think I remember seeing some other feedbakc about moving navbar items, so maybe that's not needed after the proposed changes?
      • mayhem[m]
        moving what? got a ticket?
      • monkey[m]
        I don't remember where I read it :/ Let me scratch my brain for a minute (+ coffee)
      • mayhem[m] uploaded an image: (103KiB) < https://matrix.chatbrainz.org/_matrix/media/v3/download/chatbrainz.org/hYfjKjkIYubcdGQERJAIyAZZ/image.png >
      • mayhem[m]
        ok, lets roll with that for now.
      • BrainzGit
        [metabrainz.org] 14mayhem opened pull request #498 (03master…increase-font-size): Increase font size https://github.com/metabrainz/metabrainz.org/pu...
      • mayhem[m]
        monkey: ^^
      • vardhan_ has quit
      • vardhan_ joined the channel
      • BrainzGit
        [metabrainz.org] 14mayhem merged pull request #498 (03master…increase-font-size): Increase font size https://github.com/metabrainz/metabrainz.org/pu...
      • [metabrainz.org] release 03v-2025-01-15.0 has been published by 14mayhem: https://github.com/metabrainz/metabrainz.org/re...
      • vardhan_ has quit
      • vardhan_ joined the channel
      • vardhan_ has quit
      • Gaurav993[m] has quit
      • vardhan_ joined the channel
      • vardhan_ has quit
      • vardhan_ joined the channel
      • yano has quit
      • yano joined the channel
      • suvid[m]
      • suvid[m] uploaded an image: (85KiB) < https://matrix.chatbrainz.org/_matrix/media/v3/download/matrix.org/yRxjaJLoRSUzGngnhKpeyQcN/image.png >
      • d4rk-ph0enix has quit
      • d4rk-ph0enix joined the channel
      • HemangMishra[m] joined the channel
      • HemangMishra[m]
        aerozol: jasje I have made the figma design of the 'created for you' screen to be implemented in android app. Would love to have your suggestions on the designs and improvements needed. https://www.figma.com/design/Hf46QX1kW0WK3SiGqF...
      • mayhem[m]
        monkey: I changed the cron teams, for now you should get no notifications.
      • monkey[m]
        Thanks !
      • JadedBlueEyes has quit
      • JadedBlueEyes joined the channel
      • ansh[m]
        <suvid[m]> "https://github.com/metabrainz/..." <- Sure! I'll have a look
      • monkey[m]
        suvid: (also ansh ): Looks like the error is thrown in the soundclooud javascript player. You can try updating that file at frontend/js/lib/soundcloud-player-api.js
      • But I suspect this just won't work as-is in the test framework, possibly because the node window object is missing postMessage
      • ansh: I think the apple music playlist export PR is ready, but I did leave some comment on the PR. We'll have to look into the troi issues, which are also going to apply for other music services I would guess
      • minimal joined the channel
      • But I'm fine if we merge this PR now, the button is on the page and export works (even though it only exports some tracks) so that's a success
      • jasje[m]
        <HemangMishra[m]> "aerozol: jasje I have made the..." <- Looks good on first look. I see you have created two designs. aerozol aerozol @aerozol:matrix.org: would be able to comment better here.
      • HemangMishra[m]
        <jasje[m]> "Looks good on first look. I..." <- I'll pick whichever looks better to u guys
      • * Yeah, I'll pick whichever looks better to u guys
      • BrainzGit
        [musicbrainz-server] 14reosarevok opened pull request #3449 (03beta…help-for-fat-admin-fingers): Beta: Don't show admin approval buttons on edit lists https://github.com/metabrainz/musicbrainz-serve...
      • texke has quit
      • texke joined the channel
      • mayhem[m]
        bitmap: do you know if the privs for the postgres user mbid_mapper have changed?
      • suddenly the user cannot see any tables anymore:
      • hendrix:~->docker exec -it c18d26b55c47 psql -U mbid_mapper musicbrainz_db... (full message at <https://matrix.chatbrainz.org/_matrix/media/v3/...>)
      • bitmap[m]
        mayhem[m]: weird, not that I know of, but I can take a look
      • mayhem[m]
        thx
      • bitmap[m]
        mayhem: if I connect the same way it appears there's no `search_path` set for the mbid_mapper user. if I `set search_path = musicbrainz, public;` it then shows everything there
      • d4rk-ph0enix has quit
      • for musicbrainz role we have default search_path set to musicbrainz, public
      • mayhem[m]
        hmmm. something must've changed, because our queries are now failing. I guess I better make them all fully qualified.
      • d4rk-ph0enix joined the channel
      • let me try that next.
      • monkey[m]
        mayhem, lucifer, ansh Hello! It is meeting time. Where are we doing the meeting?
      • mayhem[m]
        oh feck. I've been so slammed all day, I totes forgot. mea culpa.
      • gimme just a minute or two
      • monkey[m]
        Fine by me. I don't see the others online either
      • mayhem[m]
        well, do we bother starting a meeting when only half the team is here?
      • monkey[m]
        No. Let's wait a minute to see if they appear
      • mayhem[m]
        maybe better to just postpone to next monday at the expected time. we've been all over the place scheduling this one.
      • monkey[m]
        Yeah, that's probably why we're in this siutuation now XD
      • mayhem[m]
        let me schedule it for next week and make the zoom for it then.
      • sent
      • bitmap: I have no idea what happened, but fully qualifying the queries now works. odd.
      • bitmap[m]
        mayhem[m]: hmm, there's also a way to specify search_path when connecting, maybe some lower-level connection routine changed? I could also set it on the role if you'd like
      • mayhem[m]
        i was just curious how this change happened, but the fix is easy and in place, lets leave it at that. thanks!