#metabrainz

/

      • CJ_
        chirlu`, There is a mention of adding ./node_modules/.bin to $PATH but again, no mention of why.

        • 2015-12-22 35602, 2015

      • chirlu`
        CJ_: Yes, that is indeed a documentation bug that I stumbled over just this morning.

        • 2015-12-22 35615, 2015

      • chirlu`
        You should not run ./node_modules/.bin/gulp, but ./script/compile_resources.sh

        • 2015-12-22 35649, 2015

      • CJ_
        Will running gulp cause problems? Or can I just run the other command without having to do anything?

        • 2015-12-22 35600, 2015

      • chirlu`
        (The bug is older, but became obvious when the gulpfile started validating the DEVELOPMENT_SERVER setting.)

        • 2015-12-22 35608, 2015

      • chirlu`
        YatharthROCK_: Yes, at least for CB (and MB) you should use GitHub.

        • 2015-12-22 35623, 2015

      • CJ_
        Understood. I just want to make sure that I am correctly building this and whether I need to do anything other than run the new command.

        • 2015-12-22 35626, 2015

      • YatharthROCK_
        chirlu`: May I ask what prompted the switch?

        • 2015-12-22 35656, 2015

      • chirlu`
        CJ_: I think calling gulp directly just will error out and not actually build anything. Not 100% sure, though.

        • 2015-12-22 35620, 2015

      • chirlu`
        So you should use the wrapper script.

        • 2015-12-22 35623, 2015

      • CJ_
        So I should not have broken anything then. I will try the new command.

        • 2015-12-22 35650, 2015

      • YatharthROCK_
        Are the comments on BitBucket PRs (and whatever content unfortunately gets locked into the Git hosting service one uses) lost?

        • 2015-12-22 35655, 2015

      • CJ_
        chirlu`, "ReferenceError: Map is not defined"

        • 2015-12-22 35650, 2015

      • chirlu`
        CJ_: For that, you need a newer Node.js.

        • 2015-12-22 35611, 2015

      • spuniun
        for some reason I cannot run musicbrainz slave.sh from cron - works fine when I run from active console - what might I be missing from $PATH?

        • 2015-12-22 35611, 2015

      • chirlu`
        You can install it via npm, I think.

        • 2015-12-22 35613, 2015

      • spuniun
        http://pastebin.com/dbyt4Nin

        • 2015-12-22 35637, 2015

      • CJ_
        chirlu`, Uninstall node legacy and then place the newer node in path?

        • 2015-12-22 35650, 2015

      • chirlu`
        spuniun: You need to set up the local::lib environment.

        • 2015-12-22 35638, 2015

      • spuniun
        that's not something I came across int he docs

        • 2015-12-22 35657, 2015

      • chirlu`
        spuniun: Something like “eval $( perl -Mlocal::lib )”.

        • 2015-12-22 35659, 2015

      • CJ_
        spuniun, It is there. I just did it.

        • 2015-12-22 35619, 2015

      • opatel99
        spuniun: A note, Pastebin is blocked in some countries, and there are more elegant websites that offer pasting services with code highlighting like hastebin and paste.ee . Also, Freso doesnt like it

        • 2015-12-22 35633, 2015

      • chirlu`
        CJ_: Yes, that should work.

        • 2015-12-22 35629, 2015

      • chirlu`
        There are some recent issues with node and npm, they tend to fail if they are not quite recent (npm 3, e.g.).

        • 2015-12-22 35638, 2015

      • spuniun
        ok, that step was done already

        • 2015-12-22 35600, 2015

      • spuniun
        just checked my .bashrc and that line was already there

        • 2015-12-22 35648, 2015

      • CJ_
        chirlu`, Is there a reason not to change the script to use nodejs instead of node?

        • 2015-12-22 35614, 2015

      • chirlu`
        spuniun: Cron jobs don’t necessarily execute .bashrc.

        • 2015-12-22 35644, 2015

      • spuniun
        so I can just add that cmd to the slave.sh script?

        • 2015-12-22 35645, 2015

      • chirlu`
        CJ_: Well, it doesn’t work on older Ubuntus then. :-)

        • 2015-12-22 35626, 2015

      • CJ_
        You could also link nodejs to node.

        • 2015-12-22 35629, 2015

      • chirlu`
        spuniun: Yes, that will probably work, but may be overwritten when you check out a newer version at some point. So it may be safer to put it into the crontab entry.

        • 2015-12-22 35646, 2015

      • CJ_
        Is there a reason that MB uses Ubuntu?

        • 2015-12-22 35633, 2015

      • spuniun
        will try that

        • 2015-12-22 35602, 2015

      • CJ_
        chirlu`, I just tried the link solution and that did not work. I will try changing the PATH.

        • 2015-12-22 35655, 2015

      • chirlu`
        Reason for choosing Ubuntu: A choice had to be made. ;-)

        • 2015-12-22 35601, 2015

      • CJ_
        :D

        • 2015-12-22 35622, 2015

      • chirlu`
        I actually wasn’t around then, but I don’t think it’s because of some particular advantage.

        • 2015-12-22 35630, 2015

      • CJ_
        Understood.

        • 2015-12-22 35618, 2015

      • chirlu`
        YatharthROCK_: Generally, we used to use Bitbucket because for a long time, its review tools were ahead of GitHub’s.

        • 2015-12-22 35633, 2015

      • CJ_
        What version of node should I be using?

        • 2015-12-22 35626, 2015

      • CJ_
        Apparently nodejs-legacy just adds the node->nodejs link. :D

        • 2015-12-22 35635, 2015

      • chirlu`
        YatharthROCK_: But it had other issues (client-side rendering causing long browser stalls, the most notable one), and GitHub has mostly caught up.

        • 2015-12-22 35644, 2015

      • chirlu`
        CJ_: Yes, that’s all it does.

        • 2015-12-22 35658, 2015

      • YatharthROCK_
        chirlu`: Oh, I see. I'm not familiar with being on the reviewing end of PRs, although I'll soon have to be, so gtk :)

        • 2015-12-22 35600, 2015

      • chirlu`
        Recent version; I’m not exactly sure which one.

        • 2015-12-22 35602, 2015

      • CJ_
        chirlu`, I was not aware of that. :)

        • 2015-12-22 35645, 2015

      • CJ_
        chirlu`, What is the best way to update node?

        • 2015-12-22 35622, 2015

      • chirlu`
        YatharthROCK_: And GitHub is obviously far more popular; since we always have had a repository there, people were bound to expect being able to PR there.

        • 2015-12-22 35623, 2015

      • Mineo
        alastairp: just saw http://tickets.musicbrainz.org/browse/AB-101, 10mb/song also looks like a lot of data to send for users

        • 2015-12-22 35643, 2015

      • chirlu`
        Mineo/alastairp: I was wondering if it is high-level or low-level.

        • 2015-12-22 35648, 2015

      • alastairp
        low level

        • 2015-12-22 35600, 2015

      • Leo_Verto joined the channel

        • 2015-12-22 35601, 2015

      • alastairp
        but it's required data for people who want to do in-depth machine learning

        • 2015-12-22 35614, 2015

      • alastairp
        it was the most requested feature when we presented it at the conference

        • 2015-12-22 35644, 2015

      • alastairp
        so the question is definitely "how do we deal with it" rather than "do we collect it"

        • 2015-12-22 35608, 2015

      • alastairp
        I'm running experiments now with protobufs, msgpack, and hdf5

        • 2015-12-22 35638, 2015

      • Nyanko-sensei has quit

        • 2015-12-22 35647, 2015

      • alastairp
        if we can get it down to <0.5mb for a standard pop song, I'd be happy

        • 2015-12-22 35656, 2015

      • alastairp
        filesizes now are 100-200k normally

        • 2015-12-22 35655, 2015

      • chirlu`
        CJ_: I’m actually using Debian myself, and there I have a reasonably recent version (in testing).

        • 2015-12-22 35649, 2015

      • chirlu`
        CJ_: https://askubuntu.com/questions/426750/how-can-i-…

        • 2015-12-22 35616, 2015

      • CJ_
        chirlu`, :D I just found that page right before you linked it to me.

        • 2015-12-22 35630, 2015

      • CJ_
        This issue is not in any of the docs.

        • 2015-12-22 35625, 2015

      • chirlu`
        YatharthROCK_: Regarding the comments and pull requests for CB on BitBucket, hm. The MB BitBucket repo still exists, mostly for that reason.

        • 2015-12-22 35640, 2015

      • chirlu`
        Perhaps Gentlecat can undelete the repo?

        • 2015-12-22 35648, 2015

      • chirlu`
        YatharthROCK_: It seems exporting from BitBucket is possible in principle, see https://blog.bitbucket.org/2013/04/02/finnovation…

        • 2015-12-22 35654, 2015

      • YatharthROCK_
        chirlu`: I would suggest keeping the Bitbucket repo up as an archive and disabling PRs to encourage people to use what is now the preferred platform of GitHub to contribute code from, but, unfortunately, BitBucket doesn't seem to want to go for that: https://bitbucket.org/site/master/issues/5963/add…

        • 2015-12-22 35612, 2015

      • stanislas
        Freso: ping

        • 2015-12-22 35654, 2015

      • YatharthROCK_
        Leftmost, bitmap: Could one of you look at the following comment with regards to creating a task on GCI? https://github.com/metabrainz/critiquebrainz/pull…

        • 2015-12-22 35621, 2015

      • chirlu`
        CJ_: I agree we’ll need to write something about these issues that recently popped up.

        • 2015-12-22 35639, 2015

      • CJ_
        chirlu`, Once I ran the three commands on the page you linked it worked. I did not have to make a link.

        • 2015-12-22 35640, 2015

      • opatel99
        stanislas: Hey!

        • 2015-12-22 35645, 2015

      • stanislas
        opatel99: hi :)

        • 2015-12-22 35629, 2015

      • stanislas
        Freso, opatel99: I've claimed the task you were talking about. I need a little rest from BB

        • 2015-12-22 35608, 2015

      • opatel99
        stanislas: Which task?

        • 2015-12-22 35634, 2015

      • stanislas
        the one about displaying review language

        • 2015-12-22 35656, 2015

      • opatel99
        Oh cool

        • 2015-12-22 35658, 2015

      • stanislas
        opatel99: Are you making BB test task now ?

        • 2015-12-22 35619, 2015

      • opatel99
        Yeah. Wondering what revision you had made for the OAuth client

        • 2015-12-22 35625, 2015

      • opatel99
        and where did you get the client key?

        • 2015-12-22 35652, 2015

      • stanislas
        opatel99: You're welcome to look up in my code.

        • 2015-12-22 35605, 2015

      • opatel99
        Is it the same Client-ID?

        • 2015-12-22 35642, 2015

      • CatQuest
        hi, what do i actually do on http://stats.musicbrainz.org/login ?

        • 2015-12-22 35644, 2015

      • stanislas
        opatel99: you mean, every time ?

        • 2015-12-22 35650, 2015

      • CatQuest
        (to see stat graphs)

        • 2015-12-22 35624, 2015

      • CatQuest
        I mean, do I put in musicbrainz name and pass or what?

        • 2015-12-22 35635, 2015

      • Freso
        stanislas: Cool. Looking forwards to seeing your work. :)

        • 2015-12-22 35656, 2015

      • stanislas
        opatel99: The user is initiated in fixture.py, you might want to look there. The client_id is some random thing (but the same as in fixture.py)

        • 2015-12-22 35609, 2015

      • opatel99
        stanislas: Cool. Thanks

        • 2015-12-22 35612, 2015

      • stanislas
        opatel99: I've taken it from some forum

        • 2015-12-22 35648, 2015

      • stanislas
        opatel99: It doesn't matter really, I think it needs to be in uuid form

        • 2015-12-22 35655, 2015

      • opatel99
        ok

        • 2015-12-22 35609, 2015

      • opatel99
        Is it ok to just use your client id?

        • 2015-12-22 35644, 2015

      • stanislas
        opatel99: as long as you use it under GNU GPL :)

        • 2015-12-22 35602, 2015

      • opatel99
        :p

        • 2015-12-22 35608, 2015

      • CatQuest
        I'd liek to see the stats

        • 2015-12-22 35650, 2015

      • stanislas
        opatel99: Are you writing this in vim ? I've heard that you are using it/

        • 2015-12-22 35605, 2015

      • opatel99
        Yeah I use vim with NERDTree

        • 2015-12-22 35608, 2015

      • CatQuest
        so i'd appreciate a guide as to what to do on stats.musicbrainz.org/login

        • 2015-12-22 35609, 2015

      • opatel99
        and a few other plugins I got

        • 2015-12-22 35634, 2015

      • CatQuest
        it isn't clear, do I use my musicbrainz credentials?

        • 2015-12-22 35642, 2015

      • opatel99
        stanislas: ^

        • 2015-12-22 35643, 2015

      • stanislas
        opatel99: If you don't have any code completition plugin, I would suggest using something else. You don't want to write some very long names by hand, and there are a lot of them.

        • 2015-12-22 35634, 2015

      • opatel99
        stanislas: I got https://github.com/davidhalter/jedi-vim

        • 2015-12-22 35609, 2015

      • opatel99
        But as I mentioned when I had the conversation, I use vim for quick edits after files are made, and I use Atom for larger edits

        • 2015-12-22 35642, 2015

      • CatQuest
        Freso, bitmap, chirlu ?

        • 2015-12-22 35601, 2015

      • opatel99
        CatQuest: Why not just press the sign-up tab?

        • 2015-12-22 35612, 2015

      • chirlu`
        CatQuest: Yes, no SSO.

        • 2015-12-22 35617, 2015

      • CatQuest
        opatel99: erh because why would i do that?

        • 2015-12-22 35630, 2015

      • CatQuest
        chirlu`: uhm

        • 2015-12-22 35631, 2015

      • chirlu`
        You need to get a separate account, like on forums, JIRA, wiki, etc.

        • 2015-12-22 35612, 2015

      • CatQuest
        why did you guys change from the one we had before (which had a loginpromt that you just add to put "mb""mb" in to this with requires a new account?)

        • 2015-12-22 35644, 2015

      • CatQuest
        jeg gidder fane ikke å lage en *til* brukerkonto

        • 2015-12-22 35654, 2015

      • CatQuest
        i'll wait until you fix it

        • 2015-12-22 35629, 2015

      • chirlu`
        Det kommer förhoppningsvis att bli löst när SSO över OAuth introduceras.

        • 2015-12-22 35636, 2015

      • stanislas
        Freso: I am not very familiar with CodeBrainz, where is the review page generated ?

        • 2015-12-22 35658, 2015

      • CatQuest
        chirlu`: forhåpentligvis :) jeg venter til da ok :)

        • 2015-12-22 35602, 2015

      • chirlu`
        Det må vara tidigt för Discourse, men Grafana stats är sannolikt inte högt uppe på listan.

        • 2015-12-22 35615, 2015

      • chirlu`
        -e

        • 2015-12-22 35646, 2015

      • Freso
        opatel99: The reason I don't like pastebin is exactly because of those reason you listed. They shouldn't avoid using that site because of my like or dislike, but because of those reasons.

        • 2015-12-22 35653, 2015

      • CatQuest
        det er tidlig for discourse? I didn't quite get that

        • 2015-12-22 35605, 2015

      • Freso
        (There are plenty of things you should stay away from, if you should stay away from everything I didn't approve of.)

        • 2015-12-22 35615, 2015

      • chirlu`
        CatQuest: https://discourse.metabrainz.org/

        • 2015-12-22 35626, 2015

      • CatQuest
        wait we have codebrainz,org now?

        • 2015-12-22 35636, 2015

      • Freso
        opatel99 | Freso: I wanted to make sure it looks right to you -- I think it maybe does, but I would rather look at the image file on its own.

        • 2015-12-22 35651, 2015

      • chirlu`
        Du kan dock inte logga in eller skapa konto där ännu.

        • 2015-12-22 35626, 2015

      • chirlu`
        Back to English … or rather, back later. ;-)

        • 2015-12-22 35645, 2015

      • gcilou joined the channel

        • 2015-12-22 35607, 2015

      • D4RK-PH0ENiX joined the channel

        • 2015-12-22 35620, 2015

      • YatharthROCK_
        I just finished created this PR <https://github.com/metabrainz/critiquebrainz/pull/23> but the for some reason I can't claim the GCI task <https://codein.withgoogle.com/tasks/4647133944741888/>

        • 2015-12-22 35620, 2015

      • gcibot
        CB: Add indication of review language || 3 days || Code, User Interface || MetaBrainz Foundation || Currently claimed

        • 2015-12-22 35634, 2015

      • CatQuest
        chirlu`: jaaa. jeg er klar over at discourse er en greie vi skal få (og att det *ikke* er det samme som disqus som jeg trodde tidligere >_<)

        • 2015-12-22 35600, 2015

      • stanislas
        Freso: isn't it critiquebrainz/blob/master/critiquebrainz/frontend/templates/review/browse.html ?

        • 2015-12-22 35614, 2015

      • YatharthROCK_
        I tried abandoning my current task and claiming it, but the claim button didn't show up.

        • 2015-12-22 35639, 2015

      • CatQuest
        hjelper ikke at logoene er *noenlunde* like

        • 2015-12-22 35641, 2015

      • CatQuest
        :/

        • 2015-12-22 35648, 2015

      • stanislas
        YatharthROCK_: try again, refresh the pahe, it helps sometimes

        • 2015-12-22 35654, 2015

      • CatQuest
        chirlu`: ok, ha det bra! _o/

        • 2015-12-22 35654, 2015

      • stanislas
        *the page

        • 2015-12-22 35608, 2015

      • chirlu`
        YatharthROCK_: According to gcibot, it’s currently claimed.

        • 2015-12-22 35643, 2015

      • YatharthROCK_
        chirlu`: Oh :'(

        • 2015-12-22 35647, 2015

      • stanislas
        YatharthROCK_: I've claimed this task :)

        • 2015-12-22 35605, 2015

      • stanislas
        YatharthROCK_: really sorry

        • 2015-12-22 35633, 2015

      • YatharthROCK_
        I'd already submitted the PR externally before it was claimed

        • 2015-12-22 35647, 2015

      • YatharthROCK_
        stanislas: You couldn't have known :)

        • 2015-12-22 35609, 2015

      • stanislas
        YatharthROCK_: You've done the task already, right ?

        • 2015-12-22 35626, 2015

      • YatharthROCK_
        stanislas: Yeah, ^^^

        • 2015-12-22 35628, 2015

      • stanislas
        YatharthROCK_: ok, no problem, unclaiming it now