akshaaatt: hey!, I was getting my feed blank due to un-utilized parameters present in our model, I have removed them and refactored the top bar.
2024-02-19 05038, 2024
Schmias
how long should I wait between submission batches (of 1000 listens)?
2024-02-19 05031, 2024
lucifer
Schmias: there are ratelimiting headers in responses which provide the info
2024-02-19 05021, 2024
reosarevok
bitmap, yvanzo: can you see any reason why we wouldn't want MBS-13407 ? (so, an option on artist rename to either change all previously uncredited rels to the new name, or keep them credited to the old one)
you can enter the artist's (current) name as a relationship credit to work around that (if it's actually meant to be fixed), but I agree an option to rename blank ones may be useful (dunno about "almost always not intended")
2024-02-19 05051, 2024
Schmias has quit
2024-02-19 05058, 2024
reosarevok
I know you can, but that needs to be done on a case per case basis, so it can be a ton :)
2024-02-19 05055, 2024
reosarevok
I agree that often it's intended, for typos and such, but I can also see that "Artist renamed themselves" cases often should *not* have this change happen :)
2024-02-19 05043, 2024
yvanzo
It seems sensible when an artist is changing their name in real life, not when their name is changed in the database for some other reason such as cleanup which might be the most frequent case.
2024-02-19 05044, 2024
bitmap
sure, just if you're originally entering them you should usually set it if it's not meant to change
2024-02-19 05004, 2024
reosarevok
That's very much non-obvious to users though :)
2024-02-19 05009, 2024
reosarevok
(since it's not like that for ACs)
2024-02-19 05051, 2024
yvanzo
Maybe the UI should also be improved to make it clearer how to originally enter credits.
2024-02-19 05029, 2024
reosarevok
IMO either they should be entered by default like for ACs, or there should be an option to change it or not
2024-02-19 05035, 2024
reosarevok
Possibly both?
2024-02-19 05005, 2024
bitmap
yeah I agree it's not obvious
2024-02-19 05039, 2024
bitmap
if we have an option to change them all, entering them by default might make sense
2024-02-19 05049, 2024
kellnerd[m] joined the channel
2024-02-19 05049, 2024
kellnerd[m]
I have even seen editors arguing to empty the credited field if it is identical to the entity's (current) name.
2024-02-19 05058, 2024
bitmap
(as in we should disallow blank credits and have an option to change ones that match the current artist name)
2024-02-19 05040, 2024
aabbi15_ has quit
2024-02-19 05008, 2024
yvanzo
Is it something that would be easier implemented in React?
2024-02-19 05001, 2024
bitmap
kellnerd[m]: I guess it's understandable, since we don't have documentation for it
2024-02-19 05052, 2024
bitmap
yvanzo: not significantly, since the UI would just be a checkbox, I think
2024-02-19 05009, 2024
yvanzo
👍
2024-02-19 05038, 2024
kellnerd[m]
Yeah, at least until you had to update a lot of rels to the old name after the entity has been renamed 😅
2024-02-19 05008, 2024
kellnerd[m]
So I would definitely be in favor of filling the credit field by default.
2024-02-19 05025, 2024
bitmap
storing blank credits was mostly an implementation detail (since it'd takes less space in the DB when most would match the artist name anyway)
2024-02-19 05056, 2024
kilos_0 joined the channel
2024-02-19 05000, 2024
reosarevok
I guess this wouldn't even be a schema change, or would it?
2024-02-19 05012, 2024
bitmap
it shouldn't be
2024-02-19 05019, 2024
yvanzo
shouldn’t indeed
2024-02-19 05006, 2024
bitmap
and we could continue to store them as blank credits as long as it's the same to the user (i.e., fill the field with the artist name in the UI, and only blank it on the server if it matches)
2024-02-19 05019, 2024
reosarevok
Oh, so just show the same as the artist in the credit field if it's actually blank, but make sure the process of changing them either fills them in or leaves them blank in which case they'll show the new artist?
2024-02-19 05026, 2024
reosarevok
Well, new name
2024-02-19 05053, 2024
yvanzo
I also thought that it seems like it was an implementation detail to take less space rather than a feature. Maybe (new) blank credits should rather be used as “no specific credit”?
2024-02-19 05001, 2024
bitmap
reosarevok: yes, I think so
2024-02-19 05008, 2024
bitmap
"no specific credit" is pretty much how they work now, yeah. it's just that it's very unclear to the user
2024-02-19 05012, 2024
reosarevok
Yeah, dunno. FWIW I sometimes do something like "recording-level credits get a relationship credit but work-level credits do not so that they will match the artist name"
2024-02-19 05019, 2024
reosarevok
I guess that wouldn't work with the new understanding.
2024-02-19 05045, 2024
reosarevok
But it feels honestly more straightforward to always have a credit, like we do with ACs
2024-02-19 05056, 2024
reosarevok
aerozol: ^ opinion, when around? :)
2024-02-19 05053, 2024
yvanzo
Maybe have a checkbox “This is specifically how this artist is credited here” to fill the field even though it is the same name.
2024-02-19 05013, 2024
yvanzo
(And leave it blank as we already do otherwise.)
2024-02-19 05057, 2024
Schmias joined the channel
2024-02-19 05039, 2024
minimal joined the channel
2024-02-19 05057, 2024
Schmias
`RateLimit-Reset-In` is always a one digit number. so it would be safe to wait for 10 seconds between batches
2024-02-19 05010, 2024
pranav[m] has quit
2024-02-19 05039, 2024
outsidecontext
Schmias: ideally you should make it variable and wait for the number of seconds returned by RateLimit-Reset-In for the next call. The exact duration can vary
2024-02-19 05017, 2024
Schmias
ok
2024-02-19 05005, 2024
ApeKattQuest
[17:09] <bitmap> if we have an option to change them all, entering them by default might make sense <-- I've wantd this for ages
2024-02-19 05005, 2024
ApeKattQuest
I've seen some people actually remove ac on relationships if it was identical to the (current) name, which I thoguht annoying
2024-02-19 05009, 2024
outsidecontext
I don't know the exact configuration behind it (lucifer surely can tell), but from my use I think I have seen it significantly higher then 10 seconds in situations where the rate limit is actually exhausted.
Schmias: What did you want to check in your imports? it looks good at first glance
2024-02-19 05029, 2024
Schmias
monkey: I just wanted to make sure I used the right fields
2024-02-19 05035, 2024
monkey
I do see some playlist listens from youtube, I don't currently know of a good way to deal with them and I sometimes just decide to remove them manually https://usercontent.irccloud-cdn.com/file/wWrCC5z…
2024-02-19 05047, 2024
monkey
Not much we can do here I guess
2024-02-19 05059, 2024
monkey
Yep, the fields look fine for what I've looked at Schmias
i.e. the name of the submission client you're writing
2024-02-19 05034, 2024
Schmias
it's just a script. I could publish it on github before submission and then put a link there
2024-02-19 05044, 2024
monkey
It can be useful down the line to isolate submissions from a particular script or version thereof
2024-02-19 05052, 2024
Schmias
makes sense
2024-02-19 05007, 2024
Schmias
thanks for your input monkey
2024-02-19 05018, 2024
monkey
You're welcome :)
2024-02-19 05046, 2024
dseomn has quit
2024-02-19 05047, 2024
dseomn_ joined the channel
2024-02-19 05010, 2024
Schmias has quit
2024-02-19 05010, 2024
dseomn_ is now known as dseomn
2024-02-19 05042, 2024
KassOtsimine has quit
2024-02-19 05053, 2024
aerozol
mōrena!
2024-02-19 05017, 2024
ApeKattQuest
🐟!
2024-02-19 05042, 2024
ApeKattQuest has fish for dinner. fish curry
2024-02-19 05007, 2024
ApeKattQuest
box said to use coconut powder/milk but I usd cream and some yogurt and rømme
2024-02-19 05010, 2024
ApeKattQuest
ecellent
2024-02-19 05013, 2024
aerozol
reosarevok: bitmap: yvanzo: I think I understand the relationship credit discussion… I have also thought it can look weird when sometimes there are specific artist credits, sometimes not. Especially after renames and merges
2024-02-19 05023, 2024
aerozol
fish!
2024-02-19 05035, 2024
ApeKattQuest
tbh it has annoyed me whne i fill in the ac on rels because it's *spesifically this name* and thne someone has gone and removed it on the grounds that it's "the smae as the artist name" (sometimes that'll change you know so i wanted to helgardere myself to make sure those relAC wasn't changed)
2024-02-19 05052, 2024
aabbi15 joined the channel
2024-02-19 05020, 2024
reosarevok
Yeah, I think it would be clearer if it was always there tbh
2024-02-19 05021, 2024
aerozol
I know it might be a headache with the code (e.g. not sure if it’s worth the trouble), but there could be a default like [artist name] (maybe displayed in grey). That can be toggled/is changed upon rename. When someone explicitly types something in it becomes locked in (even if the same as the artist name). I haven’t thought it through completely though
2024-02-19 05022, 2024
twodoorcoupe joined the channel
2024-02-19 05037, 2024
aerozol
Otherwise yeah, auto setting to artist credit is probably 👍
so thne you cna whne changing or merging be like "yes I want it chnage identical artist rel AC
2024-02-19 05009, 2024
aerozol
I don’t know if that’s quite right, because sometimes there might be a specific credit for the old name, while in other cases it’s just whatever/should be renamed
2024-02-19 05013, 2024
ApeKattQuest
option. incase the rename/merge is a typo edit
2024-02-19 05027, 2024
ApeKattQuest
i mean ys
2024-02-19 05038, 2024
ApeKattQuest
hench giving the option to do or not rename credits
2024-02-19 05000, 2024
ApeKattQuest
if there are spesific ones with the old name thne will have to be edited. much like we do AC on releases
