#metabrainz

/

      • pite has quit
      • Kladky joined the channel
      • lusciouslover has quit
      • lusciouslover joined the channel
      • davic has quit
      • lusciouslover has quit
      • lusciouslover joined the channel
      • lusciouslover has quit
      • lusciouslover joined the channel
      • BrainzGit has quit
      • _BrainzGit has quit
      • lusciouslover has quit
      • lusciouslover joined the channel
      • jesus2099 joined the channel
      • zas[m]
      • Services are back
      • BrainzGit joined the channel
      • _BrainzGit joined the channel
      • __BrainzGit joined the channel
      • Maxr1998_ joined the channel
      • Maxr1998 has quit
      • jesus2099 has quit
      • __BrainzGit
        [metabrainz.org] 14mayhem merged pull request #514 (03metabrainz-notifications…authorization-decorator): Add authentication to the notification endpoints. https://github.com/metabrainz/metabrainz.org/pu...
      • Jade[m] has quit
      • pite joined the channel
      • suvid[m]
        <_BrainzGit> "[listenbrainz-server] MonkeyDo..." <- Thanks monkey for creating the page for it :)
      • <lucifer[m]> "suvid: hi! let me know when..." <- Hi... (full message at <https://matrix.chatbrainz.org/_matrix/media/v3/...>)
      • lucifer[m]
        [@suvid:matrix.org](https://matrix.to/#/@suvid:matrix.org) we decided to not use the listens importer or external service oauth table for the imports. but a separate table like we do for user data exports.
      • suvid[m]
        oh thank god 😅
      • makes life a bit easier!?
      • lucifer[m]
        great, you should be able to copy most of the logic from exports and use it in import APIs.
      • suvid[m]
        yes that was indeed the original plan
      • * yes,, * and that was
      • lucifer[m]
        The only difference is that the API to trigger the import should be uploading the files
      • suvid[m]
        * yes,, * and that was, * original plan :)
      • lucifer[m]
        And you don't need an api to download the files.
      • suvid[m]
        i am wondering should i start from scratch 🤔
      • lucifer[m]
        your call.
      • suvid[m]
        just to be clear, we aren't using the listens_importer tablr now right?
      • lucifer[m]
        Nope
      • suvid[m]
        and we're making the new imports table right?
      • lucifer[m]
        Yes
      • suvid[m]
        👍️
      • ok lemme think on how to go about it and will let you know in a few hours
      • gotta see if i should start from scratch 😅
      • rayyan_seliya123
        hey lucifer while testing my updated indexer aligining with your recent suggestions i encountered an issue that i am unable to get directly the metatdata for `artist` and `album ` they’re only present inside the `HTML` in the `description` field Should I implement a helper function like `extract_from_description` to parse these values from the description `HTML` as a fallback, or do you prefer that we only index
      • what’s available in the top-level metadata?
      • ```txt
      • lucifer[m]
        [@rayyan_seliya123:matrix.org](https://matrix.to/#/@rayyan_seliya123:matrix.org) can you share the item id or url with me
      • rayyan_seliya123 sent a txt code block: https://matrix.chatbrainz.org/_matrix/media/v3/download/chatbrainz.org/wUWJqhuPAMwkZgwLpAeSSASX
      • rayyan_seliya123
        see this
      • for some different recording i got the data as like this see the gist https://gist.github.com/RayyanSeliya/89ba8bf48e...
      • lucifer[m]
        rayyan_seliya123: i see okay, lets try to use the metadata if available directly otherwise try to extract from the html description
      • let's use beautifulsoup or some other html parsing for this task though instead of hand written regex.
      • rayyan_seliya123
        lucifer[m]: yeah for sure i will use this and check again !!
      • reosarevok[m]
        yvanzo, lucifer: any idea if there's any good reason why area and label index `F("sortname", "aliases.sort_name")` in `sir`?
      • (or, alternatively, why other entities do not)
      • I think I'll drop it from my genre impl unless there's any reason we do want it
      • lucifer[m]
        no idea.
      • holycow23[m] has quit
      • __BrainzGit
        [metabrainz.org] 14mayhem merged pull request #516 (03master…dependabot/pip/urllib3-2.5.0): Bump urllib3 from 2.4.0 to 2.5.0 https://github.com/metabrainz/metabrainz.org/pu...