[01:12:40] (PS6) Jdlrobson: Use NewEditorEdit schema for new graph [analytics/limn-mobile-data] - https://gerrit.wikimedia.org/r/105869 [12:37:45] hey nuria [12:37:58] hello average [14:22:09] (PS2) Milimetric: README improvements - httplib2 [analytics/wikimetrics] - https://gerrit.wikimedia.org/r/107710 (owner: Sharihareswara) [14:23:28] (CR) Milimetric: [C: 1] "I took out the venv suggestion because I actually prefer it to not be in a venv due to the celery issues. Once we fix those perhaps we ca" [analytics/wikimetrics] - https://gerrit.wikimedia.org/r/107710 (owner: Sharihareswara) [14:52:18] Believe it or not ... Goggle allowed me to join the Batcave again! \o/ Yippie. [14:52:33] Nothing prohibits me from haveng tons of meetings again. Yay. [14:52:41] Oh well ... :-( [15:00:32] ottomata: Scrum time :-) [15:50:02] have to run to get somethin, might be a couple minutes late for tasking [15:50:53] np [15:58:50] back [16:34:34] back in 15 [17:45:35] lunchtime, brb [17:50:46] Hi everyone. Is Dario or Toby around? (I don't know their handles) [17:51:09] sayhar: Dario is DarTar, Toby is tnegrin. Neither are online. [17:51:17] Thanks ori [17:53:04] halfak, Ironholds: http://www.amazon.com/Mining-Generated-Content-Social-Computing/dp/1466557400 [17:53:28] it showed up in a new books newsletter that I subscribe to, looks interesting [18:01:53] (PS3) Sharihareswara: README improvements - httplib2 [analytics/wikimetrics] - https://gerrit.wikimedia.org/r/107710 [18:02:51] (CR) Sharihareswara: "Took out the venv suggestions." [analytics/wikimetrics] - https://gerrit.wikimedia.org/r/107710 (owner: Sharihareswara) [18:03:08] milimetric: *wave* [18:03:37] brion: code review? [18:03:41] whoops, wrong channel [18:03:42] milimetric: I don't actually see a difference between patchset1 and patchset2 in https://gerrit.wikimedia.org/r/#/c/107710/ btw [18:11:48] Would it be a pain for me to submit a changeset for Wikimetrics where I just clean up a bunch of extra whitespace and PEP8 issues? [18:15:02] sumanah: why a pain? [18:15:37] I can help review too, to take some load off milimetric (my python is decent, so should be okay even though I don't really know wikimetrics) [18:18:48] YuviPanda: oh cool! ok [18:18:58] YuviPanda: I know some maintainers think it's a waste of time [18:19:08] ah! [18:19:25] I guess that's for milimetric to decide, but I personally don't [18:19:45] cool [18:19:45] sumanah: either way, feel free to add me as a reviewer to any python patches :) [18:19:50] thanks YuviPanda! :D [18:19:53] yw! [18:20:33] https://gerrit.wikimedia.org/r/#/c/107715/ it is :) [18:21:02] (PS2) Yuvipanda: capitalization in README [analytics/wikimetrics] - https://gerrit.wikimedia.org/r/107715 (owner: Sharihareswara) [18:21:14] (CR) Yuvipanda: [C: 2] Capitalization in README [analytics/wikimetrics] - https://gerrit.wikimedia.org/r/107715 (owner: Sharihareswara) [18:21:22] YuviPanda: if you feel like trying to help me figure out a crash I started seeing last night: http://tools.wmflabs.org/paste/view/385b0c7b [18:21:24] * YuviPanda wonders if jenkins is setup [18:21:45] sumanah: looking [18:22:49] oh hmmm I think this happened right after I also git cloned another repo and meant to install it in a subdirectory of wikimetrics/ [18:23:03] I would not be surprised if now there are sort of two competing repos or something strange like that [18:23:07] hmm, yeah, looks like a bit of a path issue to me [18:23:32] sumanah: had you run 'source bin/activate' before executing this one? [18:23:38] and I also dunno where those git fatals were coming from [18:23:59] (CR) Yuvipanda: [V: 2] Capitalization in README [analytics/wikimetrics] - https://gerrit.wikimedia.org/r/107715 (owner: Sharihareswara) [18:24:08] YuviPanda: I was already in the venv and it was active (I use virtualenvwrapper so I just do "workon [name of venv]") [18:24:13] ah, right. [18:24:30] hmm, sorry, nothing pops out from there. [18:24:49] sumanah: I merged ^ btw :) Made a tiny modification to the commit message. [18:25:38] capitalizing "Capitalization"? :) [18:26:12] (CR) Milimetric: [C: 2 V: 2] README improvements - httplib2 [analytics/wikimetrics] - https://gerrit.wikimedia.org/r/107710 (owner: Sharihareswara) [18:26:30] sumanah: :D yeah [18:26:47] You jester :) [18:26:53] :) [18:26:58] the Birbal of --amend [18:27:11] haha! [18:27:19] that's a nice one. [18:27:26] * YuviPanda files it away in head to reuse some other time [18:28:55] Man, i posted wrong again on analytics list, sorry about that , tried to repost but I think gmail reformated my e-mail funny [18:29:05] YuviPanda: :D [18:29:05] you're right sumanah, sorry 'bout that, gerrit snafu [18:29:19] and I definitely don't think reviewing wikimetrics stuff is a waste of time [18:29:19] we're planning some pretty big new features for it and the healthier it is the happier I am [18:29:19] so quite the opposite [18:29:20] If you know others that are interested, bring them on :) [18:29:20] nuria: it happens to everyone - mailing lists sometimes don't quite work right with all email interfaces :/ [18:29:45] milimetric: :D (I just meant the nitpicky whitespace-on-a-blank-line stuff can feel tedious to look at) [18:30:02] milimetric: and np about the gerrit snafu, I just wanted to double-check that I wasn't missing something! [18:31:08] I must've been offline while you were talking about nitpicky whitespace, /me reading logs [18:31:08] milimetric: if you have any ideas about this problem I started seeing last night http://tools.wmflabs.org/paste/view/385b0c7b I'd welcome your thoughts. I'm trying to google around for key phrases and not seeing an AHA moment [18:31:29] ah! I see your comment [18:31:36] "Would it be a pain for me to submit a changeset for Wikimetrics where I just clean up a bunch of extra whitespace and PEP8 issues?" [18:31:39] so we use flake8 [18:31:52] and we have a setup.cfg that has a flake8 section [18:32:05] which specifically ignores certain rules [18:32:09] oh! [18:32:11] and the whitespace on blank lines is intentional [18:32:19] btw YuviPanda check out http://www.qc4blog.com/?p=1528 - the captcha :) [18:32:27] believe it or not, a majority of people working on wikimetrics like it :) [18:32:35] looking, sumanah [18:32:55] sumanah: hah! :) [18:33:08] halfak: there? [18:33:14] milimetric: it is! ah. ok. I have an emacs rule that I'll disable ((add-hook 'before-save-hook 'delete-trailing-whitespace)) [18:33:34] trailing whitespaces are not ok [18:33:52] i think flake8 complains about that [18:33:52] or should [18:33:53] but whitespaces on blank lines are not trailing [18:33:58] they're just indented to show that they're part of the same code block [18:34:51] I think Emacs's definition treats them as similar, which sounds wrong. I can use pep8ify very granularly to fix *trailing* whitespace on nonblank lines [18:34:56] sumanah: ^ [18:35:05] cool, cool. Got it [18:35:09] :) [18:35:44] you know what? I just realized that I got so stuck on the first parts of the README that I missed explanations of this which are at the end [18:35:47] I am silly Sumana [19:09:18] milimetric: https://bugzilla.wikimedia.org/show_bug.cgi?id=22629 re reply-by-email [19:12:06] thank you sumanah! [19:12:07] http://lists.wikimedia.org/pipermail/analytics/2014-January/001509.html [19:12:10] lol