[09:37:35] I4f677f08 Update ZeroRatedMobileAccess extension to master (MERGED) preilly mediawiki/core wmf/1.20wmf5 (wmf/1.20wmf5) Jun 12 [09:37:35] Id0b125ce Update ZeroRatedMobileAccess extension to master (MERGED) preilly mediawiki/core wmf/1.20wmf4 (wmf/1.20wmf4) Jun 12 [09:37:35] I90aa5900 Update ZeroRatedMobileAccess extension to master (MERGED) preilly mediawiki/core wmf/1.20wmf3 (wmf/1.20wmf3) [09:37:41] Why was 1.20wmf3 updated? :/ [16:35:34] hey jd [16:35:36] err [16:35:36] jdlrobson: [16:35:42] Mr YuviPanda ! [16:36:22] jdlrobson: still jet lagged? [16:36:27] progress [16:36:32] went to a js meetup yesterday [16:36:39] drank lots of beer, got to sleep at a respectable midnight [16:36:43] awoke at a respectable 7 [16:36:59] ah [16:37:02] physically awoke or ? [16:42:14] New patchset: Jdlrobson; "add site language to settings page" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/10452 [16:42:31] YuviPanda: the sun made me wake up [16:42:33] that's a good sign [16:42:34] :) [16:43:46] jdlrobson: very :) [16:43:49] jdlrobson: also, https://github.com/wikimedia/WikipediaMobile/pull/253 :P [16:44:05] yep i've seen it [16:44:09] have a huuugggee backlog [16:45:17] i could guess :) [16:49:18] New review: MaxSem; "(no comment)" [mediawiki/extensions/MobileFrontend] (master); V: 0 C: -1; - https://gerrit.wikimedia.org/r/10452 [16:49:57] morning all [16:51:14] hangout is up https://plus.google.com/hangouts/_/f863ff4bf0eac08f808a00993e275424e35ec9d6?hl=en [16:51:21] we'll start in 10min [16:54:29] New patchset: Jdlrobson; "increase line height for android 2.3.6" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/11180 [16:56:19] jdlrobson: is that a cow on your t shirt? [17:09:32] YuviPanda: MaxSem : how was the background noise on our mic today? [17:09:49] it was ok [17:10:07] tfinc_meetings: a bit harder on patrick and phil. not as good as the one usually used [17:10:11] the street outside is being torn up so it was hard to hear you. im curious if the mic picked that up [17:10:19] YuviPanda: they were just farther away [17:10:25] if you didn't hear construction then were ok [17:10:33] i'll just have to find some slightly louer speakers [17:10:38] hah, no I didn't hear construction [17:10:50] midway i tried switchign to the yeti mic since it showed up as an audio output but clearly that didn't work [17:10:52] ok [17:10:56] no reason to move the location then [17:12:20] YuviPanda: i see you've already let the dolphin guys know that they should work off of 1.2 [17:12:46] the biggest point is that we don't want to support screen scraping in any new dev [17:13:01] tfinc_meetings: do you want to shoot off an email mentioning that too, or should I? [17:13:11] you already said it in your email [17:13:21] lets see what they say [17:13:24] okay [17:14:50] YuviPanda: what blockers are left on the android release ? [17:15:15] tfinc_meetings: all have been fixed, i'm just waiting for jdlrobson to look at one pull req and merge it in [17:15:30] jdlrobson: will you have a chance to look at it today? [17:16:29] tfinc_meetings: yes should do [17:16:35] i'll have a quick peek now [17:17:10] thanks [17:17:33] tfinc_meetings: looks like we got the crashing bug on windows 8 all worked out [17:17:59] brion: you know kul will be stoked to hear that [17:18:16] yeah raj is a frequent-phoner [17:18:37] and the update is up in the store ? [17:18:46] s/up/available [17:18:47] should be, i'm confirming now on my end [17:18:50] woot [17:18:55] MaxSem: [17:19:00] Line 78: $allLanguages = new languages(); [17:19:01] Still lowercase [17:19:07] should that be $all-languages ? [17:19:19] YuviPanda: were you abe to sync up with MaxSem about your WLM questions ? [17:19:30] jdlrobson: what is the link to your device detection change? [17:19:30] and also to check.. I should use $title = Title::newFromText( $code .':Main Page' ); ? [17:19:31] tfinc: no, I'm trying to reach multichill before that [17:19:43] it should be new Langauge(); [17:19:45] preilly: https://gerrit.wikimedia.org/r/#/c/9640/ [17:19:50] ahh ok [17:20:04] YuviPanda: what do we need from multi chill? he's a volunteer so we can't anticipate his time availabiltiy [17:20:24] if were blocked on him then we need to find someone else who knows the infrastructure [17:20:28] like Platonides [17:20:43] the infrastructure is pretty well documented (we did that at the hackathon) [17:20:53] so what do we need from multichill ? [17:21:01] could you guys and gals test GeoData for me? [17:21:02] http://dl.dropbox.com/u/8768784/Wikipedia-Mobile-Geo.apk [17:21:09] http://dl.dropbox.com/u/8768784/Wikipedia-Mobile-Geo.ipa [17:21:34] preilly: where is the redirection code for redirecting to m.wikipedia ? [17:21:40] MaxSem : what do you need us to test? [17:21:42] tfinc: I needed to figure out if he had implemented any of the admin stuff he was talking about at all. But yes, I think I'll just go make bugs for everything and poke MaxSem [17:21:45] a quick list would do us well [17:21:54] MaxSem: is this the same stuff from the hackathon? [17:21:56] YuviPanda: do it. lets not block if we don't have to [17:22:00] alright [17:22:10] yes [17:22:19] New patchset: Jdlrobson; "add site language to settings page" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/10452 [17:22:20] in squid [17:22:28] MaxSem: then it won't be usable much on Android 4. Let me make you new builds. [17:22:32] preilly: link me up? [17:22:38] cool, thnks [17:22:53] New review: Jdlrobson; "Also problems with native iphone handling of select menus" [mediawiki/extensions/MobileFrontend] (master) C: -1; - https://gerrit.wikimedia.org/r/10452 [17:23:31] jdlrobson: it's on fenari [17:26:30] MaxSem: https://dl.dropbox.com/u/8768784/Wikipedia-Mobile-Geo.apk and https://dl.dropbox.com/u/8768784/Wikipedia-Mobile-Geo.ipa [17:26:38] oh, same links [17:29:36] jdlrobson: pass me the contact info for the folks that you met last night that we should follow up with [17:29:47] preilly: link would make life easier? :P [17:30:55] tfinc more informal follow ups ... cramforce (https://twitter.com/#!/cramforce) was cool and I think it would be fun to see if we can host some kind of js meetup at the foundation via him (but didn't talk about anything concrete) [17:31:03] jdlrobson: I don't have one [17:31:09] jdlrobson: I'd have to hunt for it [17:31:33] https://twitter.com/#!/stubbornella would be an awesome person to bring in if we ever want some css related advice but again nothing to really follow up there [17:31:35] YuviPanda: for http://etherpad.wikimedia.org/wlm-on-labs-steps where did we archive it ? etherpad is not long term storage [17:32:10] i'm pushing the fixmes to bugzilla now [17:32:12] jdlrobson: so the bias is toward filling http://hire.jobvite.com/Jobvite/Job.aspx?j=o1H6Vfwt&c=qSa9VfwQ first then anything else [17:32:19] will archive in mw in a bit [17:32:24] tfinc: ok confirmed that new version installed from store doesn't crash. yay [17:34:57] tfinc: the wiktionary mobile app is ready for release. if you give the go ahead we can put it out to android market [17:35:07] philinje: how are we doing at defining metrics for WLM ? [17:36:09] tfinc: woot! [17:36:11] err [17:36:14] brion: woot! [17:36:16] :D [17:36:22] \o/ [17:36:23] pfhayes: fancy [17:36:36] pfhayes: which account are you guys going to put it up on ? [17:36:51] tfinc: I assumed under wikimedia foundation [17:36:56] tfinc: same as wikipedia app [17:37:17] tfinc: metrics for WLM are roughly defined, and the page has been re-organized, but refinement is needed, will work on that with Dario [17:37:23] philinje: thanks [17:37:56] philinje: we need to clarify those with dario and faulkner so that we don't have any surprsies of what we can and can't measure [17:38:24] sure [17:38:36] philinje: is this the best page to add new metrics http://www.mediawiki.org/wiki/Wiki_Loves_Monuments_mobile_application#Metrics ? [17:38:58] tfinc: YuviPanda: I will create a sub-page on the WLM app project page for the ether pad notes on DB work [17:39:17] philinje: i'm filing specific bugs on things that need fixing - there's a new product for this [17:40:19] tfinc: should be, there was an earlier page on photo upload project page, let me make sure they are in sync [17:40:39] YuviPanda: new project for the WLM DB work? [17:40:40] philinje: thanks [17:40:48] philinje: in bugzilla, yes [17:40:56] I think multichill had it craeted [17:41:07] philinje: https://bugzilla.wikimedia.org/buglist.cgi?query_format=advanced&list_id=122750&resolution=---&resolution=LATER&resolution=DUPLICATE&product=Monuments%20database [17:41:17] thanks [17:41:26] i'm setting up the FIXMEs from the etherpad into bugzilla, then creating a tracking bug. [17:41:36] ok cool [17:42:33] YuviPanda: let me know the tracking bug number when ready [17:43:13] tfinc, YuviPanda: the wiktionary code is available at https://github.com/wikimedia/WiktionaryMobile/tree/v1.0.1, and the release is the v1.0.1 branch. can one of you make a signed build and put it up on the market? [17:44:34] pfhayes: we'll need description, screenshots, high res icon, and website contact info [17:44:49] tfinc: the metrics link on the WLM App page goes to: https://www.mediawiki.org/wiki/MobileFrontend/Photo_upload/Analysis [17:45:04] tfinc: I don't think we need a separate page [17:45:15] tfinc: okay i'll put those together [17:47:01] pfhayes: awesome [17:47:20] so YuviPanda - I dont' think we should be using app.curSpinningReq in the way we are [17:47:31] yeah, that feels a little bit dirrty to me. [17:47:36] jdlrobson: other suggestions? [17:47:36] I'd prefer a get / setter for better readability as currently it's not too clear what we are doing [17:47:47] hmmm [17:47:48] are we doing showSpinner clearSpinner [17:47:51] or something else? [17:47:57] philinje: then get rid of the second page. you added it with http://www.mediawiki.org/w/index.php?title=Wiki_Loves_Monuments_mobile_application&diff=534940&oldid=534927 [17:48:01] we do have showSpinner and hideSpinner [17:48:37] also https://github.com/yuvipanda/WikipediaMobile/commit/7dcfa66c81e4bd2c7ad7901f2e090bad30097c95 < if we are moving method why not also move lang to extraOptions [17:48:58] (personally I'd just like makeAPIRequest to take one options parameters [17:49:37] jdlrobson: extraOptions is just extended and pushed to $.ajax [17:49:43] jdlrobson: lang goes to making the URL [17:49:56] jdlrobson: maybe I should call it ajaxOptions instead of extraOptions [17:50:59] YuviPanda: can't you just do options = $.extend(defaultOptions, options); [17:51:27] your basically wrapping an ajax request with some Wikipedia Mobile App specific parameters [17:51:49] which go to different URLs, yes. [17:52:03] jdlrobson: I prefer having different defaultOptions - makes it clearer, IMO [17:52:52] I'm not sure I follow YuviPanda [17:53:05] let me write some code quickly [17:55:56] YuviPanda: so do we have the ether pad instructions somewhere more permanent ? [17:55:59] so YuviPanda what would be wrong with this approach = http://pastebin.com/tZpJ7d32 [17:55:59] for wlm [17:56:46] jdlrobson: because then options would be mix of our params + whatever will be passed to $.ajax [17:57:46] well one other approach... (types code) [17:58:03] delete options.wikiLanguage; [17:58:17] since we currently only have one parameter of our own) [17:58:58] it just feels that if you're dropping the method parameter you might as well go the whole distance and drop the others as well [17:59:04] tfinc: https://commons.wikimedia.org/wiki/Commons:Monuments_database/Setup/In_labs, needs wikification. Will do once i'm done with jdlrobson and pushing fixmes to bugzilla [17:59:17] jdlrobson: i dropped method since it was never being used anywhere [17:59:42] YuviPanda: excellent. link to it from the WLM project page so that its easy to find [17:59:57] tfinc: yeah, will do! let me finish with jdlrobson's review :) [18:00:40] jdlrobson: so right now, the meaning of each param is clear - params for api, lang wikipedia to use, and any ajax options (as from jquery docs) [18:01:01] if we mash them all into one param, we'll need to figure out which ones we treat specially, and which ones we don't [18:01:04] me no likey [18:02:41] jdlrobson: also, axel found a bug with that, I just pushed another small commit to it [18:04:47] pfhayes: "320 x 480, 480 x 800, 480 x 854, [18:04:47] 1280 x 720, 1280 x 800 [18:04:47] 24 bit PNG or JPEG (no alpha) [18:04:47] Full bleed, no border in art [18:04:47] You may upload screenshots in landscape orientation. The thumbnails will appear to be rotated, but the actual images and their orientations will be preserved." [18:04:57] for the screenshots [18:06:09] tfinc: thanks, i just took some screenshots on my galaxy nexus. they are 1280 x 720 so they should suffice [18:07:56] YuviPanda: mm.. I don't know me no likely this either but I can't quite put my finger on what in particular. out of interest do you ever need to make an API request outside the current language? [18:08:22] jdlrobson: yes, interlanguage links [18:08:51] tfinc: I just sent you the release information [18:09:11] pfhayes: thanks [18:09:18] Can language be sent as a parameter YuviPanda ? [18:09:36] jdlrobson: ? [18:09:40] jdlrobson: to the server? [18:09:41] no [18:09:45] they're different subdomains [18:09:51] so we need to construct the url differentl [18:10:16] as in http://wikipedia.org/w/api.php?language=fr [18:10:37] the subdomain naming casing seems wmf centric as I suspect not all wikis will use it [18:10:37] nope, nothing :) [18:11:08] jdlrobson: you'll see it uses app.baseUrlForLanguage [18:11:16] yeh [18:11:19] I've never liked that code [18:11:28] pfhayes: i'll need you to clean up the screenshots. get rid of what various icons you have in the status bar [18:11:30] jdlrobson: the geo testing apk MaxSem was floating around a bit - i was able to modify that to return the labs url everywhere. [18:11:42] so that's much neater than how it was before [18:11:44] philinje: it looks messy when there is a hodgepodge [18:11:50] err [18:11:52] pfhayes: ^ [18:12:12] but that aside YuviPanda I really don't like app.curSpinningReq = [18:12:19] ah yes, that I agree with [18:12:22] I'd prefer named functions for readability [18:12:37] so, pass that into showSpinner? [18:12:53] or have a showSpinnerFor? [18:13:41] 1 [18:13:42] s [18:14:55] New review: MaxSem; "(no comment)" [mediawiki/extensions/MobileFrontend] (master); V: 1 C: 2; - https://gerrit.wikimedia.org/r/11180 [18:14:57] Change merged: MaxSem; [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/11180 [18:16:24] YuviPanda: would something like this work - http://pastebin.com/SvUz2bzr ? [18:17:18] jdlrobson: yeah, makes sense to me [18:17:18] tfinc: I just fixed up the screenshots [18:17:19] shall do [18:17:27] awesome [18:17:38] poke me when done and I'll merge it and test it out [18:17:58] tfinc: i emailed you the updated ones [18:18:53] meh, my Wi-Fi router doesn't rout anything and testing maps on EDGE is a $^#@#$^!#$ [18:20:18] jdlrobson: sweet. On it [18:20:21] pfhayes: thanks [18:30:51] jdlrobson, regarding the planned addition of maps to MF: it creates a dependency on external stuff (even after we switch it from geonames to GeoData) and sincde we're merging MF into core, articles around me will have to be moved somewhere else [18:31:07] I wonder where - to GeoData or a separate extension [18:33:56] MaxSem: interesting - I was thinking this could all be configurable though [18:34:07] ie. which tile server you use, which geoserver you use, if it is activated or not etc [18:34:36] hr finally got their act together and printed out were hiring cards. i have a bunch in the office and am pushing hr to put them up online too so that anyone who's remote can help out [18:34:52] here is what they look like https://twitter.com/flyingclimber/status/212973830803701760 [18:35:08] preilly: do you already have some? [18:35:15] jdlrobson, yes, but core can't depend on external services or extensions anyway [18:35:17] MaxSem: talking to phil I think we're looking at getting nearby out within next 2 months so plenty of time to think around all of that (hence why I published early and -1ed so we can have these discussions) [18:35:31] but MaxSem where would the dependency be if it's all configurable? [18:35:43] as that would allow it to be turned off etc [18:35:57] Some smaller wikis might be fine using geonames for instance [18:37:07] you can't say in core "this feature requires this extension" - same thing for external services. MediaWiki has to be self-contained [18:37:19] so that extensions _extend_ it [18:37:30] we don't want to continue using geonames [18:37:33] its not scaling for us [18:37:40] and they cant increase their capacity [18:40:06] so i don't agree that we should operate in a split world between our own geo service and geonames [18:41:18] MaxSem: but there is no requirement... [18:41:42] jdlrobson, which requirement? [18:42:44] maybe I'm misunderstanding MaxSem but the MobileFrontend provides an optional nearby page which if someone wants to make use of they enable and bring in external services. If they don't they don't configure... I don't see what the problem is [18:43:30] tfinc: I wasn't suggesting using geonames on wikipedia.org - I was just suggesting that someone with a mediawiki instance might want to configure mobilefrontend to use geonames instead of geodata) [18:44:40] jdlrobson, that optional page can't have an external dependency if it's in core. and frankly, this functionality is too narrow for core anyway [18:44:54] but what is the external dependency - it's just a url at the end of the day right? [18:45:09] I say this is the api (url) to get articles from [18:45:17] I'm free to write my own service if I want [18:45:39] I don't see that as a dependency.. [18:46:38] service means a set of APIs [18:47:34] just believe me - when MF will be migrated, nobody'll alow that feature to be in core [18:47:37] ;) [18:52:42] pfhayes: i need high res icon 512x512 pixels [18:53:02] tfinc: how big was the one I sent? [18:53:07] i thought it was large enough [18:54:25] checking [18:55:09] lolz .. pfhayes yours is 513x513 [18:55:18] tfinc: :) [18:55:42] google doesn't want your extra pixels ;) [18:55:56] * tfinc loads up gimp [18:57:55] tfinc: i just cropped it, i can send it to you [18:58:33] pfhayes: too late .. i already did it in gimp ;) [18:58:41] tfinc: okay, thanks :) [18:58:42] i have everything staged. now its just a matter of setting a date for it [18:58:50] today it too hectic for me to hit publish [18:58:56] tfinc: okay great [18:59:06] i'll likely changing the wording about to metion UCOSP [18:59:09] tfinc: just to make sure, you built the code from the v1.0.1 branch, and not master? [18:59:12] you guys deserve the credit there .. not us ;) [18:59:20] tfinc: works for me :) [18:59:29] [tfinc@Fluffy:~/Developer/Wikimedia/WiktionaryMobile]$ git checkout v1.0.1 [18:59:29] Branch v1.0.1 set up to track remote branch v1.0.1 from origin. [18:59:29] Switched to a new branch 'v1.0.1' [18:59:35] tfinc: great [18:59:46] tfinc: any date is good for us, the sooner the better i think [18:59:55] whenever it's not too hectic [18:59:58] ok .. i need to step away for a bit . but i wanted to get everything prepped [18:59:59] sure [19:03:54] MaxSem: have you seen http://www.mediawiki.org/w/index.php?title=Extension_talk:MobileFrontend&offset=20120611162104&lqt_mustshow=16250#How_Do_we_make_to_Mobile_view_stick.3F_16250 ? [19:05:38] nothing new - MF can't tell non-mobile devices from mobile ones [19:06:05] thus all the recognition lies on the shoulders of squid [19:06:42] AND SQUIDS DON'T HAVE SHOULDERS! [19:07:07] but Max surely we can [19:07:13] there are many clues [19:07:19] as that comment suggests [19:08:23] yeah, just needs code to be written:) [19:08:41] is there anything wrong with that suggested code? [19:08:42] jdlrobson: you don't make it to code on almost all of our requests [19:08:56] jdlrobson: they are mostly handled by the cache layer [19:09:06] jdlrobson: hence the detection happening at that layer [19:09:31] ...but if we want to make our stuff useable by third parties.... [19:09:35] exactly [19:09:39] jdlrobson: for an end user using MF we pretty much already have what is needed [19:10:44] jdlrobson: with the detectFormatName method in the DeviceDetection class [19:11:11] jdlrobson: you just need to call that method and store the return in a xDevice variable [19:12:12] preilly, nope [19:12:22] MaxSem: why nope? [19:12:40] it detects various desktop browsers as random mobile browsers [19:12:56] MaxSem: well then we just need to fix those bits [19:13:13] https://bugzilla.wikimedia.org/show_bug.cgi?id=36894#c4 [19:13:30] yeh was about to say the same [19:13:51] I'll fix that this afternoon (will require changes to varnish again though ;-)) [19:16:00] jdlrobson, how's living in SF been so far? [19:16:06] great [19:16:09] I just need a house [19:16:16] that would be nice [19:16:44] jdlrobson: where have you been living so far? [19:16:52] my friends sofa by the baseball stadium [19:16:59] also, updated pull req [19:17:06] good work [19:17:10] i'm stepping out for some lunch [19:17:13] then i'll sort that out [19:17:30] okay! [19:17:38] brb [19:17:45] there's been another minor regression spotted, so looking onto that [19:59:41] jdlrobson: you have another one as well :) [19:59:53] * YuviPanda steps away for post midnight snack [20:35:52] [WikipediaMobile] jdlrobson pushed 3 new commits to master: http://git.io/A6IF3w [20:35:52] [WikipediaMobile/master] Fix bug introduced with 407c639463c11b4d8c28ed6d4f2016e3124b9731 - YuviPanda [20:35:52] [WikipediaMobile/master] Bind click events to references and other links in order - YuviPanda [20:35:52] [WikipediaMobile/master] Merge pull request #254 from yuvipanda/fix-reference-reveal - Jon Robson [20:57:05] jdlrobson: fixed your line notes on that req [20:59:26] New patchset: L10n-bot; "Localisation updates from http://translatewiki.net." [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/11230 [20:59:50] thx YuviPanda [21:03:19] New review: L10n-bot; "(no comment)" [mediawiki/extensions/MobileFrontend] (master); V: 1 C: 2; - https://gerrit.wikimedia.org/r/11230 [21:03:21] Change merged: L10n-bot; [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/11230 [21:11:44] [WikipediaMobile] jdlrobson pushed 8 new commits to master: http://git.io/NaVr5A [21:11:44] [WikipediaMobile/master] Make page's requestFromTitle return the jqXHR object directly - YuviPanda [21:11:44] [WikipediaMobile/master] Cancel current page load when hitting the 'X' button - YuviPanda [21:11:44] [WikipediaMobile/master] Simplify makeAPIRequest signature, remove unused method param - YuviPanda [21:14:17] sweet, thanks jdlrobson [21:17:35] [WikipediaMobile] yuvipanda pushed 1 new commit to master: http://git.io/XjL1MQ [21:17:35] [WikipediaMobile/master] Updated CHANGELOG and bumped version numbers - YuviPanda [21:28:44] meetings meetings meetings [21:29:42] philinje: the UCOSP team has prepped their relese of the wiktionary app. do you have any preference on when i publish it ? [21:30:14] you mean, blog post or mobile-l? [21:31:24] philinje: I think he means when he publishes the application [21:32:02] well, right now is a good window because there is not a lot of other noise, so to speak [21:32:03] philinje: we'd do the release similar to the wikipedia app. publish it in store, tweet, and watch reviews. then publish a blog post in a day or so after we see things settle [21:32:19] sure [21:32:30] id want patrick hayes around for the first couple of hours so that he could collect feedback and push any fixes [21:32:38] it should be pretty light weight on us [21:32:47] yes, has it been tested at least basically> [21:32:49] ? [21:33:12] tfinc: move https://dl.dropbox.com/u/8768784/Wikipedia-iOS-v3.2RC3.ipa and https://dl.dropbox.com/u/8768784/Wikipedia-v1.2RC3.apk to dumps? [21:34:34] YuviPanda: only if you guarantee that we won't see RC4 [21:35:21] RC2 never saw the light of day, so maybe RC4 won't either :P [21:35:27] jdlrobson: how's your 20% day going. did you find a good focus area ? [21:41:11] tfinc: have pushed the beta to market. [21:41:16] should hopefully be final [21:41:57] YuviPanda: http://dumps.wikimedia.org/iOS/Wikipedia-iOS-v3.2RC3.ipa http://dumps.wikimedia.org/android/Wikipedia-v1.2RC3.apk [21:42:04] sweet [21:42:26] YuviPanda: i'm also opening up an rt ticket to figure how i can stop being the bottleneck for this [21:42:42] sweet! cc me [21:43:01] jdlrobson: preilly https://twitter.com/sayonical/status/213023527241125889 [21:43:15] #win7 mobile site issue with expanding sections [21:43:28] email sent [21:54:54] thanks YuviPanda [21:55:27] jdlrobson: lets settle on which blackberrys you want us to get [21:55:44] yes tfinc - just looking into the windows 7 phone problem.... [21:55:47] i need the model numbers [21:55:51] k [21:57:39] jdlrobson: i mailed brion about the win7 phone [21:57:46] i bet he has it in the office [22:00:14] tfinc: do you have a working windows VM [22:00:18] tfinc: that we could try http://msdn.microsoft.com/en-us/library/ff402563(v=vs.92).aspx [22:02:25] preilly: i'm already installing that [22:02:28] also http://www.redmondpie.com/standalone-windows-phone-7-series-emulator-9140536/ [22:02:31] jdlrobson: ah okay [22:02:48] this is due to our varnish change [22:03:01] we're giving them new javascript that apparently doesn't run will be interesting to see what it is :) [22:10:11] philinje: https://bugzilla.wikimedia.org/show_bug.cgi?id=37568 is tracking bug for WLM Backend [22:10:21] awesome, thanks [22:10:40] made some links from the project page, will update with this bug [22:11:11] YuviPanda: have you had a chance to dive into the app itself? [22:11:58] philinje: yes, been working on the CSS. A little bit less today/yesterday though. [22:12:30] cool, feel free to ping me or Lindsey with questions [22:12:37] sure! [22:12:52] sometimes it's easier to find Lindsey on wikimedia-design [22:14:08] oh, didn't know that channel existed! [22:14:24] * YuviPanda adds that on autojoin [22:14:30] thanks philinje [22:17:08] * YuviPanda wonders if MaxSem is around [22:17:16] he is [22:17:27] hello MaxSem! [22:17:45] MaxSem: tfinc told me to poke you about https://bugzilla.wikimedia.org/show_bug.cgi?id=37568 :) [22:21:19] * MaxSem guesses it could be moved to some place like wlm.wikimedia.org [22:21:35] jdlrobson: is it installed now? [22:21:41] question: is Labs not powerful enough? [22:21:45] MaxSem: yeah, that sounds like a nice URL [22:21:59] MaxSem: it certainly is down more often than toolserver :) [22:22:11] and I am not sure if labs is *meant* for running these things in production [22:22:24] IIRC things should 'graduate' from labs to production [22:23:18] mmm, there were discussions about moving stuff *to* Labs, e.g. Gerrit [22:23:40] you left the wrong link to sources, btw [22:23:55] ah, to the bot [22:23:58] will correct [22:24:21] MaxSem: https://svn.toolserver.org/svnroot/p_erfgoed/api/ is the correct one [22:24:31] don't run production services on labs [22:24:31] yup, alredy found it [22:24:35] thats not what its for [22:24:58] NOT at all [22:25:18] yeah, that's what I gather too. run in labs -> 'graduate' to production [22:25:20] anything on labs has to be able to die in a flaming inferno and not drastically affect the site [22:25:21] yes [22:25:25] thats the life cycle [22:25:40] you will see some things that only run in labs like bots but if they die it doesn't dramatically affect the site [22:26:41] okay, so we need to: 1) gather size/load requirements [22:26:51] 2) ask ops for machine(s) [22:27:01] MaxSem: is this for Geo ? [22:27:05] tfinc: WLM [22:27:10] k [22:27:17] throw in an rt ticket [22:27:24] s/throw/cut [22:27:34] WLM will not kill the site either:) [22:27:43] 3) review this stuff [22:27:53] 4) ... profit ? [22:27:54] :D [22:27:55] yeah, (3) wasn't pretty from my cursory look [22:29:05] * YuviPanda definitely likes tracking bugs over milestones [22:29:12] I don't see teh DB schema [22:29:56] MaxSem: yeah, that's at https://svn.toolserver.org/svnroot/p_erfgoed/erfgoedbot/ [22:29:57] sooo [22:30:01] sql/ [22:30:39] jdlrobson: do you have the windows phone emulator installed now? [22:30:41] https://commons.wikimedia.org/wiki/Commons:Monuments_database/Setup/In_labs is what i documented on setting it up locally. it was a bit twisty / hardcoded-ey [22:30:43] ehm, is the bot also supposed to work from that machine? [22:31:02] MaxSem: well, they're supposed to access the same database... [22:31:13] mmm [22:31:24] who will be its operator? [22:31:52] MaxSem: we won't be running the 'write' part of the bot [22:32:01] MaxSem: just the 'read' part, so I don't think we even need an operator [22:32:28] what about (1)? [22:32:50] philinje: do you have rough numbers on how much traffic the app will get? [22:33:13] or the entire endeavour, rather - I'd suppose the cluster's API will be used even by other WLM apps (desktop tool platonides is building, for example) [22:34:10] MaxSem: last year it was 168208 uploads over a few months. [22:35:09] that is, we need one machine, which doesn't need to be very powerful (unless there's some serious undercover DB work) [22:35:36] let's assume it can be 10 time worse, ~2M uploads [22:36:18] MaxSem: uploads don't go through this machine [22:36:23] only queries [22:36:33] * MaxSem knows [22:36:40] so assuming 2M uploads, and 10x that many queries [22:37:53] mmm, the schema is also in Dutch. delicious! [22:39:03] heh [22:39:14] MaxSem: and some of it is hand-written, some is autogen [22:41:04] aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaargh! https://svn.toolserver.org/svnroot/p_erfgoed/erfgoedbot/monument_tables.py [22:41:17] MaxSem: 20M over 30 days is 7reqs/s, according to google. a mostly meaningless number [22:41:20] yeah, I know [22:41:33] there's something a little bit worse [22:41:34] WTFWTFWTF [22:41:47] https://svn.toolserver.org/svnroot/p_erfgoed/erfgoedbot/sql/fill_table_monuments_all.sql [22:41:50] is hand maintained [22:42:16] http://commadot.com/wp-content/uploads/2009/02/wtf.png [22:42:44] * tfinc facepalms ... [22:42:48] we can make this better [22:43:09] YuviPanda: MaxSem so the key here is not to go out of control on this refactor [22:43:15] it would be easy for us to dive really deep in [22:43:18] thats not the point [22:43:33] is to not touch the things that don't some how tie into the API. [22:43:43] that py file can be mostly left alone, for example :P [22:43:50] who said refactor? I say rewrite!XD [22:44:09] * YuviPanda hands MaxSem an american sized can of worms and an opener [22:44:42] when does the campaign begin? [22:45:19] MaxSem: september, but we should have this ready (backend + app) by end of july at most [22:45:53] how close to completion is what we currently see? [22:46:18] MaxSem: it's functionally mostly complete - only feature is the country/admin level features [22:46:28] which is pretty smallish in terms of code [22:47:21] YuviPanda: later today i'm going to look the backlog for WLM as i need to get more invovled with its moving parts [22:49:11] a good in a Russian webshop http://www.ulmart.ru/goods/279275/?head=1 [22:49:40] tfinc: then fix https://bugzilla.wikimedia.org/show_bug.cgi?id=37571 when you're at it :) [22:50:06] MaxSem: are they selling a replica of the death star? [22:50:17] i see 164 км. in there [22:50:21] no, the original one [22:50:42] against Диаметр, and i'm going to assume that that is 'diameter' :D [22:50:54] it is indeed [22:50:56] New patchset: Jdlrobson; "fix toggling on WP7 (Bug 37572)" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/11254 [22:51:00] can anyone doing a quick deployment of https://gerrit.wikimedia.org/r/11254? MaxSem preilly ? [22:52:51] YuviPanda: to answer the question about usage, the DB will be used for viewing monuments, not for uploads, and is the question referring to the DB? [22:52:58] New review: preilly; "(no comment)" [mediawiki/extensions/MobileFrontend] (master); V: 1 C: 2; - https://gerrit.wikimedia.org/r/11254 [22:53:00] Change merged: preilly; [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/11254 [22:53:07] YuviPanda: MaxSem i'm going to stress. getting the app working is priority #1, making the wlm backend code better is #2 [22:53:18] we need to not optimze before we even have anything working :D [22:53:29] "working"? [22:53:52] indeed, and I've listed the bare minimum we'd need to fix to get the backend running on the cluster as dep. bugs [22:53:58] to give you an idea, a single user in Cologne could access 9000 monuments in a single search [22:54:14] the most hilarious part is customers' reviews, e.g. "After one week of use, D.Vader has choked all the admirals, the station is uncontrollable" [22:54:29] MaxSem: no 'GAS IS NOW 5$ A GALLON!' comments? [22:55:00] they complain about uranium [22:55:08] philinje: we took number of monument pics last year (~160,000), rounded it up, multiplied it by 10, then multiplied that by 10 again, and ended up at 7 reqs/sec on average [22:55:15] might be completely bogus :) [22:56:58] thanks, Thehelpfulone [23:00:34] MaxSem: can you file appropriate bugs / comment on the tracking bug on things that block a deployment? [23:00:53] will do after CR [23:02:32] hi YuviPanda [23:02:50] yeah no problem, what would you like the component description for "Browse" to be? [23:02:58] I've made some up for the other ones but I don't know what that is [23:04:24] Thehelpfulone: 'browse through monument lists' [23:04:24] ? [23:04:31] yeah that will do :) [23:04:42] MaxSem: can you push that change of Jon's live? [23:04:54] mmmm [23:05:04] do you approve the change? [23:05:09] MaxSem: yes [23:05:13] I can push it then [23:05:16] done [23:08:11] and we want roughly 100 monuments to be shown at a time [23:08:41] preilly, should I cherrypick or just update to master? [23:09:04] MaxSem: just update to master [23:20:44] YuviPanda: still there? hope you saw the comment about usage [23:21:16] MaxSem: ^ [23:21:36] MaxSem: whom do we poke on ops side for this? Just do an estimate and file an RT? [23:21:49] yes [23:22:05] I still don't have a RT account [23:22:15] tfinc ^ [23:22:19] uh, he's not around [23:28:57] YuviPanda: I will ask for some details from Maarten and others [23:31:32] in terms of DB usage last year [23:31:44] philinje: i think maarten might have logs that might help [23:31:52] he's not online right now, however [23:32:02] neither is platonides [23:32:10] ok, will send an email and copy you and Max [23:33:20] philinje: i'm finishing up an email myself. will hit send in am inute or so [23:33:41] make sure to copy Elke [23:34:04] or do you want to follow up after I send mine? [23:34:17] philinje: sent [23:34:20] i think i missed elke [23:34:25] ok [23:36:09] MaxSem: did that change get deployed? [23:36:32] deploying to 1.20wmf4 ATM [23:37:25] sweet poke me when done [23:37:28] jdlrobson, test on enwiki [23:37:34] ok ill test now [23:38:20] fixed [23:38:26] \o/ [23:38:29] confirmed MaxSem [23:38:35] jdlrobson: SWEET [23:38:38] MaxSem: thanks! [23:39:13] WP7 is actually quite nice [23:39:18] and for once this bug makes sense [23:39:58] jdlrobson: ha ha ha [23:40:21] in the future, I'd prefer not to deploy so late cause I forgot to test on test.wp.org before syncing this time [23:42:21] MaxSem: okay seems logical [23:42:45] YuviPanda: why does contact us in RC3 open in a seperate browser ? [23:42:51] on android [23:42:56] tfinc: it always has? [23:43:10] can we make it open within the app so that we keep the ua ? [23:44:22] * MaxSem looks around for nuclear fireballs [23:44:24] tfinc: the app doesn't handle submitting forms or any js, so that's not really going to work without a fair bit of work [23:44:51] well since we register wikipedia urls you can still open it with the app. now what happens when you try that is that the app spins and spins [23:44:53] thats bad [23:45:08] as anyone who's set the app as its default wikipedia handler will just get a white page [23:45:21] tfinc: are you sure you're on RC3? it doesn't list the wikipedia app in the list for me. That was recently fixed [23:45:26] i only get my browsers [23:46:36] YuviPanda: i updated to latest in the google play store which accroding to my phone is 1.2rc1 [23:46:38] stupid phone [23:46:45] let me kill the process and see if it didn't restart it [23:46:47] tfinc: google play takes a few hours [23:46:49] or so [23:46:51] to propogate [23:47:28] YuviPanda: even if it does i'll see it in manage apps as that will tell me the version [23:47:32] tfinc: i see it as RC3 on the play app though [23:47:52] ok [23:47:58] rc1 would have that, rc3 won't [23:48:12] confirmed even though i updated it android kept the old app in memory [23:48:14] android fail [23:48:26] heh! [23:49:15] time to sleep, cya [23:49:20] gnite MaxSem [23:50:02] YuviPanda: still seeing it register itself as a handler for contact us [23:51:22] tfinc: check version again in about? [23:51:31] i'm going to smash this phone [23:51:49] oh wait [23:51:50] i'm dumb [23:52:06] night MaxSem [23:52:07] i was seeing the "Wikipedia" handler ... not hte "Wikipedia Beta" handler [23:52:16] all is well .. ignore me [23:52:16] aaah, right [23:52:21] :D [23:53:37] I actually seem to enjoy working SF time much more than the weird half-time I was doing before, especially now that jdlrobson is no longer overlapping [23:55:11] :) [23:55:11] YuviPanda: its good to have you back :D [23:55:47] :) [23:56:19] now to figure out a way to make that work *without* messing up my apartment-hunting rounds on the weekends... [23:56:29] * YuviPanda goes back to filing bugs on wlm app