[01:15:38] <marktraceur>	 Everyone ready for BUGSPAAAAAAAAAM????
[01:15:43] <marktraceur>	 Too bad you're getting it anyway
[01:25:28] <grrrit-wm>	 (03PS1) 10MarkTraceur: Bump version number [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/96411 
[07:51:44] <quiddity>	 yay bugspam!  :D
[08:29:09] <grrrit-wm>	 (03CR) 10Gergő Tisza: [V: 032] Bump version number [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/96411 (owner: 10MarkTraceur)
[11:10:35] <grrrit-wm>	 (03PS1) 10Gergő Tisza: Preserve space for the description even if it is not present [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/96465 
[15:12:27] <grrrit-wm>	 (03PS1) 10Gergő Tisza: Add Attribution field to CommonsMetadata output [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/96493 
[17:46:30] <marktraceur>	 aarcos: https://gerrit.wikimedia.org/r/96152
[17:49:04] <grrrit-wm>	 (03PS1) 10Gergő Tisza: Make sure local URLs in image description are always absolute [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/96528 
[17:56:45] <bd808>	 The hangout link has disappeared from the bug triage invite
[17:57:51] <grrrit-wm>	 (03PS2) 10Gergő Tisza: Make sure local URLs in image description are always absolute [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/96528 
[18:00:00] <wmfmedia-standup>	 Stand-up!
[18:00:00] <wmfmedia-standup>	 Order: bd808 -> fabriceflorin -> marktraceur -> tgr
[18:00:45] * bd808  stands up and tells wmfmedia-standup to sit down

[18:01:16] <marktraceur>	 bd808++
[18:04:16] <bd808>	 marktraceur: When fab<tab> shows up tell him about the disappearing hangout link
[18:04:37] <bd808>	 And don't have him post the link here in the public channel :)
[18:04:37] <marktraceur>	 Uh
[18:04:42] <marktraceur>	 Kay
[18:04:50] <marktraceur>	 I don't see him as yet
[18:04:52] <bd808>	 [10:56]  <    bd808>	 The hangout link has disappeared from the bug triage invite
[18:05:00] <bd808>	 Email said "late bus"
[18:05:05] <marktraceur>	 Ah, right
[18:06:20] * bd808  goes back to more gorram code reviews

[18:07:32] <marktraceur>	 bd808: Try again?
[18:07:59] <bd808>	 Yup. Thanks
[18:08:12] <marktraceur>	 tgr: Joining us?
[18:26:10] <tgr>	 got disconnected
[19:10:43] <marktraceur>	 fabr<tab> argh
[19:12:40] <tgr>	 marktraceur: will you have time for some code review today?
[19:12:57] <marktraceur>	 Yeeeesssmaybe
[19:13:01] <marktraceur>	 tgr: Whatcha need reviewed?
[19:13:20] <tgr>	 ideally, everything :)
[19:13:35] <marktraceur>	 Fun times
[19:13:37] <marktraceur>	 I'll get on that
[19:13:57] <tgr>	 but mostly 94931 and 96288
[19:14:11] <tgr>	 not particularly urgent, mind you
[19:14:59] <tgr>	 i just want to reshuffle stuff in CMD and write more tests, and want to avoid rebase hell
[19:16:29] <marktraceur>	 'kay
[19:16:31] <marktraceur>	 Will do
[19:16:45] <grrrit-wm>	 (03CR) 10MarkTraceur: [C: 032 V: 032] "I like it." [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/96465 (owner: 10Gergő Tisza)
[19:16:57] <marktraceur>	 ...V2 was a mistake, but Jenkins will merge
[19:16:59] <grrrit-wm>	 (03Merged) 10jenkins-bot: Preserve space for the description even if it is not present [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/96465 (owner: 10Gergő Tisza)
[19:24:46] <tgr>	 thanks
[19:46:43] <grrrit-wm>	 (03CR) 10MarkTraceur: [C: 04-1] "Still not sure cc0 will be handled properly" (033 comments) [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/94931 (owner: 10Gergő Tisza)
[19:59:04] <grrrit-wm>	 (03CR) 10MarkTraceur: [C: 04-1] "-1 for line 82" (033 comments) [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/96288 (owner: 10Gergő Tisza)
[20:21:04] <grrrit-wm>	 (03CR) 10Gergő Tisza: "It won't. How should it be handled? Do we want to display the CC logo for such images? CC0 is basically just a fancy way of saying somethi" (031 comment) [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/94931 (owner: 10Gergő Tisza)
[20:21:14] <grrrit-wm>	 (03PS4) 10Gergő Tisza: Add generic CC license parsing [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/94931 
[20:21:44] <grrrit-wm>	 (03CR) 10Gergő Tisza: "Added missing docs." [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/94931 (owner: 10Gergő Tisza)
[20:26:11] <marktraceur>	 tgr: Can't exactly leave out CC0 in a patch that says "Generic CC license handling" :P
[20:26:23] <marktraceur>	 I don't see any reason not to show the CC logo for that
[20:27:05] <marktraceur>	 Hrm
[20:27:18] <marktraceur>	 Did you get my last messages?
[20:27:33] <tgr_>	 probably not
[20:28:01] <tgr_>	 contact problems with the power plug + linux failing to alert when battery is low -> not a good combination
[20:28:31] <marktraceur>	 Ah.
[20:28:35] <marktraceur>	 tgr: Can't exactly leave out CC0 in a patch that says "Generic CC license handling" :P
[20:28:39] <marktraceur>	 I don't see any reason not to show the CC logo for that
[20:29:32] <tgr>	 fair enough
[20:29:50] <marktraceur>	 Luckily it's easy to fix :)
[20:30:27] <tgr>	 also, there was another PD CC license
[20:30:42] <tgr>	 the CC public domain certification or something like that?
[20:31:36] <marktraceur>	 Waiver probably
[20:31:44] <marktraceur>	 CC-PD or something equally silly
[20:31:49] <tgr>	 yes
[20:31:53] <marktraceur>	 I wouldn't mind supporting that but it's less critical
[20:32:12] <tgr>	 less critical than CC0? :)
[20:32:18] <marktraceur>	 Yes
[20:32:24] <marktraceur>	 CC0 has some adoption on Commons
[20:32:28] <marktraceur>	 AFAIK CC-PD doesn't
[20:32:37] <marktraceur>	 Maybe I'm remembering that wrong?
[20:33:14] <marktraceur>	 400k vs 600
[20:33:17] <tgr>	 no, it seems to be used on a fair number of images
[20:33:27] <marktraceur>	 https://commons.wikimedia.org/wiki/Category:CC-PD
[20:33:51] <tgr>	 never understood the point of it
[20:34:07] <marktraceur>	 Heh
[20:34:22] <marktraceur>	 I suspect that's why CC retired it from use :)
[20:34:35] <marktraceur>	 Maybe I'll ask Kat
[20:34:53] <tgr>	 its basically saying, this image is in the public domain, except for jurisdictions where that is not legal, in which case it is in the public domain
[20:35:17] <marktraceur>	 Yeah
[20:35:21] <tgr>	 anyway
[20:35:28] <marktraceur>	 Copyright is complicated :)
[20:36:01] <tgr>	 i'll just send a 'cc0' string
[20:36:06] <tgr>	 for both of those
[20:36:10] <marktraceur>	 Works for me
[20:55:35] <grrrit-wm>	 (03PS5) 10Gergő Tisza: Add generic CC license parsing [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/94931 
[20:56:06] <grrrit-wm>	 (03CR) 10Gergő Tisza: "Added CC0 handling. Also fixed a nasty bug in the process." [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/94931 (owner: 10Gergő Tisza)
[20:56:35] <marktraceur>	 Cool
[20:56:39] <marktraceur>	 tgr: Runs? Tested?
[20:57:05] <marktraceur>	 I should add a jenkins job for running the unit tests...
[20:58:09] <tgr>	 seems to work, and test are green
[20:58:19] <marktraceur>	 Saweet.
[20:58:26] <grrrit-wm>	 (03CR) 10MarkTraceur: [C: 032] "Huzzah" [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/94931 (owner: 10Gergő Tisza)
[20:58:26] <tgr>	 i'll write some more generic tests soon
[20:58:31] <marktraceur>	 Coolio
[20:58:43] <grrrit-wm>	 (03Merged) 10jenkins-bot: Add generic CC license parsing [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/94931 (owner: 10Gergő Tisza)
[21:09:38] <chrismcmahon>	 marktraceur tgr speaking of which, at some point I assume you'll want some browser tests for multimedia beyond our one UploadWizard test? 
[21:10:04] <marktraceur>	 chrismcmahon: That sounds like A Good Idea
[21:11:35] <grrrit-wm>	 (03CR) 10Gergő Tisza: Use data provided by remote CMD only when appropriate (032 comments) [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/96288 (owner: 10Gergő Tisza)
[21:11:45] <chrismcmahon>	 marktraceur: one thing I'd like to do sooner is to put the UploadWizard browser test into the UploadWizard repo: https://gerrit.wikimedia.org/r/#/c/92875/ . Beyond that I'm a bit unclear about what would be useful to test at the UI level. 
[21:11:53] <grrrit-wm>	 (03PS2) 10Gergő Tisza: Use data provided by remote CMD only when appropriate [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/96288 
[21:11:54] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Use data provided by remote CMD only when appropriate [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/96288 (owner: 10Gergő Tisza)
[21:12:07] <marktraceur>	 Oh, I thought you guys had merged that already
[21:12:30] <marktraceur>	 I'll take a look at it during our UploadWizard Tech Debt Extravaganza
[21:14:45] <chrismcmahon>	 marktraceur: feel free to send me an invite to the Extravaganza
[21:15:31] <marktraceur>	 I don't think it's an actual thing
[21:15:56] <marktraceur>	 I'm just joking about the fact that we're organizing a blocked-out time to deal with UW tech debtn
[21:15:59] <marktraceur>	 debt*
[21:16:02] <marktraceur>	 cannot keyboard
[21:16:56] <chrismcmahon>	 marktraceur: I was thinking lack of UI tests was techdebtular
[21:17:52] <marktraceur>	 A little bit
[21:18:06] <marktraceur>	 I just don't really have a place to put tech-debt things apart from bugzilla
[21:18:10] <marktraceur>	 So...file bugs? :D
[21:19:45] <grrrit-wm>	 (03PS3) 10Gergő Tisza: Use data provided by remote CMD only when appropriate [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/96288 
[21:21:31] <grrrit-wm>	 (03CR) 10Gergő Tisza: "Rebased + fixed medadata." [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/96288 (owner: 10Gergő Tisza)
[21:32:14] <chrismcmahon>	 marktraceur: bugs under Wikimedia/Quality Assurance are welcome
[21:32:34] <marktraceur>	 'kay
[21:33:05] <grrrit-wm>	 (03CR) 10MarkTraceur: [C: 032] "I'll go with it. :)" [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/96288 (owner: 10Gergő Tisza)
[21:33:50] <marktraceur>	 tgr: Are the other two patches basically-fine for now?
[21:34:06] <marktraceur>	 I'd like to sort out the UI redesign if I can, maybe we can get it to mediawiki.org tomorrow (HAH)
[21:34:12] <marktraceur>	 I mean totally.
[21:36:54] <grrrit-wm>	 (03Merged) 10jenkins-bot: Use data provided by remote CMD only when appropriate [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/96288 (owner: 10Gergő Tisza)
[21:40:13] <tgr>	 attribution is super-minor, so easy to rebase, and the other one is really nasty, not sure if i want to actually go along with that
[21:40:27] <tgr>	 so yes, those can wait
[21:40:29] <tgr>	 a lot
[21:41:02] <marktraceur>	 Cool
[23:06:11] <marktraceur>	 OK I am getting _really_ weird results with the new controls code
[23:06:14] <marktraceur>	 But it's coming along
[23:06:46] <grrrit-wm>	 (03PS9) 10MarkTraceur: Reorganize UI to be...way sexier. [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/95081 
[23:07:05] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Reorganize UI to be...way sexier. [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/95081 (owner: 10MarkTraceur)
[23:07:40] <grrrit-wm>	 (03CR) 10MarkTraceur: "Argh. *sobs*" (032 comments) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/95081 (owner: 10MarkTraceur)
[23:39:54] <grrrit-wm>	 (03PS10) 10MarkTraceur: Reorganize UI to be...way sexier. [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/95081 
[23:39:54] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Reorganize UI to be...way sexier. [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/95081 (owner: 10MarkTraceur)
[23:40:00] <marktraceur>	 Sliiiightly better now
[23:40:07] <marktraceur>	 No mobile solution yet I guess
[23:40:15] <marktraceur>	 I feel a little dirty using click to trigger this
[23:43:47] <grrrit-wm>	 (03PS11) 10MarkTraceur: Reorganize UI to be...way sexier. [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/95081 
[23:43:49] <marktraceur>	 Ah well
[23:43:50] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Reorganize UI to be...way sexier. [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/95081 (owner: 10MarkTraceur)
[23:43:50] * marktraceur  rebases

[23:44:47] <grrrit-wm>	 (03PS12) 10MarkTraceur: Reorganize UI to be...way sexier. [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/95081 
[23:44:49] <marktraceur>	 Woo, looks better
[23:48:07] <marktraceur>	 fabriceflorin: Latest lightbox UI redesign patch is on alpha
[23:48:20] <marktraceur>	 I'm suuuuper unhappy about bits of the design, but at least it's working now