[00:02:22] brion: chrome, opera, safari… any others? [00:07:33] mhurd: Yo, 17:30 walk to the BART station sound good to you? [00:07:49] marktraceur: ya! [00:07:54] Coolio [00:11:46] New patchset: Jdlrobson; "Cleanup Skin language and variants" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58459 [00:23:21] New review: Jdlrobson; "(1 comment)" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59722 [00:23:24] New patchset: Jdlrobson; "Bug 30405: Don't special case homepage HTML in alpha" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59722 [00:23:24] New patchset: Jdlrobson; "Make homepage module run outside alpha" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59723 [00:23:32] New review: Jdlrobson; "(1 comment)" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59723 [00:57:34] Change merged: JGonera; [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58459 [01:00:03] Change merged: JGonera; [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59722 [05:36:24] hello all [06:09:23] New patchset: Jdlrobson; "Build content navigation menu" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59985 [06:10:12] jdlrobson what module of the resource loader should the fast click library be added to? [06:10:18] mobile.beta.jquery? [06:10:33] hey pragunbhutani ! you probably want to add it to mobile.alpha [06:11:18] the way we tend to do things in mobile is try them out in production on an experimental alpha version of the site , then move them to beta (mobile.beta) if we think they are useful, then big round of testing before moving to stable for everyone mobile.stable [06:11:47] New patchset: Jdlrobson; "Upstream skin changes - getLanguageUrls" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59987 [06:12:34] jdlrobson: I don't see any mobile.alpha modules right now, so should I create one? [06:12:59] it's in MobileFrontend.php - are you using latest MobileFrontend? [06:13:03] search for $wgResourceModules['mobile.alpha'] = $wgMFMobileResourceBoilerplate + array( [06:13:34] I'm pretty sure I [06:13:42] *I'm using the latest [06:13:48] let me just check that to be sure [06:14:08] pragunbhutani: latest commit is commit 293c572e3cbf847203adcf4351f918dffc169ebf [06:19:15] jdlrobson: there's the problem then [06:19:23] ? [06:19:29] I'd included the extension with installation some time back [06:19:41] ahh you'll need to upgrade.. :) [06:19:45] cloning the latest from git now :) [06:19:58] git clone https://gerrit.wikimedia.org/r/p/mediawiki/extensions/MobileFrontend.git [06:20:28] yep, that's the one I'm cloning [06:21:53] my MW installation is encountering a fatal error now [06:22:30] call to undefined method OutputPage::etProperty() [06:22:42] in /mw/extensions/MobileFrontend/includes/skins/SkinMobile.php on line 202 [06:26:42] pragunbhutani: you'll need to update mediawiki as well.. [06:26:53] hmm [06:26:57] MobileFrontend works off the latest and greatest MediaWiki version! :) [06:27:06] we spend a lot of time upstreaming changes [06:27:27] https://gerrit.wikimedia.org/r/gitweb?p=mediawiki/core.git [06:29:05] jdlrobson: I can't believe how noobishly I've set mediawiki up [06:29:12] nothing is a git repo! [06:29:16] pragunbhutani: don't worry! happens to us all :) [06:29:34] I'm going to clean up this entire thing before I make any changes :p [06:30:45] rm -rf is my friend today [06:33:49] :) [06:33:59] excited to see the results :) [06:55:24] pragunbhutani: gotta go sleep but hopefully see you tomorrow :) [07:55:50] mmm I love the smell of a fresh installation :) [13:42:17] New review: MaxSem; "(2 comments)" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59722 [14:14:50] New review: Jdlrobson; "(2 comments)" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59722 [14:14:55] Change merged: jenkins-bot; [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59710 [14:19:03] New review: MaxSem; "(1 comment)" [mediawiki/extensions/MobileFrontend] (master) C: -1; - https://gerrit.wikimedia.org/r/59987 [14:37:32] New patchset: MaxSem; "[DON'T MERGE, EXTERNAL DEPENDENCY]Upstream skin changes - getLanguageUrls" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59987 [14:37:48] New review: Jdlrobson; "Out of interest does the redirecter not take care of these rules to ensure user gets to mobile site?" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59987 [14:38:36] New review: MaxSem; "Blocking a merge until the dependency is merged." [mediawiki/extensions/MobileFrontend] (master) C: -2; - https://gerrit.wikimedia.org/r/59987 [14:39:33] New review: MaxSem; "Redirector means more requests on mobile connections, and even redirector will not help people brows..." [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59987 [15:26:22] New patchset: Jdlrobson; "Upstream skin changes - getLanguageUrls" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59987 [15:51:55] New patchset: MaxSem; "[DON'T MERGE, EXTERNAL DEPENDENCY] Upstream skin changes - getLanguageUrls" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59987 [16:47:52] MaxSem: since awjr_away is ill can you fix up https://gerrit.wikimedia.org/r/#/c/59765/ so i can merge it? [16:47:58] (setup and teardowns) [16:48:23] on it [16:49:07] New review: Jdlrobson; "(1 comment)" [mediawiki/extensions/MobileFrontend] (master) C: -1; - https://gerrit.wikimedia.org/r/59987 [16:49:11] thanks MaxSem :) [16:49:21] also got a bit confused by your tweak to mobile device detect ^ [16:49:41] owchie [16:49:48] New patchset: Jdlrobson; "Promote menu button generation to MinervaTemplate" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58990 [16:50:01] New patchset: Jdlrobson; "Define render function in MinervaTemplate to output page" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58991 [16:50:12] New patchset: Jdlrobson; "Kill navigationStart" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58992 [16:50:27] New patchset: Jdlrobson; "Promote all the banners into MinervaTemplate" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58993 [16:50:29] jdlrobson, so by enabling $wgMFVaryResources we actually get a more resonable order of stylesheets, right? [16:50:45] MaxSem: in what sense? [16:51:05] currently, device styles are loaded first [16:51:14] and thus can't override anyhting [16:51:36] so if we set the group to other as in https://gerrit.wikimedia.org/r/#/c/59512/ they will come last [16:51:52] otherwise they will be packaged up as RL sees fit :( [16:52:09] we should really reduce the number of requests [16:52:44] New review: Jdlrobson; "The other benefit of this patchset is that 'other' stylesheets are loaded last so will force loading..." [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59512 [16:52:56] MaxSem: i don't see it as such a big deal [16:53:10] the device stylesheet is a last resort [16:53:17] most of the time it is empty [16:53:34] we could remove the request altogether on devices that don't use it [16:53:38] of course! you're using it via Wi-Fi instead of GPRS;) [16:53:56] expedited request: denied [16:53:57] sigh [16:54:01] no we can't - it will require a variance in HTML [16:54:10] MaxSem: i really think this is being blown out of proportion - it's one http request - i'd worry more about our use of the api! :) [16:54:25] MaxSem: that's true yes - i guess we do always have to include it [16:54:35] jdlrobson, I will slowly chew the API usage out of you, too:) [16:55:12] jdlrobson - haha, from the mobile editor survey i put up, in answer to what kind of editing would you like to do on mobile that you can't: "Add pictures to articles that have a infobox" [16:55:20] did i send this to your sockpuppet? [16:55:22] brion, throw your MacBook, iPhone and iPad (and iPod?) out of the window:P [16:55:23] Maryana: :D [16:55:42] yeah i'll switch to microsoft surface ;) [16:56:27] * Maryana plans out iteration after next - editing, talk pages, and images in articles w/infoboxes [16:56:37] wheeee [16:57:30] Maryana, and "you have new messages" - we don't want to inflate the blocklist [16:58:16] brion: http://jonrobson.me.uk/sadbrion.jpg [16:58:16] by that point, echo should be enabled on enwiki - we can probably gank their "you have new messages" notice :D [16:58:40] jdlrobson: nice :D [17:00:14] agh standup time [17:02:20] zz_YuviPanda: poke [17:03:20] duh, no clue, will try to reboot... [17:11:51] brion: denied for expedited. bastards! [17:11:57] yuuuuup [17:12:39] brion: poke [17:12:41] back [17:12:43] hey, at least they were expedient in denying the expediation [17:13:02] YuviPanda: just checking if you were gonna be on the standup, too late now :) [17:13:12] in the meantime, what's up? [17:13:16] heh, I don't usually turn up for Standups on Friday [17:13:25] brion: head is clearer, pain's gone :) [17:13:30] brion: working on analytics stuff. [17:13:30] yay [17:13:33] awesome [17:13:45] diving into mysql indexes on the logging table right now :) [17:14:14] managed to get an order of magnitude speedup in my query by adding a namespace check to a where clause [17:14:18] stupid mysql wih stupid indexes [17:14:34] yay [17:14:38] hehe [17:15:25] jgonera: big win :) https://gerrit.wikimedia.org/r/#/c/59985/ [17:16:11] whee, that works :) [17:16:49] jdlrobson, nice [17:17:10] jdlrobson, won't this generate lots of stuff that we don't need though (isn't gonna be too costly)? [17:17:48] jgonera: on the long term we can reuse a lot of it / encourage refactoring of it [17:18:04] jgonera: for example watch list star is buried in there :) [17:18:21] and the watch list star works without javascript [17:18:26] which will be exciting.. [17:18:26] :) [17:18:34] ok, will test it today [17:18:47] New patchset: Jdlrobson; "Make the universe explode by making a desktop AND mobile skin called Minerva" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58997 [17:18:57] New patchset: Jdlrobson; "Promote addToBodyAttributes to desktop skin" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58257 [17:20:52] New patchset: MaxSem; "Make sure globals are not overwritten in MobileContextTest" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59765 [17:20:59] jdlrobson, ^^ [17:21:48] thanks MaxSem testing now! [17:23:13] New patchset: Jdlrobson; "Make sure globals are not overwritten in MobileContextTest" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59765 [17:23:25] Change merged: Jdlrobson; [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59765 [17:23:37] New patchset: Jdlrobson; "Do not display desktop view on mobile domain" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59778 [17:23:48] Change merged: Jdlrobson; [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59778 [17:24:38] jgonera: re deploying on the dashboad. [17:24:42] jgonera: something seems to be wrong with labs [17:24:58] jgonera: I tried deploying on -dev, it errored out and -dev is now in an unusable state [17:25:05] I've asked labs and analytics folks, no response yet though [17:25:09] hm, I see, I was surprised because I remember you telling me that it should autodeploy [17:25:12] so hold off on deploying the dashboard :) [17:25:18] ok [17:25:20] hmmm [17:25:23] it should [17:25:50] jgonera: I suppose this is all due to a labs issue [17:25:54] jgonera: since manual deploy is failing too [17:27:22] ok, let's wait for analytics response [17:27:39] ok [17:28:04] jgonera: https://mingle.corp.wikimedia.org/projects/mobile/cards/449 :) [17:28:16] jgonera: actually jgonera, I think adding a new graph requires a manual deploy. [17:28:22] jgonera: the auto deploy is only for fixes to existing graphs [17:28:33] also MaxSem https://mingle.corp.wikimedia.org/projects/mobile/cards/461 - let me know if you need me to poke anyone in the office to get that done [17:29:31] jgonera: http://stat1001.wikimedia.org/limn-public-data/mobile/datafiles/performance-rendering.csv is up and running, so that part of the auto deploy clearly worked [17:29:42] YuviPanda, ok, still, I'm not going to try before it works on dev because I don't want to break it ;) [17:29:44] hm [17:29:45] :) [17:29:58] yeah, me neither [17:30:05] no response from analytics *or* labs though :( [17:30:31] jdlrobson, how should I move it and where should I move it to? [17:31:22] jgonera: i just suggest mailing mobile-l with a brief summary of anything you've discovered and maybe link the limn graph you made with some suggestions for next steps? [17:31:41] then i'll move it to ready ready for sign off :) [17:31:46] bbiab [17:32:14] jdlrobson, ok, will do that today, now I have to reboot because a system update messed up my desktop a bit, possibly the mic too, brb [17:32:22] thanks jgonera [17:35:38] Maryana: any big pushes going out tomorrow for web ? [17:35:57] err, on saturday? [17:36:05] or do you mean next week? [17:36:07] s/tomorrow/next week :D [17:36:10] hehehe [17:36:12] yup, I think we're enabling editing for everyone on stable on saturday evening :P [17:36:25] yes. on 4/20 [17:36:27] great idea! [17:36:35] +1! [17:36:50] in fact, if we actually show everyone the edit view by default [17:36:53] Maryana: so anything big going out ? [17:37:00] that'll skyrocket our active editor count! :P [17:37:40] tfinc: nothing major. a lot of bug fixes. the only big-ish thing is potentially the device-targeted banner to download the commons app. but that's just for logged in beta users [17:37:50] so a pretty small population [17:38:18] but its our first use of mobile banners [17:38:37] Maryana: --^ ? [17:38:56] yes indeed [17:40:03] Maryana: when will the banner go live ? [17:40:58] the banner code is already live on testwiki. if all goes well, it will be deployed to full mobile beta during our regular deployment window [17:41:07] tuesday afternoon [17:43:13] MaxSem: what x-device changes are going out next week ? [17:43:42] tfinc, rollout of our performance work on more wikis [17:43:53] aka all of them if everything goes well [17:44:21] MaxSem: by well do you mean analyzing the current rollout ? [17:46:01] the current rollout looks good already. I mean what if there's something that breaks badly performance-wise? with the current share of requests being served the new way, performance changes are not noticeable [17:49:08] mhurd: YuviPanda : brion : retrospective in 10min reminder [17:49:13] yup [17:49:16] hey jgonera [17:49:23] MaxSem: by well do you mean analyzing the current rollout ? [17:49:57] jdlrobson - commas are your friend :) [17:50:00] tfinc: yes [17:50:02] tfinc, replied above^^ [17:50:02] YuviPanda, yes? [17:50:20] jgonera: can you V+2 a patch for me? [17:50:22] arr you weren't here [17:50:24] Maryana: which commas? [17:50:32] tfinc, the current rollout looks good already. I mean what if there's something that breaks badly performance-wise? with the current share of requests being served the new way, performance changes are not noticeable [17:50:36] ahh i see [17:50:42] jgonera: https://gerrit.wikimedia.org/r/60027 [17:51:29] Maryana: nice survey! [17:52:07] YuviPanda, has some whitespace issues but I won't be too picky ;) you tested it? [17:52:25] jgonera: I tested the data generation parts, but have no way to test the limn part :) [17:52:36] hence the request for V+2, since you're the only person I know who has a working Limn install :P [17:53:00] ok, let me check [17:53:34] tfinc: homepage special casing is disappearing from alpha which should make the likes of aharoni excited ;-) [17:53:44] jdlrobson, see my email :) [17:53:47] ok [17:53:50] and cause lots of unknown breakage of course :-) [17:54:16] Maryana: more wikis = all wikis ? [17:54:17] err [17:54:19] MaxSem: --^ [17:55:48] tfinc, yes - the plan is to deploy it everywhere [17:55:53] Maryana: Also, in order to use "special" pages like watchlists you have to disable Java. Very annoying - and it took ages for me to figure out what the problem was." o_O [17:56:01] java?! [17:56:04] yeah..... [17:56:07] i don't even know [17:56:09] ... [17:56:18] on a tablet, even [17:56:28] Maryana: \o/ "There should be a "Add topic" option for talk pages." [17:56:30] almost there.. :) [17:56:34] yep :D [17:56:50] we might need to revive our mobile nav discussion [17:56:55] we're running out of space to stick stuff [17:56:55] also haha "I like Wikipedia very much, and I use it several times a day. Thank you very much!" [17:57:07] yeah, i bet all the nice things came from new users [17:57:12] Maryana: well the add topic is orthogonal to mobile nav but we do need that discussion [17:57:15] and all the mean/snarky things came from exisiting users [17:57:27] yeah - i'm thinking user talk [17:57:27] Maryana: i've got a proof of concept that adds topics to talk pages via the overlay [17:57:28] "you guys are all right bastards" [17:57:31] you don't even have to leave the article [17:57:42] lolz "I like Wikipedia very much, and I use it several times a day. Thank you very much!" [17:57:47] wrong lolz.. [17:57:49] lolz ""I wasn't really impressed with the mobile editor." [17:57:54] heh [17:57:58] YuviPanda, I think you left some print somewhere because generate.py is outputting a lot of stuff in the terminal [17:58:00] it's amazing it works at all! be pleased ;) [17:58:09] jgonera: argh, yes. let me fix that [17:58:15] worst. mobile. editor. ever. [17:58:23] is i think what that guy was going for [17:58:25] "It is still very buggy, please get the quality assurance people on it. Also adding fair use images needs to be easier." [17:58:31] ^ jcmish think that means you ;-) [17:58:38] see what i mean about overrepresented wikipedians [17:58:45] oh fair use [17:58:51] jgonera: other than that? [17:58:59] still checking [17:59:02] okay ;) [17:59:23] so brion why can't I do Title::newFromText( $title )->getHeadings()['h2's] ? :( [17:59:43] we have to use jinja for JSON config files too I guess, because I have to change the url in datasources all the time manually [17:59:44] this would make talk pages soooooo aamaszmzmazzzzinggballs [17:59:48] jdlrobson: well if Title::newFromText returns null it will break [17:59:54] oh getHeadings() ? hah [17:59:56] yeah that'd be nice eh [18:00:02] getHeadings is what i want yes [18:00:20] brion: is that anything you'd be interested in experimenting with in your experimental time? i fear i don't know enough to do this :( [18:00:25] but i soooooo want it [18:00:39] i might take a peek later :D [18:01:08] YuviPanda, the graph shows values only for ios, for android it's always 0 and for web the graph is missing [18:01:33] hmm [18:01:34] loking [18:07:22] New review: Jdlrobson; "recheck" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59559 [18:09:58] mhurd: http://mobile-reportcard.wmflabs.org/ [18:12:03] YuviPanda still working on the commons android icons, but here's the wikipedia set for what it's worth https://www.dropbox.com/sh/hivuelx5eq5mqco/awtWO4eStx [18:14:03] jdlrobson, anything specific you want me to review? [18:14:38] MaxSem: all the skin stuff would be great starting with https://gerrit.wikimedia.org/r/#/c/58990/3 [18:14:55] i really want to see all that get merged so anything you need fixing up just let me know and i'll do it right away [18:15:54] YuviPanda, have you checked it? [18:17:56] jdlrobson: i pushed a new patchset for the spam [18:17:56] err [18:17:56] jg [18:17:56] gah [18:17:56] he left [18:18:00] sorry jdlrobson [18:18:07] you better be sorry YuviPanda :) [18:19:03] New review: MaxSem; "(1 comment)" [mediawiki/extensions/MobileFrontend] (master) C: -1; - https://gerrit.wikimedia.org/r/58990 [18:24:37] jgonera: submitted a patchset :) [18:24:42] two actually [18:25:05] New patchset: Jdlrobson; "Alpha: Talk page - allow access to lead section conversation" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59559 [18:25:05] New patchset: Jdlrobson; "Alpha: Allow users to start conversations on talk page" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59560 [18:25:26] great... I just rebooted to OS X to check the mic, it doesn't work there either, so I guess the Apple service broke it/forgot to connect it :/ [18:25:48] YuviPanda, will check in a sec [18:26:00] :D [18:26:08] ow, at mic [18:26:19] New patchset: Jdlrobson; "Promote menu button generation to MinervaTemplate" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58990 [18:26:28] ^ MaxSem [18:26:34] New review: Jdlrobson; "Done." [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58990 [18:26:42] New patchset: Jdlrobson; "Define render function in MinervaTemplate to output page" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58991 [18:30:57] Change merged: jenkins-bot; [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58990 [18:31:47] jgonera: you're right, it's not reporting deletions for iOS at all [18:31:49] * YuviPanda looks into it [18:32:02] for ios? or for android? [18:33:19] jgonera: for iOS. There was an off-by-one error in the graph definition that shifted things [18:33:23] jgonera: Web graph should be visible now [18:33:29] in latest patchset, that is [18:33:56] ok, running generate [18:34:49] we should add *.pyc to .gitignore [18:35:02] yeah [18:35:32] jgonera: they're on my global gitignore :) [18:35:50] at least python saves its byte code :P [18:36:08] yeah, snakes are careful with bites ;P [18:36:17] haha [18:36:42] ok, seems to work [18:36:44] ok i'm gonna get a sandwich and stuff and get back to categories [18:36:47] bbiab [18:37:15] I mean, I didn't verify the data but loads of deleted uploads for web seems to be right considering the village pump discussion [18:38:35] YuviPanda, merged, anything else? [18:38:40] jgonera: no, thanks :) [18:38:51] jgonera: i'm setting up with Dan about the dashboard [18:38:53] will take care of it [18:38:55] before I shut down the whole ssh tunnel and limn ;) [18:38:56] ok [18:42:45] jgonera: thanks a lot :) [18:42:51] jgonera: also, can you pass me over the commands for the ssh tunnel? :) [18:43:16] * munaf sighs. [18:43:18] YuviPanda, ssh -L 3307:s1-analytics-slave.eqiad.wmnet:3306 -L 3308:s4-analytics-slave.eqiad.wmnet:3306 stat1.wikimedia.org [18:43:24] i really want a new commons logo. [18:44:21] jgonera: http://mobile-reportcard-dev.wmflabs.org/ :) [18:44:33] munaf: the current one is all wrong proportions? :D [18:44:45] jgonera: the new graphs are tehre [18:45:06] and, well... i feel like it's just not the clearest brand [18:45:51] YuviPanda, great, thanks! [18:47:31] YuviPanda, I don't understand gerrit, I just wanted to modify .gitignore, branched off master and modified it, and git review wants to submit 5 commits that are already in master... [18:47:43] jgonera: do git fetch gerrit [18:47:46] jgonera: 'known bug' [18:48:01] wow, thanks [18:48:13] YuviPanda, https://gerrit.wikimedia.org/r/60032 [18:48:21] :D [18:49:10] jgonera: merged :) [18:49:17] thanks [18:49:20] jgonera: that's a bug in the newest version of git review [18:49:32] jgonera: so everytime it does that, do a 'git fetch gerrit' and do a git review again [18:49:34] will go away [18:50:06] thanks, I'll think two times before updating git review next time [18:50:12] heh :D [18:50:52] still no call from the airlines/airport... I'd call them but yesterday I had to hold for 1.5h to actually talk to someone... [18:56:07] munaf: the wiki icons look nice :D [18:56:21] munaf: did you notice the desktop wikipedia favicon change? [18:56:21] i did [18:56:25] and thanks! [18:57:07] yeah it's an improvement. might need to pop a bit more with a border or something because pure white sort of fades into most browser default themes [18:57:17] jdlrobson, what's the difference between prepareData() and prepareCommonData()? [18:57:51] New patchset: Jdlrobson; "Alpha: Allow users to start conversations on talk page" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59560 [18:57:56] MaxSem: prepareData can be overriden [18:58:09] is that bad?:) [18:58:19] prepareCommonData shouldn't have to (and in future shouldn't exist when SkinTemplate gets some more polyfill) [18:58:20] YuviPanda commons ios icons done :-) android coming next [18:58:24] hang on grabbing the link. [18:58:27] yay [18:58:32] argh, they just called and said that they will bring my bag before 3pm... yesterday they said it would be in the morning. I thought 3pm could be considered as morning only in Spain and even there it's far-fetched. [18:58:36] just meeting with jorm so brb [18:59:07] jdlrobson, I still don't see a need [18:59:28] MaxSem: it becomes clearer in the later commits [18:59:33] brb [19:06:16] YuviPanda here's ios for reference :-) https://www.dropbox.com/sh/f88mogxgych5g1k/Ga9D0bIKc_ brb. [19:06:38] munaf: wah neato! [19:06:45] whee [19:06:50] snazzy [19:06:56] very! [19:08:47] munaf: got flattened versions for us without photoshop? :) [19:09:51] yes brion [19:09:55] app icon squared / rounded [19:09:56] \o/ [19:10:00] those 2 folders have 'em all [19:10:03] awesome [19:10:51] jdlrobson: new dual usb charger is on our desk [19:10:54] your* [19:10:56] YuviPanda android will be ready some time after lunch :) [19:11:00] wheee! [19:11:01] munaf: the 'small' icons in the squared dir appear to be rounded :) otherwise they look awesome [19:11:09] odd [19:11:14] i shall look into this [19:12:06] tfinc: oh did we get the hdmi adaptor for the nexus4 in? i'll want to update the android video on monday or tuesday, if it's not going to be here I can just reflash my galaxy nexus, i have the adaptor for it [19:12:15] brion: YuviPanda : i just passed on a copy of TeamGeek to mhurd. lets see what kind of hand balancing he can do after [19:12:20] munaf: thanks [19:12:21] :) [19:12:22] brion: amazon was out of them when i checked yesterday [19:12:23] :) [19:12:25] heh [19:12:25] let me check today [19:12:32] tfinc: :) [19:12:36] tfinc: brion mhurd Maryana new graphs at http://mobile-reportcard.wmflabs.org/ :) [19:12:47] still need to add two more [19:12:47] i have the kindle edition, not sure if i can lend that or not these days :) [19:12:50] but there is now a 'deletions per day' [19:12:53] of mobile uploads [19:12:59] brion: ZOMG $72.89 + $13.75shipping [19:13:02] it's missing iOS, but works otherwise [19:13:15] tfinc: waaaaaaaaat that's crazy talk [19:13:23] awesome, yuvipanda [19:13:28] awesome and sad for that one day [19:13:36] brion: http://www.amazon.com/SlimPort%C2%AE-SP1002-Connect-connector--Supports/dp/B009UZBLSG [19:13:40] 300 files deleted :( [19:13:50] that's a lot of test images [19:13:50] glad to see we're past that [19:13:52] 300 files were deleted on 3 consecutive days :( [19:14:12] brion: would a generic like this http://www.amazon.com/GTMax-Micro-HDMI-Microsoft-Viewsonic-CellPhone/dp/B005KH72CM work ? [19:14:12] tfinc: daaaaamn and i thought the apple adaptors were overpriced [19:14:35] tfinc: no, that's a micro-hdmi, slightly different port [19:15:04] i'll just flash the galaxy nexus back to android :) [19:15:20] brion: google shopping has no merchant selling it [19:15:24] its like its dissapeared [19:15:28] harumph [19:15:33] brion: how badly do you want it ? [19:15:42] tfinc: i can live without it for now, no worries [19:15:49] Maryana: https://gerrit.wikimedia.org/r/gitweb?p=analytics/limn-mobile-data.git;a=blob;f=mobile/deleted-uploads.py;h=dbe073264dbe26255afe9b2390ebfb1d2346296c;hb=HEAD is the SQL used [19:15:54] i'll use the adaptor for the gnex [19:15:54] well, it's a combination of python and SQL [19:15:57] but mostly SQL [19:16:06] Maryana: do take a look, just to ensure I'm not doing something wrong :) [19:16:09] but the numbers feel right to me [19:16:15] brion: lets give it a couple of weeks for new stock to come in. if not we'll just buy it and attach a large metal spoon to it [19:16:20] so that no one steals it [19:16:24] brion: is this the same adaptor? http://www.monoprice.com/products/product.asp?c_id=108&cp_id=10833&cs_id=1083314&p_id=8675&seq=1&format=2 [19:16:36] mmmm metal spoons [19:16:55] mhurd: as far as i can tell the MHL and the Slimport are different but use the same ports. this is fairly maddening ;) [19:16:56] yuvipanda, so are we showing the number of files that were deleted on the same day that they were uploaded? [19:17:11] or are daily successful uploads and daily deletions not necessarily the same files [19:17:13] brion: gah! [19:17:15] Maryana: no, it's just showing number of files deleted that day [19:17:20] Maryana: they might be uploaded weeks prior [19:17:21] gotcha. thought so [19:17:26] yeah, that makes sense [19:17:39] it might just be a bit misleading for now, looking at the numbers [19:17:49] http://droidnerds.com/content/ces-2013-slimport-vs-mhl/ [19:17:55] because we're still catching up on a deletion backlog from when all the brand new people were uploading their selfies [19:18:00] Maryana: yup! [19:18:08] Maryana: but i suppose it'll even out over time. [19:18:17] yes [19:18:17] hopefully [19:18:24] Maryana: is there any other metric you can think of that'll track it better? [19:18:36] I quite like doing analytics stuff :) [19:18:43] hehe, i know, it's fun! [19:18:50] nah, i think this is good for now [19:18:50] brion: what about http://www.amazon.com/gp/product/B00AGFISF8 ? [19:19:01] though maybe we should think about monthly deletion instead of daily [19:19:11] brb, foodz [19:19:12] tfinc: that should work \o/ [19:19:24] brion: 1 or 2 ? [19:19:29] i'll just order it now [19:19:32] 1 is fine [19:19:39] thanks :D [19:19:58] ordered [19:20:02] will be here tuesday [19:20:47] woohoo [19:21:33] tfinc: seen http://mobile-reportcard.wmflabs.org/? [19:21:38] hey moogsi! More graphs at http://mobile-reportcard.wmflabs.org/ :) [19:22:18] YuviPanda: i see a new photos with no categories [19:22:30] uploads deleted [19:22:33] yup [19:22:53] ouch 300 day deletion [19:22:58] burtal commons admins. brutal [19:23:15] YuviPanda: what does the rendering time correspond to ? [19:23:19] under performance [19:23:24] tfinc: that's from jgonera [19:23:34] YuviPanda: it needs an explanation [19:23:38] jgonera: ^ [19:23:48] i still don't understand 'Apr 13 — Apr 13 58.96%' ? [19:24:05] tfinc, it's basically the time between the page starts loading and finishes loading [19:24:06] me neither. I'll need to ask the analytics folks where that ocmes from [19:24:28] tfinc: I've started asking milimetric / DarTar to add documentation for Limn (currently it is almost non-existant), so hopefully that'll solve this [19:24:47] YuviPanda, when you're talking to them, ask them if it's possible to show the current hovered date too, right now it shows only the month [19:25:08] jgonera: what do you mean by 'hovered date', jgonera? [19:25:11] also you're cc'd in the email :P [19:25:26] so do respond wtih the parts you found lacking [19:25:30] or confusing [19:25:31] I mean, when you move the mouse over some data point it would be nice to know from what day it is [19:26:42] jgonera: ah, right [19:41:29] jdlrobson: you might enjoy https://twitter.com/_FloridaMan [19:48:12] tfinc: Maryana started assembling notes about the NavTiming performance data here: https://meta.wikimedia.org/wiki/NavigationTiming/Logging we should add explanations there, I think [19:48:53] page_loading is the total time it took to retrieve and render the page, so "rendering time" is not a good title [19:49:15] (I should have noticed that earlier) [19:50:20] it includes time spent resolving hostname, making request, following redirects, retrieving content and rendering it on screen [19:50:36] gnite everyone [19:50:49] nice…. more retina-quality laptops coming out :D http://arstechnica.com/gadgets/2013/04/toshibas-2560x1440-kirabook-is-a-high-res-laptop-for-the-windows-world/ [19:50:52] nini yuvi [19:51:04] cya YuviPanda [19:51:34] we have more granular data that helps pinpoint specific bottlenecks; as a figure this one is rather general, but gives an overall sense of latency as perceived by users [19:53:01] MaxSem: i'm back! [19:53:08] wee [19:53:12] so the purpose of prepareCommonData is to define template data specific for our skin that works on both mobile and desktop [19:53:21] it could be in execute but i wanted to break it out for readability [19:54:00] MaxSem: if you look at https://gerrit.wikimedia.org/r/#/c/58997/3/includes/skins/MinervaTemplate.php it makes more sense [19:54:37] brion: why *do* we use binary charset in mysql? i'd ask on #wikimedia-operations but i think they'd shoot me [19:55:03] ori-l: 'utf8' in mysql is (or at least was) insufficient to store full utf8 data [19:55:14] yes non-obvious i know :) [19:55:33] but mysql's "utf8" was hardcoded to limit to BMP chars (can fit in 3 bytes) [19:55:42] while full utf8 range includes chars that encode to 4 bytes [19:55:48] ohhhhhh [19:55:51] and of course we support these rare languages [19:56:07] i remember they were introducing full utf8 support in mysql 6 or something but i don't know the current status [19:56:47] so we just said 'hell with it, let's treat everything as binary blobs and just put raw utf-8 in' [19:57:05] right, it's enough that one layer knows about encoding [19:57:13] in the olden days we had to use mysql 3.x which had NO unicode support; it thought we were speaking latin1 though we stored utf-8. that was worse :) [19:57:42] New patchset: Jdlrobson; "Alpha: Allow users to start conversations on talk page" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59560 [19:57:42] i'm going to respond to yuvi on bugzilla with "i'm going to cry if i need to migrate that database" [19:57:45] and hope that satisfies him [19:57:48] haha [19:58:07] New patchset: Jdlrobson; "Kill navigationStart" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58992 [19:58:12] New patchset: Jdlrobson; "Promote all the banners into MinervaTemplate" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58993 [19:58:49] brion: well, thanks for that explanation, i'll go read up on the current utf8ness of utf8 in mysql [19:58:58] :D [19:58:59] enjoy [20:00:58] New patchset: Jdlrobson; "Move header to generic MinervaTemplate" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58994 [20:01:18] New patchset: Jdlrobson; "banner fixup" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58995 [20:01:22] New patchset: Jdlrobson; "Code move: Lift and shift html rendering to MinervaTemplate" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58996 [20:01:58] jgonera MaxSem were you happy with https://gerrit.wikimedia.org/r/#/c/59985/ in end? [20:02:36] jdlrobson, let me just test [20:02:45] jdlrobson, I forgot where we're supposed to see the variant menu:P [20:03:04] otherwise, I have just one small nitpick [20:04:33] MaxSem: so it shows as part of languages section when you have $wgLanguageCode = "sr"; [20:05:30] MaxSem: so kaldari is ready to code away once all the skin stuff is merged [20:06:46] jdlrobson, looks good [20:07:08] jgonera MaxSem has a nit pick apparently so i'm waiting to hear that then i'll fix and hopefully you can +2 :) [20:07:47] well, you have my +1 already so feel free to self-merge when MaxSem's nitpick is fixed [20:07:59] k k [20:08:21] if you've got more time to review any skin stuff would be good to see merge. i'm like a kid waiting for christmas.. [20:08:39] I know, I will review some more ;) [20:08:43] New review: MaxSem; "(1 comment)" [mediawiki/extensions/MobileFrontend] (master); V: 2 C: 2; - https://gerrit.wikimedia.org/r/59985 [20:08:49] meh [20:09:00] let the var name match the array key [20:09:06] Change merged: jenkins-bot; [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59985 [20:09:12] thiugh array key is itself in violation [20:09:52] Change merged: jenkins-bot; [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58991 [20:09:56] hmm [20:10:09] now i'm thinking the categories list should go on the detail page in full instead of a second list [20:10:09] I think I'm getting used to "relaxed" naming conventions in MW and didn't notice that ;) [20:12:16] jgonera, relaxed in comparison to what [20:16:35] MaxSem: desktop made me do it :) [20:16:49] that's the key it uses :/ [20:17:03] but yey! \o/ one step closer to minerva [20:17:26] New patchset: Jdlrobson; "Kill navigationStart" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58992 [20:17:30] New patchset: Jdlrobson; "Promote all the banners into MinervaTemplate" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58993 [20:17:33] New patchset: Jdlrobson; "Move header to generic MinervaTemplate" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58994 [20:17:36] New patchset: Jdlrobson; "banner fixup" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58995 [20:17:39] New patchset: Jdlrobson; "Code move: Lift and shift html rendering to MinervaTemplate" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58996 [20:18:31] jdlrobson, is it safe to merge that much refactoring so close to deployment? [20:19:52] MaxSem, I mean that you can find a lot of violations [20:21:10] New patchset: Jdlrobson; "Make the universe explode by making a desktop AND mobile skin called Minerva" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58997 [20:21:10] New patchset: Jdlrobson; "Promote addToBodyAttributes to desktop skin" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58257 [20:21:28] MaxSem: your call. I think it's always safe :) [20:21:54] MaxSem: i'd only be concerned about https://gerrit.wikimedia.org/r/#/c/58997/ [20:22:03] up till there should be pretty harmless [20:24:35] btw jdlrobson, did you discuss the possibility of merging of our templates module into core with teh RL gang? [20:25:04] MaxSem: not yet. They are aware we are doing it and want it in core [20:25:10] ('they' = trevor/roan [20:25:35] i'll aim to start a conversation on wikitech within next 2 weeks [20:26:38] welcome back dr0ptp4kt [20:27:02] thanks, MaxSem [20:31:05] jdlrobson: ready when you are in R33 [20:48:17] YuviPanda Maryana brion https://trello.com/card/icon-for-android-commons-app/5112b893d0cd43cb370062bd/43 [20:48:37] ' This board may be private. If someone gave you this link, they may need to invite you to one of their boards or organizations. ' [20:48:50] :( [20:49:44] wut. [20:49:45] odd [20:49:53] thought these were all public [20:57:16] jdlrobson, what happens if i edit the site notice on test wiki? does the world explode? [20:57:57] shouldn't explode… i think.... [21:24:01] only one way to find out.. [21:28:36] hmm, it scares me a bit that the banner is showing up on desktop testwiki [21:28:41] jdrlobson ^ [21:33:51] just received my bag, I guess I'll just keep working from home today unless I'm needed at the office [21:39:35] not needed, per se. just missed :) [22:03:03] Maryana: edit away the site notice [22:03:14] Maryana: mwalker made it target desktop as well [22:03:15] i have :D nothing exploded [22:03:17] (on test wiki) [22:03:22] ah, ok [22:03:23] just checking [22:05:29] Maryana: *waves* I was going to put something up in Meta:Babel and enwiki:VillagePump\Tech about the mobile banner we're going to try next week -- it's going out to logged in/logged out users in english on en.m.wp.o for android? [22:06:43] just logged in users [22:06:52] and that would be most excellent of you, mwalker. thanks :) [22:07:47] i like that i edited my first centralnotice message on the day that megan came back to the office. that's appropriate. [22:10:35] tfinc: first pass at goals: https://office.wikimedia.org/wiki/Goals/2012-2013/Engineering/Monte_Hurd [22:10:58] mhurd: 2. Increase development and collaboration skills is pretty vauge [22:11:18] can you narrow that down a bit ? [22:12:04] excellent 'Git-fu' reference [22:12:06] tfinc: ya hehe [22:12:38] 4. Begin experimenting with Android development process <-- lets get this a bit more specific [22:30:33] tfinc: second pass [22:47:57] mhurd: taking a look [22:48:47] mhurd: why don't we add working and finishing an enhancement to #4 ? [22:48:54] that way its not just setting up your dev env [22:49:12] getting the dev env working doesn't take long [22:50:44] tfinc: i like it [22:51:11] mhurd: we don't have to say what the enhancement is but it gives you a clear dev and not just setup goal [22:53:10] tfinc: just added it [23:00:28] New review: JGonera; "(4 comments)" [mediawiki/extensions/MobileFrontend] (master) C: -1; - https://gerrit.wikimedia.org/r/58992 [23:01:08] brion: mhurd any idea on what's causing the dip in android uploads ? [23:01:19] brion: http://mobile-reportcard.wmflabs.org/graphs/30-day-uploads [23:04:02] hm, slight dip in web too [23:04:30] maybe just friday… maybe because the day's incomplete? [23:04:47] New review: JGonera; "(2 comments)" [mediawiki/extensions/MobileFrontend] (master) C: -1; - https://gerrit.wikimedia.org/r/58993 [23:06:55] tfinc, brion isn't this related to the recent EL outage? [23:08:17] New review: Jdlrobson; "(4 comments)" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58992 [23:10:30] New review: Jdlrobson; "(2 comments)" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58993 [23:19:06] New review: JGonera; "(2 comments)" [mediawiki/extensions/MobileFrontend] (master) C: -1; - https://gerrit.wikimedia.org/r/58994 [23:20:46] New review: JGonera; "(1 comment)" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58993 [23:25:06] New review: JGonera; "(4 comments)" [mediawiki/extensions/MobileFrontend] (master) C: -1; - https://gerrit.wikimedia.org/r/58992 [23:33:30] New review: Jdlrobson; "(1 comment)" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58993 [23:37:03] New review: JGonera; "This makes the star visible on special pages and you can add watchlist to your watchlist so you can ..." [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58994 [23:40:27] New review: JGonera; "Is there any reason why we print the footer on special pages then? It seems a waste of bandwidth, ev..." [mediawiki/extensions/MobileFrontend] (master) C: -1; - https://gerrit.wikimedia.org/r/58996 [23:47:35] New patchset: Jdlrobson; "Make the universe explode by making a desktop AND mobile skin called Minerva" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58997 [23:47:35] New patchset: Jdlrobson; "Code move: Lift and shift html rendering to MinervaTemplate" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58996 [23:47:35] New patchset: Jdlrobson; "Promote all the banners into MinervaTemplate" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58993 [23:47:36] New patchset: Jdlrobson; "Move header to generic MinervaTemplate" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58994 [23:48:04] Change abandoned: Jdlrobson; "This shouldn't have been submitted. Was a fixup which should have been applied in a rebase." [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58995 [23:49:07] New patchset: Jdlrobson; "Promote addToBodyAttributes to desktop skin" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58257 [23:56:45] New review: Jdlrobson; "Currently we hide footers in the design." [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58996 [23:57:51] New patchset: Jdlrobson; "Upstream skin changes - getLanguageUrls" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59987 [23:58:30] jgonera: still waiting on https://mingle.corp.wikimedia.org/projects/mobile/cards/449 changing status ;-) [23:58:48] jdlrobson,I thought you said you'd move it [23:58:49] EL ? [23:58:52] I don't know how [23:58:55] Event Logging [23:59:02] jgonera: did you mail mobile-l about it? [23:59:04] ahh [23:59:09] I mean it doesn't let me move it to sign off [23:59:14] jdlrobson, yes [23:59:29] jdlrobson, I mailed mobile-tech [23:59:34] o_O oohh sorry [23:59:34] somehow that got lost in my mail [23:59:45] i apologise :)