#metabrainz

/

      • alastairp
        C, Gama, and Preprocessing all need an independent reset button
      • 2020-06-25 17714, 2020

      • alastairp
        button addon looks good
      • 2020-06-25 17734, 2020

      • Mr_Monkey
        And for the checkboxes, I guess another list item that's a "reset" button
      • 2020-06-25 17714, 2020

      • alastairp
        it'd be nice to put it to the right of 'basic', directly under where the reset button for gamma will be. not sure
      • 2020-06-25 17736, 2020

      • alastairp
        maybe I can do x cols for gamma, and x-1 cols for preprocessing, and an extra col for reset
      • 2020-06-25 17704, 2020

      • Mr_Monkey
        You can try the class `pull-right` on your button. If you're lucky and sacrifice three live flies to the CSS gods it might align to the top-right of the row that contains the preprocessing values section.
      • 2020-06-25 17715, 2020

      • Mr_Monkey
        If not, add more flies.
      • 2020-06-25 17727, 2020

      • alastairp
        yeah, I've got experience doing that
      • 2020-06-25 17730, 2020

      • alastairp
        thanks!
      • 2020-06-25 17702, 2020

      • alastairp
        I think you're right, some things here might be much easier to do in react. I haven't even asked you about hiding fieldsets and expanding them when a button is clicked ;)
      • 2020-06-25 17757, 2020

      • Mr_Monkey
      • 2020-06-25 17707, 2020

      • alastairp
        exactly like that!
      • 2020-06-25 17734, 2020

      • alastairp looks sideways at bootstrap 2
      • 2020-06-25 17742, 2020

      • Mr_Monkey
        Oh frameworks, how much CSS would we actually have to do without you?
      • 2020-06-25 17738, 2020

      • prabal
        Mr_Monkey: which do you prefer React Class or function?
      • 2020-06-25 17755, 2020

      • Mr_Monkey
        If you don't need a state, functional class is better
      • 2020-06-25 17714, 2020

      • Mr_Monkey
        er… functional component*
      • 2020-06-25 17755, 2020

      • prabal
        :P
      • 2020-06-25 17757, 2020

      • prabal
      • 2020-06-25 17702, 2020

      • prabal
        I am converting this to class
      • 2020-06-25 17713, 2020

      • prabal
        i need to use state here
      • 2020-06-25 17718, 2020

      • prabal
        is that alright?
      • 2020-06-25 17733, 2020

      • Mr_Monkey
        If it's needed there, then no problem
      • 2020-06-25 17755, 2020

      • prabal
        okay thanks!
      • 2020-06-25 17758, 2020

      • prabal
        (Y)
      • 2020-06-25 17700, 2020

      • prabal
        (y)
      • 2020-06-25 17702, 2020

      • prabal
        huh?
      • 2020-06-25 17706, 2020

      • Mr_Monkey
        The general advice is: if you don't need state or lifecycle events, prefr functional components. If not, class it is
      • 2020-06-25 17709, 2020

      • prabal
        👍
      • 2020-06-25 17730, 2020

      • prabal
        okayy
      • 2020-06-25 17744, 2020

      • Mr_Monkey
        (I'll admit my opinion is only based on reading some articles, but…)
      • 2020-06-25 17702, 2020

      • Mr_Monkey
      • 2020-06-25 17713, 2020

      • Mr_Monkey
        prabal: Why do you need the state specifically in the footer?
      • 2020-06-25 17719, 2020

      • Mr_Monkey
        Out of curiosity
      • 2020-06-25 17721, 2020

      • prabal
        add to collection
      • 2020-06-25 17726, 2020

      • prabal
        modal
      • 2020-06-25 17738, 2020

      • Mr_Monkey
        Right, makes sense.
      • 2020-06-25 17714, 2020

      • prabal
        Yepp
      • 2020-06-25 17712, 2020

      • alastairp
        Mr_Monkey: what are your thoughts on using a checkbox as the way of toggling a collapsable block?
      • 2020-06-25 17726, 2020

      • alastairp
        the idea is that you'd have a box which says [ ] use advanced features
      • 2020-06-25 17732, 2020

      • alastairp
        checking it will show the advanced features
      • 2020-06-25 17724, 2020

      • Mr_Monkey
        alastairp: I'd say if your form has chekboxes for actual form info and you just want to show/hide some elements, checkbox isn't a good choice.
      • 2020-06-25 17712, 2020

      • Mr_Monkey
        There are toggles (checkboxes with fancy CSS), buttons, button links and I'm sure other solutions that could work
      • 2020-06-25 17703, 2020

      • iliekcomputers
        pristine___: hey, i ran your code, i think there are a couple of bugs, which is why nothing got written to the db
      • 2020-06-25 17705, 2020

      • alastairp
      • 2020-06-25 17712, 2020

      • alastairp
        Mr_Monkey: this is what I was thinking
      • 2020-06-25 17738, 2020

      • ruaok
        pristine___: but loads of notification emails being sent. 🤣
      • 2020-06-25 17745, 2020

      • Mr_Monkey
        Yeah, I think that's more confusing that it needs to be
      • 2020-06-25 17748, 2020

      • alastairp
        the "nice" thing about this is that I'd have the values of these checkboxes on the server, so it makes validation nice
      • 2020-06-25 17709, 2020

      • Mr_Monkey
        Ah, meaning if they are checked you know to use the defaults?
      • 2020-06-25 17723, 2020

      • alastairp
        yeah, the reverse. if they're unchecked I'll use defaults
      • 2020-06-25 17727, 2020

      • alastairp
        if checked, use whatever is set
      • 2020-06-25 17735, 2020

      • Mr_Monkey
        Er, right.
      • 2020-06-25 17744, 2020

      • alastairp
        so, by default it'll look like this:
      • 2020-06-25 17701, 2020

      • alastairp
      • 2020-06-25 17706, 2020

      • alastairp
        and you can just hit evaluate if you want
      • 2020-06-25 17727, 2020

      • Mr_Monkey
        Right. In this case, they're part of the form, so checkboxes make sense
      • 2020-06-25 17729, 2020

      • alastairp
        if you think there's a visual way of making it look better, we can definitely talk about that
      • 2020-06-25 17745, 2020

      • alastairp
        if that means representing these 2 checkboxes in a different way, fine
      • 2020-06-25 17720, 2020

      • ruaok
        iliekcomputers: you still about?
      • 2020-06-25 17734, 2020

      • iliekcomputers
        ruaok: yea
      • 2020-06-25 17753, 2020

      • iliekcomputers
        disclaimer: might be a little braindead
      • 2020-06-25 17701, 2020

      • ruaok
        I've received a few hunded "recommendations being written into DB" emails.
      • 2020-06-25 17713, 2020

      • iliekcomputers
        Huh
      • 2020-06-25 17722, 2020

      • ruaok
        and its still going.
      • 2020-06-25 17732, 2020

      • iliekcomputers
        Whoops
      • 2020-06-25 17740, 2020

      • ruaok
        since I've pinged you I've gotten another 60 or so.
      • 2020-06-25 17745, 2020

      • iliekcomputers
        That's the recommendation job, sorry about that.
      • 2020-06-25 17704, 2020

      • iliekcomputers
        Looks like it's not handling emails correctly
      • 2020-06-25 17706, 2020

      • ruaok
        I'll make a filter for now.
      • 2020-06-25 17729, 2020

      • iliekcomputers
        I'll open a ticket for pristine___ , that shouldn't happen.
      • 2020-06-25 17753, 2020

      • iliekcomputers
        Sorry about that, I haven't really read this part of the code, didn't know it was sending an email per user...
      • 2020-06-25 17723, 2020

      • ruaok
        yeah, I didn't catch that either.
      • 2020-06-25 17724, 2020

      • iliekcomputers
        I think it's a bug, the table has nothing.
      • 2020-06-25 17736, 2020

      • iliekcomputers
        🤷🏽‍♂️
      • 2020-06-25 17744, 2020

      • iliekcomputers
        I'll open a few tickets
      • 2020-06-25 17754, 2020

      • iliekcomputers
        And purge the queuw
      • 2020-06-25 17702, 2020

      • ruaok
        th
      • 2020-06-25 17703, 2020

      • ruaok
        x
      • 2020-06-25 17722, 2020

      • ruaok
        now that my brain fog has lifted, I'm finally going to play with recommendation stuff.
      • 2020-06-25 17732, 2020

      • iliekcomputers
        Queue has been purged
      • 2020-06-25 17749, 2020

      • iliekcomputers
        Cool!
      • 2020-06-25 17711, 2020

      • iliekcomputers
        I figure pristine___'s work should have an API endpoint on prod soon
      • 2020-06-25 17722, 2020

      • iliekcomputers
        Once the bugs are ironed out
      • 2020-06-25 17751, 2020

      • ruaok
        yeah, I've got several more support objects to write.
      • 2020-06-25 17708, 2020

      • ruaok
        tonights goal is to fetch some user stats and do the MSID->MBID translation
      • 2020-06-25 17726, 2020

      • iliekcomputers
        The user stats are in the latest dump btw
      • 2020-06-25 17735, 2020

      • iliekcomputers
        If you don't wanna go through the API endpoint
      • 2020-06-25 17753, 2020

      • ruaok
        I specifically want to go through the API endpoint.
      • 2020-06-25 17738, 2020

      • ruaok
        I'm hoping to make everything work with APIs, so that devs can download one simple python program and its deps and get rolling
      • 2020-06-25 17715, 2020

      • iliekcomputers
        Right, makes sense
      • 2020-06-25 17712, 2020

      • iliekcomputers
      • 2020-06-25 17713, 2020

      • BrainzBot
        LB-645: Recommendation pipeline bugs
      • 2020-06-25 17717, 2020

      • iliekcomputers
      • 2020-06-25 17718, 2020

      • BrainzBot
        LB-646: the recommended recordings endpoint says /artist
      • 2020-06-25 17744, 2020

      • iliekcomputers
      • 2020-06-25 17745, 2020

      • BrainzBot
        LB-648: Allow ability to ask for generation of recommendations for list of users
      • 2020-06-25 17735, 2020

      • iliekcomputers
        The sentry error came through for the recommendations stuff: https://sentry.metabrainz.org/metabrainz/listenbr…
      • 2020-06-25 17731, 2020

      • iliekcomputers
        huh looking more into it, this is probably me not doing the schema change correctly
      • 2020-06-25 17703, 2020

      • travis-ci joined the channel
      • 2020-06-25 17703, 2020

      • travis-ci
        Project bookbrainz-site build #3168: failed in 3 min 42 sec: https://travis-ci.org/bookbrainz/bookbrainz-site/…
      • 2020-06-25 17703, 2020

      • travis-ci has left the channel
      • 2020-06-25 17739, 2020

      • iliekcomputers
        ruaok: looks like it was my fault, i've disabled emails and run the job again
      • 2020-06-25 17757, 2020

      • ruaok
        thanks for chasing that down.
      • 2020-06-25 17722, 2020

      • travis-ci joined the channel
      • 2020-06-25 17722, 2020

      • travis-ci
        Project bookbrainz-site build #3170: passed in 3 min 40 sec: https://travis-ci.org/bookbrainz/bookbrainz-site/…
      • 2020-06-25 17722, 2020

      • travis-ci has left the channel
      • 2020-06-25 17727, 2020

      • ruaok
        we should some sort of an easter egg here: https://api.listenbrainz.org/1/stats/user/(user_n…
      • 2020-06-25 17739, 2020

      • ruaok
        for people who copy URLs and don't fully fix them.
      • 2020-06-25 17752, 2020

      • ruaok is suggesting this for a friend, of course
      • 2020-06-25 17732, 2020

      • iliekcomputers
        heh
      • 2020-06-25 17750, 2020

      • ruaok
        > count (int) – Optional, number of recordings to return, Default: DEFAULT_ITEMS_PER_GET Max: MAX_ITEMS_PER_GET
      • 2020-06-25 17734, 2020

      • ruaok
        taken from the stats docs. I don't see where the actual values of these are defined. it would be nice that for user's sake we listed the actual values in this place.
      • 2020-06-25 17734, 2020

      • iliekcomputers
        this reminds me
      • 2020-06-25 17747, 2020

      • iliekcomputers
      • 2020-06-25 17749, 2020

      • ruaok
        I'd use that right this second. :)
      • 2020-06-25 17710, 2020

      • iliekcomputers
        on my list for tomorrow
      • 2020-06-25 17716, 2020

      • ruaok
        does it have rate limiting support?
      • 2020-06-25 17727, 2020

      • ruaok
        if it does, I'll switch to it.
      • 2020-06-25 17733, 2020

      • iliekcomputers
        yes
      • 2020-06-25 17734, 2020

      • ruaok
        in a heartbeat
      • 2020-06-25 17756, 2020

      • ruaok
        great, then I won't put a lot of effort into this code and just slap it together for my needs tonight.
      • 2020-06-25 17707, 2020

      • iliekcomputers
        yes!
      • 2020-06-25 17712, 2020

      • iliekcomputers
        first user lol
      • 2020-06-25 17721, 2020

      • iliekcomputers
        #dogfooding
      • 2020-06-25 17723, 2020

      • ruaok
        hehehe
      • 2020-06-25 17745, 2020

      • iliekcomputers
      • 2020-06-25 17710, 2020

      • ruaok
        WOWOWOWOW!
      • 2020-06-25 17750, 2020

      • ruaok
        non-unique list. :(
      • 2020-06-25 17707, 2020

      • iliekcomputers
        i only trained the models on last 60 days for now
      • 2020-06-25 17721, 2020

      • iliekcomputers
        i'll open a ticket
      • 2020-06-25 17730, 2020

      • ruaok
        thx.
      • 2020-06-25 17752, 2020

      • ruaok
        uhm, I'm going to make that playable next. that will be interesting.
      • 2020-06-25 17700, 2020

      • ruaok
        "next"
      • 2020-06-25 17712, 2020

      • iliekcomputers
        lol
      • 2020-06-25 17731, 2020

      • ruaok
        there are 15 next items. I'm sure you understand
      • 2020-06-25 17740, 2020

      • iliekcomputers
        i get it
      • 2020-06-25 17737, 2020

      • iliekcomputers
        pristine___: "0bf9fcd2-8a96-4860-bc4c-5343dac8e2ba"
      • 2020-06-25 17741, 2020

      • iliekcomputers
        ugh
      • 2020-06-25 17704, 2020

      • iliekcomputers
      • 2020-06-25 17720, 2020

      • iliekcomputers
        it works! i'll close the ticket i opened
      • 2020-06-25 17725, 2020

      • iliekcomputers
        (and open other tickets)
      • 2020-06-25 17724, 2020

      • iliekcomputers
      • 2020-06-25 17725, 2020

      • BrainzBot
        LB-649: collaborative filtering recommendations
      • 2020-06-25 17738, 2020

      • iliekcomputers
        let's chat about how to productionize this more when you get a chance
      • 2020-06-25 17753, 2020

      • ruaok
        BINGO!
      • 2020-06-25 17709, 2020

      • ruaok
        I've been waiting to cross off productionize for eons!!
      • 2020-06-25 17719, 2020

      • ruaok
        :D
      • 2020-06-25 17739, 2020

      • iliekcomputers
        my pylistenbrainz thingy is actually used by someone lol
      • 2020-06-25 17712, 2020

      • ruaok
        noice