I haven't published it yet as it is part of my latest MB project which I've been working on. Let me try to convert it into a python function this evening.
mayhem
❤️
kellnerd
Hmm, looking at the above examples I'm not quite sure how to decide which script should be regarded as guff and which should be kept...
Do we want to keep latin / the first / the most frequent script or even return two alternative cleanup results?
e.g. "Dan No Ura" or "壇ノ浦の戦" or ["Dan No Ura", "壇ノ浦の戦"]
mayhem
keep latin is the first thing I would try.
kellnerd
👍
mayhem
two results would work too.
huh, best band name today `(not a band name, just fyi)`
not-band-name?
rishav_a2z joined the channel
jivte_ has quit
jivte_ joined the channel
rozlav has quit
rozlav joined the channel
akshaaatt: there is a LB app feedback mail in support@ for you.
akshaaatt
Please forward it to me mayhem .
mayhem
done
akshaaatt
Great feedback! Sending local listens from BP to LB is in the pipeline currently.
We should get to it soon enough
mayhem
can you please reply and cc support@?
akshaaatt
Sure thing!
mayhem
is there support or could there be support for knowing what spotify is playing?
there is a now listening API for android, yes?
kellnerd
mayhem: Unfortunately python's re module does not support unicode properties, the feature which I used in JS :/
unidecode does a pretty good job of converting things back to ascii.
akshaaatt
We do have that API available, mayhem! We use it in the app currently to show what is Now Playing in the listens page
mayhem
the problem with this approach is that we have a diff type problem of analyzing the results.
akshaaatt
Even when you play a song on web, the app syncs automatically
Monkey tested that a few days back
mayhem
does it submit the now listening to LB? or is it only kept inside the app?
akshaaatt
Not yet, mayhem. It’s only inside the app currently.
mayhem
is that on the roadmap? it could solve the problem of spotify's laggy reporting of listens.
akshaaatt
Yussss
On it
mayhem
sweeeeit!
monkey
Certainly helpful for the playing now listen
kellnerd
Heh, I just realized that python-regex was already installed as a dependency on my OS, so I will definitely try to use that :)
mayhem
kellnerd: yay!
kellnerd
Yay! s/import re/import regex as re/ and it works
mayhem
nice.
I wonder if that is as performant as the existing re lib.
could you try running my large datasets using both?
happy to share the file with you.
akshaaatt: lucifer : there is a chance I'll be arriving in Delhi the afternoon of 25 Nov. Shall we have a mini meet up in India and invite other contributors as well?
lucifer
sounds awesome to me! :D
mayhem
keep the weekend free for now.
akshaaatt
Awesome! ⚡️⚡️⚡️
What’s the occasion, mayhem?
mayhem
chhavi and samj are getting married in jaipur the days before the weekend.
akshaaatt
Damn
mayhem
so, I can fly to Delhi Saturday morning.
mayhem is stupidly excited to be invited to an indian wedding.
lucifer
ohh noice! 🎉🎉
jivte_ has quit
mayhem
param: you too! Nov 25th/26th MeB meeting up Delhi?
akshaaatt
mayhem: you’re surely gonna enjoy the big fat weddings in India
mayhem
:D
akshaaatt
They spend like they own the world when marrying
mayhem
I've heard. can't wait to experience the food. and I hope there are bollywood dancers. :D
jivte_ joined the channel
monotux has quit
jasje
akshaaatt: BP submissions work already
can you check the date when that mail was recieved?
mayhem
today
akshaaatt
Did we release that already, jasje?
mayhem
just a short while ago.
akshaaatt
I thought it was merged just a few days ago and we are yet to do a release of it
jasje
akshaaatt: did that before redesigning listen service
no no
is something f*cked again lol?
I was sending local submissions last month no way it don work
akshaaatt
Let me confirm once
jasje
yeah it works
the user might not have filled their LB token
put these questions:
akshaaatt
jasje: I think it is wrong of you to barge into a conversation without having context.
This is different
Calm down, I’ll share the email with you
Dm me your email
jasje
akshaaatt: sorry mb
i was replying just to this line btw:
<akshaaatt> Great feedback! Sending local listens from BP to LB is in the pipeline currently.
akshaaatt
Yeah I get that but the context is different. I don’t treat BP and the spotify listens separately in my head even though we deal with them separately in the app currently
Now you can either test the performance of "regex" yourself or send me the file :)
Currently only stripping foreign scripts, if the performance is good I can look into separating different scripts
mayhem
cool! will test now.
jasje
akshaaatt: oh i see
akshaaatt
Yus
jasje
now i need to search android go
akshaaatt
XD
jasje
phil says that pano scrobbler works
is there another way?
monkey
I also use pano scrobbler
(in case you need testing/info)
reosarevok
yvanzo: PR updated
jasje
monkey: LB Android doesn't work for you?
akshaaatt
He didn’t say he uses android go
Talking about monkey
jasje
i mean the submission
monkey: android version?
mayhem
from 22s to 57s for 4M rows, kellnerd.
akshaaatt
More so, we should send spotify listens atleast because we do get that callback easily on the app
jasje
akshaaatt: i remember he did say sometime back that app isnt working i think'
monkey
For me I think last time I tried local listens (not listened with the android app but with bandcamp app) weren't submitted. Not sure if that's what you're talking about
mayhem
I think I might make use of both libs in this case. (use regexp only to do the script matching)
akshaaatt
monkey, if you have a moment, could you please hop on to the LB app and send a screenshot of your screen when you click on the button present jn the Listens screen?
jasje
monkey: yes thats what i meant
akshaaatt
I just wanna confirm whether it’s a device issue or implementation issue?
bitmap
reosarevok: yvanzo: json dumps are up and running since a few hours ago
reosarevok
Thanks!
mayhem
shaved 11 seconds using both libs
monkey
Listens screen shows my most recent listens (plus as "now playing", my ringtone from this morning that plays through spotify)