#metabrainz

What does this error from the API mean? https://github.com/metabrainz/musicbrainz-serve...

trying to submit a rating with:

const root = 'https://musicbrainz.org';

const path = '/ws/2/rating/?client=chartbrainz-1.0.0';

const body = `

<?xml version="1.0" encoding="UTF-8"?>

<metadata xmlns="http://musicbrainz.org/ns/mmd-2.0#&#...;

<artist-list>

<artist id="a4cda071-2ba4-4fa9-a50d-4378c408676d">

<user-rating>100</user-rating>

</artist>

</artist-list>

</metadata>`

ah nevermind - does anyone know of any working code (in any language) that can submit a rating for an entity (specifically a release group)?

monkey: didn’t have time to do MeB stuff yet this week, but how’s it going? What do you think about the rating mockups? I hope I didn’t mess with your plans

*tags, not ratings

[musicbrainz-server] 14reosarevok opened pull request #2734 (03master…MBS-12708): MBS-12708: Report for lonely pseudo-releases https://github.com/metabrainz/musicbrainz-serve...

chancey: https://github.com/metabrainz/picard/blob/maste...

[sir] 14amCap1712 opened pull request #156 (03master…work-fix): Improve indexing works https://github.com/metabrainz/sir/pull/156

aerozol: Hi! Blehh sort of week for me, fighting with the test infrastructure, and it's taking all my time and will to live :)

So similarly, had no time to think about design. I dig the tags though! I like the simplicity of it, they look quite clean (and I was getting nowhere trying to find a color that would work so white/transparent solves that)

I'm on the fence about showing the +/- buttons to vote. I think if we don't show them it won't be clear at all that a user can vote, and also I think we'll want to keep single click interaction to navigate to a genre radio (in the future)

Not sure about showing the number of votes; not sure it's needed. Currently they are sorted by number of votes

O’Moin

moooin!

lucifer: not sure what exactly means "rank much higher in runtime"? good idea to try updating the default config! I can setup a MB Docker instance dedicated to testing SIR on wolf if that helps with?

yvanzo: i am modifying the existing one on wolf and it seems to be working fine. yes, i'll try playing around with the default config and see what comes up. i'll redo the profiling and share the results to look at.

(the current MB Docker instance is used to test other things IIUC, and has much more memory than what is required for a simple mirror)

i see, makes sense.

yvanzo: another thing, do we have a wscompat server running somewhere? otherwise can you point me to the repo having that code

the current behaviour of sir wrt artist aliases for wscompat seems to be weird. iiuc, its indexing those for release group and recordings but not for releases.

lucifer: It is used by MB mirrors that don’t run a local Solr instance: https://musicbrainz.org/doc/Development/Search_...

lucifer: here is the code:

yvanzo: i see. makes sense. if it wanted to query search.musicbrainz.org dirctly, what would be the right url? reading the code it seems to be `http://search.musicbrainz.org/ws/2/release?query=release:mismatched&type=release&fmt=xml&limit=1` but it returns a 404.

[listenbrainz-server] 14amCap1712 merged pull request #2246 (03master…fix-rg-ca): Fix release group cover art query https://github.com/metabrainz/listenbrainz-serv...

lucifer: is LB#2243 ready for review?

LB-1128: Phase out usage of mbid mapping metadata: https://github.com/metabrainz/listenbrainz-serv...

yes

k

[listenbrainz-server] 14amCap1712 merged pull request #2242 (03master…delete-old-playlists): Update existing daily jams playlist on spotify if available https://github.com/metabrainz/listenbrainz-serv...

[listenbrainz-server] 14MonkeyDo opened pull request #2247 (03master…fetch-cover-art-CAA-release): Listening Now: Use caa_release_mbid field https://github.com/metabrainz/listenbrainz-serv...

lucifer: Forgot to do that after you added caa_release_mbid to the metadata response ^

My bad ^^

monkey: no worries, i had actually made the changes already but didn't open a PR yet because those tables changes are currently not in prod

Are they not? I realized that it was broken because I was using the Listening Now page and seeing broken images

Which AFAIK uses the prod API

those fields are present in the prod database and are being served in test-api currently.

where did you face the issue? test or prod?

prod

ah right, metadata viewer automatically serves all fields from db so even the prod version has it.

for listens, fields have to added manually.

Got it.

Looks like this wouldn't work (using releaseMBID instead of caaReleaseMBID) but that's moot now I guess: https://github.com/metabrainz/listenbrainz-serv...

sorry not sure what you mean?

That string `https://archive.org/download/mbid-${releaseMBID}/mbid-${releaseMBID}-${caaId}_thumb500.jpg` was using the wrong MBID in that code, I think. Will add the proper URL construction to my PR

oh! makes sense

ah yes that should have been caaReleaseMbid

Ya. Just making sure I'm not missing anything :)

lucifer: It seems to be broken atm indeed, I cannot make it work locally either.

oh! i see

mayhem: the incremental updates when using pink are quite fast it seems. take < 1 min

very good -- glad that is bearing out

there is an issue that the full rebuild query can be killed by pg replication lags. but thats only done once a month so hopefully it'll work out fine.

otherwise we can add an hourly retry to it i guess.

lucifer: are you looking into openresty logs on kiki already?

yvanzo: i looked there sometime ago but didn't find anything usefull

zas, atj: any clue why openresty is currently returning 404 for http://search.musicbrainz.org/ws/2/release?quer... ?

i am not sure if i am reading those incorrectly or if those don't record the urls of each request.

[listenbrainz-server] 14mayhem opened pull request #2248 (03master…fix-art-again): Make test for production a bit more MeB specific and not use FLASK_ENV. https://github.com/metabrainz/listenbrainz-serv...

reosarevok: there is a screen session opened for 6 months on kiki, not sure it is still needed.

I'd assume not

[musicbrainz-server] 14reosarevok opened pull request #2735 (03master…no-extends-in-begin): Move extends out of BEGIN blocks https://github.com/metabrainz/musicbrainz-serve...

don't turn off all the microservices yvanzo!

atj but only <20% are needed, rest are bloatware!

lucifer: who needs 2fa anyway?

atj: it's a screen session under reosarevok account and it is still on atm.

indeed!

mayhem, with the way things are going in twitter-verse. thoughts on creating backup accounts somewhere else just in case?

Like mastodon?

yeah i guess or some other close alternative if there is some.

good idea!

or we could create an account in the Metaverse !!

atj: btw did you see the 1000 RPCs therad?

I've seen it all, people are helpfully compiling the slow motion car crash into threads

🤣

I don't think mayhem is a fan of mastodon from what I read :)

yvanzo: I can't see anything obvious in the openresty configs for search.

yvanzo: afaict it was doing nothing

I closed it now

https://musicbrainz.org/ws/2/release?query=rele... "release is not a recognized release-group type." ?

That is for type album or whatever

To filter results by RG types

reosarevok: great

atj: even if you drop the type query parameter: https://search.musicbrainz.org/ws/2/release?que...

atj: that query likely won't work on mb ws because it performs some modifications on the params before calling search.mb.lrh

i had modified the params manually to try to recreate what an internal call from mb ws to search.mb would look like acc to my understanding

atj: I read and article a couple of years back that talked about mastondon being rife with abusive behaviour. I can't find that article... its just buried in more current things. few of which are really taking a close look at mastondon. but there is this: https://lucumr.pocoo.org/2022/11/14/scaling-mas...

yvanzo: see https://github.com/metabrainz/openresty-gateway...

[listenbrainz-server] 14mayhem merged pull request #2248 (03master…fix-art-again): Make test for production a bit more MeB specific and not use FLASK_ENV. https://github.com/metabrainz/listenbrainz-serv...

but I don't know why it ends in 404 yet