reosarevok, akshaaatt: I fixed the broken link and some other stuff in the readmes. Thanks for finding reo
2022-09-15 25806, 2022
aerozol
also, my first time doing something direct on the main repo with direct write access! (the other updates I did, when I remembered I could) I assume that’s fine to do it directly if it’s readme tweaks and non-breaking etc?
2022-09-15 25839, 2022
darkstardevx joined the channel
2022-09-15 25835, 2022
darkstardevx has quit
2022-09-15 25800, 2022
darkstardevx joined the channel
2022-09-15 25813, 2022
darkstardevx has quit
2022-09-15 25849, 2022
lucifer
mayhem: it picked up some speed overnight. 130 K albums done now, only explanation i have is that the discovery probably slowed down as more and more artists became already seen with time. charting this data in grafana may provide more insights.
2022-09-15 25827, 2022
lucifer
although if this is indeed the case, it makes me sad that we didn't cache discovered artists/albums in redis to survive restarts.
2022-09-15 25816, 2022
darkstardevx joined the channel
2022-09-15 25834, 2022
d4rk joined the channel
2022-09-15 25842, 2022
d4rkie has quit
2022-09-15 25807, 2022
Lotheric has quit
2022-09-15 25844, 2022
Lotheric joined the channel
2022-09-15 25855, 2022
aerozol
Does anyone else use LB to checkwhat their friends/followers have been listening to? If so, do you share my annoyance with clicking on a profile to not be taken to their listens tab, but to their reports tab?
2022-09-15 25856, 2022
aerozol
tl;dr can we change it to take us to the listens tab. I’m visiting their page to see the songs they’ve listened to last, and to check if they’re listening right now which gives a nice cross-globe feeling of connection
2022-09-15 25849, 2022
aerozol
Also default the Followers/Following widget to show the ‘Following’ list (which is a bit more meaningful anyway, since you don’t pick your followers/that’s pretty random)
2022-09-15 25806, 2022
aerozol
When I pin a song it replaces the ’now listening widget’? Though that widget is unreliable so it might come back, let’s see
2022-09-15 25814, 2022
aerozol
(I’m digging through LB a little bit today)
2022-09-15 25847, 2022
aerozol
Ah yup the now listening widget is back, as well as the pinned widget, excellent
i manually browsed the file and loaded it in python as well. seems sane in both cases but pg gives the above error.
2022-09-15 25841, 2022
lucifer
\join #postgresql
2022-09-15 25849, 2022
mayhem
I have not, sorry.
2022-09-15 25841, 2022
loujine
reosarevok: ack, I'll try to take a look over the weekend (no free time for mb lately)
2022-09-15 25850, 2022
reosarevok
Thanks! :)
2022-09-15 25837, 2022
BrainzGit
[bookbrainz-site] 14MonkeyDo merged pull request #874 (03new-creation-form…feat/early-submission): Feat(UF): Submit modal entities directly to server https://github.com/metabrainz/bookbrainz-site/pul…
2022-09-15 25812, 2022
monkey
aerozol: I agree clicking on a user's name could take you to their listens page. I think this is one aspect that will be improved by redesigning a good user landing page that shows both the most recent listens, pins, etc. but also some more general stats.
Also, I contacted my HoD and asked for some flexibility with college attendance until oct 24
2022-09-15 25849, 2022
alastairp
how's that readme going?
2022-09-15 25855, 2022
Pratha-Fish
Let's see how it goes :)
2022-09-15 25831, 2022
Pratha-Fish
alastairp: re - README
2022-09-15 25831, 2022
Pratha-Fish
The restructuring part is done, I also rechecked it by setting it up again in another folder
2022-09-15 25839, 2022
Pratha-Fish
everything seems to be working!
2022-09-15 25839, 2022
Pratha-Fish
however, some older legacy scripts might not work as intended
2022-09-15 25814, 2022
elgranRoble joined the channel
2022-09-15 25835, 2022
lucifer
mayhem: alastairp: folks in PG channel told that copy doesn't support json so the error. trying another way now.
2022-09-15 25858, 2022
mayhem
sounds like a better error message might be needed.
2022-09-15 25855, 2022
lucifer
indeed! it incidentally works if there is no `"` inside the values because of csv parser working then etc but yeah not supported in general.
2022-09-15 25845, 2022
lucifer
PG was really the last thing i thought in the pipeline of `curl`, `split`, `jq` and `psql` to error but oh well.
2022-09-15 25828, 2022
mayhem
one doesn't expect pg to be the weak link, indeed.
2022-09-15 25822, 2022
alastairp
Pratha-Fish: yes, I was looking at some of your older notebooks because I wanted to check some things and saw some issues
2022-09-15 25801, 2022
alastairp
specifically, I was interested in counting the instances where the artist mbid was incorrect, and also the case where we counted all of the many different mbids referring to artists with the same name
next I'll hammer out another call that generates an image for a user based on their top-albums.
2022-09-15 25853, 2022
mayhem
but first, its time to go for a swim
2022-09-15 25855, 2022
mayhem
bbiab
2022-09-15 25858, 2022
kellnerd joined the channel
2022-09-15 25842, 2022
mayhem had some good grid API thoughts on the way home
2022-09-15 25857, 2022
mayhem
this should blow the doors off the last.fm images. :)
2022-09-15 25814, 2022
alastairp
nice grid!
2022-09-15 25828, 2022
mayhem
Ready for people to make their own. 🙂
2022-09-15 25836, 2022
alastairp
already did
2022-09-15 25846, 2022
Pratha-Fish
alastairp: I'll take a look at them
2022-09-15 25831, 2022
d4rk has quit
2022-09-15 25804, 2022
lucifer
mayhem: related to the album index, we could run all spotify data through the mapper and record all tracks that don't get matched to find issues and fix them.
2022-09-15 25844, 2022
mayhem
is this to improve our mapping?
2022-09-15 25856, 2022
lucifer
yes
2022-09-15 25822, 2022
mayhem
yes, that would work. but is it worth it?
2022-09-15 25855, 2022
lucifer
depends on how much work it is.
2022-09-15 25853, 2022
lucifer
spotify reader is a big source of listens so making the mapper better at those listens would help matching for many users.
2022-09-15 25805, 2022
Pratha-Fish
alastairp: could you specify which notebooks you specifically tried?
2022-09-15 25834, 2022
kellnerd
bitmap: I'm trying to understand the new relationship editor in order to adapt my userscripts. Currently I'm struggling to programmatically open an "Add relationship" dialog for a release.
2022-09-15 25835, 2022
kellnerd
Can you give me a pointer how to do that without having to trigger a click event for the + icon?
2022-09-15 25822, 2022
kellnerd
So far I've only found `MB.relationshipEditor.dispatch` which is apparently able to do much more complex things.
2022-09-15 25822, 2022
kellnerd
The point is that I don't want to automatically add a rel, I want the pre-filled dialog to stay open to be confirmed by the user.
2022-09-15 25856, 2022
bitmap
kellnerd: sure, I can help you there. are you performing this action from a release page (to add a relationship to a recording, say) or a different entity type?
2022-09-15 25846, 2022
bitmap
(the way to do it will mostly be the same except you have to specify a track in the function call for releases)
2022-09-15 25812, 2022
v6lur joined the channel
2022-09-15 25813, 2022
bitmap
you have to dispatch an action with type 'update-dialog-location', so something that would work on both pages is: MB.relationshipEditor.dispatch({type: 'update-dialog-location', location: {source: MB.relationshipEditor.state.entity}});
2022-09-15 25824, 2022
kellnerd
I'm in the release rel editor and want to add rels to the release, e.g. "vocals" having an artist name (or sometimes even an artist MBID).
2022-09-15 25801, 2022
bitmap
on release pages that ^ will open a dialog to add a release-level relationship
2022-09-15 25809, 2022
kellnerd
Thanks, that works, although I wouldn't have expected the action to be named 'update-dialog-location' :)
2022-09-15 25829, 2022
bitmap
it's kind of a weird internal naming convention since it also closes the dialog (by setting the location to null)
2022-09-15 25856, 2022
kellnerd
And now when I want to fill the dialog I can use `MB.relationshipEditor.relationshipDialogDispatch`, right?
2022-09-15 25824, 2022
bitmap
right. I imagine filling out the form with JS and triggering change events manually would also work, but you can control everything in the dialog with that dispatch function
2022-09-15 25842, 2022
kellnerd
Haha, it's probably a good thing that I'm blocking requests to sentry, I've already crashed the dialog a few times