monkey: to be fair, it's not like it was registered in the public ports database as "for LB use only" either ;)
2022-01-17 01731, 2022
monkey
For what it's worth, I'm in favor of the change to 8100 (or whatever port is free and generally available)
2022-01-17 01746, 2022
PrathameshG13 has quit
2022-01-17 01756, 2022
PrathameshG joined the channel
2022-01-17 01700, 2022
PrathameshG
Got it, thanks for the help!
2022-01-17 01701, 2022
PrathameshG
But for some reason its not taking the input.
2022-01-17 01722, 2022
PrathameshG
Also, I should mention that I havent setup my local development environment yet 👁
2022-01-17 01710, 2022
PrathameshG
Oh nvm, it worked somehow
2022-01-17 01701, 2022
yvanzo
reosarevok: The rebased PR has conflicts on PaginatedResults.js
2022-01-17 01727, 2022
gcrkrause3 joined the channel
2022-01-17 01754, 2022
reosarevok
Hmm. Wonder why it didn't complain
2022-01-17 01755, 2022
reosarevok
Will check
2022-01-17 01719, 2022
PrathameshG has quit
2022-01-17 01736, 2022
reosarevok
Oh, or maybe it's just new ones
2022-01-17 01710, 2022
reosarevok
Yeah. Rebased again, yvanzo :)
2022-01-17 01755, 2022
PrathameshG joined the channel
2022-01-17 01753, 2022
treeshateorcs[m] joined the channel
2022-01-17 01705, 2022
treeshateorcs[m]
hello, everyone. i can't set up a development environment for listenbrainz. it raises an exception https://0x0.st/o-nB.png
2022-01-17 01747, 2022
treeshateorcs[m]
i did everything according to the readthedocs page
2022-01-17 01734, 2022
lucifer
treeshateorcs[m]: uhh, sorry that looks like one of my recent patches gone wrong. i'll try to fix it soon.
2022-01-17 01755, 2022
lucifer
outsidecontext: will look into it.
2022-01-17 01717, 2022
treeshateorcs[m]
oh, sure! can't wait. thank you
2022-01-17 01755, 2022
outsidecontext
lucifer: thanks. it just rather randomly imported a single listen again. Interestingly I had paused playback for a while, started again. The very first listen got imported, but the rest fails again
No other player provides such field mappings for last.fm afaik
2022-01-17 01731, 2022
PrathameshG12
If you know any, please let me know I'd be happy to move👀
2022-01-17 01746, 2022
treeshateorcs[m]
you may try your luck with cmus and cmusfm (both are open source, so you can edit the cmusfm's source code to match your requirements, it's a smol program)
2022-01-17 01754, 2022
treeshateorcs[m]
but let's not offtop too much
2022-01-17 01718, 2022
PrathameshG12
oh right. Forgot this is not discord lol
2022-01-17 01712, 2022
bitmap
asubedy: hi, a SQL query is the only way to get recently-added releases at the moment. to get an MBID you need some kind of metadata to search for; we have a search API at https://wiki.musicbrainz.org/MusicBrainz_API/Sear… or if you have direct database access you can of course just do a SQL query for that too
2022-01-17 01722, 2022
EyePulp joined the channel
2022-01-17 01748, 2022
EyePulp
Howdy; Querying the musicbrainz API for recordings with a certain word in `recording` key, and when using offsets to loop thourhg pages of results we get back a whole lot more than the `count` value sent by the result. We seem to get a lot of dupes, too, judging by the recording id value. Any suggestions on how to improve? I'll post the query in a sec
2022-01-17 01750, 2022
reosarevok
bitmap: hi! looking at https://github.com/metabrainz/musicbrainz-server/… again - when you suggest doing all with SQL, would that still involve doing the cursor stuff, but just running SQL deletes instead of $c->model('Editor')->delete?
2022-01-17 01722, 2022
v6lur joined the channel
2022-01-17 01710, 2022
bitmap
reosarevok: that's correct, the cursor stuff is still needed to keep each delete in its own transaction
2022-01-17 01723, 2022
reosarevok
Ok :)
2022-01-17 01740, 2022
CatQuest
cursors?! oh no.
2022-01-17 01708, 2022
EyePulp
So when I get this back in a query: {"created":"2022-01-17T17:26:09.742Z","count":272,"offset":0,"recordings":[...]...} What does the "count" key represent? number of total items (recordings) in this case matching the result, or total number of pages of results at whatever the "limit" is?
2022-01-17 01752, 2022
reosarevok
IIRC it's total items, but IIRC also someone complained recently that the search server didn't keep the order with offset...
2022-01-17 01756, 2022
reosarevok
yvanzo, do you remember?
2022-01-17 01723, 2022
EyePulp
Also hello, and thanks for your help and any work you do related to the MB db & API. =)
2022-01-17 01702, 2022
EyePulp
The second question -- why do I get a lot of repeated "id" values for recordings in the results? I think even within a single page's worth, there are dupes. Is there a way to force DISTINCT on id or something along those lines?
2022-01-17 01710, 2022
bitmap
EyePulp: can you link to the query you're seeing this behavior on?
2022-01-17 01751, 2022
EyePulp
@bitmap I'm fine posting an example here if that's acceptable, or a gist if preferred
2022-01-17 01715, 2022
asubedy
<bitmap> "asubedy: hi, a SQL query is..." <- Oh ok then there would be no way to get the recently added releases on the MB revamp without actually connecting it with the database // akshaaatt
2022-01-17 01715, 2022
asubedy
So can we do this, instead of recently added releases we can show cover arts of the most listened musics, for which I guess there is an API or something like it through ListenBrainz. (I saw in the chat earlier). Any suggestions on this?
I think any inconsistencies you'd see otherwise are in theory due to index updates
2022-01-17 01735, 2022
reosarevok
bitmap: I think if the collections are empty, it should be fine to delete any collaborators as well, or would you rather I checked there are none in the query and left those unremoved?
2022-01-17 01723, 2022
bitmap
hmm. if they are empty I think it's fine to delete...especially since it'd have to match all these other criteria for the editor being empty, which should be rare
2022-01-17 01703, 2022
reosarevok
Hmm, although I think our idea earlier was "the collections are untouched"
2022-01-17 01720, 2022
reosarevok
So maybe we should discard collaborator ones. I expect the number difference will be 0 or 1 at most...
2022-01-17 01712, 2022
bitmap
yea, I think it will hardly matter either way, so whatever you prefer
2022-01-17 01723, 2022
EyePulp
@bitmap - beautiful! results are matching expectations now! Thanks so much for the clarification.
I have received no mailed in reviews or notices of people outside the usual crowd who want to give review, so… people up for reviews: lucifer, monkey, reosarevok, bitmap, akshaaatt, zas, alastairp, mayhem, yvanzo, CatQuest, Freso – anyone else who wish to give review, let me know ASAP. :)
2022-01-17 01702, 2022
Freso
lucifer: Go!
2022-01-17 01706, 2022
CatQuest
woa
2022-01-17 01715, 2022
CatQuest
oh
2022-01-17 01721, 2022
lucifer
hi all!
2022-01-17 01712, 2022
EyePulp has his world rocked by IRC with embedded media stills
2022-01-17 01722, 2022
lucifer
i worked on seeking and implementing feedback on various already open PRs last week. we deployed the long due listen user id PR. and started work on the next steps.
2022-01-17 01738, 2022
lucifer
another long due PR to fix listen counts in LB also progressed along nicely. other than that, worked on a few improvement of the artist origins map. when hovering over the country, it now shows the artists name as well.
2022-01-17 01726, 2022
lucifer
finally some misc bug fixes and discussions. fin.
2022-01-17 01729, 2022
lucifer
monkey: next?
2022-01-17 01735, 2022
monkey
Hi everyone !
2022-01-17 01741, 2022
monkey
Happy new year !
2022-01-17 01723, 2022
monkey
I just started working again today, so don't have a lot to say about the past three weeks. I was vacationing in the US, got married, caught covid again, managed to get back after more delays.
2022-01-17 01742, 2022
monkey
Before the holidays, I mostly worked on the Year in Music project.
2022-01-17 01751, 2022
monkey
This week, docs for BB!
2022-01-17 01706, 2022
monkey
That's it for me, bitmap go !
2022-01-17 01710, 2022
bitmap
hey
2022-01-17 01738, 2022
bitmap
last week I was mainly working on speeding up area-related queries for the website, since we've been getting alerts about those pages causing excess load
2022-01-17 01723, 2022
bitmap
my plan is to bring back area_containment, which was previously a view, but will be brought back as a materialized table. (those have worked amazingly for the artist pages)
2022-01-17 01735, 2022
bitmap
I have most of the triggers done and am writing pgtap tests now
2022-01-17 01746, 2022
reosarevok
Ooh, neat
2022-01-17 01722, 2022
bitmap
other than that that, I opened a couple other PRs to fix spammy warnings in our website container logs, and to remove support for Amazon cover art as decided last year (which lets us delete a ton of code, woohoo)
2022-01-17 01739, 2022
bitmap
that's about it I think...fin, go reosarevok
2022-01-17 01748, 2022
reosarevok
Hi! I was on holiday and traveling for most of the week
2022-01-17 01755, 2022
Freso
(People still up: akshaaatt, zas, alastairp, mayhem, yvanzo, CatQuest, Freso – anyone else who wish to give review, let me know ASAP. :))
2022-01-17 01757, 2022
reosarevok
Then I worked on a few easy tickets created while I was away
2022-01-17 01721, 2022
reosarevok
And now looking into some changes to the MBS-9356 script so we can run it
