#metabrainz

/

      • Mineo has quit
      • 2021-12-09 34350, 2021

      • MRiddickW joined the channel
      • 2021-12-09 34352, 2021

      • rdswift
        CatQuest, it should now be Thursday in your time zone, so let me be one of the first to wish you a Happy Birthday!
      • 2021-12-09 34345, 2021

      • gcrkrause has quit
      • 2021-12-09 34333, 2021

      • gcrkrause joined the channel
      • 2021-12-09 34325, 2021

      • MRiddickW has quit
      • 2021-12-09 34340, 2021

      • MRiddickW joined the channel
      • 2021-12-09 34324, 2021

      • kinduff6 joined the channel
      • 2021-12-09 34352, 2021

      • kinduff has quit
      • 2021-12-09 34352, 2021

      • kinduff6 is now known as kinduff
      • 2021-12-09 34347, 2021

      • MrClon has quit
      • 2021-12-09 34316, 2021

      • CatQuest
        rdswift: thanks!
      • 2021-12-09 34329, 2021

      • CatQuest
        :D
      • 2021-12-09 34353, 2021

      • MRiddickW has quit
      • 2021-12-09 34320, 2021

      • reosarevok
        🍰!
      • 2021-12-09 34337, 2021

      • reosarevok
        I assume you already got the MB cakes, but still :D
      • 2021-12-09 34314, 2021

      • reosarevok
        goldenshimmer, mayhem: of course, if we *do know* the format, we could still add it :)
      • 2021-12-09 34320, 2021

      • reosarevok
        (assuming that's what this was about)
      • 2021-12-09 34320, 2021

      • mayhem
        moooin!
      • 2021-12-09 34324, 2021

      • Arsen has quit
      • 2021-12-09 34320, 2021

      • Arsen joined the channel
      • 2021-12-09 34339, 2021

      • BrainzGit
        [listenbrainz-server] 14amCap1712 opened pull request #1776 (03master…validate-listen-): LB-1035: 500 error when sending an array to /1/submit-listens https://github.com/metabrainz/listenbrainz-server…
      • 2021-12-09 34338, 2021

      • mayhem
        monkey: I dont remember what you and i were looking at yesterday, but the release_mbid in playlists is actually not trivial. a few hours of work, but not trivial. which means I can't really dig until the playlists are done.
      • 2021-12-09 34300, 2021

      • lucifer
        morning
      • 2021-12-09 34306, 2021

      • lucifer
        lol this non-dict breaks the code to jsonify the exception too `TypeError: cannot convert dictionary update sequence element #0 to a sequence`
      • 2021-12-09 34346, 2021

      • nbin
        lucifer: those listens are still hanging around, so my account must be the edgiest of edge cases
      • 2021-12-09 34308, 2021

      • lucifer
        nbin: not an edge case but i had discovered another bug in listen counts yesterday evening :(. we are going to rework listen counts so that these bugs go away, listen counts are flaky till then.
      • 2021-12-09 34331, 2021

      • nbin
        no worries. if i add listens in the meantime i assume the count will eventually reflect properly?
      • 2021-12-09 34352, 2021

      • lucifer
        yes, when the counts are recalculated the correct one will be reflected.
      • 2021-12-09 34307, 2021

      • nbin
        that's all that really matters then. just figured since i was "starting fresh" that i should probably try to start from 0
      • 2021-12-09 34325, 2021

      • nbin
        maybe i'll use the "-1 when deleting an invalid listen" bug to get there :P
      • 2021-12-09 34345, 2021

      • CatQuest has quit
      • 2021-12-09 34314, 2021

      • CatQuest joined the channel
      • 2021-12-09 34322, 2021

      • monkey
        mayhem: Understood. So not a "one line change" then :p
      • 2021-12-09 34343, 2021

      • mayhem
        more like a weekend project.
      • 2021-12-09 34345, 2021

      • mayhem
        well not quite.
      • 2021-12-09 34350, 2021

      • monkey
        So about a week?
      • 2021-12-09 34356, 2021

      • mayhem
        lol
      • 2021-12-09 34310, 2021

      • mayhem
        I just need to make a new labs api endpoint for this.
      • 2021-12-09 34316, 2021

      • monkey
        Joyeux anniversaire CatQuest !
      • 2021-12-09 34349, 2021

      • mayhem
        and in the process I also figured out how to make canonical recordings, which is something that MB and BBC has desired over the years.
      • 2021-12-09 34320, 2021

      • mayhem
        effectively, if you roll in with a recording_mbid, it will look up the canonical one and return it and the associated metadata.
      • 2021-12-09 34326, 2021

      • mayhem
        including release.
      • 2021-12-09 34354, 2021

      • BrainzGit
        [listenbrainz-server] 14amCap1712 merged pull request #1774 (03master…LB-1033-fix_doc_comment): LB-1033: Fix doc comment for listenbrainz.webserver.views.api_tools.MAX_LISTEN_SIZE https://github.com/metabrainz/listenbrainz-server…
      • 2021-12-09 34329, 2021

      • MrClon joined the channel
      • 2021-12-09 34306, 2021

      • BrainzGit
        [listenbrainz-server] 14amCap1712 merged pull request #1776 (03master…validate-listen-): LB-1035: 500 error when sending an array to /1/submit-listens https://github.com/metabrainz/listenbrainz-server…
      • 2021-12-09 34346, 2021

      • akshaaatt
        Happy Birthday CatQuest ! 🎉
      • 2021-12-09 34346, 2021

      • monkey
        I managed to make that graph a bit better, thanks for the feedback riksucks and tandy1000 https://usercontent.irccloud-cdn.com/file/xXJYsI7…
      • 2021-12-09 34327, 2021

      • mayhem
        akshaaatt monkey alastairp lucifer tandy1000 reosarevok : if I have made you a playlist (via troi-bot) in the past couple of weeks and you want to save your list, clone it soon. I'm going to nuke all generated playlists soon.
      • 2021-12-09 34345, 2021

      • mayhem
        !m monkey
      • 2021-12-09 34345, 2021

      • BrainzBot
        You're doing good work, monkey!
      • 2021-12-09 34348, 2021

      • monkey
        Thanks for the heads up
      • 2021-12-09 34307, 2021

      • monkey
        Definitely want to save one of those playlists
      • 2021-12-09 34329, 2021

      • akshaaatt
        Already done ruaok 👌
      • 2021-12-09 34333, 2021

      • mayhem
        yeah, I'm saving my missed tracks one.
      • 2021-12-09 34338, 2021

      • akshaaatt
        mayhem!
      • 2021-12-09 34352, 2021

      • mayhem
        :D
      • 2021-12-09 34354, 2021

      • lucifer
        mayhem: 👍
      • 2021-12-09 34339, 2021

      • alastairp
        python logging is so confusing
      • 2021-12-09 34331, 2021

      • mayhem
        alastairp: agreed. one of my least favorite parts of python
      • 2021-12-09 34332, 2021

      • alastairp
        and it's so opaque in so many places, you have no idea _what_ is doing the logging
      • 2021-12-09 34353, 2021

      • alastairp
        I think I've finally got the double-logging stuff fixed in both dev and prod
      • 2021-12-09 34338, 2021

      • alastairp
        unfortunately, I now think my recommendation from earlier in the year to use `current_app.logger` everywhere was a stupid move and doesn't make sense, so got to put that on the future todo list again
      • 2021-12-09 34338, 2021

      • MrClon has quit
      • 2021-12-09 34359, 2021

      • mayhem
        why so?
      • 2021-12-09 34308, 2021

      • MrClon joined the channel
      • 2021-12-09 34330, 2021

      • alastairp
        flask has a single logger, named the thing that you pass in to the `Flask()` constructor. in our case this is `listenbrainz.webserver`. We print the name of the logger in log messages:
      • 2021-12-09 34331, 2021

      • alastairp
        > spotify_reader_1 | 2021-12-09 12:12:19,994 listenbrainz.webserver INFO Processed 1 users successfully!
      • 2021-12-09 34313, 2021

      • alastairp
        this means that a log message coming from `listebrainz/spotify_updated/spotify_read_listens.py` still has a name of `listenbrainz.webserver`
      • 2021-12-09 34331, 2021

      • alastairp
        not the end of the world, but makes it a bit more difficult to work out where the message comes from
      • 2021-12-09 34355, 2021

      • mayhem
        clear as mud. :)
      • 2021-12-09 34324, 2021

      • alastairp
        I was just confused why _all_ of our log messages appeared to be coming from the webserver package
      • 2021-12-09 34306, 2021

      • atj
        outsidecontext: there is something weird going on here https://i.ibb.co/BwjkBZr/Screenshot-2021-12-09-12…
      • 2021-12-09 34302, 2021

      • atj
        Picard appears to be changing electronica to electronic. I just checked the tags and it's written electronic twice.
      • 2021-12-09 34331, 2021

      • outsidecontext
        atj: how have you configured genres? is this genres from MB (the built-in ones in Picard options), or any plugins?
      • 2021-12-09 34307, 2021

      • alastairp
      • 2021-12-09 34324, 2021

      • alastairp
        let me futz with genres a bit more, and then if I have time I'll get back to https://github.com/metabrainz/listenbrainz-server…
      • 2021-12-09 34341, 2021

      • alastairp
        nice
      • 2021-12-09 34319, 2021

      • reosarevok
        alastairp: btw, do we now have the CB review filter to get only text reviews?
      • 2021-12-09 34345, 2021

      • monkey
        When are you coming to the motherland reo? Planning on stopping by here?
      • 2021-12-09 34359, 2021

      • reosarevok
        Nah, Madrid / Asturias only this time
      • 2021-12-09 34313, 2021

      • monkey
        Makes sense. You'll be missed !
      • 2021-12-09 34336, 2021

      • reosarevok
        Next Autumn, maybe? :)
      • 2021-12-09 34354, 2021

      • reosarevok
        That's for seeing you, not for the alastairp code
      • 2021-12-09 34357, 2021

      • reosarevok
        (but maybe both :D )
      • 2021-12-09 34358, 2021

      • atj
        outsidecontext: those are genres from MB. Configuration: https://i.ibb.co/Q6X6rYX/Screenshot-2021-12-09-12…
      • 2021-12-09 34341, 2021

      • outsidecontext
        and what release is that?
      • 2021-12-09 34356, 2021

      • alastairp
        reosarevok: CB managed to find itself on the backburner again :( Maybe lucifer and I can do a hack week after YIM is finished or early in the new year to finish up all of the PRs (and the new LB mini review functionality!)
      • 2021-12-09 34356, 2021

      • atj
        no plugins installed except for a custom one I wrote to define some script functions for my folder name conventions
      • 2021-12-09 34309, 2021

      • atj
      • 2021-12-09 34311, 2021

      • atj
        sigh
      • 2021-12-09 34319, 2021

      • reosarevok
        alastairp: ok, no worries :)
      • 2021-12-09 34322, 2021

      • atj
      • 2021-12-09 34326, 2021

      • reosarevok
        (just wanted to make sure re: the MB PR)
      • 2021-12-09 34332, 2021

      • outsidecontext
        that's odd. does this only happen if you have files on that release, or does it show up the same when just loaded?
      • 2021-12-09 34329, 2021

      • atj
        I just loaded the release manually by entering the MB URL in the search box and it shows electronic twice
      • 2021-12-09 34338, 2021

      • atj
        I feel like this is too odd, so it must be something I've done :)
      • 2021-12-09 34302, 2021

      • outsidecontext
        can't reproduce this here. a script maybe?
      • 2021-12-09 34348, 2021

      • outsidecontext
        for me it shows only a single genre, "Electronic", which would be expected with the given settings (because of the minimal genre usage threshold)
      • 2021-12-09 34339, 2021

      • mayhem
        alastairp: when do you think you'll leave the office today? because the first conversation is really more a request than a long discussion...
      • 2021-12-09 34341, 2021

      • atj
        outsidecontext: I'll investigate further and let you know if I find a bug
      • 2021-12-09 34303, 2021

      • mayhem
        alastairp: looks like I might return closer to 3pmish.
      • 2021-12-09 34324, 2021

      • lucifer
        alastairp: 1765 lgtm but something in frontend tests needs poking. https://github.com/metabrainz/listenbrainz-server…
      • 2021-12-09 34351, 2021

      • alastairp
        mayhem: I think we'll do lunch and then I'll be around until 4ish, so should be fine to talk then
      • 2021-12-09 34346, 2021

      • alastairp
        lucifer: good catch, thanks. urg, this will be due to /code being owned by root. will have to rethink it
      • 2021-12-09 34303, 2021

      • alastairp
        maybe just preventing pyc files is a good enough fix for now and we can look at it later
      • 2021-12-09 34344, 2021

      • lucifer
        sure sounds good.
      • 2021-12-09 34320, 2021

      • lucifer
        alastairp: 1773 looks good but will probably need a change. i realised after approving that load_config should probably come before logger init.
      • 2021-12-09 34333, 2021

      • mayhem
        +1
      • 2021-12-09 34341, 2021

      • alastairp
        urgh. I thought that I put it there, I must have been moving it around while testing stuff
      • 2021-12-09 34337, 2021

      • alastairp
        ah, that's right. I removed DEBUG from my config file and was using the FLASK_ENVIRONMENT env var to set it
      • 2021-12-09 34343, 2021

      • alastairp
        so yeah - if for some reason we set FLASK_ENV=production, but config.DEBUG=True it wouldn't necessarily work as we expected it to. will change
      • 2021-12-09 34358, 2021

      • tandy1000
        <monkey> "I managed to make that graph a..." <- lookin good
      • 2021-12-09 34339, 2021

      • tandy1000
        <monkey> "I managed to make that graph a..." <- graph is looking good! what helped?
      • 2021-12-09 34354, 2021

      • monkey
        tandy1000: Removing values on the bars, and figuring out how to massage the data to get a better looking X axis
      • 2021-12-09 34325, 2021

      • monkey
      • 2021-12-09 34353, 2021

      • mayhem
        alastairp: ready to chat when you are.
      • 2021-12-09 34347, 2021

      • alastairp
        just debugging something
      • 2021-12-09 34316, 2021

      • rdswift
        mayhem, you might want to update your IRC nick on https://metabrainz.org/team
      • 2021-12-09 34327, 2021

      • mayhem
        I did the code change, but haven't deployed yet.
      • 2021-12-09 34320, 2021

      • rdswift
        👍
      • 2021-12-09 34332, 2021

      • alastairp
      • 2021-12-09 34337, 2021

      • reosarevok
        yvanzo, bitmap: updated https://github.com/metabrainz/musicbrainz-server/… with the suggestion to allow reuse, would be good to get this ran this year :) Maybe next week even?
      • 2021-12-09 34357, 2021

      • dogga joined the channel
      • 2021-12-09 34304, 2021

      • mayhem
        lucifer: I just tried to deploy metabrainz.org and I am getting:
      • 2021-12-09 34306, 2021

      • mayhem
      • 2021-12-09 34322, 2021

      • mayhem
        anything you've encountered before?
      • 2021-12-09 34329, 2021

      • monkey
      • 2021-12-09 34313, 2021

      • monkey
      • 2021-12-09 34314, 2021

      • lucifer
        mayhem: haven't seen myself but looks like https://stackoverflow.com/questions/70068407/modu…
      • 2021-12-09 34357, 2021

      • mayhem
        I'll go deal with this in the new year. too much YIM left to be YAMmering about with this.
      • 2021-12-09 34359, 2021

      • lucifer
        quick look at https://github.com/metabrainz/metabrainz.org/blob… don't see us pinning the version so unless a dep show does. possible it pulled in a latest version which changed some imports
      • 2021-12-09 34316, 2021

      • lucifer
        if you want, i can look into it later today or tomorrow?
      • 2021-12-09 34336, 2021

      • mayhem
        sure, if you feel inclined.
      • 2021-12-09 34341, 2021

      • lucifer
        👍
      • 2021-12-09 34351, 2021

      • lucifer
        also, let me know if i can help with the left yim stuff
      • 2021-12-09 34303, 2021

      • mayhem
        k, not sure if there is a chunk that can be broken out. I only have submitting playlist excerpts to LB left.
      • 2021-12-09 34353, 2021

      • alastairp
      • 2021-12-09 34322, 2021

      • lucifer
        what's the plan? troi dumps a csv, we import the csv in LB with a script?
      • 2021-12-09 34312, 2021

      • mayhem
        I dislike the repeatability of this plan, so I am going to add a troi element that will query LB directly to fetch a list of users who do not have YIM playlists for a given troi patch. and the end of the pipeline will be a submitter element which executes an UPDATE statement on LB to insert the JSPF except into the DB. should the process get interrupted, I just start it over again.
      • 2021-12-09 34350, 2021

      • alastairp
        monkey: ` "test:ci": "JEST_JUNIT_OUTPUT_DIR=/tmp TZ='Europe/London' jest --ci --reporters=default --reporters=jest-junit",`
      • 2021-12-09 34355, 2021

      • mayhem
        that begs the question -- in which order do we process these things?
      • 2021-12-09 34329, 2021

      • mayhem
        if I do an UPDATE, then I am assuming that most of the data already exists and that I just need to add one key value pair.
      • 2021-12-09 34357, 2021

      • lucifer
        order of processing as in order of other YIM stats and playlists?
      • 2021-12-09 34308, 2021

      • mayhem
        yes
      • 2021-12-09 34311, 2021

      • lucifer
        to confirm, troi will create a playlist for the user and then add a link of it to YIM table with a specifc key?
      • 2021-12-09 34311, 2021

      • mayhem
        I expect that generating playlists might take as long as a day.
      • 2021-12-09 34339, 2021

      • mayhem
        lucifer: yes, to be specfic, monkey requested that I upload a 5 track JSPF except file to LB.
      • 2021-12-09 34349, 2021

      • mayhem
        excerpt
      • 2021-12-09 34338, 2021

      • lucifer
        i see. i would say we execute both together, and let INSERT ON CONFLICT DO UPDATE do its job. all the stats jobs are written that way.