m.amanullah7: i think its on the latest official release. can you please confirm with them what the latest release is? anyway, i'll update to latest master.
rayyan_seliya123: is the code in the PR updated with latest changes?
rayyan_seliya123
lucifer[m]: No I have not yet pushed the indexer and models script in the pr because I was refactoring it should I push it ?
lucifer[m]
yes you can push it, and i'll take a look
rayyan_seliya123
lucifer[m]: Okk sure will do it
lucifer[m]
mayhem: do you remember what compatibility issues funkwhale devs had with integrating MB/LB? they are doing a major breaking release apparently, may be they are more open to the integration now
mayhem[m]
morning! it wasn't so much issues of incompatibility, but more like issues with general approach to getting recs into funkwhale.
they resist requiring MBIDs and keep attempting solutions that... don't work. I've tried to offer help on how to do this, but my suggestions were rejected, bad things were implemented and then "look, this shit doesn't work". uh yeah.
lucifer[m]
i see :/
Maxr1998_ has quit
outsidecontext[m joined the channel
outsidecontext[m
lucifer, mayhem One of the changes in the next FW release is that it includes my patch for opensubsonic support, though. So for collections that do have MBIDs the opensubsonic API will report those, and e.g. troi will work properly with FW
mayhem[m]
I think this is a great approach -- we keep adding the functions we need in their software to deliver recommendations against their best efforts. :)
Maxr1998 joined the channel
outsidecontext[m
I haven't followed FW development very closely in recent months, and also don't know all the details about your discussions for the recommendation features. But I can see how making MBIDs a strict requirement is a bit problematic for any player, though.
But as I understand the upcoming version has an option to actually enforce this on an instance
mayhem[m]
outsidecontext[m: Yes, that was never going to fly. But the alternatives they went for, which I suggested not to do, didn't work out. and I spent a lot of time trying to help. And each time it was like: Go west, you'll find the holy land of recommendations. FW: Right, tomorrow morning we set offeast and march 30 days time.
outsidecontext[m
Btw, I had recently tried updating my FW server to the last alpha release. Encountered some issues that should be fixed now, so I will try again. If you guys still need access for development I could provide this (at least if I succeed with the upgrade).
rayyan_seliya123
<lucifer[m]> "yes you can push it, and i'll..." <- Just pushed let me know after review what am I missing in your free time !!
derat[m] has quit
lucifer[m]
m.amanullah7: updated funkwhale instance to 2.0.0.alpha2, try again using that, you will have to create a new oauth app because i reset the db when upgrading.
bitmap: when around, can you take a look? It's now quite unhappy with mutate-cow uses
Also there's a bunch of issues with phraseArgs where I'm not sure it's trivial to fix it since the array itself is not const but I don't think we can do as const for one member of the array?
Wonder if it would make sense to turn those into objects to begin with - us having true, /* forGrouping */ and the like righ tnow suggests it could be clearer
d4rkie has quit
mamanullah7[m] uploaded an image: (290KiB) < https://matrix.chatbrainz.org/_matrix/media/v3/download/matrix.org/doQkiuygiajrEUFZbhNbTmtv/Screenshot%202025-06-12%20at%203.40.30%E2%80%AFPM.png >
d4rkie joined the channel
mamanullah7[m]
lucifer: i think again we need to create a user?
lucifer[m]
[@m.amanullah7:matrix.org](https://matrix.to/#/@m.amanullah7:matrix.org) sorry I had created a user for you but different name, try without the 7.
mamanullah7[m]
lucifer: same msg! displayed ?
lucifer[m]
you can report it back to the funkwhale devs.
mamanullah7[m]
okay i'll report!
lucifer: i think we need to verify our email and then funkwhale also verify their own as well and then we can able to login in funkwhale (funkwhale.audio) is this will be same for this?
lucifer: '<petitminion@matterbridge> another user reported this behaviour, do you have backend logs ? could you try hard reloading the browser / in a private window ?'
i tried this in incognito as well getting same error!
m.amanullah7: just this in the backend logs `2025-06-12 12:17:00,636 django.request WARNING Unauthorized: /api/v1/oauth/authorize/`, i'll try debugging it later on my end.
you can also proceed with the rest of the work in the meantime.
monkey: hi! i am migrating the libre.fm importer to the backend like last.fm, the import listens page looks like this now. should i remove it completely?
lucifer[m] uploaded an image: (53KiB) < https://matrix.chatbrainz.org/_matrix/media/v3/download/chatbrainz.org/nqYpCHuWqdEHJsNYGrFwqhyC/image.png >
lucifer[m]: Yes, I saw that. I think we should keep the importer for a while as the services will still be available for existing users, and they'll want an easy way to migrate.
lucifer[m]
yup definitely keeping the importer around, once this new backend one is deployed. i'll also post a reply on the thread in case people want to migrate.
monkey[m]
Then revisit in a year or two, maybe move to importing dump files instead
mayhem: can you share the meet link for later today?
mayhem[m]
lucifer[m]: done
lucifer[m]
suvid: i just saw `external_service_oauth_id` in `listens_importer` is also NULLABLE so you can just set that to NULL when creating a listens_importer row entry.
mamanullah7[m]
<lucifer[m]> "m.amanullah7: just this in the..." <- lucifer: I tried to open auth_url and I was logged out still i got authentication error instead of login page! I think its not about the versions it's smtng bug!
I've pushed my changes, integrated frontend as well except BrainzPlayer you can test now!
Some parts I've hardcoded for testing purpose I'll change once it's working fine
* lucifer: I tried to open auth_url and I was logged out still i got authentication error instead of login page! I think its not about the versions it's smtng bug!
I've pushed my changes, integrated frontend as well except BrainzPlayer you can test now!
Some parts I've hardcoded for testing purpose I'll change once it's working fine
lucifer[m]
m.amanullah7: does it work if you are already logged into funkwhale?
i also needed to expose the 9870 port from the namenode container in docker-compose.spark.override.yml. i have already updated it on your wolf setup. and tested this there as well.
let me know if you still face any issues
mamanullah7[m]
lucifer[m]: Yes
lucifer[m]
cool
for now, let's put a message in the frontend that you need to be already logged in for this to work.
* logged in to funkwhale for this
i'll try to work with funkwhale devs to fix this issue meanwhile.
you can continue to work on the rest of your project.
<lucifer[m]> "mayhem, monkey: also, fyi libre..." <- not shutting down yet, just not open for new-sign ups, but yes. I suppose we should contact them and let them know we're happy to take the users and let them import their data. thoughts?
mayhem[m] leaves a comment
lucifer[m]
yup makes sense
monkey[m]
I see that mattl mentioned LB a few comment above, that's good.
I didn;t really want to barge in the day of the announcement and look like a music lover poacher...
mayhem[m]
monkey[m]: worry not, I did it for you. lol.
lucifer[m]
Sorry running a few minutes late for the next meeting
reosarevok[m]
Wait, that reminds me, did I miss a meeting on oauth or was that not supposed to be on Tuesday, lucifer ?
bitmap[m]
<zas[m]> "bitmap: yvanzo load on selda..." <- we already have beta containers on yamaoka, is there another alternative?
lucifer[m]
mayhem: libre.fm maintainer replied to you on that thread
reosarevok: yes but we never agreed on a time with everyone so i didn't ping that day. i'll ask again later for sometime next week.
<lucifer[m]> "i also needed to expose the 987..." <- You must have imported the `get_listens_metadata()` function right?
lucifer[m]
[@holycow23:matrix.org](https://matrix.to/#/@holycow23:matrix.org) yes. It's in the updated gist but the modifications are patching listenbrainz_spark.config to make it work outside the container.
holycow23[m] uploaded an image: (28KiB) < https://matrix.chatbrainz.org/_matrix/media/v3/download/matrix.org/lwmYTSMlcRtGwQyaipNeyNQz/image.png >
holycow23[m]
I meant the gist doesn't have anything with you importing the function, I mean fine I will import it but it does need to be imported right
I tried using get_incremental_listens_df to get the listens but getting ConnectionRefusedError: [Errno 111] Connection refused
Else could you assist me with how to fetch the listens using the document path returned by get_listens_metadata
suvid[m] has quit
Kladky joined the channel
d4rk-ph0enix has quit
d4rk-ph0enix joined the channel
d4rk-ph0enix has quit
d4rk-ph0enix joined the channel
dabeglavins6071 joined the channel
dabeglavins607 has quit
dabeglavins6071 is now known as dabeglavins607
lucifer[m]
<holycow23[m]> "I meant the gist doesn't have..." <- uhh my bad, let me fix that
holycow23: running with increased driver memory seems to fix it, note the java heap space error before the connection refused `pyspark --driver-memory 8g` seems to fix it.
