[00:01:26] <grrrit-wm>	 (03CR) 10MarkTraceur: [C: 032] "Sure!" [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/95173 (owner: 10Gergő Tisza)
[00:01:28] <grrrit-wm>	 (03Merged) 10jenkins-bot: Handle remote categories in a ForeignDBRepo setup [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/95173 (owner: 10Gergő Tisza)
[00:02:15] * marktraceur  waits for beta to catch up

[00:02:17] <marktraceur>	 Thanks tgr
[00:03:17] <aarcos>	 marktraceur: Need credentials for GitHub?
[00:03:28] <marktraceur>	 aarcos: No you don't, we don't use GitHub
[00:03:40] <marktraceur>	 You'll need credentials for Gerrit though
[00:04:10] <aarcos>	 tx !, let me continue...
[00:04:16] <marktraceur>	 aarcos: Go to https://gerrit.wikimedia.org and click the "register" link in the top right corner, I think it's pretty self-explanatory from there
[00:09:14] <tgr>	 marktraceur: thx
[00:09:46] <marktraceur>	 My pleasure
[00:09:51] <marktraceur>	 Now if only it would update on beta already
[00:10:30] <tgr>	 how do you see when that happens?
[00:10:40] <marktraceur>	 The job will run
[00:10:44] <marktraceur>	 https://integration.wikimedia.org/ci/view/Beta/
[00:10:52] <marktraceur>	 When code-update runs and finishes, we can test
[00:11:13] <marktraceur>	 I strongly encourage you to try auto-refresh, it makes for way more fun
[00:19:17] <marktraceur>	 tgr: http://en.wikipedia.beta.wmflabs.org/w/api.php?action=query&prop=imageinfo&titles=File:Title_0.6229354626122394.png&iiprop=extmetadata looks not working still
[00:19:40] <marktraceur>	 Let me just confirm it's got the latest version
[00:21:22] <marktraceur>	 Totally does
[00:22:05] <marktraceur>	 Maybe because there are two entries for it in the repo list. Weird.
[00:22:23] <marktraceur>	 Compare http://en.wikipedia.beta.wmflabs.org/w/api.php?action=query&meta=filerepoinfo and http://en.wikipedia.org/w/api.php?action=query&meta=filerepoinfo
[00:25:47] <tgr>	 looks like File::getDescriptionText() is also not working with commons, which is strange because it worked on MediaWiki as far as i can recall
[00:25:57] <marktraceur>	 Hm.
[00:26:14] <tgr>	 anyway, i'll have a closer look at that tomorrow, it's getting late
[00:26:18] <marktraceur>	 'kay
[00:26:36] <marktraceur>	 Nothing's crashing, so I won't revert
[00:29:57] <chrismcmahon>	 marktraceur: this is "-multimedia" I assume
[00:31:52] <marktraceur>	 Yarp
[00:33:03] <chrismcmahon>	 marktraceur: at some point I would like to figure out how to monitor services that multimedia (including UploadWizard) depend on.   
[00:34:41] <marktraceur>	 Sure
[00:34:50] <marktraceur>	 chrismcmahon: Services like what?
[00:36:51] <chrismcmahon>	 marktraceur: exactly.  there is something called an 'upload stash' I hear of.   
[00:37:25] <marktraceur>	 Ah.
[00:37:37] <marktraceur>	 I think that's just part of the MediaWiki API
[00:37:40] <marktraceur>	 But I could be wrong
[00:38:02] <chrismcmahon>	 marktraceur: what aspects of operation are unique to Multimedia?  thumbnails?  rendering? 
[00:38:31] <marktraceur>	 Thumbnails mostly, but also transcoding (once we start serving MP4 files, if we start)
[00:38:43] <chrismcmahon>	 marktraceur: checking that sort of thing in some random browser is not very efficient, or even useful
[00:38:52] <marktraceur>	 I guess so
[04:43:39] <bawolff>	 Hmm, extmetadata is also a prop of stashedimageinfo. I wonder if that actually works
[04:45:41] <bawolff>	 meh, that api module is already so messed up anyhow
[17:46:39] <marktraceur>	 Righto
[17:46:48] <marktraceur>	 tgr: Any luck figuring out the categories patch?
[17:49:03] <tgr>	 partially, i commented in the bug
[17:49:12] <tgr>	 everything works, except beta commons
[17:49:40] <tgr>	 i will look deeper into it, but i don't think it matters
[17:49:40] <marktraceur>	 Ah.
[17:49:45] <marktraceur>	 Well
[17:49:54] <marktraceur>	 If the betacommons setup is, like, stupidly unique
[17:50:05] <marktraceur>	 But if it's a plausible setup, we should probably support it
[17:50:16] <marktraceur>	 Oh wait no.
[17:50:23] * marktraceur  forgot this is a temporary thing

[17:50:24] <tgr>	 i don't have a shared db setup locally
[17:50:26] <marktraceur>	 IGNORE ALL THE THINGS
[17:50:36] <tgr>	 much less a loadbalanced-shared-db
[17:50:55] <marktraceur>	 tgr: So long as we can test it on testwikis before it goes out, I guess :)
[17:50:59] <tgr>	 so i can't be sure that it is not related to that
[17:51:40] <marktraceur>	 Cool, so the last things for our potential release on Thursday are, what, the UI stuff
[17:51:43] <tgr>	 but it works with the old version on mw.org, it works with the current version on my vagrant + betacommons as a remote, and any other combination i could come up with
[17:51:56] <marktraceur>	 Which is pending review...and then there's probably some other stuff that just straight up didn't get done
[17:52:02] <tgr>	 so yes, it should be tested as soon as we have released
[17:52:26] <marktraceur>	 Well, that's good then
[17:52:36] <tgr>	 and before that on testwiki, but the version on commons might also matter...
[17:52:38] <marktraceur>	 Ooh, captions, that's an easy start to the day
[17:52:45] <marktraceur>	 Urgh
[17:52:48] <marktraceur>	 We need testcommons clearly
[17:53:05] <tgr>	 but i looked at the code and i think this is something weird with beta
[17:53:15] <tgr>	 yes, testcommons would make things a lot easier
[17:53:33] * marktraceur  asks

[17:56:16] <chrismcmahon>	 marktraceur: beta commons should be a plausible setup, and if it is not, then that is a bug in beta commons
[17:57:15] <chrismcmahon>	 marktraceur: I'd really like to use beta commons for multimedia testing if at all possible, and if the current configuration makes that difficult, then let's change the current configuration
[17:57:50] <marktraceur>	 tgr: Can you explain to chrismcmahon what's up with betacommons?
[18:00:00] <wmfmedia-standup>	 Stand-up!
[18:00:00] <wmfmedia-standup>	 Order: marktraceur -> bd808 -> fabriceflorin -> tgr
[18:00:30] <marktraceur>	 stfu wmfmedia-standup
[18:00:34] <marktraceur>	 You don't know what you're saying
[18:01:06] <bd808>	 marktraceur: Are we having the weekly meeting this week?
[18:01:14] <marktraceur>	 We are, in one hour
[18:01:20] <bd808>	 +1
[18:01:35] <marktraceur>	 Exactly
[18:02:27] <chrismcmahon>	 tgr: marktraceur beta labs gets improvements when people need to use it for things it doesn't immediately support.  we are in a position to make changes there, and I'd rather do that than a new one-off env
[18:06:38] <bd808>	 chrismcmahon: Speaking of beta, have you seen the Logstash instance running in Labs that is indexing log data from beta yet?
[18:07:12] <chrismcmahon>	 bd808: I have not  (although I was hoping something would have logged the 503 errors I saw there this morning)
[18:07:39] <bd808>	 chrismcmahon: I'll pm a link
[18:08:16] <chrismcmahon>	 thanks bd808
[18:09:49] <tgr>	 chrismcmahon: i have yet to figure out why it is not working on beta
[18:10:15] <tgr>	 but regardless, we cannot currently test things after the first round of deployment
[18:10:31] <tgr>	 or can, but not very reliably
[18:12:00] <tgr>	 with beta, the main problem is that it uses commons as a ForeignDBRepo, and beta commons as a ForeignApiRepo
[18:12:15] <tgr>	 or maybe the other way around?
[18:13:19] <tgr>	 i had some trouble with that since CommonsMetadata has to handle the two types of foreign repo a bit differently
[18:15:06] <tgr>	 i guess it depends on what we want to test since ForeignAPIRepo is good for testing InstantCommons functionality, and that is also something we don't want to break
[18:16:35] <tgr>	 it would be nice to have vagrant support for a local setup with two sites which behave like wikipedia and commons
[18:18:04] <bd808>	 tgr: That would be nice. S asked at some point about vagrant support for multiple wikis too.
[18:18:57] <bd808>	 I looked at it briefly and couldn't see an "easy" way to support it, but I don't think it's impossible
[18:19:08] <grrrit-wm>	 (03PS1) 10Gergő Tisza: Use data provided by remote CMD only when appropriate [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/96288 
[18:20:39] <tgr>	 recent versions of vagrant support for multiple VMs in one file
[18:20:56] <tgr>	 so it should definitely be possible
[18:21:05] <chrismcmahon>	 tgr: would you file a bugzilla ticket for that, under Wikimedia-labs/deployment-prep?
[18:21:16] <tgr>	 i will
[18:21:28] <grrrit-wm>	 (03CR) 10Brion VIBBER: [C: 04-1] "Looking awesome! A couple problems:" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/95081 (owner: 10MarkTraceur)
[18:22:08] <chrismcmahon>	 thanks tgr that would be really helpful
[18:22:56] <marktraceur>	 Woo a brionreviewer
[18:25:53] <grrrit-wm>	 (03CR) 10Brion VIBBER: "Mouseover display of the close/zoom controls and the detail panel don't work at all in IE 11 on Windows RT 8.1 tablet (pure touch mode, no" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/95081 (owner: 10MarkTraceur)
[18:26:32] <brion>	 somebody's gotta test these windows tablets :D
[18:28:59] <marktraceur>	 brion: Sure as hell won't be me
[18:29:03] <marktraceur>	 Wait, that's weird
[18:29:04] <brion>	 :D
[18:29:17] <marktraceur>	 brion: Do they not get the .mobile display? I set up a CSS rule just for body.mobile.
[18:29:33] <marktraceur>	 Not for the details display, I haven't figured that out, but the controls should be visible
[18:29:43] <brion>	 windows 8 IE gets desktop site...
[18:30:06] <marktraceur>	 Argh.
[18:31:03] <grrrit-wm>	 (03CR) 10MarkTraceur: "Excuse me while I swear at Gerrit's reply features." [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/95081 (owner: 10MarkTraceur)
[18:31:29] <marktraceur>	 brion: Is there some way I can tell, in CSS or JS, that you're running in touch-only mode?
[18:31:32] <marktraceur>	 'cause I mean
[18:31:41] <marktraceur>	 "Desktop IE8" could mean a lot of things
[18:31:44] <brion>	 marktraceur: supposedly there is, let me look it up
[18:31:52] <marktraceur>	 'kay
[18:32:04] <marktraceur>	 OOh, sorry, Windows 8 IE11.
[18:32:11] * marktraceur  conflated some version numbers there

[18:34:38] <brion>	 hmmm some general advice at http://msdn.microsoft.com/en-us/library/ie/jj583807%28v=vs.85%29.aspx but i don't see the 'identify that there's no pointer device' thing. lemme search some more
[18:34:54] <brion>	 oh -- accepting click to toggle hover state would probably do it though :D
[18:34:59] <brion>	 but lemme find that i'm sure it's out there
[18:35:18] <grrrit-wm>	 (03PS6) 10MarkTraceur: Reorganize UI to be...way sexier. [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/95081 
[18:35:50] <marktraceur>	 brion: It might just be that we need a different design for the controls
[18:35:53] <marktraceur>	 I'd be fine with that
[18:36:08] <brion>	 *nod*
[18:36:39] <marktraceur>	 pg<tab><tab> god DAMN it
[18:36:49] * marktraceur  frustrated with people who sign out of IRC

[18:36:53] <brion>	 bah. http://blog.stevelydford.com/2012/03/detecting-touch-hardware-in-ie-10/ <- shows how you can identify presence of touchscreen, but that doesn't tell you if there's no keyboard/mouse
[18:37:26] <brion>	 lots of laptops have touch now so that's meaningless
[18:37:34] <marktraceur>	 Yeah it kinda is
[18:37:54] <marktraceur>	 I'm just going to tell pginer that we're not blocking the image AND we're not waiting for hover, so he can just deal with it
[18:38:39] <brion>	 :D
[18:38:50] <brion>	 overall though it's looking way sexier indeed :D
[18:39:45] <brion>	 woops
[18:39:53] <marktraceur>	 Well that's great
[18:47:38] <marktraceur>	 Oh, hm, I should +V aaron_arcos
[18:48:06] <marktraceur>	 Hm, wrong one
[18:49:05] <grrrit-wm>	 (03CR) 10Brion VIBBER: "Per IRC discussion..." [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/95081 (owner: 10MarkTraceur)
[18:49:57] <marktraceur>	 aarcos: I meant, if you type /nick aarcos, you don't need to restart the app :)
[19:18:23] <dan-nl>	 bd808 just saw your comment on the mediafile-job-array commit. do you want me to make that change? i like the explicitness of using count() > 0, but if you think it's better to use the falsi test instead i'll use it
[19:19:25] <bd808>	 dan-nl: I don't think it really matters either way. I'd write it the other way, but it's probably personal preference more than anything else.
[19:19:49] <dan-nl>	 k, i'll leave it for now, thanks
[19:23:21] <dan-nl>	 bd808: thanks for the rebase … did you rebase against mediafile-job-array or master?
[19:23:58] <bd808>	 I just pushed the rebase button. I think that should have rebased against the ancestor patch
[19:24:33] <bd808>	 I was seeing bleed from the ancestor in the diff but the rebase seems to have fixed that
[19:26:48] <dan-nl>	 ah ok
[19:27:00] <dan-nl>	 where's the rebase button?
[19:32:46] <bd808>	 dan-nl: It shows up in the same area as the "Review" button when the changeset can/should be rebased
[20:36:00] <grrrit-wm>	 (03PS3) 10Gergő Tisza: Add generic CC license parsing [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/94931 
[20:36:22] <tgr>	 marktraceur: could you have a look at ^^?
[20:38:21] <marktraceur>	 tgr: After deploying VectorBeta, I think
[20:45:24] <tgr>	 thanks
[22:41:56] <grrrit-wm>	 (03PS7) 10MarkTraceur: Reorganize UI to be...way sexier. [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/95081 
[22:42:04] <marktraceur>	 Fixed the background thing
[22:42:08] <marktraceur>	 (in fullscreen)
[22:42:53] <marktraceur>	 Fixing the metadata hover effect next...probably by just shifting to using JS events instead of CSS
[23:36:07] <grrrit-wm>	 (03PS8) 10MarkTraceur: Reorganize UI to be...way sexier. [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/95081 
[23:36:14] <marktraceur>	 Bam
[23:36:17] <marktraceur>	 Scrolls
[23:36:24] <marktraceur>	 ...the controls are gone again though
[23:36:25] * marktraceur  fixes

[23:36:59] <marktraceur>	 Oh, no they aren't
[23:37:02] <marktraceur>	 I just didn't see them
[23:50:56] <grrrit-wm>	 (03CR) 10MarkTraceur: [C: 04-1] "So, I've stopped the hover effect on the metadata div, it's just scrolled off the screen now. I also stop the background article from scro" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/95081 (owner: 10MarkTraceur)
[23:53:25] <marktraceur>	 All: I've deployed the latest version to -alpha