#metabrainz

/

      • reosarevok
        Oh, you just asked. Then yes :D
      • 2023-08-16 22811, 2023

      • mayhem
        I'm working on the metadata cleaner again. and I am looking for examples of common useless guff that is found at the end of track names in parens.
      • 2023-08-16 22817, 2023

      • mayhem
        e.g. (2018 remaster)
      • 2023-08-16 22827, 2023

      • mayhem
        live, demo, master, remaster.
      • 2023-08-16 22829, 2023

      • reosarevok
        (bonus track)
      • 2023-08-16 22838, 2023

      • reosarevok
        or just (bonus)
      • 2023-08-16 22839, 2023

      • mayhem
        great, please send me more.
      • 2023-08-16 22802, 2023

      • reosarevok
        Arguably live or demo are not useless but I assume this is for some sort of canonical thing?
      • 2023-08-16 22822, 2023

      • mayhem
        not useless, but often not useful for finding a match.
      • 2023-08-16 22836, 2023

      • mayhem
        this is to resolve a list of MBIDs to a local collection of files
      • 2023-08-16 22842, 2023

      • arsh
        mayhem: Sure I will starting on the feedback
      • 2023-08-16 22836, 2023

      • mayhem
        great.
      • 2023-08-16 22810, 2023

      • mayhem
        and when you're done with that and are ready to improve the CSS, please ping monkey. He's volunteered to show you a few tricks to make the page look lots better with just a few changes.
      • 2023-08-16 22841, 2023

      • monkey
        "broad strokes" was the term I used I believe
      • 2023-08-16 22816, 2023

      • mayhem
        ok...
      • 2023-08-16 22823, 2023

      • arsh
        That’s great I will
      • 2023-08-16 22834, 2023

      • mayhem
        He's volunteered to show you a few tricks to make the broad strokes lots better with just a few changes.
      • 2023-08-16 22838, 2023

      • mayhem
        better monkey?
      • 2023-08-16 22829, 2023

      • monkey
        He's volunteered to broad strokes a few strokes to broaden your strokes with just a few strokes.
      • 2023-08-16 22845, 2023

      • mayhem
        He's broadly stroked the broad strokes a few strokes to stoken your broads with just a few brokes.
      • 2023-08-16 22857, 2023

      • arsh
        😂
      • 2023-08-16 22807, 2023

      • monkey
        Im broaken, having a strokes
      • 2023-08-16 22831, 2023

      • mayhem
        borked.
      • 2023-08-16 22810, 2023

      • mayhem
      • 2023-08-16 22837, 2023

      • mayhem
        kellnerd: do you know a python way to implement the above to work on non-ascii word characters?
      • 2023-08-16 22817, 2023

      • reosarevok
        mayhem: ETI stuff (that we always put in parens if at the end):
      • 2023-08-16 22828, 2023

      • reosarevok
      • 2023-08-16 22835, 2023

      • mayhem
        ETI?
      • 2023-08-16 22838, 2023

      • reosarevok
        Whether all of that should be ignored? probably not all, but
      • 2023-08-16 22846, 2023

      • reosarevok
        "extra track information"
      • 2023-08-16 22850, 2023

      • mayhem
        thanks very useful!
      • 2023-08-16 22806, 2023

      • reosarevok
        Then there's this
      • 2023-08-16 22809, 2023

      • reosarevok
      • 2023-08-16 22806, 2023

      • reosarevok
        Some might still be of interest
      • 2023-08-16 22808, 2023

      • mayhem
        the first list is great, the second has a couple of good ones I'll adopt.
      • 2023-08-16 22826, 2023

      • Maxr1998_ joined the channel
      • 2023-08-16 22828, 2023

      • Maxr1998 has quit
      • 2023-08-16 22815, 2023

      • reosarevok
        Yeah, the second is supposed to be more "miiight be this, might not be" so that makes sense
      • 2023-08-16 22814, 2023

      • reosarevok
        yvanzo: do you think I should just not support www./event links? Since they're kinda hidden anyway?
      • 2023-08-16 22800, 2023

      • kellnerd
        mayhem: Should be possible by matching `\p{Letter}`, but only with the "regex" library, not with the standard "re".
      • 2023-08-16 22817, 2023

      • kellnerd
        We are already using "regex" here: https://github.com/metabrainz/listenbrainz-matchi…
      • 2023-08-16 22818, 2023

      • mayhem
        ohhh, will tryh that.
      • 2023-08-16 22828, 2023

      • mayhem
        yes, very convenient.
      • 2023-08-16 22833, 2023

      • reosarevok
        yvanzo: also, https://tickets.metabrainz.org/browse/MBS-13233 seems a lot smaller than the others, wonder if it makes sense to do the autoselect - are we going to do that for every country's platform?
      • 2023-08-16 22834, 2023

      • BrainzBot
        MBS-13233: Autoselect the ticketing relationship for Eventfinda links
      • 2023-08-16 22841, 2023

      • reosarevok
        aerozol: ^ how big is that down there?
      • 2023-08-16 22819, 2023

      • reosarevok
        I'm thinking maybe it's enough for us to have a ticketing rel and people can hand-select it...
      • 2023-08-16 22819, 2023

      • milkii has quit
      • 2023-08-16 22849, 2023

      • milkii joined the channel
      • 2023-08-16 22850, 2023

      • BrainzGit
        [musicbrainz-server] 14reosarevok opened pull request #3012 (03master…MBS-13237): MBS-13237: Clean up Twitch mobile links https://github.com/metabrainz/musicbrainz-server/…
      • 2023-08-16 22813, 2023

      • BrainzGit
        [listenbrainz-server] 14amCap1712 merged pull request #2550 (03master…refactor-cache): Refactor Spotify Metadata Cache https://github.com/metabrainz/listenbrainz-server…
      • 2023-08-16 22801, 2023

      • BrainzGit
        [musicbrainz-server] 14reosarevok opened pull request #3013 (03master…MBS-13233): MBS-13234 / MBS-13235: Handle ticketing links https://github.com/metabrainz/musicbrainz-server/…
      • 2023-08-16 22852, 2023

      • BrainzGit
        [listenbrainz-server] 14MonkeyDo opened pull request #2551 (03master…LB-1320): LB-1320: replace cover art fallback image https://github.com/metabrainz/listenbrainz-server…
      • 2023-08-16 22834, 2023

      • BrainzGit
        [listenbrainz-server] 14MonkeyDo opened pull request #2552 (03master…react-testing-library): Install and set up React Testing Library, update Jest https://github.com/metabrainz/listenbrainz-server…
      • 2023-08-16 22854, 2023

      • monkey
        lucifer: I made some progress yesterday using react-testing-library along with Enzyme in our tests, as we discussed, and converted one test file (which took hours as I'm learning to use the library). Works a treat !
      • 2023-08-16 22834, 2023

      • lucifer
        oh awesome monkey!
      • 2023-08-16 22838, 2023

      • monkey
        And I wanted to say sorry, because… https://usercontent.irccloud-cdn.com/file/ICoV4Uf…
      • 2023-08-16 22848, 2023

      • mayhem
        oy
      • 2023-08-16 22803, 2023

      • monkey
        And that's mainly one single test file and some gaff around it
      • 2023-08-16 22832, 2023

      • monkey
        And of course tests are failing in CI 😠
      • 2023-08-16 22819, 2023

      • monkey
        `2 snapshots failed from 2 test suites.` grrrr
      • 2023-08-16 22838, 2023

      • monkey
        Follow-up PR: nuke snapshots
      • 2023-08-16 22811, 2023

      • jasje
        Can't find these icons on fontAwesome https://usercontent.irccloud-cdn.com/file/mXPlcjQ…
      • 2023-08-16 22820, 2023

      • jasje
        aerozol^^
      • 2023-08-16 22812, 2023

      • monkey
        Which icons jasje ?
      • 2023-08-16 22851, 2023

      • jasje
        monkey: dropdown icons in above img
      • 2023-08-16 22829, 2023

      • monkey
        Sorry, I don't understand. You are looking for what exactly? SVGs? the icon definition on the FA website?
      • 2023-08-16 22811, 2023

      • mayhem
        totallly OT monkey , but the three stacked buttons on the listen cards is really hard to hit on a large monitor or mobile.
      • 2023-08-16 22833, 2023

      • mayhem
        I would love something like a burger on a diet. (fat dots)
      • 2023-08-16 22844, 2023

      • monkey
        mayhem: Yeah, I think one of the recent updates to the icon library made that worse too. I think there's a ticket open for it
      • 2023-08-16 22851, 2023

      • mayhem
        great
      • 2023-08-16 22847, 2023

      • monkey
        jasje: If it's the SVGs you want, you can simply inspect the web page in your browser console https://usercontent.irccloud-cdn.com/file/BhmQ3Hl…
      • 2023-08-16 22855, 2023

      • jasje
        monkey: no way to download it but i guess copying path should do
      • 2023-08-16 22838, 2023

      • monkey
      • 2023-08-16 22851, 2023

      • monkey
        It's there on the FA website as well
      • 2023-08-16 22851, 2023

      • monkey
      • 2023-08-16 22810, 2023

      • monkey
      • 2023-08-16 22819, 2023

      • jasje
        where's the above photo from?
      • 2023-08-16 22827, 2023

      • monkey
      • 2023-08-16 22811, 2023

      • jasje
      • 2023-08-16 22822, 2023

      • jasje
        how do i navigate to this page
      • 2023-08-16 22854, 2023

      • jasje
        or is it some custom software?
      • 2023-08-16 22828, 2023

      • monkey
      • 2023-08-16 22821, 2023

      • jasje
        monkey: ohh you went to font awesome i thought you picked it up from inspect
      • 2023-08-16 22845, 2023

      • monkey
        I mean… I did send the URL twice :p
      • 2023-08-16 22809, 2023

      • jasje
        anyways i found this cool project thatll just convert svg path from inpect to downloadable svg
      • 2023-08-16 22803, 2023

      • jasje
        thanks monkey!
      • 2023-08-16 22829, 2023

      • jasje
        very quick demonstration tho
      • 2023-08-16 22805, 2023

      • monkey
        Why not use the download button from the FA page?
      • 2023-08-16 22823, 2023

      • jasje
        monkey: cos i need to look it up again and again
      • 2023-08-16 22832, 2023

      • jasje
        for the correct size
      • 2023-08-16 22800, 2023

      • jasje
        the site im using give alot of options
      • 2023-08-16 22844, 2023

      • jasje
        plus the extra copyright tag
      • 2023-08-16 22858, 2023

      • iconoclasthero joined the channel
      • 2023-08-16 22853, 2023

      • iconoclasthero has quit
      • 2023-08-16 22854, 2023

      • derat joined the channel
      • 2023-08-16 22855, 2023

      • ssam has quit
      • 2023-08-16 22856, 2023

      • derat
        hey, would it be possible to get reviews and/or feedback for the various MBS PRs that i've sent?
      • 2023-08-16 22856, 2023

      • derat
      • 2023-08-16 22857, 2023

      • derat
      • 2023-08-16 22857, 2023

      • derat
      • 2023-08-16 22805, 2023

      • derat
        (i also had an open question on https://github.com/metabrainz/musicbrainz-server/… about whether the change would be accepted if i added more event handlers to completely emulate normal focus-cycling)
      • 2023-08-16 22829, 2023

      • mayhem
        zas: outsidecontext ^^
      • 2023-08-16 22852, 2023

      • yvanzo
        hi derat, thanks for contributions, will look into it
      • 2023-08-16 22812, 2023

      • derat
        yvanzo: thanks, much appreciated!
      • 2023-08-16 22850, 2023

      • ssam joined the channel
      • 2023-08-16 22822, 2023

      • derat
        (i've just been focusing on the UX things that i find most annoying when editing so far :-) )
      • 2023-08-16 22811, 2023

      • yvanzo
        (3006 depends on 3008 so I guess both will be reviewed at the same time)
      • 2023-08-16 22853, 2023

      • derat
        i think that they can be merged in an arbitrary order, but i wasn't sure if there's any way to express that. the roman numerals surprised me a bit, to be honest!
      • 2023-08-16 22845, 2023

      • BrainzGit
        [bookbrainz-site] 14architbaurai opened pull request #1008 (03master…BB-749): BB-749: Links that include "https://" as part of the non-protocol of the URL in Annotations break. https://github.com/metabrainz/bookbrainz-site/pul…
      • 2023-08-16 22815, 2023

      • yvanzo
        derat: Ok but #3006 alone doesn’t resolve any issue, does it?
      • 2023-08-16 22818, 2023

      • derat
        yvanzo: #3006 enables tab, enter, tab, enter, etc. to be used to select the first recording for each track, which is the primary functionality i care about for MBS-13207. #3008 is nice to have since once the first recording has the focus, it fixes the normal radio button behavior of being able to use the arrow keys to select other recordings
      • 2023-08-16 22819, 2023

      • BrainzBot
        MBS-13207: Radio buttons on Add Release page's Recordings tab aren't keyboard-focusable https://tickets.metabrainz.org/browse/MBS-13207
      • 2023-08-16 22844, 2023

      • yvanzo
        derat: Ok I tested it locally and found a way to reproduce this specific issue separately :)
      • 2023-08-16 22804, 2023

      • iconoclasthero joined the channel
      • 2023-08-16 22817, 2023

      • yvanzo
        derat: I tried the two patches together now and it makes funny things when using arrow keys to select the recording: the order of the suggested recordings may change.
      • 2023-08-16 22848, 2023

      • derat
        yvanzo: i'm not sure how #3006 could be contributing to that -- is it possible that you also see that just with #3008? i haven't been able to figure out how to test with a long list of recordings locally (probably because i don't have working search in my local setup), but i did notice some flickering that suggested to me that another search is
      • 2023-08-16 22849, 2023

      • derat
        being performed whenever a new recording is selected. i'll try to see if i can repro what you're seeing locally...
      • 2023-08-16 22848, 2023

      • yvanzo
        I added a release duplicating http://localhost:5000/release/0d9e2d13-3a3b-4823-… which is part of the sample data.
      • 2023-08-16 22830, 2023

      • yvanzo
        Then track #1, there are 3 recordings for “Mr Self Destruct”.
      • 2023-08-16 22807, 2023

      • yvanzo
        You can switch the search field to use direct search (below when scrolling).
      • 2023-08-16 22819, 2023

      • iconoclasthero has quit
      • 2023-08-16 22803, 2023

      • yvanzo
        (Direct search is a PG fallback when Solr isn’t available.)
      • 2023-08-16 22826, 2023

      • derat
        there must be something strange with my musicbrainz-docker setup. when i do the same thing, i see recordings listed in the dropdown after doing a direct search, but when i click one, it often doesn't show up as a radio button (although it *does* get associated with the track). this is without my changes
      • 2023-08-16 22826, 2023

      • derat
        tbh, this seems a bit weird even on musicbrainz.org. when i create a new release as a dupe of an existing one, all of the tracks are associated with the expected recordings but the "edit" bubble for each only has a radio button for "Add a new recording". when i search for a track's name and click a recording, the track association is updated and
      • 2023-08-16 22827, 2023

      • derat
        now there are radio buttons for both the original recording from the duplicated release and for "Add a new recording". if i click the original recording's radio button, the track association is updated and the recording that i selected previously gets a radio button too.
      • 2023-08-16 22837, 2023

      • derat
        that is, i think i'm seeing exactly the behavior that you described on musicbrainz.org too. it's just a bit harder to trigger since you need to repeatedly click on radio buttons to focus them since the search field keeps stealing the focus
      • 2023-08-16 22815, 2023

      • derat
        every time i click on a recording and then on "Add a new recording", the radio buttons get reshuffled. it looks like it's moving the last-selected recording up to the top of the list
      • 2023-08-16 22835, 2023

      • yvanzo
        derat: Correct, it’s just harder to trigger at the moment than with your patches, but definitely not cause by those. Will fill a separate issue about it.
      • 2023-08-16 22840, 2023

      • yvanzo
        bitmap: Please double-review derat’s PRs #3011, #3010, #3008 & #3006 when you have some time.
      • 2023-08-16 22832, 2023

      • derat
        yvanzo: thanks for all the comments! applying them now...
      • 2023-08-16 22855, 2023

      • bitmap
        i will try to check them today :)
      • 2023-08-16 22845, 2023

      • aerozol
        jasje: what new icon library are you using? Didn’t we already agree to using the same one as BookBrainz where possible, so we can start getting consistency across our platforms
      • 2023-08-16 22809, 2023

      • aerozol
        The mention of animations causes me more concern than excitement ;D
      • 2023-08-16 22856, 2023

      • aerozol
        Ah I see you and monkey have discussed it more (it looks), just catching up on chat now
      • 2023-08-16 22820, 2023

      • derat has quit
      • 2023-08-16 22826, 2023

      • aerozol
        outsidecontext: tweeted and reddit-ed re. 2.9.1, great job on the quick update! That shift+drag issue is a wild one!
      • 2023-08-16 22801, 2023

      • tux0r
        now mastodon it!
      • 2023-08-16 22849, 2023

      • aerozol
        tux0r: I was actually going to add some re. that to the next meeting :) But because of timezones and reo being away it would probably have to be the next one
      • 2023-08-16 22802, 2023

      • aerozol
        (I have been reposting stuff to my personal mastodon account to test the waters)
      • 2023-08-16 22812, 2023

      • tux0r
        :-)
      • 2023-08-16 22832, 2023

      • tux0r
        now that tweetdeck is dead as well, mastodon seems to be the last remaining alternative
      • 2023-08-16 22823, 2023

      • tux0r
        (yes, kbin/firefish/misskey/.. exist and bluesky exists blah blah.. honestly, our bubble is on mastodon :p)
      • 2023-08-16 22842, 2023

      • aerozol
        I think there’s still a bunch of services vying to be the replacement, but basically I’ve decided that it’s not too much work to post to a bunch. But it does create more profiles for MeB to upkeep longterm so it’s not a no-brainer
      • 2023-08-16 22801, 2023

      • tux0r
        most of them use activitypub anyway
      • 2023-08-16 22806, 2023

      • tux0r
        except bluesky
      • 2023-08-16 22827, 2023

      • aerozol
        Yeah, mastodon seems like “our” audience. But let’s not forget that we have a huge silent audience as well (the non-tech one). But I think we can hit both just fine
      • 2023-08-16 22817, 2023

      • tux0r
        does the non-tech audience read changelogs?
      • 2023-08-16 22830, 2023

      • tux0r
        or even care about updates?
      • 2023-08-16 22858, 2023

      • aerozol
        No, and yes :)