#metabrainz

/

      • pavlovsfrog6 joined the channel
      • pavlovsfrog has quit
      • pavlovsfrog6 is now known as pavlovsfrog
      • lusciouslover has quit
      • lusciouslover joined the channel
      • Kladky joined the channel
      • rayyan_seliya123 joined the channel
      • rayyan_seliya123
        <lucifer> "rayyan_seliya123: you can push..." <- Hey lucifer: I have pushed the latest changes on that pr waiting for your review to move ahead !
      • _BrainzGit
        [musicbrainz-server] 14reosarevok closed pull request #3571 (03master…pr/3514): Pr/3514 https://github.com/metabrainz/musicbrainz-serve...
      • Kladky has quit
      • Kladky joined the channel
      • Maxr1998_ joined the channel
      • Maxr1998 has quit
      • JadedBlueEyes has quit
      • JadedBlueEyes joined the channel
      • mayhem[m] joined the channel
      • mayhem[m]
        julian45: hey! today is the day I check in with you about the test VM we made a few weeks back. still need it?
      • julian45[m] joined the channel
      • julian45[m]
        <mayhem[m]> "julian45: hey! today is the..." <- hey! i think i need it for 1 more week, and then we can probably get rid of it. would that work for you?
      • suvid[m] has quit
      • lucifer[m] joined the channel
      • lucifer[m]
        rayyan_seliya123, suvid, m.amanullah7 will answer your questions later today.
      • mayhem: gentle reminder about similarity data.
      • mayhem[m]
        on it. (very soon anyways)
      • rostiku[m] joined the channel
      • rostiku[m]
        Hey guys, I'm working on joining release_first_release_date on release to determine the earliest released track. This is what SQL script I have so far:... (full message at <https://matrix.chatbrainz.org/_matrix/media/v3/...>)
      • s/released track/release/
      • Do the release ids have any significance? I am thinking to just stick to the one with the lowest release id. But to me that makes the whole join process irrelevant.
      • reosarevok[m]
        If the date is the same, by definition any of them are equally earliest :)
      • Jade[m] joined the channel
      • Jade[m]
        Heya, I won't be at the meeting today, being on holiday :)
      • reosarevok[m]
        Release ids will just depend on what was added first, which has nothing to do with the actual date
      • If you don't really care about earliest as such but want a canonical release for each group, the LB people have code for that
      • mayhem: ^
      • But that's a different thing of course :)
      • Jade: enjoy!
      • Jade[m]
        reosarevok[m]: Thank you ❤️
      • rostiku[m]
        Yeah, I guess you're right, it's the "earliest" release and there can be multiple "earliest" releases. I am just trying to decide how to find the "best" earliest release.
      • I'm thinking if I combine the two processes, get the "earliest" releases, then see what was added first (by release ID), it'll be satisfactory. But I'll be happy to look into the "canonical release" determination code as well
      • reosarevok[m]
        bitmap: do you know if there's any real reason we don't show ACs in duplicates? (MBS-14065)
      • BrainzBot
        MBS-14065: Show artist credits in release editor duplicates tab https://tickets.metabrainz.org/browse/MBS-14065
      • mayhem[m]
        <reosarevok[m]> "If you don't really care about..." <- See this blog post: https://blog.metabrainz.org/2023/06/12/new-data...
      • rostiku[m]
        Thanks guys! ❤️
      • loving this open project
      • reosarevok[m]
        bitmap, zas, outsidecontext: not sure what changes we need to do for MBS-14062 - maybe you know more about it? (I'm not sure why it shouldn't work when allowing it, but that's what the ticket claims)
      • BrainzBot
        MBS-14062: Tagger link on tag lookup no longer works with Chrome's local network access restrictions https://tickets.metabrainz.org/browse/MBS-14062
      • outsidecontext[m joined the channel
      • outsidecontext[m
        reosarevok: not sure, I also think it should work once permission was granted. I can test
      • I looked into this recently for my magic tagger button user script. There it still works without prompt due to user scripts having different permissions
      • But it generally looked like the normal button on the page should work once granted
      • zas[m] joined the channel
      • zas[m]
        This is more or less the same issue than with Safari right? Do we really need the Tagger button ? I know it's convenient but it is more and more a problem it seems
      • outsidecontext[m
        I think safari is different
      • reosarevok: the difference between https://musicbrainz.org/taglookup and the other pages is that everywhere else the tagger buttons have a click event and the call to the local networks is controlled by scripting. That makes Chrome show the prompt to grant access. On taglookup it is just a link and Chrome disallows the navigation.
      • outsidecontext[m uploaded an image: (61KiB) < https://matrix.chatbrainz.org/_matrix/media/v3/download/matrix.org/wFBbfmGgYhLHTmObqHILdhdq/image.png >
      • Actually reloading that page then works (but this is not pretty anyway)
      • So the fix would be to enable the script handling of the tagger buttons on the taglookup page as well
      • reosarevok[m]
        Ok, I wonder if it's a React vs TT thing
      • I'll check
      • mayhem[m]
        lucifer: labs.api is running quite slowly -- the similarity data takes eons to return, making eval hard. are there some missing indexes, perhaps?
      • rostiku[m]
        hey, is there a way to populate the cover_art_archive schema after setting up the musicbrainz-server? I am currently retrieving cover art using the CAA API and caching it. Wondering if I could just populate the schema and not need to query the API anymore
      • reosarevok[m]
        outsidecontext: I don't have Picard installed here - what's the link you get when tag-lookuping from Picard?
      • Does it have the ?tport=8000 bit or? (any example works)
      • outsidecontext[m
        reosarevok: I think it is not possible to directly open the taglookup page from within Picard. But yes, there are multiple cases where Picard can open a MB website, and it will always append the `tport` parameter
      • reosarevok[m]
        Hmm, ok - I thought that page was made specifically to use it from Picard :D
      • AFAICT the same code is used for tagger everywhere, weird
      • outsidecontext[m
        reosarevok: I was wrong. Looking up clusters uses the taglookup page. Such as https://musicbrainz.org/taglookup?artist=Savana...
      • reosarevok: and the code in both cases includes the click handler?
      • reosarevok[m]
        Checking
      • https://github.com/metabrainz/musicbrainz-serve... the handler is defined inside TaggerIcon which is what the search uses...
      • lusciouslover has quit
      • lusciouslover joined the channel
      • outsidecontext: what search has links that fail for you in taglookup? Release? Recording? Both?
      • (and the same do not fail for normal not tag-lookup searches?)
      • outsidecontext[m
        reosarevok: they behave both the same
      • on taglookup they are just html links, on search pages and elsewhere they have the click handlers
      • but you should be able to reproduce this. it's independent of browser
      • also no need for picard
      • for the working case, open https://musicbrainz.org/search/textsearch?limit... . Look at the network tab of the developer tools, then click on a tagger button. You should see a request to http://127.0.0.1:8000/openalbum?...
      • when clicking on the tagger button the browser will navigate to the the local address
      • reosarevok[m]
        Hmm,ok, yeah, I see the difference
      • Now, where does it come from :)
      • buildTaggerLink always returns http://127.0.0.1:${tport}/${path}?id=${gid}
      • Aaah
      • The hydration is missing for the taglookup code, heh
      • I think anyway. Let me test locally
      • Yeah. Trivial fix. Thanks!
      • _BrainzGit
        [musicbrainz-server] 14reosarevok opened pull request #3573 (03master…MBS-14062): MBS-14062: Add TaggerIcon manifest to TagLookup results https://github.com/metabrainz/musicbrainz-serve...
      • BrainzBot
        MBS-14062: Tagger link on tag lookup no longer works with Chrome's local network access restrictions https://tickets.metabrainz.org/browse/MBS-14062
      • reosarevok[m]
        bitmap: ^ if you can review that when around we can put it in beta later today
      • holycow23[m] joined the channel
      • holycow23[m]
        <lucifer[m]> "holycow23: running with increase..." <- Hey, sorry was off for a little bit, was a little under the weather
      • What exactly is fixed with this specific extension to the script, is the working of the `get_incremental_listens_df` function solved with it?
      • lucifer[m]
        holycow23: it runs the spark/pyspark processes with extra memory. the stack trace of your error failures had out of memory errors. running with extra memory makes it work for me.
      • holycow23[m]
        Okay will give it a try
      • reosarevok[m]
        <BANG>
      • Welcome to another MetaBrainz Monday Meeting!
      • Jade is on holiday so no notes today!
      • And monkey sent me... "My Monday Meeting Motes"
      • Which say:
      • '''
      • Hello one and all!
      • I spent most of last week working on moving our CSS preprocessing from Less to Sass, following our migration from Bootstrap 3 to 5.
      • It is not complicated, but a lot of details to go through.
      • Also had a team meeting with the LB team discussing work and goals for the coming month or two.
      • Some miscellaneous tasks and that's about it for me.
      • '''
      • More Monday, more motes, monkey, much merriment!
      • That's it for the mailed-in motes
      • On my list for today: suvid, lucifer, bitmap, zas, Gautam_coder, kellnerd, reosarevok, Hemang Mishra, MonkeyPython (Jigen (IRC)), outsidecontext, holycow23, jasje, julian45, ansh, yvanzo, m.amanullah7, @fettuccinae:matrix.org, mayhem
      • I see zas around, want to start?
      • zas[m]
        Hey
      • Last week, mostly worked on Picard, patches & PR reviews, Picard dev is quite active at the moment :)
      • Plus usual stuff, system supervision & maintenance, various minor tasks & fixes, user support, etc.. fin. julian45
      • ?
      • julian45[m]
        hello!
      • wasn't able to get much done last week :( had to support 2 high-impact go-lives for $dayjob
      • did usual system maintenance on auth servers though, as well as some peering at arcane jira docs
      • fin, go kellnerd ?
      • kellnerd[m] joined the channel
      • kellnerd[m]
        Hey all!
      • For the last two weeks I've been working on a couple of Harmony improvements which I really wanted to get done.
      • Recording link types are now properly detected for Bandcamp tracks (thanks to arsinclair!) and Apple Music.
      • I finally did the massive refactoring of the MBID resolver to make use of MBS-13943
      • BrainzBot
        MBS-13943: Lookup multiple URLs at once with the API https://tickets.metabrainz.org/browse/MBS-13943
      • kellnerd[m]
        Usage of the MB API went down from up to a few dozen requests (for compilations) to one request per release lookup on average.
      • Only very few releases need more than 1 req (100 URLs included) and as a bonus I could finally implement a search for release links as well to detect potential duplicates (without any additional requests).
      • Today I submitted a PR for PICARD-2828 because I was still annoyed by it forcing me to change Picard configuration between releases sometimes.
      • BrainzBot
        PICARD-2828: Separate options for standardized instrument and vocal credits https://tickets.metabrainz.org/browse/PICARD-2828
      • kellnerd[m]
        That's it, go suvid!
      • reosarevok[m]
        Still on my list for today: lucifer, bitmap, Gautam_coder, reosarevok, Hemang Mishra, MonkeyPython (Jigen (IRC)), outsidecontext, holycow23, jasje, ansh, yvanzo, m.amanullah7, @fettuccinae:matrix.org, mayhem
      • suvid[m] joined the channel
      • suvid[m]
      • reosarevok[m]
        suvid: it's best if you post one line at a time :) (it will be less messy for IRC users)
      • fettuccinae[m] joined the channel
      • fettuccinae[m]
        Hi!
      • reosarevok[m]
        Anyway, no big deal, for the future :)
      • suvid[m]
        oh ok I'll keep that in mind
      • reosarevok[m]
        @fettuccinae:matrix.org: go on :)
      • fettuccinae[m]
        Last week ive worked on adding authentication for notification endpoints and spent too much time writing tests.
      • This week ill work on adding cron jobs for sending and deleteing notifications.
      • Thats it for me, next ansh ?
      • ansh[m] joined the channel
      • ansh[m]
        Hi Everyone!
      • Last week, I worked on refactoring the routing to improve the load times. And rewriting some components to handle promises better
      • Apart from them, I’ve been working on fixing re-rendering issues on the website which might cause performance issues down the line.
      • And had the LB team meeting discussing our current progress and work for the coming month.
      • That’s it from my side.