[00:03:19] gi11es: Nice work with the metrics dashboards! They are very cool, and I can’t wait to see what we learn from all the new actions. Much, much appreciated! [00:05:53] yup, the new dashboards are super-useful [00:06:41] and the 99th percentile results are scary [00:07:14] 50 sec for image loading and so [00:07:28] obviously a single outlier, but still [00:08:18] yeah you need a decent sample size for that one to be meaningful [00:08:28] but the 90th numbers are decent [00:08:31] that's why I turned it off by default [00:11:19] the maps seem to be broken [00:18:18] hmmm indeed, this is empty: http://multimedia-metrics.wmflabs.org/graphs/mmv_performance_country_api.json [00:19:15] maybe I accidentally deleted something [00:19:44] appears that way :) [00:25:10] marktraceur: https://gitorious.org/analytics/multimedia-limn/merge_requests/10 [00:26:23] (03PS1) 10Gilles: Fix Chrome bug where popstate could still reset scroll position [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/129082 [00:27:53] (03CR) 10Gilles: "Media Viewer bugfix: https://gerrit.wikimedia.org/r/129082" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/127477 (owner: 10Gilles) [00:31:55] however the maps, when restored, don't work locally, so maybe something else is up [00:32:42] but I can't remember if I got the config fixed locally after all the shuffling last time [00:34:10] ah, yeah, that's it, locally I had it mounted on "multimedia", needs to be "mmv" to match production [00:38:07] it's the other way around, I had already fixed it, gah... [00:40:59] another limn map mystery [00:43:04] probably has to do with the updated geo tsvs, rolling back prior to today's multimedia-limn changes doesn't fix it [00:46:30] (03CR) 10Gilles: [C: 032] Add mediawiki.jqueryMsg as a dependency for mmv.ui.fileUsage [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/129016 (owner: 10Amire80) [00:49:08] (03Merged) 10jenkins-bot: Add mediawiki.jqueryMsg as a dependency for mmv.ui.fileUsage [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/129016 (owner: 10Amire80) [00:51:01] (03CR) 10Gilles: [C: 04-1] Skip userinfo request if no message depends on the gender. (031 comment) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/127459 (owner: 10Gergő Tisza) [00:59:36] (03PS8) 10Gergő Tisza: Skip userinfo request if no message depends on the gender. [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/127459 [01:04:13] marktraceur: https://gitorious.org/analytics/multimedia/merge_requests/10 it never stops :) [01:17:54] tgr: hah, just left an almost identical comment on that mingle card minutes after you [01:18:10] it's definitely the local cachebusting done on purpose to avoid loading stale code while developing, I think [01:19:20] I imagine it's set up this way for people who want to develop without turning off their cache entirely [01:19:40] makes sense [01:23:18] mmmm now that I'm playing with beta and my cache turned on, with the bigger progress bar it animated quickly even when the image is loaded [01:23:32] the big progress bar flashing by for a fraction of a second is really distracting [01:23:42] the older one probably did this but it was hard to spot [01:29:49] (03PS1) 10Gergő Tisza: Show caption on thumbnails [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/129089 [01:31:16] tgr: do you have some test page/wikicode to test this change? [01:31:26] I've never made a gallery [01:31:45] i used this locally [01:31:51] Image:Newcityhall.jpg|City Hall Image:city.hall.london.arp.jpg|City Hall from the high walkway on Tower Bridge Image:City Hall at night with long exposure.jpg Image:BJs Office IJA.PNG [01:31:55] [01:31:57] uh [01:32:08] irssi is not great about linebreaks apparently [01:32:23] http://pastebin.com/V58EDBXb [01:33:12] i want to set up a proper test page with all the different ways if including an image on beta, but haven't yet gotten around to do it [01:36:39] (03CR) 10Gilles: [C: 032] Show caption on thumbnails [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/129089 (owner: 10Gergő Tisza) [01:37:21] (03Merged) 10jenkins-bot: Show caption on thumbnails [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/129089 (owner: 10Gergő Tisza) [16:33:20] gi11es: rillke was asking me about GSoC planning. Eventually we realized we should probably be talking here where you can tell us what you'd like to do too. [16:33:57] bd808: I haven't been on melange as of late, what's the current state of the process? [16:34:35] They announced Monday, so rillke's project is a go :) [16:35:07] And my other project ended up being dropped, so I should have time to help mentor [16:35:27] (assuming I can learn things fast enough to keep up) [16:35:27] nice, congrats rillke! [16:35:40] Thanks :) [16:35:55] And hi gi11es. [16:35:57] so this is probably going to be a 3-headed mentoring? bawolff was also interested [16:36:14] That would be great. [16:36:42] Having bawolff help would be awesome. [16:36:43] We are wondering whether we should have some weekly hangouts ? [16:36:59] I am UTC + 2 [16:37:07] sounds good to me, I prefer any time before 8pm CET [16:38:00] I was thinking a daily irc standup/checkin and a longer weekly voice/video call where we could do screen sharing and planning [16:38:12] But I'll let gi11es call the shots [16:39:08] I'm UTC-6 which might make me less useful for a standup if rillke prefers to do such things near the start of his day. [16:42:14] I think it makes sense to find a time that works for everyone [16:42:21] I'd suggest finding somewhere to keep a task tracking board too. Maybe in the phabricator testing instance? [16:46:57] phabricator sounds good to me [16:48:07] I think it's possible to find a time that is to everyone's contentment. http://doodle.com/7gumuank5zivf6p3 [16:49:18] Yeah, phabricator would be cool. Is it considered stable enough? [16:50:12] For certain values of "stable" :) I don't think Chad's going to randomly delete the instance. [16:50:39] rillke: Who's timezone are those times in? Yours I assume. [16:51:41] bd808: I was assuming doodle converting them automatically ... [16:52:13] It's starting UTC 2:00 PM [16:52:15] * bd808 sees 16:00-19:00,20:00 [16:55:10] bd808: corrected that [16:55:18] now all are UTC [17:31:09] fabriceflorin: I don't know if Jan or Denis have already proofread it, but's it's here: https://www.mediawiki.org/w/index.php?title=Multimedia/Media_Viewer/Survey/de&diff=977890&oldid=977605 [17:32:14] se4598: Thanks so much for this translation, much appreciated! I will update the survey shortly and let you guys know when it’s up. :) [17:43:01] gi11es: the percentile/histogram ticket was accepted, but i don't think we have histograms yet [17:43:16] are we giving up on those, or should i split out to a separate card? [18:26:55] fabriceflorin: "(note we propose to shorten ‘Discuss this feature’ down to just ‘Discussion’ to make more )" you don't think that makes it too vague? [18:27:05] Like it would get misinterpreted as "discuss this file"? [18:28:03] marktraceur: I don’t see that as an issue, since it’s coming right after ‘About Media Viewer’ — otherwide the whole line gets too cluttered. [18:28:49] But if you think it all fits well, I can go either way. Just consider that it will be twice as long on German Wikipedia, for example. [18:29:03] Hrm. [18:29:59] Not a show-stopper either way. Just an idea to keep things shorter. [18:30:44] I mean, I guess it would be fine, but I see it as a danger [18:30:54] I don't want to end up with AFT again [18:42:03] fabriceflorin: I think I will add the link and worry about the message change as a separate card. [18:43:16] marktraceur: That’s fine for now. Let’s see how it looks in practice. We should also ask pginer about this. [18:43:31] *nod* [18:43:37] Argh not on IRC [19:04:27] marktraceur: how strongly are you feeling about the valueless variable declarations first thing? [19:04:35] seems kind of pointless to me [19:06:30] I think there's a convention, so...medium-strong? [19:06:41] If it turns out there isn't then I can lower it to "low-strong" [19:07:19] i didn't see any, might have looked for it in the wrong place [19:08:07] tgr: For JS it's a Wikimedia code convention. [19:09:15] ideally, coding conventions would show up in https://www.mediawiki.org/wiki/Manual:Coding_conventions/JavaScript , right? [19:09:57] i think this one is not mentioned [19:19:47] fabriceflorin: I posted to eight of the nine wikis, MatmaRex posted to Polish in Polish for us https://pl.wikipedia.org/wiki/Wikipedia:Tablica_og%C5%82osze%C5%84/Og%C5%82oszenia [19:20:14] Keegan: Wonderful, thank you so much! [19:20:30] So now I'm going to try to hunt down a Slovak translator annnnddddd we're ready from the community side (less the Slovak translator) [19:21:16] Keegan: Splendid, thanks. Do you mind adding the relevant announcement links to our release plan, for the record? https://www.mediawiki.org/wiki/Multimedia/Media_Viewer/Release_Plan [19:21:44] No need, they're reminders under the announcement for the most part [19:22:08] 'cept for vi.wp and he.wp where the Village Pumps are really active [19:22:15] I think it's okay [19:31:17] tgr: I've added it to https://www.mediawiki.org/wiki/Manual:Coding_conventions/JavaScript#Declarations but I'm sure Krinkle will have thoughts and may revert/reword when I get back. :-) [19:32:28] thx [19:32:45] although it is kind of self-contradictory now [19:34:13] or does that mean that unassigned variables should be declared in rough order of being used? [20:29:41] (03PS1) 10Gergő Tisza: Make License.isCc() really return booleans [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/129295 [20:30:21] (03CR) 10jenkins-bot: [V: 04-1] Make License.isCc() really return booleans [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/129295 (owner: 10Gergő Tisza) [20:31:20] (03PS2) 10Gergő Tisza: Make License.isCc() really return booleans [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/129295 [20:32:10] (03PS1) 10MarkTraceur: Add preferences link after "help" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/129296 [20:32:41] (03CR) 10jenkins-bot: [V: 04-1] Add preferences link after "help" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/129296 (owner: 10MarkTraceur) [20:42:52] (03PS7) 10Gergő Tisza: Use sinon.js mocks to test the thumbnail fetching [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/126912 [22:58:56] (03CR) 10MarkTraceur: [C: 032] "Thanks!" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/129295 (owner: 10Gergő Tisza) [22:59:26] (03Merged) 10jenkins-bot: Make License.isCc() really return booleans [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/129295 (owner: 10Gergő Tisza) [23:25:41] I felt like this needed to be done so I did it: https://www.mediawiki.org/wiki/Extension:MultimediaViewer/Extension:Metadata [23:26:49] 1. First code that modifies MMV from the outside 2. Useful dev tool 3. Fun as hell to write [23:27:28] i thought we wanted to discourage making MMV freely modifiable [23:27:32] It could be cleaner [23:28:04] tgr: Maybe, but that's a long-term concept, and we'd definitely want to allow people to add new UI elements [23:28:10] So I can port it if necessary [23:28:45] this will break any time an internal detail is changed [23:29:11] which is fine for a debug tool, but telling people to write gadgets this way would be... scary [23:29:22] Yeah [23:29:36] I can note this on the pages. [23:34:04] So exciting. [23:43:26] Hah, catrope is our fake user [23:43:27] Awesome [23:47:04] (03CR) 10MarkTraceur: [C: 032] "Seems good! Thanks, tgr!" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/127459 (owner: 10Gergő Tisza) [23:47:35] (03Merged) 10jenkins-bot: Skip userinfo request if no message depends on the gender. [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/127459 (owner: 10Gergő Tisza) [23:47:57] tgr: https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/435 looks done to me, just confirming [23:49:01] yeah, the userinfo query skipping was the last part [23:52:16] Cool