#metabrainz

/

      • Freso shrugs :)
      • 2015-12-21 35504, 2015

      • Freso
        Leftmost: When are you normally around?
      • 2015-12-21 35521, 2015

      • CJ_
        :D
      • 2015-12-21 35511, 2015

      • Freso
        (It would actually be really neat to be able to ask BrainzBot when people are normally around based entirely on its IRC log data.)
      • 2015-12-21 35535, 2015

      • dpmittal has quit
      • 2015-12-21 35551, 2015

      • chirlu`
        CJ_: Where are you, timezone-wise?
      • 2015-12-21 35505, 2015

      • CJ_
        chirlu`, EST
      • 2015-12-21 35523, 2015

      • chirlu`
        U.S. Eastern Standard Time (UTC–5)?
      • 2015-12-21 35533, 2015

      • chirlu`
        Or Egypt Standard Time (UTC+2)? :)
      • 2015-12-21 35541, 2015

      • CJ_
        The former. :D
      • 2015-12-21 35507, 2015

      • djpretzel joined the channel
      • 2015-12-21 35542, 2015

      • chirlu`
        Leftmost is in UTC–6, I’d expect him to be around in a few hours’ time.
      • 2015-12-21 35509, 2015

      • CJ_
        I will be here. :)
      • 2015-12-21 35529, 2015

      • zas
        Freso: make this BrainzBot improvement a GCI task ? ;)
      • 2015-12-21 35547, 2015

      • CJ_
        GCI?
      • 2015-12-21 35522, 2015

      • chirlu`
        Google Code-in.
      • 2015-12-21 35528, 2015

      • dpmittal joined the channel
      • 2015-12-21 35545, 2015

      • Leo_Verto joined the channel
      • 2015-12-21 35534, 2015

      • kepstin
      • 2015-12-21 35551, 2015

      • kepstin
        whee, my silly clock.
      • 2015-12-21 35558, 2015

      • CJ_
        O.o
      • 2015-12-21 35500, 2015

      • MBJenkins
        Project critiquebrainz build #153: SUCCESS in 3 min 50 sec: http://ci.musicbrainz.org/job/critiquebrainz/153/
      • 2015-12-21 35542, 2015

      • weeksio joined the channel
      • 2015-12-21 35550, 2015

      • Leo_Verto has quit
      • 2015-12-21 35554, 2015

      • Leo_Verto joined the channel
      • 2015-12-21 35531, 2015

      • Leo_Verto has quit
      • 2015-12-21 35531, 2015

      • Leo_Verto joined the channel
      • 2015-12-21 35550, 2015

      • gcilou joined the channel
      • 2015-12-21 35529, 2015

      • stanislas
        Freso, CJ_ : If you want to find out when Leftmost is normally around, you can always look on the logs : http://chatlogs.metabrainz.org/brainzbot/metabrai…
      • 2015-12-21 35544, 2015

      • CJ_
        stanislas, That sounds like work. :)
      • 2015-12-21 35516, 2015

      • stanislas
        CJ_: Not much really.
      • 2015-12-21 35526, 2015

      • stanislas
        CJ_: You can look on last monday
      • 2015-12-21 35510, 2015

      • gcilou
        reosarevok: I have a question about the task "CB: Add a note that will warn users that drafts are not private"; The related ticket has a link to a merged PR (6 days ago) .. I was wondering if that meant that the problem was fixed. Sorry about the long message
      • 2015-12-21 35526, 2015

      • reosarevok
        Gentlecat ^
      • 2015-12-21 35545, 2015

      • Gentlecat
        yes, that was fixed
      • 2015-12-21 35549, 2015

      • reosarevok
        But it might be - we have a couple cases of those, I think I hid them all from GCI but one might have remained behind
      • 2015-12-21 35552, 2015

      • Gentlecat
        is there still a task for it?
      • 2015-12-21 35513, 2015

      • gcilou
        Yes there is still a task
      • 2015-12-21 35551, 2015

      • Gentlecat
        removed
      • 2015-12-21 35558, 2015

      • Gentlecat
        thanks for noticing
      • 2015-12-21 35528, 2015

      • gcilou
        :)
      • 2015-12-21 35514, 2015

      • reosarevok
        gcilou: looking for a task? You could take a look at https://codein.withgoogle.com/tasks/5606593248886… if you feel like it
      • 2015-12-21 35514, 2015

      • gcibot
        MB: Add username to verification mails [MBS-6407] || 3 days || Code || MetaBrainz Foundation
      • 2015-12-21 35532, 2015

      • gcilou
        Yes, I'm still looking
      • 2015-12-21 35548, 2015

      • gcilou
        reosarevok: That task looks interesting. I've looked at musicbrainz-server/root/account but I couldn't seem to find the page for the verification email. Would this be given if I claimed the task?
      • 2015-12-21 35534, 2015

      • reosarevok
        We can find it together, heh
      • 2015-12-21 35545, 2015

      • reosarevok
        I don't remember where it is, honestly, but I can give you the email text to git grep
      • 2015-12-21 35556, 2015

      • reosarevok
        That should find it
      • 2015-12-21 35513, 2015

      • gcilou
        OK, I'll claim it then. lol. git grep searches for files or something?
      • 2015-12-21 35548, 2015

      • reosarevok
        git grep searches for something inside the files in the repository
      • 2015-12-21 35500, 2015

      • reosarevok
        In this case... *looks for verification mail*
      • 2015-12-21 35523, 2015

      • reosarevok
        "This is a verification email for your MusicBrainz account"
      • 2015-12-21 35533, 2015

      • gcilou
        Right, so 'grep' is the git command ?
      • 2015-12-21 35552, 2015

      • gcilou_ joined the channel
      • 2015-12-21 35508, 2015

      • gcilou_
        urg sorry, chrome just crashed
      • 2015-12-21 35525, 2015

      • gcilou_
        Sorry, I claimed the wrong task and then abandoned it
      • 2015-12-21 35554, 2015

      • gcilou has quit
      • 2015-12-21 35517, 2015

      • reosarevok
        Yeah, just git grep "something"
      • 2015-12-21 35538, 2015

      • gcilou_
        Ok, how do I abandon a CB ticket?
      • 2015-12-21 35527, 2015

      • chirlu`
        Click “Assign”, then choose “Unassigned” as the assignee.
      • 2015-12-21 35508, 2015

      • gcilou_
        Ok thank you.
      • 2015-12-21 35501, 2015

      • gcilou_
        reosarevok: Is this the right place? https://github.com/metabrainz/musicbrainz-server/…
      • 2015-12-21 35515, 2015

      • gcilou_
        line 137 ish
      • 2015-12-21 35530, 2015

      • stanislas
        Leftmost, LordSputnik: When I change a gender using PUT, the gender in the database doesn't appear. None is in instance_of_creator.master_revision.entity_data.gender. Should I create some 'gender' object first ?
      • 2015-12-21 35530, 2015

      • reosarevok
        No :)
      • 2015-12-21 35545, 2015

      • reosarevok
        Although there's a chance you'll need to modify this too
      • 2015-12-21 35549, 2015

      • reosarevok
        This is a test file
      • 2015-12-21 35511, 2015

      • reosarevok
        Yeah, you'll need to change this too I think :)
      • 2015-12-21 35529, 2015

      • gcilou_
        Oh ok..that's what grep found..I'll try something else then
      • 2015-12-21 35534, 2015

      • reosarevok
        Basically anything in the /t directory is a test (I think)
      • 2015-12-21 35547, 2015

      • reosarevok
        Try with a smaller substring then
      • 2015-12-21 35552, 2015

      • gcilou_
        That makes sense
      • 2015-12-21 35538, 2015

      • gcilou_
      • 2015-12-21 35509, 2015

      • reosarevok
        File name is promising :)
      • 2015-12-21 35525, 2015

      • reosarevok
        Yup, that should be it
      • 2015-12-21 35538, 2015

      • weeksio has quit
      • 2015-12-21 35541, 2015

      • gcilou_
        ok cool
      • 2015-12-21 35514, 2015

      • gcilou_
        So I'm assuming that adding the username will be similar to the ip address "my $ip = $opts{ip};" ?
      • 2015-12-21 35548, 2015

      • reosarevok
      • 2015-12-21 35532, 2015

      • travis-ci joined the channel
      • 2015-12-21 35533, 2015

      • travis-ci
        musicbrainz/picard#1271 (master - 2be1572 : Philipp Wolfer): The build passed.
      • 2015-12-21 35533, 2015

      • travis-ci
      • 2015-12-21 35533, 2015

      • travis-ci
      • 2015-12-21 35533, 2015

      • travis-ci has left the channel
      • 2015-12-21 35532, 2015

      • ruaok
        Freso: I do have presentations. Mail me to remind me to make them public.
      • 2015-12-21 35518, 2015

      • dpmittal has quit
      • 2015-12-21 35512, 2015

      • gcilou_
        reosarevok: I'm assuming that I have to add a line for username in the last link you gave, then add another based on the previous in the one I found..Is this correct?
      • 2015-12-21 35553, 2015

      • dpmittal joined the channel
      • 2015-12-21 35507, 2015

      • gcilou_
        "usernm => $user_id " ; then " my $userid = $opts{usernm};" ?
      • 2015-12-21 35531, 2015

      • reosarevok
        I think you want $editor->name
      • 2015-12-21 35523, 2015

      • reosarevok
        (also feel free to write "username" in full, it makes things more clear when checking the code in the future)
      • 2015-12-21 35550, 2015

      • gcilou_
        for the second statement? Ok
      • 2015-12-21 35529, 2015

      • reosarevok
        Mind, I am not very familiar with this code and I'm a mentor there mostly because I entered the original feature request
      • 2015-12-21 35553, 2015

      • reosarevok
        So we'll want bitmap to take a look at this when he's around and make sure we're not doing stuff very wrongly :p
      • 2015-12-21 35556, 2015

      • spideyonthego joined the channel
      • 2015-12-21 35505, 2015

      • gcilou_
        Alright
      • 2015-12-21 35530, 2015

      • spideyonthego
        Is there anyone here right now who has installed CB with vagrant?
      • 2015-12-21 35553, 2015

      • Gentlecat
        I tried to do that again yesterday and it worked fine
      • 2015-12-21 35520, 2015

      • Gentlecat
        stanislas might've done that before as well
      • 2015-12-21 35544, 2015

      • Gentlecat
        still same issues with downloading packages?
      • 2015-12-21 35546, 2015

      • stanislas
        spideyonthego: I did, thaks Gentlecat.
      • 2015-12-21 35504, 2015

      • gcilou_
        reosarevok: I see lower in the code that this statement is used " my $user_name = $opts{user}->name;" Is this the statement I should use too>
      • 2015-12-21 35510, 2015

      • spideyonthego
        Gentlecat: Yup.
      • 2015-12-21 35522, 2015

      • spideyonthego
        Gentlecat: This is all I do. I start the VM using vagrant up (after which a lot of files are downloaded). Then I use Putty to actually open the VM. And then when I try running run.py, it shows the errors.
      • 2015-12-21 35533, 2015

      • stanislas
        spideyonthego: Can you gist your error.
      • 2015-12-21 35554, 2015

      • spideyonthego
        stanislas: It's just that the werkzeug module couldn't be found.
      • 2015-12-21 35503, 2015

      • reosarevok
        gcilou_: Oh, that does seem like something similar to what you want. See also send_editor_report later
      • 2015-12-21 35503, 2015

      • stanislas
        Aaah.
      • 2015-12-21 35513, 2015

      • spideyonthego
        So, I installed everything from the requirements text using pip
      • 2015-12-21 35522, 2015

      • spideyonthego
        But still it doesn't work.
      • 2015-12-21 35530, 2015

      • stanislas
        spideyonthego: I've done the same, and it worked
      • 2015-12-21 35550, 2015

      • stanislas
        spideyonthego: pip install Werkzeug
      • 2015-12-21 35526, 2015

      • spideyonthego
        stanislas: But that's not the only one right? There are many more modules left.
      • 2015-12-21 35540, 2015

      • gcilou_
        reosarevok: Ok, what's that?
      • 2015-12-21 35551, 2015

      • Gentlecat
        pip install -r requirements.txt
      • 2015-12-21 35512, 2015

      • dpmittal has quit
      • 2015-12-21 35513, 2015

      • Gentlecat
      • 2015-12-21 35521, 2015

      • Gentlecat
        this is what vagrant runs anyway
      • 2015-12-21 35524, 2015

      • spideyonthego
        Gentlecat: It still doesn't work after doing all that. Is postgres part of the requirements txt?
      • 2015-12-21 35529, 2015

      • spideyonthego
        Gentlecat: Okay.
      • 2015-12-21 35538, 2015

      • Gentlecat
        no, take a look at that script
      • 2015-12-21 35522, 2015

      • reosarevok
        gcilou_: that's another "send an email that includes the username of editors" bit of code in the same file
      • 2015-12-21 35536, 2015

      • reosarevok
        It'd probably make sense to use basically the same ideas as those other ones
      • 2015-12-21 35538, 2015

      • spideyonthego
        Gentlecat: So I do all the commands in that?
      • 2015-12-21 35552, 2015

      • gcilou_
        reosarevok: Ok
      • 2015-12-21 35555, 2015

      • Gentlecat
        depends on where provisioning fails
      • 2015-12-21 35506, 2015

      • Gentlecat
        but sure, you can try doing that
      • 2015-12-21 35521, 2015

      • spideyonthego
        Gentlecat: Okay. :)
      • 2015-12-21 35533, 2015

      • Gentlecat
        or just run it automatically instead of copy-pasting
      • 2015-12-21 35543, 2015

      • gcilou_
        reosarevok: So how many files am I supposed to change, or should I ask someone else that question?
      • 2015-12-21 35505, 2015

      • opatel99
        LordSputnik: Leftmost Submitted task btw
      • 2015-12-21 35523, 2015

      • reosarevok
        I think asking bitmap (or Gentlecat if he has the time) would be better for that :) Sorry that I'm not super helpful here :(
      • 2015-12-21 35536, 2015

      • gcilou_
        nah, it's fine!
      • 2015-12-21 35549, 2015

      • opatel99
        gcilou_: What is going on right now?
      • 2015-12-21 35520, 2015

      • gcilou_
        opatel99: Task to include username in account verification email
      • 2015-12-21 35532, 2015

      • opatel99
        gcilou_: Yes. I meant what is wrong?
      • 2015-12-21 35549, 2015

      • gcilou_
        I was wondering how many files needed to be edited to make all the changes necessary
      • 2015-12-21 35520, 2015

      • opatel99
        I would imagine only the email.pm would be changed once you properly install.
      • 2015-12-21 35558, 2015

      • gcilou_
        And some syntax..yeah, but there are some test files and email.pm builds on Account.pm
      • 2015-12-21 35505, 2015

      • gcilou_
        i think
      • 2015-12-21 35549, 2015

      • opatel99
        Leftmost: LordSputnik Idk why i hadnt requested a static tunnel url. https://bookbrainz.localtunnel.me/ has latest updates
      • 2015-12-21 35536, 2015

      • gcibot has quit
      • 2015-12-21 35551, 2015

      • gcibot joined the channel
      • 2015-12-21 35530, 2015

      • opatel99
        gcilou_: I dont even see an Accounts.pm
      • 2015-12-21 35541, 2015

      • gcilou_
      • 2015-12-21 35500, 2015

      • gcilou_
        It's in a different file opatel99
      • 2015-12-21 35534, 2015

      • spideyonthego
      • 2015-12-21 35531, 2015

      • stanislas
        spideyonthego: run sudo apt-get update
      • 2015-12-21 35553, 2015

      • gcilou_
        So opatel99, this is the section in email.pm that i'm changing " my $ip = $opts{ip}; my $body = <<EOS; This is a verification email for your MusicBrainz account. Please click on the link below to verify your email address:" and here are my changes so far " my $ip = $opts{ip}; my $userid = $opts{user}->name; my $body = <<EOS; Hello [$userid], This is a verification email for your MusicBrainz account. Please cl
      • 2015-12-21 35556, 2015

      • stanislas
        this vm may have some old archives