#metabrainz

/

      • yvanzo
        I don’t know.
      • But it is safe to use the suggested option here.
      • reosarevok
        Will do (I assumed it was safe, just wanted to make sure it wouldn't make a difference for the measurements)
      • Ok, on it
      • huhridge joined the channel
      • huhridge
        mayhem: okay, just so i can confirm everything, we wanna store the url of the track on IA, if found, in the MB database right?
      • in the external links section right?
      • mayhem
        if tracks are found on the IA, we want to store it in a seperate DB, much like how we store the caches for spotify and apple music. so, no MB involved.
      • huhridge
        oh okay! i'll take a look into it
      • pranav[m]
        akshaaatt (IRC): I have applied the migration.. so things are working fine now.. I just checked on a real device.. there is no data loss. The playlist dao tests are failing with the exception that unconsumed events were found .. I am not sure what is causing this.. I tried googling if I have to do the migrations in the tests as well, I did those but still it is failing due to the same reason..
      • munishk joined the channel
      • discordbrainz
        <07munishk> Hi lucifer
      • <07munishk> @lucifer
      • huhridge
        mayhem: you are referring to the metadata-cache right? for the internet archive, how should i proceed with it? the 78 rpm collection has around 520k items, do we want to crawl through all of them?
      • mayhem
        yep
      • that's small potatoes. :)
      • lucifer
        huhridge: you can take a look at the spotify metadata cache in the LB codebase on how to do that.
      • huhridge
        lucifer: i'll look into it, where is the cache used btw?
      • lucifer
        munishk: for the websockets test, 1. you need to add the websockets container to the test docker compose file, you can probably just copy paste from dev docker compose file. 2. you need to a dev dependency to LB for a python socket-io client and use it to connect to the websockets container in your test. 3. you need to submit a listen to LB API (listenbrainz/tests/integration/test_api.py has examples on how to do that). 4.
      • verify that the socket.io client received the listens from the websockets container.
      • huhridge: in the content resolver, basically we have a metadata cache of the entire catalogue of tracks in LB. then we build indexes over it. given a track, artist and album name the content resolver queries the cache using the indexes and returns a set of matching items. at the moment this is used for exporting playlists only.
      • it is similar to searching the external service using their APIs but much faster for bulk lookups.
      • huhridge
        lucifer: okay makes sense!
      • i've worked out the player implementation fully, will look into the content resolver now!
      • munishk
        lucifer: thanks for the detailed steps, can you also pls tell how i can verify no 4 ? (very new to testing)
      • huhridge
        i'll add the player implementation part to the proposl EOD
      • pranav[m]
        akshaaatt (IRC): the tests are fixed.. should work fine now.. I am working on light theme rn..
      • U can test the pr, it shud work well in dark mode.. ill fix the light mode now
      • lucifer
        munishk: you can look at api tests i mentioned above.
      • those test submit and get apis, instead of using the get api you'll use the websocket client.
      • munishk
        ohkk, will start working on it
      • Thanks a lot
      • iconoclast_hero joined the channel
      • monkey[m] has quit
      • reosarevok
        yvanzo: indexes loaded
      • What next? :)
      • theflash__
        akshaaatt: I've fixed up proposal with the required changes,
      • yvanzo
        reosarevok: just the same: checking disk usage
      • reosarevok
      • I guess the main ones being
      • iconoclast_hero has quit
      • Satyaraj[m] joined the channel
      • Satyaraj[m]
        lucifer (IRC): Could you please review my proposal on. the Subsonic integration project?
      • s/.//
      • iconoclast_hero joined the channel
      • lucifer
        Satyaraj[m]: mayhem took a look already, we are waiting for aerozol to discuss some parts about it.
      • for the BrainzPlayer part, i'll ask monkey to check once as well.
      • monkey[m] joined the channel
      • monkey[m]
        wha? huh? where?
      • lucifer
        i think DM'ed your IRCCloud account.
      • monkey[m]
        Yus
      • huhridge has quit
      • huhridge joined the channel
      • minimal joined the channel
      • munishk has quit
      • huhridge has quit
      • huhridge joined the channel
      • theflash__ has quit
      • pranav[m] has quit
      • BrainzGit
        [listenbrainz-server] 14MonkeyDo merged pull request #2825 (03master…added_timeline_to_fresh_releases_for_you_section): LB-1552: Add timeline to Fresh Releases 'For you' section https://github.com/metabrainz/listenbrainz-serv...
      • Satyaraj[m]
        <lucifer> "Satyaraj: mayhem took a look..." <- Cool, thanks :)
      • Tarun_0x0 joined the channel
      • yellowhatpro
        @yvanzo, I figured I can add more to the rust binding crate, prolly adding more abstraction over sqlx queries, for example creating methods to write more complex sql queries, error handlings, which I actually forgot writing in the proposal.
      • Any suggestions on this front?
      • yvanzo
        yellowhatpro: Not really, we are more expecting that the suggestions come from you :)
      • yellowhatpro
        cool! I will work on some helpful traits that can come handy with the entities.
      • yvanzo
        👍
      • ursa-major_ joined the channel
      • fletchto99_ joined the channel
      • Zhele has quit
      • fletchto99 has quit
      • ursa-major has quit
      • fletchto99_ is now known as fletchto99
      • ursa-major_ is now known as ursa-major
      • relaxoMob
        mayhem: Is it possible to create playlists with more than 25 entries with LB radio?
      • Is there a Limit for prompt parameters?
      • The query hints / playlist name are a little bit too much if someone like me comes and "abuses" it.
      • 88d3b-7714-411b-a75f-727e0894ca1d::nosim%20artist:a59755a8-b518-499d-be4b-4935a721a72e::nosim%20artist:6383add7-47ff-4e7e-9ba5-328e9b197b1e::nosim%20artist:43dc147b-a289-4e64-b115-6b86f9e47de9::nosim%20artist:6268d4d3-e46a-439e-9f4e-92f366305d26::nosim%20artist:2af0b211-04a8-4fa4-b81a-b9898f7274ba::nosim%20artist:75210e6c-96d1-445e-ac95-6fa06c09e9d5::nosim%20artist:8ebd161e-f45e-41b9-8019-fcbd094c327f::nosim%20artist:b879af63
      • ff6-49b3-a556-1af86e753119::nosim%20artist:df80ae94-1738-403a-9385-f0a1f1546a48::nosim%20artist:d91b1a0e-8276-471c-99fd-3e162f5b6012::nosim%20artist:d10daa36-2392-476b-b3ce-f843bb4aeff4::nosim%20artist:0aa7e8fd-ba08-4a2d-93fc-91baf635072d::nosim%20artist:57cece42-3171-4c34-8e8b-83cbdba1d2cc::nosim%20artist:330f31eb-820e-4b60-b1e2-0a713baabe6c::nosim%20artist:6cd76c7a-757f-4bba-b88c-32b138a506a7::nosim%20artist:fc38b6c6-860e-4
      • 8-adf9-0e951d0bee77::nosim%20artist:7a9e07cd-33bf-44b6-9479-884561fcc97a::nosim%20artist:b53d1842-a95d-43cf-a106-3048da304d00::nosim&mode=easy
      • Oh sorry, that link was too much for irc.
      • mayhem
        why are you using so many seed artists?
      • the whole point of LB radio is to give one seed artist and it finds similar artists for you.
      • what is your use case?
      • and by default LB radio is supposed to output 50 tracks.
      • relaxoMob
      • Creating a Festival Playlist to discover the performing artists to decide which ones I want to see live.
      • Zhele joined the channel
      • ApeKattQuest
        🐟
      • prout has quit
      • prout joined the channel
      • mayhem
        I wonder is reosarevok is *still* putting together idea furniture
      • kellnerd
        Probably lost the instruction sheet :D
      • ApeKattQuest
        you can dl those online nowl, I think
      • monkey[m]
        Wondering where that extra screw goes
      • mayhem
        relaxoMob: the lb radio is not the right tool
      • monkey[m] also has a screw loose
      • mayhem has extra screws
      • ApeKattQuest
        reminds me of wanting to get extra ones for somehing ad hey sent those and more
      • there was a (one) missing scrw
      • and they sent double, but extra and did so the ext day with a currier
      • for free
      • theflash__ joined the channel
      • that was aazing kundeservice
      • amazing*
      • mayhem
        <BANG>
      • lets get this meeting started.
      • ApeKattQuest: go!
      • ApeKattQuest
        Arararara~! hi!
      • Finished finally the dūdmaišis and bock/dudy! (INST-1080)
      • BrainzBot
      • ApeKattQuest
        Also did INST-1081 and INST-1083
      • BrainzBot
      • aerozol[m] joined the channel
      • TOPIC: MetaBrainz Community and Development channel | MusicBrainz non-development: #musicbrainz | BookBrainz: #bookbrainz | Channel is logged and not empty as it is bridged to IRC; see https://musicbrainz.org/doc/IRC for details | Agenda: Reviews
      • ApeKattQuest
        Tis week is easter and I'll be taking a break for a while. Back week after that!
      • next can be aerozol !
      • aerozol[m]
        someone else first please :) go mayhem?
      • mayhem
        ok, np.
      • I didn't do a whole lot in the last week, since I am traveling.
      • just the usual background emails and helping with random questions.
      • has anyone found me yet?
      • ApeKattQuest
        you are here
      • :)
      • C:
      • mayhem
        I've been posting clues to IG/FB, in case you were wondering.
      • fin.
      • monkey[m]: go
      • monkey[m]
        Hi!
      • mayhem
        people still up: reosarevok, bitmap, zas, mayhem, yvanzo, monkey, lucifer, akshaaatt, atj, aerozol, ansh, pranav, rimskii, theflash__, Tarun_0x0
      • monkey[m]
        Last week was pretty similar to the last, with lots of PR revies, ticket triage, continuing work on the single-page-app with ansh
      • and reviewing GSOC proposals and such
      • mayhem
      • monkey[m]
        That is me! rimskii go !
      • ApeKattQuest
        poosibly our indians are afk today
      • monkey[m]
        Or if rimskii is not available, go pranav
      • mayhem
        yvanzo: ceiling is higher than ever.
      • reosarevok
        mayhem: I was making dinner and forgot to set an alarm. Can take it from here
      • pranav[m] joined the channel
      • pranav[m]
        Hi everyone!
      • mayhem
        ok, slacker!
      • ApeKattQuest
        reosarevok: <3
      • pranav[m]
        Last week, I worked on overview screens for BrainzPlayer revamp .. those are almost done. I have opened a PR. Turns out I messed up a part of it and have rectified that. After review, I’ll work on album pages
      • That’s it for me go theflash__ (IRC) ?
      • theflash__
        hey
      • reosarevok
        people still up: reosarevok, bitmap, zas, mayhem, yvanzo, lucifer, akshaaatt, atj, aerozol, ansh, rimskii, Tarun_0x0
      • theflash__
        last week I explored new state management, SwiftData for making models