#metabrainz

/

      • discordbrainz
        <02UltimateRiff> @Max (Maxr1998) yeah, MovieBrainz was moved to ChatBrainz (or whatever the current thing is)
      • <02UltimateRiff> (Discord link, because I'm not in IRC https://discord.com/channels/937555302251905084...)
      • pite has quit
      • pranav[m] joined the channel
      • pranav[m]
        akshaaatt: was a bit busy with intern prep, the user pages Pr shud be ready in 3-4 days, I’ll ping u once it’s done. Listens tab is done as of now
      • Kladky joined the channel
      • natrius[m] joined the channel
      • natrius[m]
        <Maxr1998[m]> "Any comment on this question..." <- Can you link me to the channel? I might be able to asnwer.
      • lucifer
        aerozol[m]: all processed now.
      • Maxr1998[m]: should be fixed now
      • lucifer[m]
        <Maxr1998[m]> "Another question, I still have..." <- Should be fine to keep them, alternatively you can delete them.
      • aerozol[m]
        Thanks lucifer - what happened?
      • Maxr1998[m]
        <lucifer> "Maxr1998: should be fixed now" <- Can confirm, works again. Thanks!
      • <lucifer[m]> "Should be fine to keep them..." <- Alright, thanks.
      • lucifer[m]
        aerozol: the component (spark reader) processing playlists taking a long time to process, which made a different component (rabbitmq) assume that the first one was broken and kept trying to restart and process playlists again. setting up a cycle blocking the processing of any other messages.
      • aerozol[m]
        Rough! Thanks for the info
      • BrainzGit
        [listenbrainz-server] 14amCap1712 opened pull request #2918 (03master…fix-spark-reader): Add a background job processor for Spark Reader https://github.com/metabrainz/listenbrainz-serv...
      • Jade[m] has quit
      • reosarevok[m]
        Argh.
      • yvanzo, bitmap: I'm kinda confused. For https://github.com/metabrainz/musicbrainz-serve... apparently we were not even showing translated names and descriptions here earlier
      • For some reason, this works for name but not description in Attribute.js:
      • lp_attributes(attribute.name, attribute.entityType)
      • I'm not even sure why it works, because https://github.com/mwiencek/musicbrainz-server/... isn't actually setting contexts to some attributes in theory but they still work, like artist_type
      • Is there any good way I'm missing to have this show properly translated?
      • Maxr1998_ joined the channel
      • Maxr1998 has quit
      • minimal joined the channel
      • monkey[m] joined the channel
      • monkey[m]
        lucifer: Hello! I see that LB prod image v-2024-06-21.0 was published, but not deployed. Can I deploy it on prod?
      • lucifer[m]
        monkey: probably wait for some time, i have a pr to release today. will update prod after that
      • monkey[m]
        OK great, thanks !
      • I'll try to merge some PRs before that too
      • lucifer[m]
        awesome
      • mayhem: will you have time to review the spark reader PR today?
      • mayhem[m] joined the channel
      • mayhem[m]
        I managed to catch covid, so I"m running at about 10% right now... but let me see.
      • lucifer[m]
        ah damn 😢
      • take care!
      • mayhem[m]
      • lucifer[m]
        yes, its working fine in prod already fwiw so fine to review or merge later
      • mayhem[m]
        the list of steps for dealing with syncronization seems to be on point.
      • glad to see you're not afraid of threads, lol.
      • clever copy pasta on the PR desc, lol.
      • lucifer[m]
        not afraid lol, they just complicate things so try to avoid them unless necessary.
      • :)
      • mayhem[m]
        there is a time and a place for threads. just some basic thinking at the beginning and you can avoid problems
      • lucifer[m]
        yup makes sense
      • BrainzGit
        [listenbrainz-server] 14amCap1712 merged pull request #2918 (03master…fix-spark-reader): Add a background job processor for Spark Reader https://github.com/metabrainz/listenbrainz-serv...
      • lucifer[m]
        thanks mayhem !
      • yvanzo[m]
        Hi reosarevok, having detailed edit type descriptions is still helping with translating edit type names faithfully.
      • Just pushed those to the branch beta.
      • Hi yellowhatpro: It is easiest to start with testing the functions that don’t call any service (PG, WM), such as most of the functions in `src/poller/utils.rs`.
      • For each function, a common use case can be added through https://doc.rust-lang.org/rust-by-example/testi...
      • Whereas corner cases can be tested through https://doc.rust-lang.org/rust-by-example/testi...
      • yellowhatpro[m]
        Hi yvanzo, thanks. I'll come up with the tests soon
      • Toasty joined the channel
      • theflash__ joined the channel
      • theflash__
        akshaaatt: please have a look I have fixed the search bug
      • zerodogg has quit
      • revi has quit
      • BrainzGit
        [listenbrainz-server] 14amCap1712 merged pull request #2916 (03master…dependabot/npm_and_yarn/multi-1729a3ee87): build(deps): bump ws and engine.io-client https://github.com/metabrainz/listenbrainz-serv...
      • zerodogg joined the channel
      • [listenbrainz-server] 14MonkeyDo merged pull request #2905 (03master…apple-music-search): Improve Apple Music search with fuzzy matching https://github.com/metabrainz/listenbrainz-serv...
      • [listenbrainz-server] release 03v-2024-06-24.0 has been published by 14github-actions[bot]: https://github.com/metabrainz/listenbrainz-serv...
      • reosarevok[m]
      • I left the [data science] field, as did most of my smarter friends, and my salary continued to rise a reasonable rate and sustainably as I learned the wisdom of our ancient forebearers. You can hear it too, on freezing nights under the pale moon, when the fire burns low and the trees loom like hands of sinister ghosts all around you - when the wind cuts through the howling of what you hope is a wolf and hair stands on end, you can
      • strain your ears and barely make out:
      • "Just Use Postgres, You Nerd. You Dweeb."
      • * "I left, * You Dweeb.""
      • bitmap[m]
        <reosarevok[m]> "For some reason, this works..." <- by doesn't work, do you mean "no translation key found"? because some of the descriptions are empty, e.g., artist_type: Other, which would cause that
      • reosarevok[m]
        Yup
      • bitmap[m]
        <reosarevok[m]> "I'm not even sure why it works..." <- if `$ctx` is `NULL` it default to using the table name as the msgctxt
      • pite joined the channel
      • reosarevok[m]
        Aah, ok
      • So for null descs, should they just be doing '' if no description and that'd work?
      • Or does '' still fail?
      • bitmap[m]
        that fails (I think they are already '')
      • something like nonEmpty(attribute.description) ? lp_attributes(attribute.description, context) : null should work
      • reosarevok[m]
        Hmm. I mean, we should have descriptions for all :p
      • But last time I tried that I killed solr so :D
      • Ok, I'll do that for now
      • descriptions for some, miniature ternary operators for others
      • theflash__ has quit
      • monkey[m]
        <reosarevok[m]> "descriptions for some, miniature..." <- "Miniature Ternary Operators" is not a bad band name...
      • Lotheric has quit
      • Lotheric joined the channel
      • BrainzGit
        [listenbrainz-server] 14amCap1712 opened pull request #2919 (03master…spark-fies): Spark development env fixes https://github.com/metabrainz/listenbrainz-serv...
      • Lotheric
        hi, testing SchildiChat and it seems to work except the IRC bridge
      • "Your connection to the IRC network 'irc.libera.chat' has been lost. "
      • how can I join the bridge ?
      • BrainzGit
        [listenbrainz-server] 14amCap1712 merged pull request #2919 (03master…spark-fies): Spark development env fixes https://github.com/metabrainz/listenbrainz-serv...
      • Lotheric[m] joined the channel
      • Lotheric[m]
        test #3
      • Lotheric
        yay!, fixed
      • Lotheric has quit
      • BrainzGit
        [musicbrainz-server] 14reosarevok opened pull request #3297 (03master…MBS-13638): MBS-13638: Remove duplicate schema on in_use query https://github.com/metabrainz/musicbrainz-serve...
      • yvanzo[m]
        bitmap, reosarevok: Is there any special task to do while releasing MBS?
      • reosarevok[m]
        Probably not? No new reports IIRC but do make sure
      • Not from my side anyway, bitmap might have some EAA bits
      • bitmap[m]
        there is one related to the ro-connector branch: need to make a few small changes to the production DBDefs
      • yvanzo[m]
        bitmap: Some changes to docker server configs I guess?
      • bitmap[m]
        yeah, and there are a couple commits that need to be reverted for the EAA too (for the edit type stubs)
      • I'll open a PR for docker-server-configs
      • yvanzo[m]
        Do you prefer to handle this release maybe?
      • I’ll start drafting the blog post instead.
      • bitmap[m]
        sure, can do
      • reosarevok[m]
        bitmap: does anything else need to happen regarding MBS-13632 ?
      • BrainzBot
        MBS-13632: Add a script to rebuild indexes using musicbrainz_unaccent https://tickets.metabrainz.org/browse/MBS-13632
      • reosarevok[m]
        (for prod/beta I mean)
      • BrainzGit
        [musicbrainz-server] 14reosarevok merged pull request #3297 (03master…MBS-13638): MBS-13638: Remove duplicate schema on in_use query https://github.com/metabrainz/musicbrainz-serve...
      • reosarevok[m]
        zas: when you have some time see the "Website Connection Issue - StarLink" email, I dunno myself :)
      • bitmap[m]
        reosarevok[m]: not for this release, it's just something we should do for the future
      • reosarevok[m]
        Oh, I guess there's some report changes in the release
      • So make sure to update the appropriate container please :)
      • bitmap[m]
        should I merge master to beta one last time for your in_use fix?
      • actually there are a couple other things in master that weren't beta-tested: MBS-13630, MBS-13624
      • BrainzBot
        MBS-13630: Improve reason for not being able to vote on closed auto-editor election https://tickets.metabrainz.org/browse/MBS-13630
      • MBS-13624: ISE when browsing old relationship type edits https://tickets.metabrainz.org/browse/MBS-13624
      • reosarevok[m]
        yvanzo: is the thumbs up "these two are small and seem fine to put out without beta testing" or "we should wait wth these"? :D
      • (I'm fine with either)
      • bitmap[m]
        I didn't merge them
      • (already building docker images)
      • reosarevok[m]
        Works for me :)
      • tykling_ is now known as tykling
      • yvanzo[m]
        Blog post is just missing an intro if any of you want to write it, I’ll prepare the MB Docker update.
      • bitmap[m]
        updating MB prod servers in a minute
      • BrainzGit
        [listenbrainz-server] 14MonkeyDo opened pull request #2920 (03master…dashboard-ws-listens): Dashboard ws listens https://github.com/metabrainz/listenbrainz-serv...
      • bitmap[m]
        there were some circleci failures on the production branch, checking those first
      • BrainzGit
        [listenbrainz-server] 14MonkeyDo opened pull request #2921 (03master…playlist-duration): LB-1512: Display total duration of playlist https://github.com/metabrainz/listenbrainz-serv...
      • theflash__ joined the channel
      • bitmap[m]
        fixed that, but I'll proceed with the release after the meeting (and let Selenium tests run in the meantime)
      • reosarevok[m]
        <BANG>
      • Hi, and welcome to another MetaBrainz Monday Meeting
      • Magnificent Metadata Mastery!
      • mayhem[m] hopes to go soonish
      • I have received 3 mailed-in reviews today
      • (plus Jigen (IRC) who said he has nothing for the week)
      • ericd says:
      • "This week I corrected the sorting order of fresh releases, by release date. Also added fresh releases for a specific user."
      • aerozol says:
      • '''
      • Did a little bit of everything last week - App mockups, tickets, socials, wiki updates, moderation, EAA stuff...
      • The only thing that stood out was working with Jade to finalize the email design for their GSoC project. Went surprisingly smoothly, seems like nobody had complaints 🤯😁
      • Long holiday weekend in NZ this week (Matariki), but I'll probably still be around. Smell ya later!
      • '''
      • And rimskii says:
      • '''
      • Hi everyone!
      • Starting w great news, my PR for importing Spotify playlists was successfully merged into Troi and LB repo(huge thanks to @lucifer) we fixed the bug and also optimized the code to reduce the number of calls needed to handle requests
      • Also I've started working with wolf and have set up the database there. Well It was challenging, but with the help of @lucifer, we managed to get it done.