#metabrainz

/

      • ruaok
        reosarevok: agreed, but we're hoping to get there too
      • reosarevok
        If you want to do what we do, you should release every 2 weeks or 1 mo even if it's just one small patch :)
      • ruaok
        1 mo is what I would like.
      • reosarevok
        (tiny improvements > no improvements anyway!)
      • iliekcomputers
        reosarevok: how do you deal with incomplete stuff? Just not merge it?
      • Like stuff thats basically done but has small things that need to be fixed
      • ruaok
        or "get this merged, so this PR can go away. smaller fixes PR coming very soon."
      • reosarevok
        Usually not merge it until it's fixed, but depends on what "fixed" means
      • iliekcomputers
        Yeah, I think that makes sense to me.
      • reosarevok
        Is it "this is an improvement but we could improve it even more with X"? Merge, send second PR
      • ruaok
        hmmm. maybe we should approve PRs and then make a new branch off the approved branch for the fixes PR
      • reosarevok
        Is it "this is kind of a mess but we can improve it"? Then fix before merge :p
      • ruaok
        but not merge into master until follow up PR is complete.
      • iliekcomputers
        Let's merge stuff to master only when we're completely ready to release it next cycle.
      • ruaok
        which is what I am trying with the lb-labs tests with pristine__ right now
      • agreed.
      • iliekcomputers
        And aim for monthly releases?
      • ruaok
        oh, that brings about a natural next question: what is the least amount of stuff we need to do in order do do a release?
      • that should inform the goal of our next release.
      • iliekcomputers
        I think having a schedule and not caring about the amount of work done is a better way to go here.
      • ruaok
        I dont think the two are mutually exclusive.
      • iliekcomputers
        Keeping goals independent of releases.
      • ruaok
        my point is more "lets not start more stuff, but push a release soon"
      • iliekcomputers
        That I agree with
      • ruaok
        that was my main point there.
      • iliekcomputers
        Hmm.
      • ruaok
        but lets just agree on the "if it aint release for release, do not merge to master" right now, shall we?
      • iliekcomputers
        +1
      • ruaok
        then we work towards the first release with the goal of soon thereafter doing monthly releases.
      • iliekcomputers
        Yep, sounds like a plan.
      • ruaok
        which also means that 1 release = 2 work periods.
      • nice cadence, I think.
      • iliekcomputers
        So right now, I have two things incomplete.
      • Stats and Incremental dumps
      • ruaok ponders priorities
      • Incremental dumps is a cron job that we can just stop until we get around to fixing it.
      • So it isn't exactly blocking
      • ruaok
        what is the problem with the incremental dumps?
      • how broken are they?
      • iliekcomputers
        There were some listens missing/more in a dump.
      • ruaok
        yes. anything else?
      • iliekcomputers
        That's it.
      • ruaok
        I mean, this is a needle in a hackstack problem, but just tyring to get the scope.
      • iliekcomputers
        Yeah
      • I spent a few days looking at it
      • But didn't really find anything
      • ruaok
        let take another evening and have you talk me through the process.
      • iliekcomputers
        Plus it's a PITA to debug because it needs actual dumps which are created really slowly.
      • ruaok
        and I'll look at code as we go. I think a second set of eyes might make this a lot easier.
      • iliekcomputers
        Yeah, having a new set of eyes on it would be good.
      • ++
      • ruaok
        I hear that -- the MSID mapping has been frustrating because each full run takes 1-2 hours.
      • and there are a lot of little things to try -- at least now I can use partial data sets that run much faster.
      • ok, lets plan to do that walk through in the comings days.
      • user stats. lots of open items in the doc.
      • shall we make a Week of Dec 16 section and start looking for things to toss in?
      • iliekcomputers
        Or put in trello?
      • ruaok
        I see bugs, completing features and needed features as three categories.
      • my brainpower is dwindling TBH -- that flight home was rough.
      • let's make a new section and then tomorrow I'll move them to trello.
      • iliekcomputers
        sure, sounds good.
      • ruaok
        (50 screaming teenagers caused 20 babies to freak out, causing teenagers to scream louder because they couldn't hear each other)
      • I think I found a new worst case scenario for noise cancelling head-phones.
      • iliekcomputers
        if you want, we can just stop here. the process discussion was really helpful and I could easily prioritize work for these 2 weeks myself.
      • noise cancelling headphones are ⭐⭐
      • ruaok
        that would be nice, but it feels incomplete to me. shall we do more tomorrow?
      • and pick a time that is earlier?
      • noise cancelling headphones were only so effective!
      • SothoTalKer
        but what about noise as music genre?
      • iliekcomputers
        hmm. sure! we can pick it up tomorrow. would 7-ish work?
      • ruaok
        might be closer to 8 for me, but round then, yeah.
      • iliekcomputers
        that works better for me too, tbh. :D
      • thanks ruaok, the process discussion has been really helpful
      • ruaok
        for both of us!
      • oh, did a new volunteer pop in over the weekend?
      • if not, I'll follow up. did you see his email to support@?
      • iliekcomputers
        Don't think I did
      • sbvkrishna has quit
      • n1tr0maverick joined the channel
      • rahul24 joined the channel
      • BrainzGit
        [musicbrainz-server] reosarevok opened pull request #1331 (master…MBS-9919-pt-2): MBS-9919: Convert the place map tab to React https://github.com/metabrainz/musicbrainz-serve...
      • BrainzBot
        MBS-9919: Convert the place public pages to React https://tickets.metabrainz.org/browse/MBS-9919
      • n1tr0maverick has quit
      • rahul24 has quit
      • n1tr0maverick joined the channel
      • thomasross_ joined the channel
      • thomasross has quit
      • thomasross_ is now known as thomasross
      • yvanzo
        reosarevok: sure :)
      • thomasross has quit
      • rahul24 joined the channel
      • thomasross joined the channel
      • sarthak_jain has quit
      • rahul24 has quit