there's a huge difference in total number of artists and sum of the artists in artist origin map
2020-08-06 21909, 2020
iliekcomputers
yeah...
2020-08-06 21925, 2020
slriv_ joined the channel
2020-08-06 21928, 2020
shivam-kapila
ruaok's all time has less data than year
2020-08-06 21930, 2020
iliekcomputers
let's add logs on what we send to the labs API and what it returns.
2020-08-06 21939, 2020
ishaanshah
yep
2020-08-06 21948, 2020
iliekcomputers
i think we might be sending too many artists in one request.
2020-08-06 21901, 2020
iliekcomputers
or something like that.
2020-08-06 21906, 2020
ishaanshah
hmm
2020-08-06 21916, 2020
ishaanshah
that may be the issue
2020-08-06 21925, 2020
ishaanshah
where should we log this
2020-08-06 21926, 2020
iliekcomputers
In the meanwhile, I'm gonna add a beta tag to the graph and release to prod
2020-08-06 21929, 2020
ishaanshah
sentry?
2020-08-06 21936, 2020
iliekcomputers
ishaanshah: sentry sounds reasonable.
2020-08-06 21945, 2020
ishaanshah
Even if it is not an error right
2020-08-06 21908, 2020
iliekcomputers
yeah. sentry is really the most convenient.
2020-08-06 21912, 2020
ishaanshah
ok
2020-08-06 21922, 2020
ishaanshah
I will open the PR tommorrow
2020-08-06 21933, 2020
ishaanshah
we will get to know more then
2020-08-06 21942, 2020
ishaanshah
also one more thing
2020-08-06 21900, 2020
ishaanshah
is it possible to bypass the limit on variable output in sentry
2020-08-06 21909, 2020
iliekcomputers
Not sure.
2020-08-06 21923, 2020
ishaanshah
that may cause some trouble
2020-08-06 21942, 2020
iliekcomputers
I'll have to look into it. Maybe just implement multiple requests anyways if it makes logging easier
2020-08-06 21949, 2020
ishaanshah
nginx has an upper limit of 1MB for post requests
2020-08-06 21954, 2020
ishaanshah
by default
2020-08-06 21944, 2020
ishaanshah
we might be clipping some artists off
2020-08-06 21944, 2020
slriv_ has quit
2020-08-06 21957, 2020
ishaanshah
I think we should do multiple post requests
2020-08-06 21907, 2020
ishaanshah
~100 artists per requests ig
2020-08-06 21915, 2020
iliekcomputers
yeah, me too.
2020-08-06 21922, 2020
iliekcomputers
100 sounds low though.
2020-08-06 21939, 2020
slriv_ joined the channel
2020-08-06 21902, 2020
iliekcomputers
let's make it a value in config.py and we can change as needed.
2020-08-06 21916, 2020
ishaanshah
ok, sounds good
2020-08-06 21928, 2020
iliekcomputers
i'm gonna leave this in beta for now.
2020-08-06 21911, 2020
ishaanshah
yeah I think that makes sens
2020-08-06 21921, 2020
iliekcomputers
if we don't fix this soon-ish, we should just not render the graph on the reports page until the bug gets fixed.
2020-08-06 21926, 2020
yvanzo
Leo_Verto: not really, even though it works with less than that, 16c/16GB is the minimum to be safe, for now.
2020-08-06 21927, 2020
slriv_ has quit
2020-08-06 21949, 2020
ishaanshah
yep yep
2020-08-06 21951, 2020
slriv_ joined the channel
2020-08-06 21915, 2020
iliekcomputers
ruaok: we've caused some 500s on labs.api.listenbrainz.org, in case you have detailed logs there. we suspect it's because we're sending too much data in one request.
Hey yvanzo! Thanks for the answer, I suppose the alternative then would be to keep using the MeB servers and to donate. Is just including the API key in the application possible or would we have to use some kind of proxy?
2020-08-06 21926, 2020
Leo_Verto
ruaok: Bavaria or Austria?
2020-08-06 21946, 2020
iliekcomputers
ruaok: np, just pinged you so that you're aware. LB-687
ishaanshah: for tomorrow, another way to debug would be to get the artist list from the prod API and send the request to labs.api locally with that artist list.
2020-08-06 21940, 2020
shivam-kapila
Qiestio: Can I use falticon icons?
2020-08-06 21959, 2020
shivam-kapila
Question*
2020-08-06 21906, 2020
shivam-kapila
Mr_Monkey: iliekcomputers ^^
2020-08-06 21952, 2020
iliekcomputers
what icon do you want to use? does it have an alternative in fontawesome?
