#metabrainz

Hey all, I'm not sure if it's because my internet connection is slow being a tethered connection atm, but I was curious if there was a direct link to Last Week's Exploration for me in ListenBrainz

Both this week and the last week bring up the exact same playlist, and the modified dates are both showing as the 8th, being only two minutes apart

mayhem: monkey: lucifer: I just found the ‘watch only releases’ subscription in github - I will start posting updates for LB releases, no need to ping! Sorry for being slow to get there :P

Binzy: Wow, your created for you page is interesting indeed. I wonder if it’s because you just started submitting listens recently? https://listenbrainz.org/user/Binzy_Boi/recomme...

[metabrainz.org] 14dependabot[bot] opened pull request #462 (03master…dependabot/pip/idna-3.7): Bump idna from 3.6 to 3.7 https://github.com/metabrainz/metabrainz.org/pu...

[musicbrainz-server] 14yvanzo merged pull request #3211 (03master…docker-imp): Improvements to the Docker images https://github.com/metabrainz/musicbrainz-serve...

hi yvanzo, things are going good with sql-gen, the cube extension support may get added to sqlx, if not, jayy told he would release a crate for this. Btw I realised I haven't contributed to MBS, so I decided to work on MBS-13533.

MBS-13533: Release group pages should prioritize non-bootleg album art https://tickets.metabrainz.org/browse/MBS-13533

hi yellowhatpro, you’re very welcome to, I assume that it’s mainly about modifying an SQL query.

yeah right!

That's good, now I need to get the cover art tables.

The development setup doesn’t have those by default, so you have to dig a bit into how to download this additional dump and load it into the database.

(those = CAA tables)

I wanted to know, if there is any way to run multiple instances of musicbrainz_db,

I would need one for running mb-server, which will contain the regular data (sample db), and the other for edit_* tables

It is likely possible now using https://github.com/metabrainz/musicbrainz-docke...

But the simplest way is probably to have two separate working copies of `musicbrainz-docker` under different paths.

For example `mb-sample/` and `mb-edit/` (so that the Compose project have different names).

project*s

oh nice, second approach seems easier.

on it then!

!m monkey, ansh

You're doing good work, monkey, ansh!

brainzplayer is finally behaving for me. <3

[troi-recommendation-playground] 14mayhem merged pull request #134 (03main…fix-artist-object): Use ArtistCredit, cleanup old cruft and use MB recording cache https://github.com/metabrainz/troi-recommendati...

[metabrainz.org] 14mayhem merged pull request #462 (03master…dependabot/pip/idna-3.7): Bump idna from 3.6 to 3.7 https://github.com/metabrainz/metabrainz.org/pu...

[metabrainz.org] release 03v-2024-04-12.0 has been published by 14github-actions[bot]: https://github.com/metabrainz/metabrainz.org/re...

mayhem (IRC): Hello! Can I deploy a PR to test.LB?

Yes, feel free. I am no longer using test.

Kewl

zas: can you deploy a new VM on the same network as the solr 9 cluster please?

a CX51 is probably slightly over kill but we'll only need it for a little whiel

atj: that's for performance tests?

yep

i'm going to try running that java based solr benchmarking tool which will probably use all 32GB RAM :P

I'll not create dns records for it

since it's temporary

yep, no need

reosarevok / yvanzo: does MBS make Solr queries with fmt=mbjson or fmt=mbxml?

Looking at the code, seems both?

my $format = ($args->{fmt} // '') eq 'json' ? 'mbjson' : 'mbxml';

what about for internal lookups e.g. in the release editor?

ok, looks like it defaults to mbxml

only mbxml and mbjson are used in MB

any of the solr offered defaults cannot be used because the data format of the documents stored in solr is unwieldly

ansh (IRC): Hi! When you have a moment, could you please do a final review of LB#2509 ? Should be ready to merge. Deployed and tested on test.LB

Integrate Apple Music playback into ListenBrainz : https://github.com/metabrainz/listenbrainz-serv...

Sure, rn I'm reviewing LB#2832, after which I'll review this one

Update global context when disconnecting music service: https://github.com/metabrainz/listenbrainz-serv...

Can you please share the `APPLE_MUSIC_KEY` config on DM

[listenbrainz-server] 14mayhem opened pull request #2833 (03master…improve-lb-radio-artist-validation): Add more validation for lb-radio artist endpoint https://github.com/metabrainz/listenbrainz-serv...

monkey[m]: I do have a use case for test -- is it free now?

Unless ansh (IRC) is testing on the test server, it is free for me

[listenbrainz-server] 14MonkeyDo merged pull request #2832 (03master…music-services-update-auth): Update global context when disconnecting music service https://github.com/metabrainz/listenbrainz-serv...

mayhem: I'm not using it right now

thx!

hye monkey[m] , I've got a couple of minor PRs that would be good to get out. will you review two minor PRs and I will do release?

Shoot

https://github.com/metabrainz/listenbrainz-serv... is one. the other coming soon

Well that's a simple one

yerp

next one wont be much harder.

though it touches SQL!

[listenbrainz-server] 14mayhem merged pull request #2833 (03master…improve-lb-radio-artist-validation): Add more validation for lb-radio artist endpoint https://github.com/metabrainz/listenbrainz-serv...

bitmap: did you see MBS-13538 ?

MBS-13538: Removing all rows in the artist credit editor makes it disappear https://tickets.metabrainz.org/browse/MBS-13538

yes, didn't get a chance to look into it yet though

Ok, nice

and I think I discovered another issue I need to open a ticket for (but it's not user visible)

[listenbrainz-server] 14mayhem opened pull request #2834 (03master…add-artist-to-lb-radio): Add similar_artist_name to LB Radio artist element https://github.com/metabrainz/listenbrainz-serv...

monkey[m]: ^^ the PR

Done

thx. I ran it on test. all good.

though a test fails. let me investigate.

There is a consistently failing server test for about a week or so

delete listens test

yeah, I've gotten adept at ignorning it, lol

ansh (IRC): Is LB#2769 ready for review and eventually merge into the feature branch?

Migrate Entity Pages to SPA: https://github.com/metabrainz/listenbrainz-serv...

Yes

I've been testing the branch over the past week and a half, I think it's ready for merging

And in any case, we'll have more testing before we merge the feature branch

[listenbrainz-server] 14MonkeyDo merged pull request #2769 (03single-page-app…spa-entity-pages): Migrate Entity Pages to SPA https://github.com/metabrainz/listenbrainz-serv...

One more down :)

no I fear doing a release this late on friday.

lol

*now

That last one is merged into a separate feature branch, no worries :)

ok, back to only one deletedlistens test, so good to go.

[listenbrainz-server] 14mayhem merged pull request #2834 (03master…add-artist-to-lb-radio): Add similar_artist_name to LB Radio artist element https://github.com/metabrainz/listenbrainz-serv...

[listenbrainz-server] release 03v-2024-04-11.1 has been published by 14github-actions[bot]: https://github.com/metabrainz/listenbrainz-serv...

second release oft he day

just in time for simon to get some email spam lol

release!

hey monkey, can i get your opinion on a mockup for LB-1558?

LB-1558: user similarity stats on user pages https://tickets.metabrainz.org/browse/LB-1558