[00:06:10] probably so we'd stop using all their bandwidth on empty rooms :) [00:10:41] awjr: alolita and i just talked with andrew about remote participation for video conferences [00:10:56] awjr: andrew already had new wide angle usb camera on order [00:11:04] \o [00:11:05] those should come in by the end of the week [00:11:10] awesome [00:11:13] we'll have to figure out a platform for them [00:11:14] im excited to try them [00:11:29] awjr: were also going to buy one of these http://www.google.com/intl/en/chrome/devices/chromebox.html [00:11:39] as the perma server so that no one has to lose a laptop [00:12:16] pp [00:12:18] er [00:12:19] oo [00:12:22] awesome [00:12:45] alolita spoke very highly of it and andrew said we can just order it [00:12:59] we're going to make r31 tele conference awesome [00:13:49] watch out teleconference ... your about to get more awesome [00:14:02] now with more awesome! [00:14:54] awjr: the biggest compaint about the chrome box is that its too simple ... which in our case is exactly what we want [00:15:20] perfect :) [00:17:45] awjr: http://toolserver.org/~erfgoed/api/api.php?action=search&sradm0=be&sradm1=be-vlg&sradm2=be-van&sradm3=Baarle-Hertog&sradm4=be-van&format=json [00:17:46] should that work? [00:18:27] define 'work' [00:18:30] return results [00:18:32] awjr: the first thing i'll want to test on the chromebox is vnc support. that way you'll be able to vnc in over vpn and get everything setup before the meeting ever starts [00:18:42] vnc/rdp/etc [00:18:54] tfinc hawt! [00:19:00] jdlrobson mmm [00:19:12] not sure - does anything have adm4=be-van? [00:19:22] isn't be-van an adm2 value? [00:20:13] jdlrobson: [00:20:14] mysql> select * from monuments_all where adm4='be-van'; [00:20:14] Empty set (0.00 sec) [00:20:17] so, i think it works :) [00:20:20] ok cool [00:20:44] this is exposing lots of weirdness with the api [00:20:57] impossible [00:21:14] http://toolserver.org/~erfgoed/api/api.php?action=adminlevels&format=json&admtree=us-ri < awjr wikitext names! [00:21:27] yep [00:21:36] to be expected [00:21:49] http://toolserver.org/~erfgoed/api/api.php?action=search&sradm0=us&sradm1=us-ri&sradm2=%5B%5BBristol+County%2C+Rhode+Island%5D%5D&sradm3=us-ri&format=json [00:21:56] is the url that results in.. [00:22:35] with no results [00:22:43] there should be 23 monuments, i think, for that county. [00:23:37] mm [00:24:03] it seems to be getting sradm3 from somewhere.. [00:24:23] (this is because the code has got more complicated now i need to maintain a tree [00:24:36] sigh [00:25:08] this is getting extremely unmanageable.. [00:25:29] jdlrobson: lame [00:26:28] awjr: is there anyway we can fix up the api so i can just give sradm= [00:26:45] it's really going to screw up the whole ecosystem.. back button behavior etc [00:27:04] jdlrobson: doing just sradm= will not necessarily give you expected results [00:27:23] that is not really an API problem per se but rather a data problem [00:27:27] sradm doesn't work as far as i know [00:27:33] you have to specify the level as well [00:27:36] e.g. sradm2 [00:27:37] correct [00:27:47] [WLMMobile] preillyme pushed 1 new commit to master: https://github.com/wikimedia/WLMMobile/commit/70bcd66b454b4dab13c27dadabe893f529ab4122 [00:27:47] [WLMMobile/master] fix spelling - Patrick Reilly [00:28:09] but doing what you are proposing will not always behave as you might expect [00:28:45] adm values are not necessarily unique [00:29:03] you might have 2 adm2 values that are the same, but have different parents [00:30:37] also awjr why does http://toolserver.org/~erfgoed/api/api.php?action=adminlevels&format=json&admtree=Brixen+im+Thale return itself? [00:30:41] we either need to come up with a way to map values to unique keys that will remain constant, or always follow the full tree [00:30:48] awjr: adm2 should be unique [00:30:53] otherwise how are we going to translate? [00:31:04] jdlrobson because you are using the admtree parameter incorrectly [00:31:15] awjr: how so? [00:31:22] logically that should work [00:31:24] admtree expects a pipe-delimited tree [00:31:29] arggh [00:31:32] but this is what i'm talking about [00:31:33] starting from the topmost node [00:31:36] it sucks :P [00:31:41] i do not disagree [00:31:48] is this really not fixable [00:32:05] because this is going to require some massive rewriting of how back button behavior works unless i have a brainwave [00:32:57] based on the way the data gets generated and regenerated, plus the fact that adm values are sometimes custom, manually entered values, i do not think we could set things up to have consistent unique keys [00:33:06] im not really sure what the alternative would be [00:33:11] ok [00:33:21] i will have to revisit this tomorrow then with a fresh mind [00:33:32] but i am definitely open to suggestions, because it sucks at the api level as well [00:33:44] i will keep moaning though [00:33:50] :p [00:38:29] jdlrobson that last link you sent me should not have returned results - you were getting results because of a bug [00:38:31] fixing now [00:40:21] k i'm giving up for the day [00:40:37] https://github.com/jdlrobson/WLMMobile/tree/monument-regions [00:40:41] explains what i was trying to do [00:47:27] [WLMMobile] preillyme pushed 3 new commits to master: https://github.com/wikimedia/WLMMobile/compare/70bcd66b454b...059a75c8f3a9 [00:47:27] [WLMMobile/master] First stab at bug 38922/card 146: Wikipedia article link on monument detail page - Brion Vibber [00:47:27] [WLMMobile/master] Prettify message; use explicit divs instead of CSS that turns a into block - Brion Vibber [00:47:27] [WLMMobile/master] Merge pull request #102 from brion/monument-article-link - Patrick Reilly [01:05:02] awjr: should i sit in for "Story prioritization for iteration 2" ? [01:05:28] tfinc: it's optional [01:05:30] for you [01:05:42] i think we'll be ok without you [07:10:52] hi notnarayan [07:10:58] you should get a proper IRC client :) [07:11:08] rather than using the web interface every time [07:11:19] recommend one sir yuvi. [07:11:21] I suggest XChat Azure [07:12:02] good on OS X [07:12:04] and free as well :) [07:12:12] xchat azure downloading... [07:12:42] nice [07:12:54] notnarayan: do you remember what Icon we decided to use for the 'Archives' button? [07:13:10] yes [07:13:23] do you want the icon? [07:15:21] notnarayan: yes [07:15:27] it currently just uses the text 'archive' [07:15:32] this one http://s1-05.twitpicproxy.com/photos/full/632293846.png?key=500380 [07:17:55] these icons are part of the standard rep, so, does it come with the os? [07:31:02] notnarayan: yes it does [07:31:08] notnarayan: need to find the name of it :P [07:36:29] Yuvi: its called '4 collections view as list' in the layers [07:36:46] i can send you a png if you want? [07:38:33] notnarayan: yes, in various sizes according to https://developer.android.com/guide/practices/ui_guidelines/icon_design_action_bar.html [07:39:09] notnarayan: also, type 'yu' and hit , should autocomplete my name and notify me of messages for me [07:39:39] yuvipanda: :) [07:47:32] yuvipanda: sent you the icons. [07:48:15] notnarayan: thanks :) [07:49:10] yuvipanda: you will need the share and the search ones, ill mail in ten. [07:50:02] notnarayan: they're in built [07:50:03] no need [07:50:10] if you saw the screenshots I already have share :) [07:50:14] and no search for v1.0 [07:50:18] needs a lot of backend work [07:51:57] yuvipanda: allright! let me set u my emulator [07:59:52] yuvipanda: how difficult would getting the comments be? [08:00:12] notnarayan: pretty :) [08:00:15] not for 1.0 [08:01:09] and as we are not doing search for this version, there is no way well be able to do related articles and previous article with the same tag? [08:06:50] notnarayan: no [08:07:01] notnarayan: it's okay. Baby steps :) [08:07:30] notnarayan: I'll see if I can hook up search, but there are lots of other things that need fixing first [14:45:45] Jarry1250: hello! Random question - how popular are signpost polls? [15:28:14] Hey yuvi - popular as in number of responses? [15:32:19] (yuvipanda) [15:53:47] Jarry1250: yes [15:54:10] Oh, not that many. ususally 30-50? [15:54:23] ah, okay [16:53:18] hey guys [16:53:26] hi MaxSem [16:55:23] awjr, I wrote an update script /var/wlm/update.sh - any objections against puting it into puppet for cron? [16:55:49] MaxSem let me take a look in a minute [16:55:52] but probably not :) [17:14:45] MaxSem i have one big concern with that shells cript [17:14:57] sed -e 's/monuments_all/monuments_all_tmp/' -e 's/admin_tree/admin_tree_tmp/' [17:15:42] in the event that monuments.sql changes it's table statements to use 'monuments_all_tmp' instead of 'monuments_all', that sed command would result in changing the table names to 'monuments_all_tmp_tmp' [17:16:53] obviously, changes in input data may require an update of processing code:) [17:17:14] MaxSem we don't have total control over that input data if it's getting downloaded from toolserver [17:17:53] and what happens if a change occurs and you and i are out sick/on vacation/etc? better to keep this a little more foolproof [17:17:55] we're using the dump made specifically for our needs [17:18:04] trust no one! [17:18:44] agreed! (puts everyoone on /ignore) [17:18:48] lol [17:19:02] perhaps just enclose the table names in ` [17:19:24] I tried, sed seemed to grumble [17:19:38] you may need to escape the ` [17:21:05] hmm, or was that because I forgot s/ in one place at that time? [17:21:19] sed -e 's/\`monuments_all\`/\`monuments_all_tmp\`/' [17:21:23] seems to do the trick [17:26:40] done [17:27:48] * yuvipanda looks around [17:28:02] MaxSem cool, go ahead and add it to puppet [17:28:07] * awjr waves at yuvipanda [17:31:57] hello awjr [17:32:04] how's it going yuvipanda? [17:32:37] awjr: pretty sweet, I'm quite enjoying writing Android code in Java vs PhoneGap (so far) [17:33:11] plus my house has turned into some sort of 'learn-random-stuff' hub, so people randomly keep popping in to learn CSS/HTML/How-to-use-a-DSLR/ellipse-drawing-algorithms. [17:33:22] so it is all good, plus there's the beach next door :) [17:33:23] hahaha sounds awesome [17:33:29] what are you working on for android? [17:33:42] awjr: https://www.mediawiki.org/wiki/Signpost_Mobile_App [17:33:51] ah right, awesome [17:42:15] awjr: hi, saw your email [17:44:19] you there? [17:44:45] hi philinje [17:45:05] a couple of stories are more properly part of this iteration [17:45:29] philinje what do you mean by 'properly'? [17:45:51] they are part of other work or are in progress already [17:46:39] oh - which are in progress already? [17:47:16] 136, 153, 154 - the first 3 [17:47:42] 153 is a spike [17:48:54] the last 4 are related to 133 Save for later [17:48:57] do you mean 136 and 154 are in progress because folks are already working on other stories, or someone's actually implementing those cards right now? [17:49:15] related cards do not necessarily need to go into the same iteration [17:49:22] 136 is part of the license tasks, and 154 is being worked on by Brion [17:50:09] i think we could move the last 4 to iteration 2 [17:51:00] er sorry, which last 4? [17:51:08] do you mean 138, 139, 155, 156? [17:51:15] yes [17:51:19] what about 152, 147, 141, 142? [17:51:39] all those depend on the spike, but will be in iteration 2 [17:51:46] ok that sounds reasonable [17:52:09] do they need to be moved anywhere? [17:52:19] can we add https://bugzilla.wikimedia.org/show_bug.cgi?id=39088 to the current iteration? [17:52:45] philinje i think they just need to have the 'iteration' property updated [17:53:52] MaxSem: makes sense to me, what do you think, Arthur? [17:55:01] MaxSem, philinje: it is generally preferable to not add new work to a current iteration, but it's up to Phil. if getting the app to use the new API location is of higher priority than any of the other stories, then it probably makes sense. but if it can wait to be implemented for the next iteration, i think that would be better [17:55:49] i think the issue is that Max would do this as part of other work [17:56:09] actually i think there's already a story for this iteration that captures that bug [17:56:27] yeah [17:56:28] https://mingle.corp.wikimedia.org/projects/wlm_android_app/cards/145 [17:56:39] "Given I am using the WLM app When I take an action that requires an API request to the monument or adminlevels API, Then the API request is handled by a server hosted on the WMF architecture." [17:57:10] so in order for that story to be complete, the app need to be making API requests against the WMF-hosted api [17:57:11] the bug is about using tool server as a back-up - the title is a little misleading [17:58:00] no, the bug is about switching to our server first of all [17:58:37] using TS as backup will require more work - i think it should be made into a separate card and prioritized against upcoming work for future iterations [17:59:13] MaxSem: sorry, is this covered in 145? [17:59:19] no [18:00:07] 145's title is misleading [18:01:10] I've renamed it to "Make app use wlm.wikipedia.org" [18:01:16] away [18:01:50] MaxSem makes sense [18:02:05] MaxSem: ok, so where should the bug go? [18:02:14] i need to go into a meeting [18:04:05] philinje i made refernce to the bug in the story, and referenced the story in the bug. [18:12:54] awjr: should i be moving things to 'complete development' or mvd to 'ready for review'? and should i do that when i've got code ready to merge, or only once it's been merged? [18:14:05] brion: just move the cards to 'ready for review' once you've met the acceptance criteria [18:14:19] ok [18:14:41] i guess we havn't really talked about how to handle the next bit [18:14:56] my feeling is that whoever is reviewing your code should then grab the card and put it 'in review' [18:15:11] if your code does what it's supposed to do, then it should be merged and the card moved to 'ready for signoff' [18:15:22] if it doesn't, it should be pushed back to 'ready for development' [18:15:35] hmm, sensible [18:15:49] i'll send out an email and propose that [18:30:43] good news awjr [18:30:50] i managed to get round the api nastiness [18:32:12] bad news … http://toolserver.org/~erfgoed/api/api.php?action=adminlevels&format=json&admtree=us returns duplicates us-or and us-OR ( <--- multichill / awjr ) [18:34:34] oh, right, awjr, you online? Where is the script that fills this? [18:34:56] The uppercase/lowercase seems to be causing problems [18:35:55] jdlrobson: Do you have any exact duplicates (same case)? [18:36:08] brion: rmoen: jdlrobson: were playing around with a boot to gecko device in r36 talking with one of the mozilla guys . we'll come with it later [18:36:16] nice [18:36:42] multichill: not that i know of just upper/lower [18:36:48] jdlrobson: And what kind of api nastiness exactly? [18:36:54] Ok, do you have a bug? I'll fix it [18:36:56] tfinc: sounds fun [18:37:12] not yet multichill … will raise as soon as my current pull request is in a nicer state [18:37:32] Don't bother, should have fixed it by than :P [18:37:40] the api nastiness was just working with the api - I was venting some frustrations with awjr yesterday.. it didn't work how I expected it to work.. mostly because codes are not uniqu [18:38:41] tfinc: nice! neither me nor rmoen were able to get an emulator/simulator running, sadly [18:39:31] multichill: hi yes im here [18:43:54] jdlrobson, multichill i can investigate the dupes (due to caps issues) and see about a quick fix. i imagine it's related to how the data gets populated/recorded and how the API code handles queries (since ca-or != ca-OR) [18:44:05] er us-or != us-OR [18:44:16] i'll take a look after lunch [18:44:22] awjr: I'm thinking about just doing a LOWER() on input [18:44:36] Running the query right now [18:45:08] multichill makes sense - but what about for the adm levels that people are adding input to as wikitext? [18:45:29] Just the adm levels with iso in the source field name [18:45:39] For the other fields it should be case sensitive anyway [18:45:46] k [18:45:47] so awjr how's 143 coming along? [18:45:55] Mine is done except for 143 [18:46:11] should I push it for review with a note saying that some changes are needed based on the work you do? [18:46:26] jdlrobson i've had almost 0 time to look at it aside from the research i've done on it last week and yesterday [18:46:31] also can campaigns-data.js be thrown away? [18:46:45] (i've stopped using it now it's dynamic) [18:46:54] jdlrobson: brion rmoen: we'll come by around noon so don't run off to lunch just yet ;) [18:47:01] and thus generate_campaigns.py [18:47:17] noon gerrit lunch meeting [18:47:25] noooooo! [18:47:32] ;_; [18:47:35] jdlrobson why did you switch to dynamic campaign stuff? it doesn't get updated particularly often [18:48:36] jdlrobson also if we're switching to dynamic campaigns, code will need to be updated elsewhere that makes use of the campaign info [18:49:12] well i'm using the api for all the drilling down [18:49:14] including top level [18:49:23] i guess it's going to be needed for license information etc? [18:50:00] jdlrobson and a few other things, i think - i'd have to dig, but i know it gets used for populating a variety of things in the photo desc on upload [18:50:26] brb phoe [18:50:41] awjr: Ok. Easy fix. In fill_table_monuments_all.sql replace all `iso` AS admX with LOWER`iso` AS admX. [18:51:05] That way we have all iso codes in lower in the main table and also in the admintree table [18:51:56] Can you apply that (after lunch) or do I need MaxSem for that? :-) [18:53:42] ok back [18:53:58] multichill the only fields using the iso codes are adm0 and adm1, right? [18:54:14] Sometimes adm2 too. [18:54:24] But that's easy to spot in the query [18:54:25] but sometimes not? [18:54:47] Hi MaxSem, were the changes to sitematrix ever deployed? [18:54:49] adm2 is a municipality in the Netherlands ([[Haarlem]]), but an iso code in Belgium [18:55:06] multichill how would you recognize it in the query? [18:56:48] Most if not all source fields have "iso" in their name [18:57:55] multichill specifically for adm2? [18:59:17] multichill looking quickly through fill_table_monuments_all.sql, it does not appear to be applied consistently. [18:59:32] https://fisheye.toolserver.org/browse/erfgoed/erfgoedbot/sql/fill_table_monuments_all.sql?hb=true ? [18:59:43] What isn't applied consistently? [18:59:59] the use of 'iso' in adm2 fields [19:00:41] Hehe, that's because countries are very inconsitent ;-) [19:00:55] take a look at https://en.wikipedia.org/wiki/ISO_3166-2:US and https://en.wikipedia.org/wiki/ISO_3166-2:BE [19:01:44] You're basicly saying that the ISO 3166-2 system is inconsistent :P [19:03:22] multichill in fill_table_monuments_all.sql, there are 20 uses of -iso and 10 uses of _iso in adm values [19:04:43] l [19:04:50] err, sorry [19:05:24] Search replace `(.)iso` -> LOWER(`\1iso`) ? [19:05:51] Ok. http://toolserver.org/~erfgoed/api/api.php?action=adminlevels&format=json&admtree=us looks clean now [19:05:58] multichill ok got it, just wanted to make sure we hit the right ones [19:07:07] hmm, dealing with the pre-honeycomb / post-honeycomb bits in Android is also a pain. It's like all the hassles of a stubbornly statically typed language without any of the benefits [19:07:43] multichill please let me know when you commit that change [19:08:07] awjr: Oh, right, I just backed off because I thought you were doing this :P [19:08:56] oh, i thought you just did it on TS [19:09:31] Ok. committing [19:09:49] thnx [19:09:51] lunch time [19:10:38] * yuvipanda goes to sleep, so he doesn't miss college like he did y'day [19:10:41] gnite folks [19:11:49] Ok. Updating the database now [19:14:56] multichill: hi, did you see my email about Wikipedia articles in the monument DB? [19:16:29] Yes, I think so, let's see [19:17:04] So you're wondering how we got the article links? [19:17:38] Some Wikipedia's have a field for that, so we just take that. For other Wikipedia's we apply a bit of logic: We extract the first [[ ]] link in the text of the name [19:18:00] Will the article exist? We don't know [19:21:04] philinje: See for example https://en.wikipedia.org/w/index.php?title=National_Register_of_Historic_Places_listings_in_Sauk_County,_Wisconsin [19:21:25] Some red links and some blue links [19:22:16] [WLMMobile] preillyme pushed 2 new commits to master: https://github.com/wikimedia/WLMMobile/compare/059a75c8f3a9...74c2b064ae05 [19:22:16] [WLMMobile/master] Card 130 / bug 38869: link to license code on upload confirmation page - Brion Vibber [19:22:16] [WLMMobile/master] Merge pull request #105 from brion/license-link - Patrick Reilly [19:22:41] [WLMMobile] preillyme pushed 2 new commits to master: https://github.com/wikimedia/WLMMobile/compare/74c2b064ae05...caba9440525c [19:22:41] [WLMMobile/master] Bug 38744 / card 121: fix map attribution - Brion Vibber [19:22:41] [WLMMobile/master] Merge pull request #107 from brion/map-attrib - Patrick Reilly [19:23:49] MaxSem: Ahum, "ERROR 1054 (42S22) at line 51: Unknown column 'lat_int' in 'field list'" [19:24:07] caxthelm, hi - it isn't live on Wikipedia yet, but you can take a look at it on mw.org: https://www.mediawiki.org/wiki/Special:ApiSandbox#action=sitematrix&format=json [19:24:17] multichill, where? [19:25:22] MaxSem, thank oyu [19:25:24] you* [19:26:11] MaxSem: In fill_table_monuments_all.sql [19:26:45] What is monuments_all_tmp ? [19:27:04] Ah, found it, monuments_all_tmp is not the same layout as the main one [19:27:18] FAIL [19:27:40] You seem to have introduced it today in https://fisheye.toolserver.org/browse/erfgoed/erfgoedbot/sql/fill_table_monuments_all.sql?r2=813&r1=811 [19:27:41] yeah, there's a leftoff from my previous experiments [19:27:50] Didn't you test it? :-( [19:28:37] on my machine it worked fine, however on ts there's monuments_all_tmp already [19:30:12] r815 [19:31:53] multichill, what happened with cron reports via email? [19:33:54] Looks like /home/project/e/r/f/erfgoed/bin/update_monuments_cron.sh writes it to a log file [19:34:51] Wow, these are quite large. 7M [19:35:43] is someone actually able to comprehend what's in these logs? :P [19:37:41] wow, TS is seriously overloaded [19:37:55] MaxSem: I understand it, but I'm not happy with it, see https://bugzilla.wikimedia.org/show_bug.cgi?id=37420 [19:38:01] Are you running the full update? [19:38:51] just fill_table_monuments_all.sql, aka almost full:) [19:39:58] Can you run php populate_adm_tree.php when you're done? [19:41:59] sure [19:43:41] done [19:45:14] Great, jdlrobson: Have fun with all the lowercase data http://toolserver.org/~erfgoed/api/api.php?action=adminlevels&format=json&admtree=nl :-) [20:01:22] philinje: https://play.google.com/store/apps/details?id=org.wikipedia.wlm <-- when it goes live [20:02:10] philinje im ready when you are to discuss prioritization for sprint 2 [20:03:48] ok, one sec [20:05:35] thanks multichill [20:14:50] "Wikipedia Mobile View bug? [20:14:52] Accessing Wikipedia pages by Google search on my up-to-date iPhone 4S using Safari shows up in desktop view instead of automatically switching to mobile view. Can anyone confirm this for me? — Preceding unsigned comment added by Scott523 (talk • contribs) 20:07, 7 August 2012 (UTC)" [20:14:58] VP/T [20:16:19] MaxSem: hi, have you started the research on the sorting? [20:16:42] philinje, not yet - have to redo DB import [20:17:04] ok [20:22:03] philinje: What do you want to sort exaclty? [20:23:08] multichill, monument list [20:23:25] multichill, https://mingle.corp.wikimedia.org/projects/wlm_android_app/cards/145 [20:23:42] multichill: http://toolserver.org/~erfgoed/api/api.php?action=adminlevels&format=json&admtree=us%7Cus-ok [20:23:47] lots of names contain [[ and ]] [20:23:50] can these be removed? [20:24:16] basically, I need to determine what sort options can be implemented w/o melting down the DB [20:24:18] In the source? No, in the output, yes [20:24:59] jdlrobson: In the admintree the field contents for the computer and for the human should be two different fields [20:25:32] Computer gets "us-ca" or "[[Alfalfa County, Oklahoma]]", human gets "California" or "Alfalfa County, Oklahoma" [20:25:53] * multichill looks at awjr [20:27:15] awjr: If we add a value field we can just stay clear of the name field and do all the i18n and cleanup on the value field [20:27:24] in the output [20:27:41] {name:[[Oklahoma County, Oklahoma]], level:2} [20:27:50] name should not have those double quotes in my opinion [20:28:34] bike9 [20:28:59] sorry, disregard [20:31:02] jdlrobson: It's used for example at http://toolserver.org/~erfgoed/api/api.php?action=search&format=html&sradm0=us&sradm1=us-ok&sradm2=[[Oklahoma%20County,%20Oklahoma]] [20:31:49] Hi MatthewARoth [20:32:30] ahh ok [20:32:31] owch [20:32:50] multichill: hi Mr Dammers! [20:32:54] multichill: there's no way we can fix that field so it would be http://toolserver.org/~erfgoed/api/api.php?action=search&format=html&sradm0=us&sradm1=us-ok&sradm2=Oklahoma%20County,%20Oklahoma [20:33:15] That would break other functionality [20:33:51] So that's why I'm suggesting, what I've stuggested before, a field for the human readable output [20:34:04] We need that for the iso i18n anyway [20:34:39] * aude waves [20:36:46] aude: hi! [20:39:34] MatthewARoth: we have a walrus meeting tomorrow evening, fyi [20:39:47] walrus = us wikimedians [20:39:57] meta.wikimedia.org/wiki/WALRUS/August_2012 [20:40:10] wlm is top of the agenda [20:41:22] aude: you guys should chat about the android app :D [20:41:45] tfinc: heh [20:42:01] i've tried it in the eclipse simulator and it's cool from what i can tell [20:43:36] aude: it'll make WLM so much easier to participate in [20:43:47] yes indeed [20:52:13] aude: thanks for the advice. I'll check it out. [20:53:40] aude: I can definitely participate [20:54:54] MatthewARoth: great [20:56:59] See you later :-) [21:34:37] [WLMMobile] jdlrobson pushed 1 new commit to master: https://github.com/wikimedia/WLMMobile/commit/ddeb7e20471be54fcb067c17d7037281aa6ccaf3 [21:34:37] [WLMMobile/master] Update cordova.js for 2.0.0 on iOS - Brion Vibber [21:37:15] [WLMMobile] jdlrobson pushed 1 new commit to master: https://github.com/wikimedia/WLMMobile/commit/6d0341ebbc833dff955a02bdd3d742e50843932d [21:37:15] [WLMMobile/master] fix whitespace in file description template (bug 39107) - Brion Vibber [21:45:44] jdlrobson: awjr: maxsem preilly: brion: pchang: reminder that the mobile hangout lunch is set for tomorrow [21:45:55] whee [21:46:20] awesome [21:46:48] jdlrobson: your on to disucss inline styles and why you hate them [21:47:16] awjr: your listed as talking about bugs that are coming in that may not be on our radar yet. [21:47:20] i put you on that [21:47:35] tfinc for wlm app in particular or in general? [21:49:32] and pchang_ youre on for talking about the kind of feedback we've been getting about all of our products [21:49:35] awjr: in general [21:49:44] ok [21:51:06] awjr: anything that were just not looking at that could be important down the line [21:51:48] sounds goo [21:51:49] d [22:01:06] awjr: ready on skype when you are [22:37:20] Ryan_Lane, is import from text files with mysqlimport more appropriate? [22:38:30] preilly: I don't know [22:38:35] Ryan_Lane: regarding https://gerrit.wikimedia.org/r/#/c/17964/1/files/misc/wlm/update_from_toolserver.sh [22:38:37] MaxSem: ... [22:38:38] well [22:38:54] it would be nice if rather than dropping everything and reimporting everything, that it would import the changes [22:39:00] which would be smaller [22:39:07] dropping the tables will lead to a service outage, right? [22:39:41] nope, the API knows how to handle a temporary absence of tables [22:40:49] delta updates don't make much sense as daily changes could esily reach tens of percent [22:41:05] and the DB isn't replicated anyway [22:42:00] these datasets are somewhat maddening. [22:43:06] MaxSem: ok, if the api can handle missing tables (which seems like a hack), then you should do a mysqldump with drop tables included [22:43:12] and then just import the dump [22:43:30] I'm also not a huge fan of the wget [22:43:34] it's using http [22:43:39] across the wan [22:43:47] that's a problem [22:44:08] it needs to use https, or ssh, or something with encryption and verification [22:44:50] ssh could be good [22:53:11] ok, I'll switch to https [22:55:22] MaxSem: can you add MD5 sums to it too [22:56:13] eh? if attacker can fake the TS, xe can fake checksums, too [22:58:42] MaxSem: you could salt them too [23:14:14] here's what we have: http://dpaste.org/cx8DS/ [23:15:08] Ryan_Lane: ^^ [23:17:59] MaxSem: why the need for drop tabled? [23:18:00] *tables [23:18:09] make the dump on the other side add the drops into the dump [23:18:23] then you don't need drops or creates [23:18:26] it'll do it for you [23:18:47] Ryan_Lane, now it doesn't use .sql at all, just plaintext files [23:18:55] .... [23:18:58] we *want* sql [23:19:35] why? [23:20:40] mysqldump --add-drop-table [23:20:55] then you don't need all that extra crap [23:20:59] searched on commons for "money lol" and got a picture of a vulva #fail - [23:21:36] jdlrobson, vandalised description? [23:22:13] and maybe --compact [23:22:15] MaxSem: you do realize you've now put me in the awkward position of viewing a vulva in the office [23:22:41] MaxSem: and please add an MD5 of the file so that we can verify it on the other side [23:22:47] it doesn't look like vandalsm.. just a weird description [23:22:57] MaxSem: and --extended-insert [23:23:49] Ryan_Lane, maybe something less homebrewed e.g. PGP? [23:24:43] MaxSem: we're just checking for validity of the data. we don't need encryption [23:25:34] md5sum -b $DUMP > $DUMP.md5 [23:25:35] if [ -f $PREV.md5 ] && diff $PREV.md5 $DUMP.md5; then [23:25:36] rm $DUMP $DUMP.md5 [23:25:37] fi [23:26:49] MaxSem: can you also download the file into a name with a date? [23:26:55] so that we can rollback if we get shit data? [23:27:16] MaxSem: something like DUMP="$DB.`date '+%Y%m%d'`.sql.gz" [23:29:51] MaxSem: and PREV could be something like PREV="$DB.`date -d'1 day ago' '+%Y%m%d'`.sql.gz" [23:33:10] I'm still afraid of security issues with import of .sql files [23:34:15] no matter how you import it, it could issue shell commands [23:34:18] New review: Krinkle; "Haven't tested it, but RL wise looks ok." [mediawiki/extensions/MobileFrontend] (master); V: 0 C: 1; - https://gerrit.wikimedia.org/r/14419 [23:41:21] Ryan_Lane, "no matter how you import it, it could issue shell commands" - when importing from tab-delimited text files it can't [23:47:01] * Ryan_Lane shrugs [23:47:07] push your change in, and asher will review it [23:47:38] okay