#metabrainz

/

      • chirlu`
        Regarding Redis, it seems the script got the KEYS result alright.
      • 2015-12-22 35604, 2015

      • chirlu`
        I could then let it MGET/DEL the values in chunks, without needing SCAN.
      • 2015-12-22 35639, 2015

      • opatel99
        Any idea how to test a Flask redirect route? Could I just use assert301?
      • 2015-12-22 35647, 2015

      • Nyanko-sensei joined the channel
      • 2015-12-22 35632, 2015

      • D4RK-PH0ENiX has quit
      • 2015-12-22 35630, 2015

      • ariscop joined the channel
      • 2015-12-22 35601, 2015

      • JonnyJD_ has quit
      • 2015-12-22 35619, 2015

      • JonnyJD_ joined the channel
      • 2015-12-22 35605, 2015

      • darwin
        chirlu`: you're not supposed to use "keys" in production, or in general, fwiw
      • 2015-12-22 35610, 2015

      • darwin
        it's ok for... some... uses
      • 2015-12-22 35629, 2015

      • chirlu`
        darwin: It’s a one-time cleanup script.
      • 2015-12-22 35637, 2015

      • darwin
        that would be one of those cases.
      • 2015-12-22 35653, 2015

      • chirlu`
        And the Redis version we use doesn’t have SCAN.
      • 2015-12-22 35603, 2015

      • chirlu`
        That’s why there was talk about updating it.
      • 2015-12-22 35616, 2015

      • darwin
        you can also parse the rdb
      • 2015-12-22 35618, 2015

      • darwin
        into json
      • 2015-12-22 35622, 2015

      • darwin
        and get keys / values that way
      • 2015-12-22 35653, 2015

      • chirlu`
        KEYS actually did succeed, but apparently the connection timed out when retrieving the values with MGET.
      • 2015-12-22 35645, 2015

      • chirlu`
        So unless it was accidental that KEYS worked, it should be enough to subdivide the MGET into multiple smaller ones.
      • 2015-12-22 35637, 2015

      • chirlu`
        And after one run, we’ll have perhaps 1% of the current number of keys left, so there won’t be issues anymore. ;-)
      • 2015-12-22 35602, 2015

      • opatel99
        How do I only run one test individually in CB?
      • 2015-12-22 35628, 2015

      • inversechi has quit
      • 2015-12-22 35607, 2015

      • inversechi joined the channel
      • 2015-12-22 35605, 2015

      • YatharthROCK_ joined the channel
      • 2015-12-22 35637, 2015

      • YatharthROCK_ has quit
      • 2015-12-22 35653, 2015

      • YatharthROCK_ joined the channel
      • 2015-12-22 35653, 2015

      • YatharthROCK_ has quit
      • 2015-12-22 35653, 2015

      • YatharthROCK_ joined the channel
      • 2015-12-22 35616, 2015

      • Freso
        opatel99: Pong
      • 2015-12-22 35626, 2015

      • Mineo has quit
      • 2015-12-22 35630, 2015

      • YatharthROCK_
        Freso: As I was saying yesterday about the "Warn users drafts are private" task, while I did see a GitHub PR to add a note to the website, the task also suggested considering excluding draft reviews from the public dumps as a perhaps a more elegant solution.
      • 2015-12-22 35633, 2015

      • opatel99
        Freso: Can you test the redirect is what you mant for it to b?
      • 2015-12-22 35649, 2015

      • Freso
        opatel99: Sure. Link?
      • 2015-12-22 35654, 2015

      • Freso
        YatharthROCK_: I replied to you last night.
      • 2015-12-22 35656, 2015

      • YatharthROCK_
        Do you think it would be better? If so, I could submit work I did to achieve that.
      • 2015-12-22 35603, 2015

      • Mineo joined the channel
      • 2015-12-22 35604, 2015

      • YatharthROCK_
        Freso: Ah, sorry; I'll check the logs.
      • 2015-12-22 35621, 2015

      • YatharthROCK_
        Thanks for letting me know :)
      • 2015-12-22 35644, 2015

      • Freso
      • 2015-12-22 35601, 2015

      • opatel99
      • 2015-12-22 35600, 2015

      • YatharthROCK_
        Freso: That was quick; thanks!
      • 2015-12-22 35603, 2015

      • YatharthROCK_
        So I'll make the PR, and Leftmost or bitmap can look into if/how a GCI task could be made for it
      • 2015-12-22 35641, 2015

      • Freso
        opatel99: Except for the Flask Debug Toolbar intercepting it, it looks like it's behaving as I suggested. :)
      • 2015-12-22 35652, 2015

      • opatel99
        Freso: Thx. Will PR
      • 2015-12-22 35621, 2015

      • Freso
        (And I had even opened my browser console to be sure I would be getting a HTTP 301 response. :p
      • 2015-12-22 35623, 2015

      • Freso
        )
      • 2015-12-22 35635, 2015

      • Freso
        Huh.
      • 2015-12-22 35637, 2015

      • opatel99
        :)
      • 2015-12-22 35641, 2015

      • Freso
        Why is it using that MusicBrainz logo though?
      • 2015-12-22 35648, 2015

      • opatel99
        hmm?
      • 2015-12-22 35648, 2015

      • Freso
        Oh.
      • 2015-12-22 35656, 2015

      • Freso adds a new ticket
      • 2015-12-22 35620, 2015

      • D4RK-PH0ENiX joined the channel
      • 2015-12-22 35621, 2015

      • opatel99
        explain?
      • 2015-12-22 35621, 2015

      • Freso
        It has nothing to do with your PR.
      • 2015-12-22 35646, 2015

      • opatel99
        I know. It has to do with a different MB logo. Still concerned about what you mean
      • 2015-12-22 35635, 2015

      • Nyanko-sensei has quit
      • 2015-12-22 35639, 2015

      • Freso
      • 2015-12-22 35646, 2015

      • Freso
        Why?
      • 2015-12-22 35656, 2015

      • Freso
        It's just an icon...
      • 2015-12-22 35622, 2015

      • opatel99
        Open ends.. Gotta close them
      • 2015-12-22 35646, 2015

      • opatel99
      • 2015-12-22 35655, 2015

      • YatharthROCK_ has quit
      • 2015-12-22 35604, 2015

      • opatel99
        Oh.. low key.. hmm...
      • 2015-12-22 35606, 2015

      • Freso
        No. That's the full‐/high‐detail version.
      • 2015-12-22 35618, 2015

      • Freso
        Don't include this in the same PR.
      • 2015-12-22 35631, 2015

      • Freso
        Different ticket, different PR.
      • 2015-12-22 35639, 2015

      • opatel99
        Oh yeah. I make a different branch on everything
      • 2015-12-22 35659, 2015

      • opatel99
        Each Jira task is a branch
      • 2015-12-22 35610, 2015

      • Freso
        All the new official MeB logos are available at https://github.com/metabrainz/metabrainz-logos/
      • 2015-12-22 35647, 2015

      • mndhck joined the channel
      • 2015-12-22 35653, 2015

      • mndhck is now known as YatharthROCK__
      • 2015-12-22 35655, 2015

      • Freso
        I don't really ever use the PIXEL versions though. I go grab SVGs and export them to resolution I want/need in Inkscape.
      • 2015-12-22 35634, 2015

      • opatel99
      • 2015-12-22 35658, 2015

      • YatharthROCK__
        Are all string extractions to be done on master branch and not in respective PRs?
      • 2015-12-22 35629, 2015

      • Freso
        Yes.
      • 2015-12-22 35633, 2015

      • Freso
        YatharthROCK__: ^
      • 2015-12-22 35605, 2015

      • Freso
        They clutter up the PR diffs and are not really needed.
      • 2015-12-22 35629, 2015

      • Freso
        opatel99: GitHub doesn't show me what it is and FF will just show me XML.
      • 2015-12-22 35641, 2015

      • Freso
        But I think it's the right one.
      • 2015-12-22 35659, 2015

      • Freso
        It should be two-coloured hexagon without any "markings".
      • 2015-12-22 35610, 2015

      • opatel99
        Yeah
      • 2015-12-22 35618, 2015

      • opatel99
        Interesting. It renders on my browser.. Chrome
      • 2015-12-22 35635, 2015

      • Freso
        Yeah. It's probably a setting I've set (or hasn't set) somewhere.
      • 2015-12-22 35601, 2015

      • Freso
        I also need to clean up those SVGs.
      • 2015-12-22 35621, 2015

      • Freso
        opatel99: You will probably want to make the box square before exporting it. Seems like it's 27x30px right now.
      • 2015-12-22 35635, 2015

      • Freso
        (According to the XML.)
      • 2015-12-22 35625, 2015

      • opatel99
        Freso: Scale or trim?
      • 2015-12-22 35639, 2015

      • YatharthROCK__ has quit
      • 2015-12-22 35658, 2015

      • Freso
        ?
      • 2015-12-22 35605, 2015

      • Freso
        Neither?
      • 2015-12-22 35611, 2015

      • Freso
        You will want to make it 30x30px, re-center the logo in the image, then export to a 16x16px PNG (or 32x32 for retina?).
      • 2015-12-22 35615, 2015

      • Freso
        Anyway. Gotta run!
      • 2015-12-22 35619, 2015

      • opatel99
        kk
      • 2015-12-22 35616, 2015

      • UmkaDK joined the channel
      • 2015-12-22 35631, 2015

      • opatel99
        Freso: When you get a chance... http://i.imgur.com/N55oEHV.png
      • 2015-12-22 35608, 2015

      • triggerwarning has quit
      • 2015-12-22 35615, 2015

      • dcentral has quit
      • 2015-12-22 35626, 2015

      • dcentral joined the channel
      • 2015-12-22 35632, 2015

      • D4RK-PH0ENiX has quit
      • 2015-12-22 35600, 2015

      • D4RK-PH0ENiX joined the channel
      • 2015-12-22 35647, 2015

      • Freso
        opatel99: ?
      • 2015-12-22 35608, 2015

      • chirlu`
        The MB icon, I think.
      • 2015-12-22 35610, 2015

      • Freso
        Sure, but I'm not sure what he wants me to say about it.
      • 2015-12-22 35625, 2015

      • Freso
        It's there?
      • 2015-12-22 35612, 2015

      • chirlu`
      • 2015-12-22 35621, 2015

      • chirlu`
        Not sure if you get notified about it.
      • 2015-12-22 35652, 2015

      • Nyanko-sensei joined the channel
      • 2015-12-22 35615, 2015

      • D4RK-PH0ENiX has quit
      • 2015-12-22 35646, 2015

      • kurros joined the channel
      • 2015-12-22 35644, 2015

      • zas
      • 2015-12-22 35609, 2015

      • zas
        any idea why number of 503s suddenly reduced at 9:54 UTC ?
      • 2015-12-22 35610, 2015

      • triggerwarning joined the channel
      • 2015-12-22 35642, 2015

      • triggerwarning has quit
      • 2015-12-22 35642, 2015

      • triggerwarning joined the channel
      • 2015-12-22 35623, 2015

      • rvedotrc has quit
      • 2015-12-22 35641, 2015

      • rvedotrc joined the channel
      • 2015-12-22 35627, 2015

      • Mineo
        that guy who was hammering mb with requests went on christmas break?
      • 2015-12-22 35649, 2015

      • zas
        perhaps ;)
      • 2015-12-22 35602, 2015

      • CJ_
        What is the link for installing the MB search? I forgot to save it.
      • 2015-12-22 35600, 2015

      • opatel99
        Gentlecat:
      • 2015-12-22 35613, 2015

      • opatel99
        Freso: I wanted to make sure it looks right to you
      • 2015-12-22 35655, 2015

      • dcentral has quit
      • 2015-12-22 35610, 2015

      • triggerwarning has quit
      • 2015-12-22 35607, 2015

      • regagain has quit
      • 2015-12-22 35609, 2015

      • zas
      • 2015-12-22 35642, 2015

      • CJ_
        zas, Thanks. :)
      • 2015-12-22 35637, 2015

      • triggerwarning joined the channel
      • 2015-12-22 35603, 2015

      • dcentral joined the channel
      • 2015-12-22 35639, 2015

      • CJ_
        "Almost certainly not what you want," :D
      • 2015-12-22 35622, 2015

      • CJ_
        Who wrote the install instructions for MB?
      • 2015-12-22 35630, 2015

      • CJ_
      • 2015-12-22 35649, 2015

      • opatel99
        CJ_: Upgrade npm and node
      • 2015-12-22 35601, 2015

      • CJ_
        opatel99, I just did a fresh install.
      • 2015-12-22 35613, 2015

      • chirlu`
        CJ_: It’s addressed in INSTALL.md.
      • 2015-12-22 35615, 2015

      • CJ_
        But I suddenly recalled that there was a mention of node legacy package for ubuntu.
      • 2015-12-22 35625, 2015

      • chirlu`
        Yes, exactly.
      • 2015-12-22 35633, 2015

      • CJ_
        chirlu`, Yes, but not explained.
      • 2015-12-22 35655, 2015

      • CJ_
        The instructions just say I might need the legacy package, but not why or how to tell if I do.
      • 2015-12-22 35612, 2015

      • chirlu`
        Well, the intention is that you install all the packages mentioned.
      • 2015-12-22 35632, 2015

      • chirlu`
        It’s just singled out because it may fail on old Ubuntu releases.
      • 2015-12-22 35635, 2015

      • YatharthROCK_ joined the channel
      • 2015-12-22 35611, 2015

      • chirlu`
        The problem is that there is another node program, unrelated, with older naming rights.
      • 2015-12-22 35634, 2015

      • CJ_
        Understood. I was just pointing out that the instructions were unclear.
      • 2015-12-22 35640, 2015

      • CJ_
        "Depending on your Ubuntu version, another package might be required, too:"
      • 2015-12-22 35600, 2015

      • chirlu`
        What wording would you suggest?
      • 2015-12-22 35607, 2015

      • CJ_
        I would move that down to where gulp is first called and point out that if it does not work then to install that pacakge.
      • 2015-12-22 35614, 2015

      • CJ_
        Or something to that effect.
      • 2015-12-22 35656, 2015

      • YatharthROCK_
        From the IRC logs, it seems BitBucket went down sometime yesterday; however, now the repo seems to just have been deleted. Is this temporary? Previously, BitBucket was the preferred platform for PRs; is GitHub the only (and thus preferred) platform to make PRs on now?
      • 2015-12-22 35606, 2015

      • CJ_
        Also, I am getting the "DEVELOPMENT_SERVER should be set to either 0 or 1" error. I set the variable in DBPrefs but it still errors out.