#metabrainz

/

      • smach joined the channel
      • 2022-12-07 34117, 2022

      • smach has quit
      • 2022-12-07 34116, 2022

      • smach joined the channel
      • 2022-12-07 34102, 2022

      • jivte has quit
      • 2022-12-07 34136, 2022

      • GibusWearingMann joined the channel
      • 2022-12-07 34102, 2022

      • jivte joined the channel
      • 2022-12-07 34159, 2022

      • BrainzGit
        [listenbrainz-server] 14jivteshsingh opened pull request #2278 (03master…cover-art-bug): Cover art bug https://github.com/metabrainz/listenbrainz-server…
      • 2022-12-07 34151, 2022

      • jivte
        lucifer: Hey I fixed the bug and created a pr can u please review it and tell further changes to implement
      • 2022-12-07 34136, 2022

      • GibusWearingMann has quit
      • 2022-12-07 34136, 2022

      • jivte has quit
      • 2022-12-07 34128, 2022

      • lucifer
        aerozol: it didn't get mapped and the links are there because the user submitted mbids.
      • 2022-12-07 34130, 2022

      • aerozol
        Okay that sounds like it makes sense. I don’t quite know how but you know what you’re doing :P
      • 2022-12-07 34158, 2022

      • lucifer
        aerozol: can you try checking the page again now? :)
      • 2022-12-07 34104, 2022

      • smach has quit
      • 2022-12-07 34117, 2022

      • aerozol
        ooh, nice one! Can we manually match these from the backend?
      • 2022-12-07 34124, 2022

      • lucifer
        kind of. the issue was that the listen was matched by the mapper but the recording had been merged in MB. currently the mapper cannot resolve redirects. so i invalidated the match and the mapper matched it again this time to correct recording.
      • 2022-12-07 34109, 2022

      • lucifer
        i'll add an automated check to detect such cases so that mapper can redo matches as needed.
      • 2022-12-07 34144, 2022

      • aerozol
        Nice one! Thanks!
      • 2022-12-07 34147, 2022

      • Aswin joined the channel
      • 2022-12-07 34158, 2022

      • jivte joined the channel
      • 2022-12-07 34153, 2022

      • Aswin has quit
      • 2022-12-07 34157, 2022

      • jasje joined the channel
      • 2022-12-07 34107, 2022

      • Aswin joined the channel
      • 2022-12-07 34140, 2022

      • Aswin has quit
      • 2022-12-07 34123, 2022

      • jivte
        lucifer: Hey can u tell how to find caa_id and caa_release_mbid og a recording
      • 2022-12-07 34144, 2022

      • wargreen has quit
      • 2022-12-07 34145, 2022

      • lucifer
        jivte: you can pass the recording mbid to this endpoint, https://api.listenbrainz.org/1/metadata/recording…
      • 2022-12-07 34128, 2022

      • jivte
        okk thanks
      • 2022-12-07 34108, 2022

      • CatQuest
        fish
      • 2022-12-07 34138, 2022

      • wargreen joined the channel
      • 2022-12-07 34116, 2022

      • jivte has quit
      • 2022-12-07 34140, 2022

      • jivte joined the channel
      • 2022-12-07 34133, 2022

      • BrainzGit
        [musicbrainz-server] 14reosarevok opened pull request #2773 (03master…MBS-12779): MBS-12779: Treat "n-bit" and "loop" as ETI if in parens https://github.com/metabrainz/musicbrainz-server/…
      • 2022-12-07 34144, 2022

      • v6lur joined the channel
      • 2022-12-07 34132, 2022

      • jivte has quit
      • 2022-12-07 34123, 2022

      • reosarevok
        How's SpamBrainz coming along? Because the newly registered editors report is... painful to look at :D
      • 2022-12-07 34116, 2022

      • v6lur has quit
      • 2022-12-07 34144, 2022

      • jasje has quit
      • 2022-12-07 34132, 2022

      • monkey
        Hi everyone, I'm back in EU timezone and out of vacation !
      • 2022-12-07 34132, 2022

      • jasje joined the channel
      • 2022-12-07 34136, 2022

      • jivte joined the channel
      • 2022-12-07 34114, 2022

      • jivte
        monkey: lucifer: Hey I implemented the final changes as you asked in the comment can u please review it and suggest further changes :)
      • 2022-12-07 34146, 2022

      • monkey
        Don't worry jivte, we get updates and we will get to in in time :)
      • 2022-12-07 34138, 2022

      • jivte
        okk sure and thanks for reviewing it and rectifying my mistakes :)
      • 2022-12-07 34152, 2022

      • lucifer
      • 2022-12-07 34140, 2022

      • lucifer
        alastairp, monkey, mayhem: apple music data export ^ has timestamp to ms precision and all other possibly needed data but their api omits the timestamp entirely :(
      • 2022-12-07 34127, 2022

      • monkey
        Well that's annoying!
      • 2022-12-07 34144, 2022

      • lucifer
        i guess the best we can do for apple music integration is allowing people to import this archive + BP support for musickit.
      • 2022-12-07 34147, 2022

      • monkey
        What are the chances of a feature getting anywhere, do you think?
      • 2022-12-07 34103, 2022

      • lucifer
        sorry not sure what you mean?
      • 2022-12-07 34125, 2022

      • monkey
        I meant we could try to request that they add the timestamp to the API response, but I'm guessing the emails will go straight to the shredder
      • 2022-12-07 34122, 2022

      • lucifer
        yeah true. i have zero hopes of apple adding this anytime unless they need it for some own product.
      • 2022-12-07 34157, 2022

      • alastairp
        hi lucifer!
      • 2022-12-07 34102, 2022

      • lucifer
        hi!
      • 2022-12-07 34105, 2022

      • alastairp
        that's... sigh
      • 2022-12-07 34110, 2022

      • alastairp
        hi monkey!
      • 2022-12-07 34122, 2022

      • monkey
        👋
      • 2022-12-07 34140, 2022

      • alastairp
        hi reosarevok, interesting question. what do you remember as the current status of spambrainz?
      • 2022-12-07 34100, 2022

      • reosarevok
        Not much, tbh :D
      • 2022-12-07 34125, 2022

      • alastairp
        (I ask, because I did some really basic image classification last week, tensorflow + the image classification tutorial, and good results are really easy to get)
      • 2022-12-07 34107, 2022

      • alastairp
        I recall large plans for some generic system that could be used for lots of different things
      • 2022-12-07 34112, 2022

      • reosarevok
        Honestly, even a non-AI check for "does this have the same IP as a user marked spammer, or does it contain a bunch of dodgy words" could even stop half these users
      • 2022-12-07 34124, 2022

      • alastairp
        yeah, right
      • 2022-12-07 34156, 2022

      • reosarevok
        I mean, we'd need to be able to let people through if someone is a legit user who *also* happens to mention being a dentist on their bio or something :p
      • 2022-12-07 34159, 2022

      • reosarevok
        But still
      • 2022-12-07 34132, 2022

      • alastairp
        yeah, right. I'm looking at https://github.com/metabrainz/spambrainz_ml/blob/…, and while this is a very impressive looking model, keyword detection might be faster and easier :)
      • 2022-12-07 34141, 2022

      • reosarevok
        I mean, it might be this would also do the trick, but dunno how far we are from it actually being usable and used
      • 2022-12-07 34140, 2022

      • BrainzGit
        [musicbrainz-server] 14reosarevok opened pull request #2774 (03master…MBS-12771): MBS-12771: Convert Change release quality page to React https://github.com/metabrainz/musicbrainz-server/…
      • 2022-12-07 34116, 2022

      • jivte has quit
      • 2022-12-07 34108, 2022

      • reosarevok
        Re that spam-ninja thing, I really should try to get https://github.com/metabrainz/musicbrainz-server/… merged - it's a year old now :/
      • 2022-12-07 34113, 2022

      • reosarevok fixes the new conflicts
      • 2022-12-07 34152, 2022

      • reosarevok
        Quite a few. Oh well
      • 2022-12-07 34131, 2022

      • Rishabh joined the channel
      • 2022-12-07 34136, 2022

      • Rishabh has quit
      • 2022-12-07 34148, 2022

      • lucifer
        alastairp, monkey: when you have time, can you please try to build https://github.com/metabrainz/listenbrainz-now-pl… on mac?
      • 2022-12-07 34101, 2022

      • alastairp
        lucifer: sure thing
      • 2022-12-07 34104, 2022

      • lucifer
        i tested on windows and linux and it works there.
      • 2022-12-07 34104, 2022

      • monkey
        Can do
      • 2022-12-07 34133, 2022

      • lucifer
        mac os config will likely need to be tuned but i can try to help do that. current one fails in ci
      • 2022-12-07 34144, 2022

      • lucifer
        thanks
      • 2022-12-07 34109, 2022

      • monkey
        install + npm start did work
      • 2022-12-07 34137, 2022

      • lucifer
        yeah that should work. try `npm run dist`
      • 2022-12-07 34143, 2022

      • monkey
        Running `npm run-script dist` now
      • 2022-12-07 34109, 2022

      • alastairp
        I got an error but it seems to be due to a download failure when getting the electron bas image
      • 2022-12-07 34116, 2022

      • jasje has quit
      • 2022-12-07 34116, 2022

      • lucifer
        alastairp: also, this repo has different perms than other meb ones? please add this to core-team as well.
      • 2022-12-07 34140, 2022

      • alastairp
        it skipped notarisation, and successfully built a dmg
      • 2022-12-07 34155, 2022

      • lucifer
        ah nice
      • 2022-12-07 34110, 2022

      • lucifer
        the script skips notarization unless running in CI.
      • 2022-12-07 34116, 2022

      • alastairp
        although unsure if the entitlements are properly working
      • 2022-12-07 34130, 2022

      • alastairp
        version number is 0.0.0 too
      • 2022-12-07 34146, 2022

      • lucifer
        yes version is configured to that intentionally.
      • 2022-12-07 34152, 2022

      • alastairp
        icon in the dmg is default electron icon
      • 2022-12-07 34126, 2022

      • lucifer
        i had copied the entitlements from https://github.com/electron-react-boilerplate/ele…
      • 2022-12-07 34137, 2022

      • lucifer
        and rest of the mac setup as well from there
      • 2022-12-07 34159, 2022

      • alastairp
        links on https://listenbrainz.org/listening-now/ shouldn't open in the electron, they should open in system browser
      • 2022-12-07 34106, 2022

      • alastairp
        (though I guess that isn't a mac specific thing)
      • 2022-12-07 34153, 2022

      • lucifer
        ah no, that's something to be fixed yet.
      • 2022-12-07 34123, 2022

      • lucifer
        electron recently changed some behaviour around it: https://github.com/metabrainz/listenbrainz-now-pl…
      • 2022-12-07 34132, 2022

      • lucifer
        i haven't looked into fixing it yet
      • 2022-12-07 34101, 2022

      • monkey
        Looks like building the app worked for me lucifer
      • 2022-12-07 34116, 2022

      • lucifer
        i see, so something wrong in ci setup.
      • 2022-12-07 34128, 2022

      • monkey
        And app works
      • 2022-12-07 34148, 2022

      • alastairp
        lucifer: right
      • 2022-12-07 34110, 2022

      • smach joined the channel
      • 2022-12-07 34136, 2022

      • alastairp
        lucifer: how are you finding couchdb?
      • 2022-12-07 34102, 2022

      • alastairp
        I have a project where I need to just dump a bunch of json for dealing with later
      • 2022-12-07 34148, 2022

      • lucifer
        alastairp: haven't received any reports of stats breaking down so i'd say working fine. if its not much json data i'd probably go with pg still.
      • 2022-12-07 34110, 2022

      • alastairp
        lucifer: it's quite a bit, n00,000+ docs
      • 2022-12-07 34131, 2022

      • lucifer
        the main motivation for couchdb in our case was writing ~10-20 GB data daily with few reads.
      • 2022-12-07 34104, 2022

      • alastairp
        right
      • 2022-12-07 34136, 2022

      • alastairp
        if I put it in postgres then at the very least I think I'll put it in a different separate db
      • 2022-12-07 34112, 2022

      • lucifer
        i see. makes sense.
      • 2022-12-07 34100, 2022

      • lucifer
        for anything long term, we are still using postgres. couchdb should be reliable but i haven't looked into that aspect much.
      • 2022-12-07 34159, 2022

      • alastairp
        yeah, I think this usecase is going to be quite similar to the LB case
      • 2022-12-07 34122, 2022

      • alastairp
        I geuss there's a lot of writes, not much reads, but we're keeping historical data each day rather than dropping it
      • 2022-12-07 34131, 2022

      • alastairp
        anyway, I'll have a look. thanks
      • 2022-12-07 34122, 2022

      • reosarevok
        yvanzo, bitmap: do you feel pages like https://musicbrainz.org/doc/Edit_Types/81 are at all useful anymore? Every edit is accepted on expiration now on the same amount of days, so the only kinda useful info is what edit types can be approved...
      • 2022-12-07 34102, 2022

      • reosarevok
        Was going to convert it and it got me wondering
      • 2022-12-07 34100, 2022

      • BrainzGit
        [listenbrainz-server] 14amCap1712 opened pull request #2279 (03master…mapping-redirect-invalidate): Invalidate mbid_mapper matches for redirects and deleted recordings https://github.com/metabrainz/listenbrainz-server…
      • 2022-12-07 34156, 2022

      • zas
        I found an issue with Bandcamp importer I'm not sure how to fix
      • 2022-12-07 34121, 2022

      • zas
        with the Bandcamp Importer enabled go to https://silvertalon.bandcamp.com/album/decadence-…
      • 2022-12-07 34157, 2022

      • zas
        this release already exists in MB, with this URL
      • 2022-12-07 34100, 2022

      • zas
      • 2022-12-07 34112, 2022

      • zas
        but usually we have icons showing up to tell that
      • 2022-12-07 34116, 2022

      • zas
        here they don't
      • 2022-12-07 34134, 2022

      • reosarevok
        Because the URL is not the same - it's a redirect
      • 2022-12-07 34141, 2022

      • reosarevok
      • 2022-12-07 34158, 2022

      • zas
        yes
      • 2022-12-07 34103, 2022

      • zas
      • 2022-12-07 34105, 2022

      • reosarevok
        Which is a know thing about the script - it breaks if the URL is not the same, because it has no way to know
      • 2022-12-07 34120, 2022

      • zas
        nope, it's actually the reverse
      • 2022-12-07 34143, 2022

      • zas
      • 2022-12-07 34154, 2022

      • reosarevok
        Huh
      • 2022-12-07 34156, 2022

      • zas
        but this URL comes from DOM TrAlbumData
      • 2022-12-07 34158, 2022

      • reosarevok
      • 2022-12-07 34107, 2022

      • zas
        ah... hmm
      • 2022-12-07 34130, 2022

      • reosarevok
        And clicking https://silvertalon.bandcamp.com/album/decadence-… actually does redirect me to the other
      • 2022-12-07 34116, 2022

      • reosarevok
        In any case, it's always good to search because I've seen issues where the release is in MB but was imported from the label's bandcamp and I'm at the artist's or vice versa
      • 2022-12-07 34126, 2022

      • reosarevok
        I guess there's only that much it can do :)
      • 2022-12-07 34103, 2022

      • zas
        the .us URL returns a 303
      • 2022-12-07 34122, 2022

      • zas
      • 2022-12-07 34135, 2022

      • zas
        but for sure the script uses URL as defined in DOM in TrAlbumData (and that's the .us)
      • 2022-12-07 34121, 2022

      • zas
        reosarevok: can you check which response you get when going to https://silvertalon.bandcamp.com/album/decadence-… ?
      • 2022-12-07 34119, 2022

      • reosarevok
        Status Code: 301 Moved Permanently
      • 2022-12-07 34126, 2022

      • reosarevok
        And sends me to .us
      • 2022-12-07 34108, 2022

      • zas
        that's very weird, I get a 200
      • 2022-12-07 34134, 2022

      • reosarevok
        I get a 200 with the .us :D
      • 2022-12-07 34137, 2022

      • zas
      • 2022-12-07 34138, 2022

      • lucifer
        200 on first, 301 on second.