== Meeting information == * #ubuntu-touch-meeting: Weather meeting, 24 Jul at 20:18 — 20:51 UTC * Full logs at [[http://ubottu.com/meetingology/logs/ubuntu-touch-meeting/2015/ubuntu-touch-meeting.2015-07-24-20.18.log.html]] == Meeting summary == ''LINK:'' http://popey.mooo.com/mirror/daily_clicks/ many builds! == Vote results == == Done items == * (none) == People present (lines said) == * ahayzen (96) * popey (50) * vthompson (40) * balloons (17) * meetingology (3) == Full Log == 20:18 #startmeeting Weather meeting 20:18 Meeting started Fri Jul 24 20:18:41 2015 UTC. The chair is popey. Information about MeetBot at http://wiki.ubuntu.com/meetingology. 20:18 20:18 Available commands: action commands idea info link nick 20:18 \o/ 20:18 weee \o/ 20:18 Ok, who's here for the weather meeting? :) 20:19 so we have https://code.launchpad.net/~ahayzen/ubuntu-weather-app/reboot-ap-create-fake-env/+merge/265191 20:19 pick me o/ 20:19 ahayzen, I think we should be able to land that today 20:19 I'll rerun the tests and I think we're good 20:19 excellent. 20:19 \o/ i'm going out in a bit so if there are any issues it'll be tomorrow but if its cool land it! 20:19 What's our list of to-do for weather before we can submit it to QA? realistically. 20:20 then if someone wants to think of test cases write detailed bugs .. i don't mind writing the AP tests 20:20 From then on out, we should be able to support people who want to help fix the autopilot bugs we have 20:20 * ahayzen will take the hit of writing AP tests as long as the bugs are detailed ;-) 20:20 and ofc community folks can write them :-) 20:20 :) 20:20 so what do we need for MVP ? 20:21 ahayzen, agreed, and I'll hopefully help with a few as well. I think it'd be good to see if other people want to contribute too 20:21 i think alot more things are now working ? 20:21 Yea, I have a handful of mps that I think finish off the last of what needs to be in the app 20:21 yeah hopefully 20:21 then just 'use it' for a few days and i'm sure we'll find mini issues 20:22 I have a bug written against the need for weather alerts, but that isn't a regression from the current app, obviously 20:22 also there is a sdk listitems bump from nik waiting until the same sdk bugs are fixed 20:22 this should improve the scrolling up/down performance (which is slow atm) 20:22 ah, and realistically we prob won't be able to land before OTA6 anyway 20:22 (or i hope it will) 20:22 yeah i'd say this is a between OTA6->7 job 20:23 Okay. 20:23 if we aim *for* OTA6 then we can land shortly after? 20:23 Sure, 20:23 but I think we really have a rather full plate already 20:23 :-) 20:24 If we could get community contributions for tests I think that'd help us out a bit 20:24 its much closer than it was i was using it on the bus back to tell me how wet i was getting/going to get :-) 20:24 i feel MVP is close now though :-) 20:24 I agree. Tests are the elephant in the room 20:25 yup 20:25 if both of us just write a test every 2/3 days we'll have like 6 tests by next week 20:25 lol 20:25 :) 20:25 just we need to actually think of the tests to write as well 20:25 Glutton for punishment 20:26 and i vote to put each test into its own MP so we don't block the landing of any 20:26 popey, do you know the state of jenkins AP on vivid ? 20:26 So, I think we can just do some soliciting for assistance, but we certainly should be able to finish them off in between ota6 and 7 20:26 i saw an email saying WIP ? 20:26 no, i know balloons_ is on it. 20:26 ah cool 20:26 but I think he's not on irc due to network issues 20:26 hehe 20:26 ahayzen, yea, 1 test per mp 20:27 'network issues' .. he is hiding from my questions 20:27 yeah we need to give this to QA at some point and to other community people to test and most likely find bugs 20:27 there are some weird things like double results when searching i've done before 20:28 but mostly it is functional 20:28 I think I can see you? 20:28 balloons, o/ 20:28 I have clones everywhere it seems 20:28 balloons, that qmltestrunner thing on device worked btw thanks :-) 20:28 Yea, let's include weather in our social solicitations this wkend. Maybe I'll write a blog post about getting it installed via my PPA... but I would need to find a way to include the API key 20:28 yea, pretty slick 20:28 :-) 20:29 vthompson, just tell them to use the openweathermap ? 20:29 we could build a click that only has that as an option? 20:29 That'd work, but I think they'd be plagued by the start up issues 20:29 startup issues? 20:29 Things getting just a white screen 20:30 i mean build a click that is hardcoded to openweathermap 20:30 just fix the setting to the value 20:30 then it should work? 20:30 I suppose, and we don't really need desktop testers 20:30 nope 20:30 ok we'll investigate that as well :-) 20:30 * ahayzen senses many TODOs 20:31 ok is that all? vthompson popey ? 20:31 hmmm 20:31 I do daily builds of the clicks on my laptop 20:31 This kind of brings up an interesting question. Should we fix this per nik90's request? : https://bugs.launchpad.net/ubuntu-weather-app/+bug/1471329 20:31 and I inject the key when I do that 20:31 Can we? 20:31 we could use that click and use twc? 20:32 ohh right 20:32 isn't the key canonical's? 20:32 the key file is important as I'm setting up the new jenkins 20:32 yeah it is :-) 20:32 :) 20:32 can we magically avoid having to inject it? 20:32 * popey comments on that 20:32 balloons, this is the WIP AP tests for weather btw https://code.launchpad.net/~ahayzen/ubuntu-weather-app/reboot-ap-create-fake-env/+merge/265191 :-) 20:32 popey, i thought before we were hiding it for a reason ? 20:33 I wonder if we should have code that automatically sets as OWM if the key variable is blank 20:33 we were 20:33 'were' or 'are' ? 20:33 we hid it because TWC asked us to 20:33 hah 20:33 and still do because that hasn't changed 20:33 ooh, I like that idea vthompson 20:33 ok so yeah why don't we ... if the key is empty use OWM 20:33 but that would hide problems in the build process 20:33 agreed 20:33 put that in the bug report 20:33 ...assign to vthompson ;-) hehe 20:34 \o/ 20:34 we'd need a test to prove TWC is working, and fail if not 20:34 yeah we can do that? 20:34 or have that as a manual test? 20:34 Well, we mock the service anyway 20:34 I mean, if the key fails to get injected, some test should fail somewhere 20:34 we have had that fail before 20:34 oh, true 20:35 vthompson, 'mock' .. inject values into our db that happen to come from TWC 20:35 ohh, you know what, I don't have a weather job in the new jenkins 20:35 wow 20:35 I was going to let you know if things were building ok or not. The status on the overall project is we're working on getting devices up and running for testing. Building clicks, checking out the code all works 20:36 awesome 20:36 So, if we build a click once things like good to test for the community, how do we want to coordinate that? 20:36 Mailing list? 20:37 i can build the click and put it on people.canonical.com 20:37 which is where we usually put it 20:37 you just tell me what bzr rev to build and I'll put it there 20:37 awesome :-) 20:38 right is that all? 20:38 k. I think we should land the remaining feature mps before soliciting the public though 20:38 +1 20:38 if we mail the list, others will pick up on it and we can spread socially 20:38 yup and at least get AP landed 20:38 Yea that sounds good 20:38 (so that people can contribute tests) 20:39 agreed 20:39 ahayzen, did you plan on having some lower level tests as well.. are there any? 20:39 balloons, 'lower level' we have 2 atm .. test empty state.. test mocking 20:40 lookie here https://code.launchpad.net/~ahayzen/ubuntu-weather-app/reboot-ap-create-fake-env/+merge/265191 20:40 then hope to build on them to test each of the views 20:40 i see only tests/autopilot/ 20:40 oh you mean /unit 20:40 ^^ 20:40 probably not for weather yet 20:41 music i'm WIP but as discussed they *may* need to be manual but gonna try and make a script which can do it automatically 20:41 Yea, it'd be good to get unit tests in there, but for now we are concentrating on getting back up to parity of the old app's test coverage 20:41 exactly AP first then unit 20:43 k. So I'll try and build weather now. Should I use the reboot branch then? 20:43 Do we care about trunk? 20:43 -reboot please 20:43 we're basically not touching trunk, only reboot 20:44 although I still build a click every day :) 20:44 oh btw me and victor have a new policy of keeping the changelog upto date as well so if either of us haven't updated it then feel free to slap us/mark as needs fixing :-) 20:44 kk 20:44 also AUTHORS :) 20:44 heeh :-) 20:44 and that 20:44 * ahayzen has a list of many things to check for each MP now 20:44 * ahayzen wishes he could somehow add that checklist to LP so it could have the checklist built into the MP itself 20:45 Copy paste into the description each time? 20:45 * balloons creates job 20:45 effort 20:45 i see thats what the canonical guys do with unity etc 20:45 "Have you made design changes, if yes have they been approved [y/n]" 20:46 but having that built into LP would be *so* useful 20:46 you can add a txt file when you bzr push can't you? 20:46 like this https://wiki.ubuntu.com/Process/Merges/Checklists/Unity8 20:46 no i mean a checklist for the reviewer to go through 20:47 - if new UI strings have been added, has the person rebuilt the pot? [y/n] 20:47 sweet, it builds fine 20:47 of course! :D 20:47 (with the key?) 20:47 http://popey.mooo.com/mirror/daily_clicks/ many builds! 20:47 no key.. so heh, I wonder if the click works :p 20:47 did we remove the depends on c++ in the end to be pure QML? 20:47 vthompson, ^^? 20:48 yep, thankfully 20:48 yeah, it's at thin package now 20:48 but IIRC is there left over stuff? 20:48 hm, maybe a backend directory? 20:48 no cpp in the reboot branch 20:48 * ahayzen thought it did something the other day when he ran $ click-buddy . 20:48 no, I think things are good 20:49 Ok, anything else? 20:49 Not from me 20:49 maybe the cmake needs updating http://bazaar.launchpad.net/~ubuntu-weather-dev/ubuntu-weather-app/reboot/view/head:/CMakeLists.txt#L14 20:49 not from me :-) 20:49 yeah. 20:49 it builds fine as a thin package, just needs a little tidy 20:49 yup 20:50 Ok, same time, same place next week, shorter meetings? 20:50 maybe and yes 20:50 vthompson, did you want weekly or bi-weekly? 20:50 Agreed 20:50 or shall i just turn up weekly anyway and bounce info to you? 20:50 Bi-weekly would work for me, but next week is good 20:50 awesome! 20:51 * ahayzen sense beer o'clock 20:51 ok, see you next week chaps! 20:51 have a good weekend 20:51 \o/ BEER! 20:51 you too 20:51 \o/ 20:51 So most likely I can attend bi-weekly starting next week 20:51 ok, thats cool 20:51 cool 20:51 Rats it's only 4 here... no beer for me 20:51 o/ 20:51 #endmeeting Generated by MeetBot 0.1.5 (http://wiki.ubuntu.com/meetingology)