#metabrainz

[listenbrainz-android] 14dependabot[bot] opened pull request #210 (03main…dependabot/gradle/io.sentry.android.gradle-3.12.0): Bump io.sentry.android.gradle from 3.11.1 to 3.12.0 https://github.com/metabrainz/listenbrainz-andr...

monkey: mayhem: lucifer: Isn’t this cool - https://mastobrainz.retiolus.net/

[listenbrainz-android] 14akshaaatt merged pull request #204 (03dev…listen-submission-improvement): Cleanup and subscribed to listen's list https://github.com/metabrainz/listenbrainz-andr...

[listenbrainz-android] 14akshaaatt merged pull request #206 (03main…dependabot/gradle/com.android.tools.build-gradle-8.1.0): Bump com.android.tools.build:gradle from 8.0.2 to 8.1.0 https://github.com/metabrainz/listenbrainz-andr...

aerozol: that is a very interesting development. but the UI confuses me. I followed the steps. and now what?

mayhem: your Mastodon profile should show what you're playing on LB

nuthing happening

aerozol: got the stats art creator going. will run on test in a bit.

Heh, nice ! More ListenBrainz integrations !

monkey: I co-opted test for a bit.

aerozol: https://test.listenbrainz.org/explore/art-creator/

Hah! I was just wondering why the love/hate buttons looked broken !

sorry.

I doubt aerozol needs to look long.

No worries, I didn't need it anymore really

I'm going to quickly fix the mode selection thing on the LB radio PR

sweet. I'll address the PR feedback after that.

Thanks mayhem!

monkey: Lb radio PR feedback improvements pushed.

mayhem: great. Just pushed some small changes myself, using a typescript enum to define the mode values a bit more safely (and generate the select options programmatically from the enum values)

+ default value for the select to fix the reported issue

I hope the smoke detectors in the office are working

Because you guys are on FIRE

huhu

I'm sure we would have found out by now if there was a working smoke detector at the office.

lol, indeed.

aerozol: did you see that I used your sticker?

I didn’t!!

Ooooh, I love it. I wouldn’t mind having pages like that open to the public if we have a ‘beta’ section in explore (I think it’s in the initial mockups even)

Oh, I like that idea. how do we do that? do you have that mocked up>

?

Doesn’t look bad at all, already

Let me dig

monkey was overseeing it. could perhaps use more whitespace and larger buttons.

It's mockep up. IIRC it was just a separator of sorts with a bit of a title

The mockup has this bit at the bottom: https://usercontent.irccloud-cdn.com/file/pfL8j...

ninja’d

Nothing complicated

says the resident CSS wizard.

if not complicated to you, wanna take a stab at it?

aerozol: what image would we use for LB radio?

I was just about to offer to make a picture! I’ll get onto it. Since it’s beta maybe some kind of a wireframe look

sweet.

<h3>Beta</h3>

yw

heh.

aerozol: I'm thinking we can drop the dropup/dropdown indicator to the left of "Beta" and the associated feature for now. Revisit it when we have more items in there when it would make sense to want to hide them

monkey: agreed

I'll do that bit while you work on the image :)

♥️

I'll keep playing with my CSS toys and you go adulting? Works for me !

aerozol: barring the CSS you were pretty close :D

stealing my work as always

monkey: something like this? easy theme for future beta stuff https://usercontent.irccloud-cdn.com/file/Qd9mS...

Perfect !

Thanks aerozol !

!m monkey

You're doing good work, monkey!

(feedback update works like a charm)

Thanks for your help steering me in a better direction :)

monkey: also, when you have some time. could use your help on fixing the tests here https://github.com/metabrainz/listenbrainz-serv...

Yeah, confirming one phantom failure, but also this `BrainzPlayer › failedToPlayTrack › tries playing the current listen with the next datasource`

yeah i tried to fix that one with the behaviour i think that is correct but doesn't seem to work.

OK, I can have a look in a bit

thanks!

Is musicbrainz.org still being updated? That's what the banner says still (since yesterday).

it's not showing on beta

but I see the banner on stable MB

[musicbrainz-server] 14derat opened pull request #3005 (03master…a_focus): MBS-11947: Stop hiding focus indicators on links. https://github.com/metabrainz/musicbrainz-serve...

monkey: I just realized that I was showing the playlist desc as a title. and we were not showing title at all. Adding one more state and one div. done soon.

👍

pushed.

When we revisit, we'll most likely transform it into something like the created for you page? https://usercontent.irccloud-cdn.com/file/caVeS...

with the side bar

yes, makes total sense. there is too much horizontal whitespace now.

building the image for beta now

$no_filter_recent -- to not filter recent listens, which otherwise is the default.

monkey: ^^ not great, but I am struggling to do better.

and $no_filter_hate

How about something along the lines of `$filter:recent:false` ?

yeah, better.

$filter:hate:false

$filter:country:spain

yeah, lets do that.

There's a subtlety in there which is that the filters can be either inclusive or exclusive: `$filter:country:spain` keeps only `spain`-ish recordings, while `$filter:hate:true` keeps only non-hated recordings

But eh. I don't think we need to dig too deep.

agreed. not sure that level of consistency serves anyone.

How to make queries take longer for no reason: `$filter:country:worldwide`

Actually, that would probably keep only ~ the digital releases, no?

Aaaaaanyway.

I wonder how long until someone requests '$filter:country:worldwide - $filter:country:(north america)'

Yeah, I was just thinking that.

$filterout:country:russia

yeah.

ok, three leavels gives us enough control. let me implement that.

There's some ambiguity with stuff like `$filter:country:spain`: how is the recording associated with the country? Are we talking album release country? Artist birthplace? Artist country of residence?

A problem for another day

thats not a parsing problem. that a data problem for lucifer. :)

[musicbrainz-server] 14derat opened pull request #3006 (03master…rec_focus): MBS-13207: Make recording focusable when adding release. https://github.com/metabrainz/musicbrainz-serve...

what about the failing front end tests monkey ?

lucifer: you about?

mayhem: hi yes

just wondering about front end tests failing and if they a phanthom failures. let me wait for this pass to finish.

lucifer: https://github.com/metabrainz/listenbrainz-serv...

ah, the output makes more sense this time. not related to lb-radio, it seems

mayhem: phantom. feel free to merge.

great, thanks!

you can fix the lint issues by running static builder and commiting if you want.

[listenbrainz-server] 14mayhem merged pull request #2537 (03master…lb-radio-testing-interface): Lb radio testing interface https://github.com/metabrainz/listenbrainz-serv...

ah, good to know for the next time.

[listenbrainz-server] release 03v-2023-08-09.0 has been published by 14github-actions[bot]: https://github.com/metabrainz/listenbrainz-serv...

aerozol: I’m up for a call with shared screen this week or next week to help you with managing PRs and setting a local MB Server for your tests?

s/?/, when would it work for you?/

LB Radio is now officially in beta: https://listenbrainz.org/explore/lb-radio/ !!

thanks lucifer and thanks monkey for making it happen!

monkey: meh -- for some reason there is a mode select issue (not a modeselektor issue) that was not present on beta.

I'll leave it for tomorrow

https://science.slashdot.org/story/23/08/09/155... :D

read somewhere online: ai getting dumber over time is the closest it can get to mimic human behavior.

the joy of machine learning. you tweak one thing for strength, another gets worse. probably a perfect parallel for humanity, if you think about it.

yvanzo: iiuc the tmp file location changes in https://github.com/metabrainz/musicbrainz-serve... didn't prevent the alerts but might've caused a slight decrease in io latency?

perhaps those make sense to keep but not sure we'll end up using the ionice script

bitmap: nothing did prevent the alerts on its own in this PR