#metabrainz

[listenbrainz-server] 14jivteshsingh opened pull request #2278 (03master…cover-art-bug): Cover art bug https://github.com/metabrainz/listenbrainz-serv...

lucifer: Hey I fixed the bug and created a pr can u please review it and tell further changes to implement

aerozol: it didn't get mapped and the links are there because the user submitted mbids.

Okay that sounds like it makes sense. I don’t quite know how but you know what you’re doing :P

aerozol: can you try checking the page again now? :)

ooh, nice one! Can we manually match these from the backend?

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.

i'll add an automated check to detect such cases so that mapper can redo matches as needed.

Nice one! Thanks!

lucifer: Hey can u tell how to find caa_id and caa_release_mbid og a recording

jivte: you can pass the recording mbid to this endpoint, https://api.listenbrainz.org/1/metadata/recordi...

okk thanks

fish

[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-serve...

How's SpamBrainz coming along? Because the newly registered editors report is... painful to look at :D

Hi everyone, I'm back in EU timezone and out of vacation !

monkey: lucifer: Hey I implemented the final changes as you asked in the comment can u please review it and suggest further changes :)

Don't worry jivte, we get updates and we will get to in in time :)

okk sure and thanks for reviewing it and rectifying my mistakes :)

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 :(

Well that's annoying!

i guess the best we can do for apple music integration is allowing people to import this archive + BP support for musickit.

What are the chances of a feature getting anywhere, do you think?

sorry not sure what you mean?

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

yeah true. i have zero hopes of apple adding this anytime unless they need it for some own product.

hi lucifer!

hi!

that's... sigh

hi monkey!

👋

hi reosarevok, interesting question. what do you remember as the current status of spambrainz?

Not much, tbh :D

(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)

I recall large plans for some generic system that could be used for lots of different things

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

yeah, right

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

But still

yeah, right. I'm looking at https://github.com/metabrainz/spambrainz_ml/blo..., and while this is a very impressive looking model, keyword detection might be faster and easier :)

I mean, it might be this would also do the trick, but dunno how far we are from it actually being usable and used

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

Re that spam-ninja thing, I really should try to get https://github.com/metabrainz/musicbrainz-serve... merged - it's a year old now :/

Quite a few. Oh well

alastairp, monkey: when you have time, can you please try to build https://github.com/metabrainz/listenbrainz-now-... on mac?

lucifer: sure thing

i tested on windows and linux and it works there.

Can do

mac os config will likely need to be tuned but i can try to help do that. current one fails in ci

thanks

install + npm start did work

yeah that should work. try `npm run dist`

Running `npm run-script dist` now

I got an error but it seems to be due to a download failure when getting the electron bas image

alastairp: also, this repo has different perms than other meb ones? please add this to core-team as well.

it skipped notarisation, and successfully built a dmg

ah nice

the script skips notarization unless running in CI.

although unsure if the entitlements are properly working

version number is 0.0.0 too

yes version is configured to that intentionally.

icon in the dmg is default electron icon

i had copied the entitlements from https://github.com/electron-react-boilerplate/e...

and rest of the mac setup as well from there

links on https://listenbrainz.org/listening-now/ shouldn't open in the electron, they should open in system browser

(though I guess that isn't a mac specific thing)

ah no, that's something to be fixed yet.

electron recently changed some behaviour around it: https://github.com/metabrainz/listenbrainz-now-...

i haven't looked into fixing it yet

Looks like building the app worked for me lucifer

i see, so something wrong in ci setup.

And app works

lucifer: right

lucifer: how are you finding couchdb?

I have a project where I need to just dump a bunch of json for dealing with later

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.

lucifer: it's quite a bit, n00,000+ docs

the main motivation for couchdb in our case was writing ~10-20 GB data daily with few reads.

right

if I put it in postgres then at the very least I think I'll put it in a different separate db

i see. makes sense.

for anything long term, we are still using postgres. couchdb should be reliable but i haven't looked into that aspect much.

yeah, I think this usecase is going to be quite similar to the LB case

I geuss there's a lot of writes, not much reads, but we're keeping historical data each day rather than dropping it

anyway, I'll have a look. thanks

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...

Was going to convert it and it got me wondering

[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-serv...

I found an issue with Bandcamp importer I'm not sure how to fix

with the Bandcamp Importer enabled go to https://silvertalon.bandcamp.com/album/decadenc...

this release already exists in MB, with this URL

but usually we have icons showing up to tell that

here they don't

Because the URL is not the same - it's a redirect

So it's now at https://music.silvertalon.us/album/decadence-an...

yes

Which is a know thing about the script - it breaks if the URL is not the same, because it has no way to know

nope, it's actually the reverse

https://music.silvertalon.us/album/decadence-an... redirects to https://silvertalon.bandcamp.com/album/decadenc...

Huh

but this URL comes from DOM TrAlbumData

Not to me, it just stays in https://music.silvertalon.us/album/decadence-an...

ah... hmm

And clicking https://silvertalon.bandcamp.com/album/decadenc... actually does redirect me to the other

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

I guess there's only that much it can do :)

the .us URL returns a 303

but for sure the script uses URL as defined in DOM in TrAlbumData (and that's the .us)

reosarevok: can you check which response you get when going to https://silvertalon.bandcamp.com/album/decadenc...

Status Code: 301 Moved Permanently

And sends me to .us

that's very weird, I get a 200

I get a 200 with the .us :D

can someone else check? https://music.silvertalon.us/album/decadence-an... and https://silvertalon.bandcamp.com/album/decadenc...

200 on first, 301 on second.