Although it looks like the recordings are from another album matched by the same query, so probably a simple off-by-one error
2021-10-13 28656, 2021
ruaok
woo, nice. YT installed, playing right away. so cool!
2021-10-13 28657, 2021
ruaok
monkey: will fix soon
2021-10-13 28608, 2021
monkey
👍
2021-10-13 28610, 2021
lucifer
ruaok: any work for me to do?
2021-10-13 28625, 2021
peterhil joined the channel
2021-10-13 28606, 2021
ruaok
not that I can think of right now. I have a lot of backend work to do to improve the script to keep things in sync, but I won't get to that until tomorrow.
2021-10-13 28631, 2021
ruaok
but, thanks for being the usual rock-start for getting us so far!
2021-10-13 28637, 2021
ruaok
*star
2021-10-13 28651, 2021
lucifer
cool, let me know if something else comes up. yeah, i see some cleanup can be done but probably best to wait till things finalize.
right, thats what i meant we dont have a newer image yet.
2021-10-13 28617, 2021
alastairp
yeah, unmerged PR, isn't really ready for prime time :)
2021-10-13 28617, 2021
opal has quit
2021-10-13 28634, 2021
opal joined the channel
2021-10-13 28616, 2021
ruaok
monkey: any news on that webpack issue?
2021-10-13 28654, 2021
monkey
I couldn't reproduce locally, it's gotta be due to available memory one way or another. Let me push what I think could be a fix
2021-10-13 28609, 2021
ruaok
k
2021-10-13 28654, 2021
monkey
Pushed. You're gonna have to rebuilt static_builder
2021-10-13 28625, 2021
ruaok
k
2021-10-13 28659, 2021
ruaok
works again, thanks!
2021-10-13 28632, 2021
ruaok
the data returned from end endpoint is still a hash, but hey.
2021-10-13 28646, 2021
monkey
lucifer: I ended up stealing and adapting the code from the react-colour-wheel library, because it was just too much of a headache to get it to compile properly.
2021-10-13 28628, 2021
monkey
Basically it didn't like that there was some JSX (`<canvas…`) in a file with .js extension
2021-10-13 28600, 2021
monkey
Oh, i'm getting this SQL error now when I run the colorplay PR:
2021-10-13 28602, 2021
monkey
sqlalchemy.exc.OperationalError: (psycopg2.OperationalError) could not translate host name "db" to address: Name or service not known
we can only ever have one instance at a time right now.
2021-10-13 28655, 2021
ruaok
would you like to run yours or should I update from git and restart periodically?
2021-10-13 28631, 2021
ruaok brings up his instance with the latest
2021-10-13 28602, 2021
ruaok
up
2021-10-13 28615, 2021
monkey
Thanks :)
2021-10-13 28633, 2021
monkey
I'm almost there with the color wheel, just needed to test
2021-10-13 28638, 2021
monkey
OK, tested committed and pushed. It's pretty damn sweet
2021-10-13 28610, 2021
monkey
I think I broke the background color change, gonna fix that now
2021-10-13 28609, 2021
ruaok
let me update. hang on everyone
2021-10-13 28628, 2021
ruaok
back up
2021-10-13 28629, 2021
ruaok
ooooh!
2021-10-13 28608, 2021
ruaok
!m monkey
2021-10-13 28608, 2021
BrainzBot
You're doing good work, monkey!
2021-10-13 28624, 2021
bitmap
reosarevok: I think we can potentially remove the 500 recording limit, but we'd have to make some changes to how we time out requests. e.g. right now we don't distinguish between a single db query taking 60s vs. 60 db queries taking 1s each
2021-10-13 28652, 2021
bitmap
the latter isn't as problematic since we know it's still doing work and might complete (though there should still be some kind of overall limit)
2021-10-13 28609, 2021
shivam-kapila
Oh the color play looks really good. GG devs :)
2021-10-13 28631, 2021
lucifer
monkey: oh! nice :D
2021-10-13 28602, 2021
akshat
monkey coming at it with Colour. looks great! 💯
2021-10-13 28638, 2021
monkey
I do want to point out I didn't make the color wheel but shamelessly stole it from that one unmaintained library
