#metabrainz

/

      • Maxr1998 joined the channel
      • Maxr1998_ has quit
      • akshaaatt suddenly wakes up after realising he randomly slept
      • akshaaatt
        If anyone is interested is my updates, I have just been reviewing so many gh PRs and making releases everywhere that gh will soon start to haunt me 😂
      • Love the work that pranav has been doing with yim in the app!!! It’s great to see someone come with such enthusiasm and own a project to completion :)
      • Thanks jasje for reviewing the PRs
      • gchound joined the channel
      • gchound has quit
      • minimal has quit
      • pranavkonidena_ joined the channel
      • Major_Lurker has quit
      • Major_Lurker joined the channel
      • lucifer
        aerozol: about logging, it is a per-channel basis setup, so yes we can get that done but it would require some effort to do it for all channels and keep it updated when channels are created or destroyed.
      • should be possible but just not simple.
      • about the [off] thing, we'll have to fork the bridge codebase and modify it. the downside is that we have an additional codebase to maintain, which i would prefer not to unless its an absolute requirement.
      • discordbrainz has quit
      • discordbrainz joined the channel
      • hi!
      • aerozol, MonkeyPython: i saw the comments about telegram later, i forked the bridge and have added support for [off]
      • relaxoMob has quit
      • relaxoMob joined the channel
      • aerozol
        lucifer: already!!! !m lucifer!
      • !m lucifer!
      • BrainzBot
        You're doing good work, lucifer!!
      • relaxoMob has quit
      • relaxoMob joined the channel
      • lusciouslover has quit
      • lusciouslover joined the channel
      • aerozol
        ansh: cool that you’re picking up the home pages! Let me know if anything’s giving you trouble, as always I am happy to adjust mockups if they are impractical
      • param
        hey
      • my pypi 2fa is lost
      • lemme see if I can recover the account easily
      • aerozol
        ooh bummer. Hopefully just your IRC account?
      • Just remembered, I have a question regarding fresh releases that I keep forgetting. Does it pull ‘For you’ from my entire history of listens, or just a certain timeframe? It’s quite empty, and it seems to not be entirely reliable in giving me new releases from artists I’ve listened to a lot
      • relaxoMob has quit
      • aabbi15 joined the channel
      • aabbi15
        hi!
      • I have been working on the BB-770 issue recently.
      • BrainzBot
        BB-770: "Stay on beta" style cookie https://tickets.metabrainz.org/browse/BB-770
      • tykling has quit
      • aabbi15
        I wanted to know is there any existing hyperlink on the bookbrainz site which redirects to beta mode or should i add it in the footer like with musicbrainz site
      • relaxoMob joined the channel
      • I am guessing adding it to the footer would be convinient right?
      • something which says "stay on beta" the same as musicbrainz
      • if monkey you could let me know that would be cool :)
      • abhis_ joined the channel
      • pranavkonidena_
        Hey akshaaatt after YIM, I would love to start contributing to ListenBrainz ios so that I get to learn swift as well hehe so is there an issue tracker or smtg for the same?
      • aabbi15 has quit
      • I’ll be posting the attributions and other changes by 8 today.. have two practical’s in b/w..
      • abhis_ has quit
      • gcrkrause joined the channel
      • tykling joined the channel
      • yvanzo
        Hi zas: Something is going wrong with mb-solr-7 atm, see the email I just forwarding. It doesn’t seem to affect MB search for now, thanks to the clustering :)
      • s/ing/ed/
      • I don’t think there is much we can do but asking Hetzner for more information if you think it is worth it and if you know which channel is best to use.
      • gcrkrause has quit
      • gcrkrause joined the channel
      • BrainzGit
        [musicbrainz-docker] 14yvanzo merged pull request #266 (03master…doc-file-perm-issues): Document file permissions issue https://github.com/metabrainz/musicbrainz-docke...
      • [musicbrainz-server] 14yvanzo merged pull request #3140 (03master…email_del_space): Fix deleted entity spacing in subscription email https://github.com/metabrainz/musicbrainz-serve...
      • zas
        yvanzo: links in emails lead to 404s, and there's no information about this issue in Robot interface (nor on Status page), I sent an email to Hetzner to obtain details and to notify the issue with the provided link. Everything seems back to normal on our side anyway.
      • yvanzo
        Thank you! I also noticed that the status wasn’t available for mb-solr-7 in Hetzner Console.
      • zas
        outsidecontext: check https://github.com/metabrainz/picard/actions/ru... (I enabled debug log)
      • yvanzo: Hetzner answered about 404 link (toggle language on 404 page to fix it.... it worked)
      • But not much info the outage itself
      • yvanzo
        zas: and mb-solr-7 is back to "Running" in Hetzner console :)
      • I’m able to SSH into it again, and the Solr logs look good.
      • outsidecontext
        zas: I think I authorized the wrong workflow file on PyPI. I'll update the config and try again
      • adhawkins
        Just updated my local mirror server to v2024-01-08, and the web server doesn't seem to be running. Getting a log of this in my logs: "musicbrainz-docker-musicbrainz-1 | 2024/01/09 10:46:23 Problem with dial: dial tcp: lookup mq on 127.0.0.11:53: server misbehaving. Sleeping 1s"
      • Any suggestions?
      • yvanzo
        Hi adhawkins, please check the status and logs of the service `mq`.
      • adhawkins
        Aha. "Failed to write pid file". Further investigation shows host disk is full. :)
      • yvanzo
        (`docker-compose ps mq` and `docker-compose logs --tail=42 mq`)
      • adhawkins
        And we're back. Sorry for the noise!
      • yvanzo
        adhawkins: `docker system prune` should free you some disk space.
      • adhawkins
        That's exactly what I did. Freed up about 16 Gig :)
      • yvanzo
        reosarevok: I pushed one more commit but it should be the last one. Failing tests are unrelated. I will rebase on master and apply amending commits once approved.
      • petitminion joined the channel
      • mayhem
        param: any luck with pypi?
      • param
        mayhem: they've asked me to create a github issue. I think it'll take time for them to follow up.
      • any chance we can just re-upload a new name on the meb account
      • gcrkrause
        This certainly works, but then we need to change the package name
      • mayhem
        I'm sure we can -- not sure what sort of ramifications that has.
      • eek, thats a bad ramification
      • MonkeyPython
        !m lucifer
      • BrainzBot
        You're doing good work, lucifer!
      • gcrkrause
        we could rename it to "listenbrainz"
      • pranavkonidena_ has quit
      • or "liblistenbrainz"
      • but yes, this is a rather huge change
      • mayhem
        the server uses pacakge name listenbrainz
      • it would have to be liblistenbrainz, but this is a silly reason in the grand scheme of things.
      • is a pypi release critical for now gcrkrause ?
      • gcrkrause
        Not exactly. This is just to solve https://github.com/metabrainz/troi-recommendati...
      • In the long run I'd appreciate a pypi package, but we have workarounds in place and can keep them for the time being
      • BrainzGit
        [troi-recommendation-playground] 14petitminion opened pull request #117 (03main…main): Update requirements.txt https://github.com/metabrainz/troi-recommendati...
      • gcrkrause
        param: May I ask what the issue is about pypi?
      • mayhem
        lost 2FA, I believe.
      • gcrkrause
        Oh, well, happens. I guess we can wait for pypi to resolve the issue
      • MonkeyPython
        lucifer: so, is telegram now piped to #musicbrainz? :o
      • lucifer: seems telegram and #musicbrainz-telegram are unlinked for sure
      • BrainzGit
        [troi-recommendation-playground] 14petitminion closed pull request #117 (03main…main): Update requirements.txt https://github.com/metabrainz/troi-recommendati...
      • [troi-recommendation-playground] 14petitminion opened pull request #118 (03main…patch-2): Update requirements.txt https://github.com/metabrainz/troi-recommendati...
      • gcrkrause
        outsidecontext Is there an issue for poetry in picard by any chance?
      • lucifer
        MonkeyPython: I didn't do anything about the telegram bridge, just the discord one.
      • I don't know where and how the telegram one is running
      • MonkeyPython
        hmm
      • its maintained by JWF
      • jwflory
      • i'll try to get a hold of him
      • and i'll ask him to tunr over the reins ot you. freso also had some reins
      • pranavkonidena_ joined the channel
      • petitminion has quit
      • Maxr1998
        lucifer: now that YIM seems to not keep you as busy anymore, would you have some time to look at https://github.com/metabrainz/listenbrainz-serv... And whether that change makes any sense in the first place? 😅
      • The long term idea is to reduce the storage requirements for spark development to make it more accessible and easier to use with your own data (because that's usually where you'd want to debug things, I believe).
      • minimal joined the channel
      • petitminion joined the channel
      • pranavkonidena_
        jasje: could u pls review my PR once.. thx
      • outsidecontext
        gcrkrause: not yet
      • but I started to experiment with converting the dependencies yesterday. it's WIP as I did not have the time to finish it. but I put my work up on https://github.com/phw/picard/commits/poetry/
      • gcrkrause
      • BrainzBot
        PICARD-2816: Migrate to poetry
      • gcrkrause
        Oh, you already started.
      • I'd suggest to get rid of setup.py as well
      • outsidecontext
        that will be more complicated. the setup.py is doing important build tasks like building the translations, desktop file, app manifest
      • the last time I checked this was not really replaceable without loosing the ability to build the package in a standardized way
      • petitminion has quit
      • but I don't also see no need to get rid of the setup.py
      • gcrkrause
        Alright, I wasn't aware there are complex tasks, I just recommend moving the metadata to pyproject.toml
      • outsidecontext
        that should be possible. a bit annoying is that the version then needs to be specified twice (in pyproject.toml and the source). with setup.py we could have it defined only once
      • gcrkrause
        It doesn't need to be
      • reading the version from pyproject.toml is trivial: https://dev.funkwhale.audio/funkwhale/funkwhale...
      • its quite nice to have native setuptools integration after all :)
      • outsidecontext
        ok, if that works for all deployments that would solve it
      • what I also noticed yesterday that running "poetry build" does not seem to run the setuptools build, even though setuptools is specified as the buildsystem in pyproject.toml
      • so you get different results when running "poetry build" and "python -m build"
      • reosarevok
        lucifer: ccd you in a LB support issue, do have a look when you have some time :)
      • outsidecontext
        gcrkrause: thanks for creating the story
      • overall I was mostly experimenting yesterday to see if it is feasible (which I think now it is). but we have quite a few places in the build scripts and github workflows that need to be adjusted.
      • gcrkrause
        I'd have to dive deeper into the builds to help there, its likely a difference between the pyproject.toml and setup.py
      • How do you determine there are differences?
      • outsidecontext
        the "python -m build" runs the gettext conversions and the resulting files contain the translations. "poetry build" does not
      • the resulting wheel is also significantly smaller
      • petitminion joined the channel
      • gcrkrause
        I am not fully aware how the translations are packaged, but generally speaking poetry isn't supposed to be a task runner and doesn't really support something like triggering other tasks before the build
      • the setup.py is indeed quite complex, this would be a fun to migrate
      • outsidecontext
        well, poetry would not be expected to run the tasks. setuptools would. I would have expected that poetry build runs the build system defined in pyproject.toml. but I think this was my wrong assumption, as poetry provides its own PEP 517 compatible build system and seems to run that and not the one defined in pyproject
      • I guess one should consider "poetry build" just as deprecated as "setup.py build"
      • gcrkrause
        Isn't python -m build also running PEP 517 builds?