/srv/irclogs.ubuntu.com/2014/09/15/#ubuntu-app-devel.txt

=== sephiap_ is now known as sephiap
ahoneybunback00:19
=== Laney is now known as Guest69163
ahoneybundoes anyone know what are the packages for the SDK's api modules?00:20
=== tsimpson_ is now known as tsimpson
=== wgrant_ is now known as wgrnat
=== wgrnat is now known as wgrant
mhall119ahoneybun: you mean the runtime?01:00
ahoneybunmhall119: no the modules01:50
ahoneybunmhall119: module "NotesApp.Plugins" is not installed01:50
ahoneybunI'm trying to run the notes app01:50
mhall119ahoneybun: the default installed one?02:19
mhall119ahoneybun: it looks like it's included in the notes-app package, so I would assume the sourcecode for the plugin is in the same bzr branch02:22
mhall119if you run it locally with qmlscene, you'll need to pass it -I with the path to the built plugin02:22
=== chihchun is now known as chihchun_afk
ahoneybunI took it from bzr02:30
ahoneybunmhall119: I took it from bzr02:31
ahoneybunbut the module is still not fount02:32
ahoneybun*found02:32
ahoneybunmhall119: got to tell you the emulator is awesome02:38
mhall119ahoneybun: is it working again? There was a bug with adb last week02:59
mhall119ahoneybun: is the C++ source for the plugin in the bzr branch?03:00
ahoneybunmhall119: adb?03:08
ahoneybunmhall119: I see a components.cpp file in src03:08
ahoneybunnotes-app/Component mhall119 ?03:10
ahoneybunI emailed the dev so I'll see tomorrow I'll be on here tomorrow even during class03:10
mhall119ahoneybun: might be03:13
mhall119adb = Android DeBugger, it's what lets the SDK talk to a device or emulator03:13
ahoneybunmhall119: oh I know what it is, yea it works kinda flappy bird will not start and the updates to music and afew other apps makes them not launch as well03:15
ahoneybunlol03:15
Akiva-Thinkpadahoneybun, hey03:17
Akiva-Thinkpadhow goes the sdk?03:17
* Akiva-Thinkpad heads out again 03:22
* ahoneybun looks on ebay for a nexus 4 after being burnt out trying to port it to his current phone03:24
=== chihchun_afk is now known as chihchun
Akiva-Thinkpadhey all03:51
Akiva-ThinkpadE: Unable to locate package libqt35d504:15
Akiva-Thinkpadughhh, the software updater needs to be threaded04:45
Akiva-Thinkpadthis is so painful on a slow connection04:46
Akiva-Thinkpadnik90, Just to let you know, that branch you and mihir worked on came in really handy.06:21
justCarakasmorning all06:59
Akiva-ThinkpadjustCarakas, hey morning07:42
Akiva-Thinkpadnik90, mihir ping07:42
mihirAkiva-Thinkpad: pong07:42
Akiva-Thinkpadmihir, hey just want to make sure we are not doing the same work07:42
Akiva-Thinkpadmihir, nik90  linked me the branch you too were working on; I used some of the revisions and merged it07:43
Akiva-Thinkpadmihir, do you want to take a look?07:43
Akiva-Thinkpad(merged it with mine) ; this is for the day and week view07:43
mihirAkiva-Thinkpad: yes , exactly we shouldn't do same work..lets collaborate and finish it , do you have screenshots ?07:44
mihirAkiva-Thinkpad: i guess, i have shared all the screenshots where we have been working right?07:45
Akiva-Thinkpadmihir, yah, although it might be easier if you just ran it07:45
mihirAkiva-Thinkpad: i can't do now because at work place..can you share screenshots ?07:45
Akiva-Thinkpadmihir, hard to tell;07:45
Akiva-Thinkpadmihir, yah what I'll do, is I'll have your branch on the right, and my branch on the left, and will do screenshot comparisons07:46
Akiva-Thinkpadgive me 5 minutes07:46
justCarakaso/ Akiva-Thinkpad07:49
Akiva-ThinkpadjustCarakas, you just screwed up my screenshot!07:50
Akiva-Thinkpad:P07:50
Akiva-Thinkpad a notifcation popped up and covered my app07:50
justCarakashahaha :D07:54
justCarakasperfect timing07:54
=== ogra_ is now known as ogra
=== ogra is now known as ogra_
=== Guest69163 is now known as Laney
Akiva-Thinkpadmihir, nik90 http://imgur.com/a/QRRdA#008:11
mivoligomzanetti: hi08:14
Akiva-ThinkpadThere are a few things that need tweaking. as written out08:15
mzanettimivoligo: hi08:23
mivoligomzanetti: about the svg files for towers: should we go with one file for each level or put all levels in one file?08:24
mzanettimivoligo: not sure I understand the question08:25
mzanettimivoligo: what do you want to put into one single file?08:26
mivoligomzanetti: we have one tower with couple upgrades/levels08:26
mzanettiah, right...08:26
mivoligomzanetti: so one line for each level?08:27
mzanettihmm... I guess they could be done in one file. however I thought about having different rotations for towers too.08:27
mzanettimivoligo: if you don't mind I'd prefer one file per level08:27
mivoligomzanetti: I don't mind :)08:28
mzanettimivoligo: so in the future we could add different perspections for towers... fabio's propsal for a level pack would want that support08:28
mzanettimivoligo: did you see the bug I reported for you?08:29
mivoligomz08:29
mivoligomzanetti: yes08:29
mzanettiok08:29
mivoligomzanetti: I'll get the towers ready and upload them asap08:30
mzanettimivoligo: awesome. I have created proper level.json files for the first 5 levels08:30
mzanettihad quite a bit of fun playing it yesterday already :D08:30
mzanettiits challenging08:30
mivoligothat's most important :)08:31
mzanettimivoligo: one thing I noticed is that board wise, level 5 is much easier than, say level 408:32
nik90Akiva-Thinkpad: hey,08:32
Akiva-Thinkpadnik90, heya08:32
mzanettimivoligo: level4 is the hardest one. no good places for effective towers08:33
mivoligomzanetti: we can change the order I think08:33
Akiva-ThinkpadI fixed the agenda view08:33
Akiva-Thinkpadso they now look identical; both branches08:33
mzanettimivoligo: I did compensate with harder/easier waves, so I guess we're fine08:33
nik90Akiva-Thinkpad: ah ok08:33
mivoligomzanetti: ok then :)08:33
nik90Akiva-Thinkpad: I really like your Week view (without changing the background colors)08:33
mzanettimivoligo: just an interesting fact to keep in mind. You can influence those things already when designing the path08:33
Akiva-Thinkpadnik90, yah; thats easily reversable08:34
Akiva-Thinkpadi read your post08:34
Akiva-Thinkpaddesigners are busy08:34
Akiva-Thinkpadetc08:34
Akiva-Thinkpadif I submit a branch, it will be with the standard background as is08:34
nik90Akiva-Thinkpad: ack08:34
nik90Akiva-Thinkpad: also mihir and I were using the orange font color only for the current day.08:35
Akiva-Thinkpadthe day view needs work, due to the redundancy. I do however think it is superiour in one way08:35
Akiva-Thinkpadnik90, oh yah thats being fixed08:35
Akiva-Thinkpadthat was just temp because of the background.08:36
nik90Akiva-Thinkpad: Ah cool08:36
Akiva-Thinkpadbasically it will go back to how it was; grey background with white text08:36
Akiva-Thinkpadthat is what you are talking about right?08:36
nik90Akiva-Thinkpad: btw in the day view, do we need to show the 3 days instead of what we had before?08:36
nik90Akiva-Thinkpad: yes08:36
dholbachdavidcalle, salut mon ami - comment ça va?08:37
mzanettimivoligo: ah, also, did you pull today already?08:37
mzanettimivoligo: have seen the new MainPage?08:37
Akiva-Thinkpadnik90, I think there is one benefit; if you have multiple events side by side, it lets people know that they are part of different days08:37
nik90Akiva-Thinkpad: I was thinking of changing that to coolGrey to improve the contrast08:37
=== chihchun is now known as chihchun_afk
Akiva-Thinkpadnik90, other than that, I'm willing to drop it in favour of what you have.08:37
mivoligomzanetti: yes, just now08:37
mzanettimivoligo: what do you think?08:37
JamesTaitGood morning all; happy Monday, and happy International Dot Day! :-D08:37
nik90Akiva-Thinkpad: I am trying to see where we can improve the vertical space saving in the day view.08:37
Akiva-Thinkpadnik90, ah yah; that can be done in another branch though easy peasy. If you want, I can do it in this one08:37
nik90Akiva-Thinkpad: tbh your looks good as well.08:38
mivoligomzanetti: need some work :D08:38
mzanettimivoligo: sure... its just some proposal08:38
nik90Akiva-Thinkpad: what happens when you have 2 all day events on the same day?08:38
Akiva-Thinkpadnik90, well i'm of the opinion to just make it hideable; I find it hard to imagine that someone needs to look at the month day all the time.08:38
mivoligomzanetti: what settings do actually need?08:38
Akiva-Thinkpadnik90, oh good question08:39
mzanettimivoligo: so far its mostly developer/level pack creator tools08:39
Akiva-Thinkpadnik90, basically the button, will say "2 events"08:39
Akiva-Thinkpadrather than have two buttons08:39
nik90Akiva-Thinkpad: smart..I like it08:39
Akiva-Thinkpadwhen you click on the button, a bubble pops up, and lets you select which one08:39
mivoligomzanetti: I mean in the final product08:39
Akiva-Thinkpadnik90, don't thank me; that was how it used to be done. I just converted the rectangle into a button, and moved it around a bit08:40
mzanettimivoligo: I guess we should have at least the "Reset saved scores" one, no?08:40
Akiva-Thinkpadnik90, i'll work on day view, because I'm not happy with it atm for obvious reasons08:40
mivoligomzanetti: sure, and sound and info about the game but not sure what else08:41
nik90Akiva-Thinkpad: oh can you fix the stuff like the background color, font color etc etc and then ping me. We can then considering merging them into mihir's original branch.08:41
Akiva-Thinkpadweek view looks great though in my opinion. And month, well its basically going to be the same as before minus the month header, which will be in the actual header.08:41
Akiva-Thinkpadyah sure08:41
Akiva-Thinkpadwill do08:41
mzanettimivoligo: and I think we should also have some in-game hints on how to create level packs. at least some small text and a link that brings you to a how to page in the web or so08:42
nik90Akiva-Thinkpad: I am fully convinced yet about the the day and dates shown since it duplicates what the actual header shows08:42
nik90(in the day view)08:42
nik90Akiva-Thinkpad: but let's see08:42
Akiva-Thinkpadnot fully convinced, or fully convinced?08:42
nik90not*08:42
Akiva-Thinkpadah08:42
nik90I will keep an open mind08:43
nik90Clean and polish up your branch and then I will test it out on the phone08:43
davidcalleSalut dholbach, bien et toi ?08:43
Akiva-Thinkpadwell the day view is redundant the way I have it now. If it were my way, I'd just have the month and year there, make it hidable.08:44
mivoligomzanetti: so, do you think it will be more different kinds of levelpacks in the game? Or it will come out under different name?08:44
Akiva-Thinkpadyah08:44
Akiva-Thinkpadsure08:44
dholbachdavidcalle, très bien, merci :)08:44
Akiva-Thinkpadi'll make a day view branch which uses yours, and my way08:44
mzanettimivoligo: I'm not sure about that yet...08:44
Akiva-Thinkpadso two branches; you can decide08:44
mzanettimivoligo: what would you suggest?08:44
mihirAkiva-Thinkpad: sorry was away for lunch08:44
Akiva-ThinkpadUnforgivable!08:45
mihirAkiva-Thinkpad: i won't prefer all to orange tbh08:45
* Akiva-Thinkpad gets to tinkering08:45
nik90Akiva-Thinkpad: cool thnx08:45
Akiva-Thinkpadignore the orange08:45
nik90mihir: yup told him08:45
mihirAkiva-Thinkpad: reason, it is eye catchy , and i won't prefer all day to eye catchy, except today..what do you say?08:45
nik90he will fix that08:45
Akiva-Thinkpadorange day names were just temp; its going back to white on grey08:45
nik90mihir: I really like week view (last screenshot)08:46
mivoligomzanetti: don't know. I wonder how many people are interested in making the levelpacks08:46
mzanettiyeah... me too...08:46
nik90mihir: he goes back to "Day \n Date" while still saving vertical space.08:46
* mihir skips to the week view 08:46
dholbachdavidcalle, did you get feedback from everyone you reached out to? like Loïc, Pat, Jamie?08:46
=== chihchun_afk is now known as chihchun
mihirnik90: do you prefer to have two rows, for day and date ?08:47
mivoligomzanetti: you mentioned fabio, can we see his stuff somewhere?08:47
nik90mihir: well previously (as we discussed) I wanted to chnage to one row since it saves space. However ...08:48
nik90mihir: Akiva-Thinkpad nicely simplified the all day event bar, and hence having two rows looks nice08:48
davidcalledholbach, after waiting 10 days I've given up on Loïc, but yes for the rest. I've just published frameworks, device kits in a moment.08:48
nik90mihir: we can re-evaluate soon08:48
mihirnik90: okay make sens :)08:49
dholbachdavidcalle, nice - can you give me the links once you're done? just so I can let the world know about your work, see if any links need bending and update the spreadsheet08:49
dholbachdavidcalle, good work08:49
mihirAkiva-Thinkpad: nice :)08:49
davidcalledholbach, frameworks : http://developer.ubuntu.com/apps/platform/guides/frameworks/08:49
dholbachdavidcalle, I'll also ping Loïc again and see if he can probably do a review and maybe file a bug if necessary08:49
dholbachthanks davidcalle08:50
davidcalledholbach, thanks08:50
Akiva-Thinkpadmihir, or nik90  http://pastebin.ubuntu.com/8349006/  just a question; is there anyway to set the border image to be behind the label, while keeping it as part of the label component?08:50
nik90Akiva-Thinkpad: did that not work?08:50
dholbachdavidcalle, maybe we should file a reminder bug on ubuntudevelerportal, so we update the frameworks doc once the first stable framework is announced08:51
Akiva-Thinkpadnik90, its sits above the label, so you can't see the day names all too well08:51
davidcalledholbach, good idea, doing it this morning08:51
dholbachrock and roll08:51
nik90Akiva-Thinkpad: may try setting z: -1 for the border image?08:51
* Akiva-Thinkpad checks if there is a background property08:51
Akiva-Thinkpadnik90, oh great thanks08:52
Akiva-ThinkpadAH THATS WHAT Z does :P08:52
Akiva-Thinkpadnik90, perfect08:52
Akiva-Thinkpad!cookie08:52
ubot5Wow! You're such a great helper, you deserve a cookie!08:52
nik90:D08:53
t1mp:)08:53
t1mpdoes it work like this?08:53
t1mp!cookie | nik9008:54
ubot5nik90: Wow! You're such a great helper, you deserve a cookie!08:54
t1mpyeah :)08:54
nik90hehe08:54
mihirnik90:  !cookie08:54
nik90t1mp: hey, Good Morning08:54
t1mpnik90: hello08:54
mihirdidn't work for me :( hehehe08:54
t1mpmihir: see what I typed08:54
nik90t1mp: I had a small question08:54
mihir!cookie | nik9008:54
ubot5nik90: Wow! You're such a great helper, you deserve a cookie!08:54
mihirahahah :D got it :P08:55
nik90t1mp: Is it possible to move actions: [], to a separate page so that we can use it multiple pages without duplicating code?08:55
t1mpmihir: now we should stop doing this, nik90 will get fat ;)08:55
nik90t1mp: for instance common actions like "Add new event" will be shown in the year, month, week and day view.08:55
* nik90 feels bloated with too much cookies ::P08:56
mihirt1mp: heheh true :P08:56
t1mpnik90: that should be possible. you can put action ids in the action list08:56
t1mpnik90: it is a very valid use case08:57
mihirahh great, nik90 if that works we can reduce redundant code from all pages.08:57
nik90t1mp: I tried ActionList{ common actions here } and saved it as CommonEventActions.qml. However when I referenced this inside head.actions {}, it said something about not using a list.08:57
t1mpnik90: ActionList is not used any more for page.head.actions08:58
dholbachdavidcalle, just went ahead and filed bug 136945808:58
ubot5bug 1369458 in Ubuntu App Developer site "Go into more detail once the first stable framework is out" [Undecided,New] https://launchpad.net/bugs/136945808:58
davidcalledholbach, :)08:59
nik90t1mp: ah, how do you define a list of Actions{} in a separate page then?08:59
t1mpnik90: it should be an actual list<Action>08:59
dholbachbrb08:59
nik90t1mp: you mean like a ListModel ?08:59
t1mpnik90: I'm wondering if you can add an Item { id: actionPool; Action { id: action1 } } to (for example) your MainView and then in your Page.head.actions have [mainView.actionPool.action1]09:00
t1mpsome aliases would have to be made to make the actions public properties to be accessed from the Page09:00
t1mpnik90: but I haven't tried that yet09:00
nik90t1mp: I think that should work09:01
* nik90 tries09:01
nik90thnx09:01
t1mpnik90: good luck :) and let me know if it works09:01
t1mpnik90: something like that may be useful to add as an example to our documentation09:01
nik90t1mp: true, If something works I will let you know and then propose a branch with documentation aid09:02
mihirnik90: one more question , currently in day view it partially display next and previous day..are we continuing that or we should move to just a single day?09:03
nik90mihir: in Akiva-Thinkpad09:04
Akiva-Thinkpadmmm?09:04
nik90mihir: in Akiva-Thinkpad's branch, he shows 3 days09:04
Akiva-Thinkpadmihir, yah09:04
nik90mihir: not sure yet what we will do yet..ideally it should only show that day's events to differentiate between day view and week view, dont you think?09:04
nik90Akiva-Thinkpad: ^^09:04
mzanettiis there a way to lock an application to landscape mode?09:04
Akiva-Thinkpadnik90, one thing I was thinking of09:05
mihirnik90: in day view i would prefer to remove that header , and show only day09:05
mihirso no partial views09:05
Akiva-Thinkpadnik90, well i am working on it at this very moment09:05
nik90mzanetti: not that I am aware of...a lot of people are looking for it though09:05
Akiva-Thinkpadit will look unified with the week view09:05
mzanettinik90: ack, thanks09:06
Akiva-Thinkpad30 mins, I'll provide another set of screenshots09:06
mzanettinik90: just wanted to confirm that I didn't just miss it before triaging some bugs09:06
nik90mihir: let's wait to see Akiva-Thinkpad 's branch09:06
mihirnik90: okay :)09:06
mihirAkiva-Thinkpad: coffee and code , make great combination :D09:06
Akiva-Thinkpadheh09:06
nik90mzanetti: http://askubuntu.com/questions/516947/how-to-lock-ubuntu-device-orientation-in-portrait-or-landscape-from-user-input09:07
mzanettinik90: thanks09:07
=== davmor2_ is now known as davmor2
mivoligomzanetti: ok, I've pushed the towers and proposed for merge09:24
davidcalledholbach, http://developer.ubuntu.com/apps/sdk/tutorials/click-targets-and-device-kits/09:25
mzanettimivoligo: awesome :)09:25
* mzanetti merges09:25
kalikianamzanetti: you'll have to bribe some unity dev for that ;-)09:26
mivoligomzanetti: towers.json needs an update too09:26
davidcalledpm : http://developer.ubuntu.com/apps/sdk/tutorials/click-targets-and-device-kits/  , http://developer.ubuntu.com/apps/platform/guides/frameworks/09:26
mzanettikalikiana: for what?09:26
kalikianamzanetti: for the orientation09:26
mzanettikalikiana: to add some property to MainView?09:26
dholbachdavidcalle, thanks a bunch - are you going to look into 1357777 as well?09:26
dholbachah, you're assigned already - great :-D09:26
kalikianamzanetti: no. to implement it09:27
davidcalledholbach, ;)09:27
dholbachdavidcalle, 1369173 too09:27
mzanettikalikiana: ah yeah... I think dandrader and greyback are on it already09:27
mzanettikalikiana: still the MainView API needs to be extended a little. I guess that's your call though. At least telling us what you want it to be API wise :)09:27
davidcalledholbach, oh, missed this one, interesting, the new article should fix that.09:28
* mzanetti wonders how the Dronfly hack will break when proper rotation support comes in :D09:28
kalikianamzanetti: there was an api proposal… if I could remember. it had something like [ Portrait ] in an enum with lots of "supported" things09:29
kalikiananeed to find where that one went09:29
mzanettinik90: btw. here's a temporary hack on how to lock to landscape until the platform gains support for it: http://bazaar.launchpad.net/~mzanetti/dronefly/trunk/view/head:/app/dronefly.qml#L5209:30
dpmdavidcalle, nice work. I started the review of the click targets doc and I had a few minor comments, but they can be addressed afterwards09:30
nik90mzanetti: ooh cool thnx09:30
davidcalledpm, sorry about that, did I miss comments in the doc?09:30
dpmdavidcalle, no, you didn't, I didn't manage to finish the review, but I didn't want to block their publication on the site, so all good09:31
davidcalledpm, ok :)09:31
dholbachdavidcalle, thanks a bunch09:32
Akiva-Thinkpadnik90, mihir http://i.imgur.com/LQ0dNe3.jpg09:33
Akiva-Thinkpadhows that?09:33
nik90Akiva-Thinkpad: in the week view, I think "Sun, Mon" etc could use a small top margin..around 0.5-1 gu unit09:34
nik90Akiva-Thinkpad: top and bottom actually with a 0.5 gu margin would be nice..otherwise it feels too restricted imo.09:35
Akiva-Thinkpadso the day name and day number side by side?09:35
Akiva-Thinkpadnik90, oh09:35
nik90Akiva-Thinkpad: no no ... the current layout you did looks good09:35
Akiva-Thinkpadsorry09:35
Akiva-Thinkpadyou want it a bit bigger09:35
Akiva-Thinkpadyah09:35
Akiva-ThinkpadI can do that09:35
nik90Akiva-Thinkpad: just that the top and bottom margin of that could use a little spacig09:35
Akiva-Thinkpadyah09:35
Akiva-Thinkpadsec09:35
nik90yup09:35
nik90Akiva-Thinkpad: year view, agenda view looks good09:36
Akiva-Thinkpadmonth view is... I actually like your month view09:37
Akiva-Thinkpadthe shape takes up quite a bit of space unfortunately.09:37
nik90Akiva-Thinkpad: not sure about the month view..the spacing between the ubuntushape and the sun,mon header doesn't look that grt.. the one we had before in the branch was better I feel09:37
Akiva-Thinkpadyah thats the other thing09:37
nik90Akiva-Thinkpad: yup exactly09:37
Akiva-Thinkpadi'll move it down a bit09:38
nik90Akiva-Thinkpad: Why not just remove the ubuntu shape?09:38
Akiva-Thinkpadnik90, hmmmm,09:38
nik90if you move it down, it won't make it any better though09:38
nik90since there would be that awkward spacing which is unused09:38
Akiva-Thinkpadnik90, yah I agree in removing it09:38
nik90I already did that yesteday..so if you revert the monthview, they should return back to what they were before.09:39
Akiva-Thinkpadis that all you did? I am just wondering if that will be straightforward, in that the year view seems to use the same components09:39
Akiva-Thinkpadnik90, oh have we decided if we are removing the year view?09:39
Akiva-ThinkpadI think you mentioned something about it.09:39
nik90Akiva-Thinkpad: no we are keeping the year view (after looking at the comments in the bug report)09:39
Akiva-Thinkpadokay09:40
nik90Akiva-Thinkpad: However I patched the year view to show the ubuntu shape09:40
mihirAkiva-Thinkpad: also , don't use all CAPS09:40
Akiva-Thinkpadalright, give me another lil bit09:40
Akiva-Thinkpadall caps?!09:40
mihirfor me , again the ornage buttons looks weird09:40
Akiva-Thinkpadoh09:40
nik90Akiva-Thinkpad: also I am with mihir on the day view. The day view is meant to be about that day alone. I feel the implementation of mihir was better09:40
Akiva-Thinkpadthe all day events?09:40
mihirJust be like Sun , Mon Tue , instead of SUN MON TUE09:40
Akiva-Thinkpadoh09:41
Akiva-Thinkpadyah09:41
nik90Akiva-Thinkpad: so the day view shouldn't show events from the other days...09:41
Akiva-Thinkpadsure; I can fix that; don't know why it is like that.09:41
nik90mihir: don't you agree? ^^09:41
nik90Akiva-Thinkpad: I remember seeing the code date.toUpperCase() which does that09:42
Akiva-Thinkpadnik90, yah, so set the model to 1 then? i don't disagree with thad decision09:42
mihirnik90: yup agreed :) lets keep day view simple , lets put all all day events on top followed by time frames09:42
Akiva-Thinkpadokay I'll search the code09:42
Akiva-Thinkpadmihir, agreed09:42
Akiva-Thinkpadokay I'll do that right now09:42
mihirAkiva-Thinkpad: i have removed in that branch , you can remove .toUpperCase , it should resolve your issue :)09:42
mzanettimivoligo: just noticed the shots are missing. but I guess you know already ;)09:43
mihirAkiva-Thinkpad: you propose branch , nik90  or I will merge with the existing branch , as it will require manual merge09:43
Akiva-Thinkpadokay09:44
Akiva-Thinkpadcool beans09:44
nik90Akiva-Thinkpad: make sure when you propose the merge, you target our branch and not trunk :)09:44
nik90I tend to forget that09:44
Akiva-Thinkpadsure thing :P09:44
mihirnik90: i tried to remove the ubuntushap from monthview, did you see that ?09:45
mihirhehe09:45
nik90mihir: yup09:46
nik90mihir: but it also removed the ubuntushape in the year view09:46
nik90mihir: but I managed to fix it09:46
mihirAhh, crap :| i forgot that09:46
nik90mihir: so it shows the ubuntushape only in the year view09:46
mihirohkie..:)09:46
Akiva-Thinkpadnik90, okay that was a question in the back of my mind09:46
Akiva-Thinkpadhow did you do that?09:46
Akiva-Thinkpadwas it just a visible: property?09:47
mihirnow in day view , I guess I need to remove partial views of next/ previous day and then i'll try to move times to the left corner09:47
mihirfun a part, for GOT fan heehe , https://www.youtube.com/watch?v=3vI_7os2V_o09:48
Akiva-Thinkpadmihir, I'm doing that atm09:55
Akiva-Thinkpadunless you really want to do it, i'll leave it to you09:55
nik90Akiva-Thinkpad: in the yearview.qml file, I modified the delegate09:57
nik90Akiva-Thinkpad: this way the ubuntu shape stays within the year view alone09:57
Akiva-Thinkpadnik90, okay thanks09:57
mzanettimivoligo: towers merged and towers.json adjusted10:11
mzanettimivoligo: one warning: the new setupdata.py script will delete the levelpacks directory before starting to extract it again10:12
mzanettimihir: so if you have work done inside the machines-vs-machines repository for testing, be careful with that script10:12
mzanettisorry mihir... I meant mivoligo10:12
=== davidcalle is now known as davidcalle_afk
Akiva-Thinkpadmihir, oh sorry what I said was not quite right10:17
Akiva-ThinkpadIgnore what I said, and carry on with the day view and putting the times on the left10:18
Akiva-ThinkpadI read that completely wrong10:18
* Akiva-Thinkpad is sleepyish10:18
Akiva-Thinkpadsorry :S10:18
nik90t1mp: it doesn't work...I tried http://paste.ubuntu.com/8349526/, however within head.actions{}, I cannot refer to it as actionPool.newEventAction as it complains about missing "{". If I add "{}", then it is invalid.10:28
nik90I tried several other stuff like only including Action{} in a separate file. But even that cannot be used as such10:29
t1mpnik90: how do you do head.actions? It should be head.actions: [ actionPool.newEventAction, actionPool.reloadAction ]10:30
nik90that's what I did10:30
t1mpweird10:31
nik90http://paste.ubuntu.com/8349548/10:32
nik90more specifically10:32
nik90not sure why10:32
t1mpnik90: does it work with only ids in the [], so no Action { } definitions there?10:32
t1mpnik90: I gotta run now, back in 1h10:32
nik90t1mp: ok tty later10:33
t1mpzsombi: ^ maybe you have an idea for the actions? see nik90s pastebins10:33
mihirahhh, what did you say Akiva-Thinkpad  ?10:34
mihirdidn't get yu10:34
Akiva-Thinkpadmihir, oh lol10:34
Akiva-Thinkpadmihir, I said, to ignore what I said10:34
mihirlol okay10:34
Akiva-Thinkpadkeep doing your stuff10:34
nik90t1mp: that worked!10:34
zsombit1mp: nik90: that kind of mix doesn't work :(10:34
nik90t1mp: using only ids in head.actions[]10:34
nik90zsombi: ah so I cannot mix Action{} and id in a head.actions[]..strange but should be documented (which I will do so)10:35
zsombinik90: nope :(10:36
zsombinik90: well, not even Qt documents that, but I realized that this setup doesn't work with ListItemOptions either :/10:36
zsombit1mp: ^10:36
nik90ah :(10:36
zsombinik90: but perhaps it's a bug, I remember we had been thinking using that some time ago, and IINM it was working on teh prototype...10:37
nik90zsombi: app devs can easily workaround this...but they should be shown (documented) this so that they do not ponder why their code doesn't work.10:37
zsombinik90: agreed10:38
nik90mihir: that "game of codes" is hilarious...dave is awesome :P10:41
mihirnik90: yeah it's hilarious :)10:47
nik90mihir: btw found a way to remove the duplicate header actions code :) ... will push when I am done10:49
mihirnik90: that's cool :D10:56
t1mpnik90, zsombi ok good that it is clear now what works :) and it is not a huge workaround to define a ("local") Action just outside of the page.actions10:57
nik90t1mp: yup that's exactly what I did :)10:57
dpmJamesTait, on bug 1343129 and bug 1358794 - what do we need to do to show the translatable strings to translators?10:59
ubot5bug 1343129 in Ubuntu Translations "Department names are not translatable" [High,Triaged] https://launchpad.net/bugs/134312910:59
ubot5bug 1358794 in Click Package Index "Ubuntu Store: "Top apps", "favourite", "app of the week" strings not localized" [Critical,Fix committed] https://launchpad.net/bugs/135879410:59
JamesTaitdpm, they're managed in devportal itself, so I'm not sure.11:00
JamesTaitdpm, I have limited access to that side of the store myself (and would like to keep it that way), but AFAIK the translations are only editable by devportal admins.11:02
JamesTaitdpm, OTOH, if we just want to allow people to see what needs to be translated, we have at least one informal way of making that information available.11:04
JamesTaitdpm, which I'm just sorting out for you now, in case you're waiting. :)11:06
dpmJamesTait, on the phone, IÇll come back to zou in a bit, thanks!11:07
ogra_for anyone concerned (and in case you didnt notice yet) adb in the emulator works again since yesterday (devel-proposed only atm, trying to land this in rtm)11:19
nik90ogra_: was there a new devel-proposed emulator image?11:20
ogra_there is one every day11:20
nik90ogra_: I just tried image 237, and when i do adb shell, it says emulator-5554 offline11:21
nik90ogra_: let me create a new devel-proposed emulator and check again11:21
ogra_you did build with --developer-mode ?11:21
nik90ogra_: why would I? I created this emulator using qtcreator. Shouldn't it do it automatically?11:21
ogra_oh, indeed11:21
nik90ogra_: let me try both ways.11:21
ogra_sudo ubuntu-emulator create --channel=ubuntu-touch/devel-proposed --arch=i386 --use-raw-disk newadb11:22
nik90ogra_: if with qtc it doesnt work, I will know where to report it11:22
ogra_thats what i used to verify it works11:22
nik90ogra_: what's --use-raw-disk newadb for?11:22
ogra_newadb is the name :)11:22
nik90ogra_: I was planning to add --developer-mode11:22
nik90lol11:22
ogra_use-raw disk only means it doesnt create loop images11:22
ogra_so you can mount them directly to check logs and hack on them11:23
mihirt1mp: it would be good to have this info in http://developer.ubuntu.com/api/qml/sdk-14.10/Ubuntu.Components.PageHeadConfiguration/11:23
ogra_shouldnt have any impact11:23
nik90ogra_: ok, but I will try with --developer-mode11:23
ogra_i dont thinnk thats needed11:23
nik90ack11:23
ogra_it should set a default pw of 0000 and enable dev mode automatically11:24
* ogra_ re-tries too ... takes 20min to pull the image here though 11:24
nik90ogra_: it seems latest is image 40011:25
nik90bah 240 I mean11:25
ogra_yeah11:25
ogra_i tested with 23911:25
ogra_which was fine11:25
nik90I tried in qtc with 237, that could have been the issue11:25
nik90will let you know in 10 mins11:25
ogra_(note i only test tzeh emulator, i dont have the sdk installed or in use)11:25
=== MacSlow is now known as MacSlow|lunch
nik90yup that's what I am there to help with :)11:26
ogra_so if it shows fine in adb devices and adb shell lets you in but the SDK still doesnt see it we might need fixes on the SDK side11:26
nik90let me confirm on my end as well and then I will take it up with zbenjamin :)11:27
* zbenjamin looks why he was highlighted11:27
nik90zbenjamin: tl;dr -> ubuntu-sdk might not recognise emulator is running with new developer mode.11:28
nik90zbenjamin: also ubuntu-sdk does not allow running apps in emulator with devel image..11:28
nik90zbenjamin: so essentially app development in emulator is a no-go ... But I am still confirming that...11:29
ogra_i dont see why iit would work on a device with the new dev mode .... but not on the emulator ...11:29
ogra_there is no difference (at least once both come up and are visible for adb)11:30
nik90ogra_: yeah I am puzzled as well11:30
zbenjaminnik90: we were requested to make the devel channel the default ... you should use the devel-proposed though11:30
ogra_well, thats the only channel that has the fixes yet anyway :)11:30
zbenjaminogra_: yep ;)11:31
nik90zbenjamin: but that's the thing...both devel and devel-proposed dont work...give 2 mins to confirm11:31
dpmJamesTait, can we put all the translations in a .pot file and upload that file to https://translations.launchpad.net/click-package-index ? I think that'd be the easiest thing to allow translators to do their work11:36
dholbachbzoltan, Mirv: looks like click-reviewers-tools is a dep of q-p-u now, but ubuntu-emulator is still a recommends (bug 1367246)11:37
ubot5bug 1367246 in qtcreator-plugin-ubuntu "ubuntu-sdk doesn't pull in emulator on 14.04" [Critical,Confirmed] https://launchpad.net/bugs/136724611:37
dholbachshall I reopen the bug?11:37
nik90zbenjamin, ogra_ : false alarm11:37
zbenjaminnik90: don't you scare me like that ;)11:37
nik90zbenjamin, ogra_: image 240 emualtor devel proposed works properly with ubtuntu-sdk11:37
zbenjaminnik90: i think there was a emulator version where adb was not working at all11:38
nik90zbenjamin: image 237 doesn't work...and there was no image during the weekend11:38
nik90zbenjamin: which scared me11:38
nik90zbenjamin: but all is good :D11:38
zbenjaminnik90: did you checkout the sourcecode completion? Should work now11:38
nik90zbenjamin: for devel? or develproposed?11:39
nik90zbenjamin: nvr mind11:39
zbenjaminnik90: thats not related to the image channel :)11:39
nik90misread the question11:39
bzoltandholbach:  the ubuntu-emulator had problems with multiarch  so the qtc ubuntu plugin was failing on autopkgtest whenit was on the dependencies.11:39
nik90I will check now..updating system to ensure I got latest packages11:39
dholbachbzoltan, ah ok - did we file a bug for that issue?11:39
zbenjaminnik90: beware for qml only projects you need to remove the import paths property from the project file to make it work11:40
* ogra_ hugs nik90 11:40
bzoltandholbach:  I talked to cjwatson about it, he  told me that it would not be trivial to fix it.11:40
ogra_thanks for confirming11:40
nik90zbenjamin: is that done in the default templates that it is shipped with?11:40
dholbachI see11:40
zbenjaminnik90: its done inside the plugins11:40
bzoltandholbach:  so the Recommends was the best I could come up with.11:40
dholbachbzoltan, in that case I'll close the bug again11:40
zbenjaminnik90: so the cmake plugin needs to get a update11:41
nik90ogra_: yw :)... sry for raising hell when all I had to do was be patient :P11:41
nik90zbenjamin: cool11:41
dholbachbut maybe there should be a bug report for it11:41
JamesTaitdpm, I'm really not the best person to ask, tbh.  pindonga is your best bet, he's more intimately involved with devportal - I'm more of a consumer of it. ;)11:41
ogra_nik90, well, i was sure i fixed it for me ... but had no external confirmation yet, so that helped a lot :)11:41
JamesTaitdpm, ah, I see he's not on this channel, but he is on #ubuntu-touch. :)11:42
dpmJamesTait, ok, thanks, I'll ask him. What channel do you guys usually hang out at?11:42
dpmok11:42
nik90zbenjamin: do you know which version the autocomplete landed in? I don't seem to have it.11:42
nik90zbenjamin: may be its still in a silo and not yet released?11:43
zbenjaminnik90: trusty or utopic11:43
zbenjaminbzoltan: did you backport the cmake plugin to trusty already ?11:43
nik90zbenjamin: trusty (but with experimental sdk tools ppa)11:43
nik90which gets the latest stuff very quickly11:44
nik90zbenjamin: let me quickly also check in my utopic lxc container11:44
bzoltanzbenjamin:  yes11:44
zbenjaminnik90: try it on the desktop in utopic first, the problem with the chroots is that they might lack some of the description files11:45
nik90ok11:47
nik90zbenjamin: code autocompletion works on utopic lxc container11:56
t1mp13:23:14 < mihir> t1mp: it would be good to have this info in  http://developer.ubuntu.com/api/qml/sdk-14.10/Ubuntu.Components.PageHeadConfiguration/11:56
zbenjaminnik90: good :)11:57
t1mpmihir: if that's about the action [ ids ] then I agree. I think nik90 is going to propose an MR for that11:57
Akiva-Thinkpadmihir, nik90 what is the branch I am proposing a merge to?11:57
nik90zbenjamin: would it be possible to do it in chroots for trusty?11:57
nik90t1mp: yes I will11:57
nik90Akiva-Thinkpad: one sec11:57
nik90Akiva-Thinkpad: https://code.launchpad.net/~ubuntu-calendar-dev/ubuntu-calendar-app/ViewRedisign11:58
zbenjaminnik90:  it should already do that11:58
Akiva-Thinkpadthanks11:58
zbenjaminnik90: can you try that with a scope template?11:58
zbenjaminnik90: so if you press F2 on any C++ symbol it should open the files in the chroot11:59
zbenjaminnik90: any scopes c++ API symbol of course11:59
* nik90 tries11:59
nik90zbenjamin: I got http://paste.ubuntu.com/8349989/12:01
nik90zbenjamin: I created a Unity Scope Project12:01
zbenjaminnik90: yeah the new scope template, needs packages that are not in the chroot if you did not create it right now12:01
nik90zbenjamin: ah so if I update the chroot it should be fixed?12:03
zbenjaminnik90: nope they won't pull them, you need to either completely recreate it or install the missing deps manually12:06
nik90ah ok12:06
Akiva-Thinkpadmihir, nik90 https://code.launchpad.net/~akiva/ubuntu-calendar-app/akivaDayWeekViewMergeMihirNik/+merge/23467612:14
mihirAkiva-Thinkpad: thanks :)12:14
Akiva-Thinkpadenjoy :P its in your hands now12:14
Akiva-Thinkpadalso btw, I reverted to the regular ui12:14
Akiva-Thinkpadno solid greys12:14
nik90Akiva-Thinkpad: thnx12:14
mihirAkiva-Thinkpad: that's what i was supposed to ask..:) so now they have normal colors right?12:15
nik90Akiva-Thinkpad: I noticed there are code conflicts in your MP. Can you fix them pls12:15
Akiva-Thinkpadafaik12:15
Akiva-Thinkpadhummm?12:15
nik90Akiva-Thinkpad: do "bzr merge lp:~ubuntu-calendar-dev/ubuntu-calendar-app/ViewRedisign" and then fix the conflicts.12:15
Akiva-Thinkpadsure12:15
=== davidcalle_afk is now known as davidcalle
=== mpt_ is now known as mpt
vitimitiI wanted to say: that problem I had with the Ubuntu SDK where it didn't highlight the Ubuntu Components because they weren't found? A new update fixed it, so it must have been some issue with the update before it. Thanks to the people that tried to help me anyway12:23
nik90zbenjamin: hey, I removed all kits, devices, emulators etc and started fresh (for the fun of it). And now things work as expected (autocomplete, sdk recognising the emulator) etc etc..12:39
=== MacSlow|lunch is now known as MacSlow
nik90zbenjamin: btw just curious, but how does the app dev know when to update and recreate his kit that he created? I already knew about it by talking to you12:40
zbenjaminnik90: he does not, thats the drawback of chroots. But the 14.10 chroot is unstable , the idea is that once the framework is released there won't be added anymore dependencies to it12:48
nik90zbenjamin: ah ok12:48
mivoligomzanetti: just submitted merge request for the shots13:00
mivoligomzanetti: also slightly changed the shot for tower-1 as you mentioned is to small13:01
mivoligomzanetti: that's me for today, tomorrow I'll try to upload some enemies in svg13:02
om26er_when I run phablet-click-test-setup I get this http://paste.ubuntu.com/8350365/ any help ?13:07
dholbachbzoltan, who's our contact for the go story?13:17
bzoltandholbach: from the point of the SDK it is me and zbenjamin13:17
dholbachbzoltan, and "everything else" in terms of go our story? :)13:18
bzoltandholbach:  but we have problems with the SDK parts since the new adbd policy landed13:18
bzoltandholbach: Gustavo13:18
dholbachthanks bzoltan13:18
dholbachbzoltan, I was just asking so so we can get some replies to https://lists.launchpad.net/ubuntu-phone/msg09588.html13:19
t1mpbzoltan: that answer is ambiguous13:19
=== chihchun is now known as chihchun_afk
bzoltant1mp: how? Are you interested to be contact point for go sdk? :D13:20
t1mpbzoltan: we have several gustavos :)13:20
t1mpyou need Gustavo Niemeyer in this case13:21
bzoltant1mp: when the context is Go and the question is a person .. then there is only one and only one Gustavo :)13:21
bzoltanHe is  Gustavo Niemeyer indeed13:21
t1mpbzoltan: okay :)13:22
=== boiko_ is now known as boiko
dholbachdavidcalle, thanks13:34
davidcalledholbach, np13:35
mzanettimivoligo: cool, thanks. will look at the shots in a bit13:42
=== _salem is now known as salem_
mivoligomzanetti: you said before I should consider how easy is the path in my next levels. (Un)fortunately I have almost finished 5 more levels and I don't want to change them now. :D So I'll do my best next time :)13:49
mzanettimivoligo: no worries14:02
mzanettimivoligo: btw. how many levels did you intend to create?14:02
mzanettiI was planning for 30 now14:02
mzanettimivoligo: I've added a concept.txt file into the level pack directory14:03
mzanettiin there I describe in which level new enemies appear, and in which level I intend to user to unlock which tower etc14:03
mzanettijudging by the amount of towers and enemies we have, I think 30 levels would fit perfectly14:04
davidcalledholbach, dpm: I've just finished the screenshots+text update of http://developer.ubuntu.com/apps/sdk/tutorials/creating-an-sdk-app-project/14:12
dholbachdavidcalle, maybe we could add something like "Once you have the Ubuntu SDK installed and set up (link to article), you can very easily start an app. Read below how to do this in five steps." or something14:15
dholbachwhat do you think?14:15
dpmdavidcalle, ok, cool, thanks14:17
davidcalledholbach, good idea14:18
dholbachcool14:20
dholbachbzoltan, Mirv: about that click backport... 0.4.32.1 would be really nice to have14:54
bzoltandholbach:  I will do that14:59
nik90zsombi: ping15:00
dholbachfantastico15:00
zsombinik90: pong15:00
nik90zsombi: I just looked at https://code.launchpad.net/~bzoltan/ubuntu-ui-toolkit/new_list_item/+merge/234625 and it looks like it has the new list items!15:00
bzoltandholbach:  how can a version number be 4.32.1? I though  you are kidding ... but not :)15:00
nik90zsombi: I thought it will only after RTM?15:00
bzoltannik90:  I should have hided it better :)15:01
zsombinik90: it was always meant to be post RTM :)15:01
zsombinik90: actually it can only come to 1.215:01
nik90bzoltan: lol15:01
=== Randy_O is now known as Randy__O
bzoltannik90:  it is a long landing porcess ... 18 or so MRs, heavy testing15:01
nik90zsombi: but If I bump clock to 1.2, I would get it right?15:01
dholbachbzoltan, a quick-fix release for 0.4.32 :)15:01
=== Randy__O is now known as Randy___O
zsombinik90: if you do that, yes. Asuming 1.2 to you means 15.04 :)15:02
bzoltandholbach:  still ... 4 3 2 115:02
=== Randy___O is now known as Randy_O
dholbachhaha, great15:02
dholbachyou're right15:02
nik90bzoltan: true, but atm we are using a custom implementation. So if the sdk offers a testing version of it, I will grab it :)15:02
nik90zsombi: wait so it is in trunk in 14.10 as far as I can see. But I can't use it until 15.04?15:03
zsombinik90: it's not there yet :)15:03
zsombinik90: and even if it will be, it's an unstable API15:03
nik90zsombi: oh15:03
bzoltannik90:  and it will warm your beer if you use it ...15:03
davmor2zsombi: wait there is a stable api ;)15:03
zsombinik90: so it will most likely land by Oct, but that doesn't nmean we can call it stable15:04
nik90bzoltan, zsombi: Damn I hyped myself too fast ...:P...15:04
nik90bzoltan: and yes you should have hidden it better15:04
zsombinik90: we are in feature freeze, so no more API to 1.115:04
zsombidavmor2: there is a stable and sloooooooooooooooooooow API set, yes15:05
davmor2zsombi: hahaha15:05
zsombinik90: but good you saw it, you can check it right now... but as we speek, I'm changing teh API :D15:05
nik90lol15:05
nik90I read "Brand new ListItem component. (spreadsheet row 53)." in the silo 009 and I rushed to see what it was15:06
nik90that ain't hiding..that's like hey focus a spotlight on me15:06
bzoltannik90: I would say that it is a super long shot to land it soon... not impossible thou. If all the 18 MRs got reviewed by t1mp super quickly and all the APIs will get proper tests from zsombi, and all the 900+ tests pass for both Utopic and RTM images then we actually might consider to land it. But that is an optimistic scenario15:08
nik90wow 900+ tests :)15:08
bzoltannik90: that is how I prevent regressions ... I am running 24/7 all available autopilot tests against the UITK staging15:09
balloonsnik90, happy monday. I approved your manual test mp. I'm curious about how the qml test article is coming15:18
nik90balloons: hi :), a bit slow since I wasn't able to come up with a good example to base it on. So I am going back and forth between the clock app and the ubuntu-sdk-example that is used for explaining AP tests on developer.ubuntu.com15:19
balloonsnik90, ahh.. yes, there are qml tests written in there.. And I found an old MP that never got merged that should be15:20
nik90yup15:20
=== gatox is now known as gatox_lunch
mihirnik90: ping15:51
nik90mihir: yo :)15:51
mihirnik90: ahh, did you merge akiva's MR..15:51
nik90mihir: no I didn't see it has code conflicts with our branch.15:52
nik90mihir: I don't think he merged our branch into his and worked on top of it.15:52
mihirnik90: unfortunately i forgot my external at workplace, in which i have installed ubuntu :(15:52
nik90mihir: oh :(15:53
nik90mihir: I can manually copy parts of his branch into ours?15:53
mihirnik90: apart from that, I am still skeptical regarding  those all events buttons :|15:54
nik90mihir: the color or the whole concept?15:54
nik90mihir: it does some some vertical space though15:54
mihirHmmm , tbh the buttons is killing the design , i am not sure just my thought15:55
nik90Do you have any ideas on how to fix it?15:55
nik90mihir: because the all day event bar is shown in all days when it shouldn't be since users don't have all days events every day15:56
nik90it is waste of precious 4-5 gu units of vertical estate15:56
mihirnik90: i would prefer as we have discussed, like lets have first row to all events, followed by time15:56
mihirif you notice it is same in Google Android calendar15:56
nik90mihir: actually in google android calendar, the all day events bar is shown *only* on those days that have an all day event.15:57
nik90mihir: in day view15:57
nik90mihir: In week view, I agree it is shown on all days.15:58
mihiryup agreed, we can do like that , if there are any lets show it, else  hide it15:58
nik90yeah we can do that15:58
nik90I was hoping you could fix the issue with the day view showing partial events from the other days.15:58
nik90but I guess that will have to wait until tomorrow15:58
mihirnik90: i planned for that...and in my to do list  :( but unfortunate and my bad i forgot :(16:00
nik90alrite no worries16:00
nik90mihir: btw how's https://imgur.com/Ze10MBn16:00
nik90mihir: I replaced textfields with labels and removed uses of the highlighted property which was going to be removed16:00
mihirahhh , amazing16:01
mihirnik90: can't we put Event Details header on top and follow rest of thing ?16:02
mihirthat's good that you removed , date and time string..16:02
nik90Event Details header?16:02
nik90ah16:02
nik90well I did that and I remember you telling me to change to how it looks now :)16:03
nik90during the new-event page redesign16:03
mihirahhh ,16:03
nik90:D16:03
mihirnik90: you done with yearview fix ?16:05
nik90yeah yesterday nite itself16:05
mihirokay great...:) so this and then restructure and we are good to go with both the MR.16:07
mihirnik90: balloons  any idea what is the reason , http://91.189.93.70:8080/job/generic-mediumtests-utopic-python3/lastSuccessfulBuild/artifact/calendar_app.tests.test_monthview.TestMonthView.test_monthview_go_to_today_prev_year.ogv ?16:13
mihiror it is just random fail ?16:13
nik90mihir: which MP?16:14
balloonsmihir, that's what we discussed last week16:14
balloonsremember, increase the timeout? the app isn't loading for some odd reason before the timeout finishes16:15
nik90balloons: he did propose Mp for it at https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/increaseWaitTime/+merge/23459616:16
balloonsnik90, heh.. I wonder if the issues are from that mp:-)16:17
nik90balloons: I think it16:17
nik90is16:17
balloonsyep16:18
balloonsso, that's quite odd.. what else is odd is that it fails on one test only16:18
mihirballoons: i added wait time but it is failing :(16:19
balloonsmihir, right.. that's good though. that means it's something else16:20
balloonsare there some qml errors that might be causing the app troubles?16:20
mihirballoons: it runs fine locally  without any errors.16:21
mihirballoons: i re-approved this , lets see if it passes16:22
nik90balloons: it only fails sometimes, so not specific to the branch16:22
nik90looks like a flaky test16:22
mihirnik90: regarding your MR , shouldn't we put date format to i18.tr ??16:23
mihirendDateInput.text = Qt.formatDateTime(endDate, "dd MMM yyyy"); ?16:24
mihiror this FormatDateTime will take care?16:24
nik90mihir: hmm, let see if I can automate, since translator won't know what code it is for their country.16:24
nik90its not correct to translate it manually16:25
balloonsnik90, the test never runs, and afaict the failure is not just on one test right?16:26
mihirHmmm balloons sometime it happens , sometime it doesnt :|16:28
balloonssorry I was rebooting my pc; it's going nuts today16:28
balloonsmihir, let's see what the next autolanding does.. do we have the old failures as well so we can see if it's just one test?16:30
balloonsI'm really confused to see any failures; I went through the entire testsuite last week and everything was running fine. what happened?16:31
ahayzenfcole90, ping16:33
fcole90hello :)16:34
ahayzenfcole90, how you getting on with https://code.launchpad.net/~fcole90/ubuntu-docviewer-app/ubuntu-docviewer-app/+merge/234438 ?16:35
ahayzenfcole90, i see it is still failing.... i assume due to this part of the log? "could not import package ubuntu_docviewer_app: No module named ubuntu_docviewer_app Loading tests from: Did not find any tests"16:36
nik90mihir: I fixed the locale issue automatically16:36
fcole90ahayzen: yes, I'm still investigating why it doesn't find the tests :/16:37
ahayzenfcole90, i'll compare to what music does and see if i can spot anything16:39
ahayzenfcole90, you can see it installing the files "-- Installing: /tmp/buildd/ubuntu-docviewer-app-0.1.1+autopilot0/debian/tmp/usr/lib/python3/dist-packages/ubuntu_docviewer_app"16:40
ahayzenfcole90, although this path looks strange "-- Installing: /tmp/buildd/ubuntu-docviewer-app-0.1.1+autopilot0/debian/tmp//ubuntu-docviewer-app.qml"16:40
fcole90ahayzen: thank you very much :) ATM I'm doing things similar to calendar app, most of the code is the same but changed to work with the files and folder structure of docviewer app :)16:40
ahayzenfcole90, compared to this from music "-- Installing: /tmp/buildd/music-app-1.2ubuntu2/debian/tmp/usr/share/music-app/music-app.qml"16:41
fcole90ahayzen: looks like I'm writing the path in a wrong way, maybe some variable in cmake :/16:42
ahayzenfcole90, yeah :/16:42
fcole90ahayzen: looking for that now, your comparison helped me a lot, thanks :)16:43
ahayzenfcole90, no problem i'm just scanning over your cmake as well now to see if i can spot anything16:43
fcole90Thanks :)16:43
balloonsahayzen, howdy16:44
ahayzenballoons, o/16:44
fcole90ahayzen: line 34 maybe :/16:46
ahayzenfcole90, hmm but on jenkins it is running in non-click mode right balloons ?16:47
=== gatox_lunch is now known as gatox
ahayzenfcole90, our equivalent is set(EXEC "qmlscene -qt5 ${MAIN_QML} --url=%u -I ./plugins")16:48
ahayzen...not sure we need the ./plugins bit anymore? as that was for grilo IIRC16:48
balloonsahayzen, yes, on jenkins it's installed via deb atm16:49
ahayzenballoons, you any good at cmake ? ;)16:49
ahayzenballoons, we're trying to get this running https://code.launchpad.net/~fcole90/ubuntu-docviewer-app/ubuntu-docviewer-app/+merge/23443816:50
ahayzenballoons, but it seems to fail stating that there are no tests http://91.189.93.70:8080/job/generic-mediumtests-utopic/2010/console16:50
fcole90ahayzen: if I understand cmake I feel that line 34 is executed if click mode is off, because it's just beyond the else. Is it or I am misreading? :)16:53
balloonsfcole90, oO hello!16:54
balloonsyou are working on docviewer, excellent16:54
popeyhi fcole90!16:54
popeywelcome to the clan16:54
ahayzenfcole90, are you referring to line 34 in the diff or the actual file?16:54
fcole90hello guys, nice to be all here there :)16:55
fcole90ahayzen: in the actual file :)16:55
balloonsso, fcole90 ahayzen we think the mp is ready and just want it to run happily in jenkins/16:55
ahayzenah..16:55
balloons?16:55
ahayzeni was looking at line in the diff lol16:55
fcole90ahayzen: line 38 in the diff :)16:56
ahayzenfcole90, we have the same in music "set(DATA_DIR ${CMAKE_INSTALL_DATADIR}/${APP_HARDCODE})"16:56
ahayzenballoons, are you rerunning now?16:57
fcole90ahayzen: ok, seems fine then :)16:58
fcole90one more thing..16:58
ahayzenfcole90, must be something else :)16:58
ahayzenpopey, is Victor correct with point 1) in the comments for https://code.launchpad.net/~andrew-hayzen/music-app/fix-1369050-content-hub-fix-translations/+merge/234572 ?17:00
fcole90under tests/autopilot/ubuntu_docviewer_app I have two folders: files and tests. The files folder doesn't contain any CMakeLists.txt because I didn't know how to handle that :/17:00
ahayzenfcole90, we have the same...17:01
fcole90do you know if its fine just this way or I shoud make cmake aware of that? :)17:01
fcole90ahayzen: good :)17:01
ahayzenfcole90, see the 'content' folder here http://bazaar.launchpad.net/~music-app-dev/music-app/trunk/files/head:/tests/autopilot/music_app/17:01
balloonsahayzen, fcole90 I see at least one potential issue in the tests17:03
balloonsthese haven't been touched in awhile :-)17:03
fcole90balloons: where? :)17:03
ahayzenballoons, we know that ;)17:03
balloonsfcole90, __init__.py is quite outdated; but in addition, it hardcodes python2: sample_dir = "/usr/lib/python2.7/dist-packages/"17:04
ahayzenballoons, ah yes good point17:05
balloonsfcole90, might be wise to clean that whole file up17:06
balloonsLet me try and propose something quickly17:06
ahayzenballoons, thanks17:06
fcole90balloons: the strange thing is that until the cmake change it was working correctly :/17:07
fcole90balloons: yes, good idea :)17:08
balloonsfcole90, you switched to python3 in the cmake change17:08
balloonsfcole90, you could make it work right now I think but changing the autopilot cmake file to python from python317:08
balloonsand actually you probably should until jenkins is configured to run as py317:08
fcole90balloons: mmh, my fault then ;D17:08
balloonsfcole90, mm yep, it's definitely running as py217:09
balloonssee in the log; + timeout --kill-after=30 --signal=2 1200 python2 -m autopilot.run run -v -o /tmp/test_ubuntu_docviewer_app.xml -f xml -r -rd /tmp/ ubuntu_docviewer_app17:10
fcole90balloons: yes, I saw that but didn't recognize the problem as such because I was focusing on understanding what test_ubuntu_docviewer_app.xml was XD17:11
balloonsfcole90, :-) no worries, we should switch to python3.. just a later merge ;-)17:12
balloonsand the test setup still needs those changes17:12
fcole90Ok, then I should change the debian/install and update the init file to use python3? :)17:12
balloonsfcole90, we can add docviewer to the metabug: https://bugs.launchpad.net/ubuntu-weather-app/+bug/136600717:13
ubot5Launchpad bug 1366007 in Ubuntu Weather App "Debian and cmake packaging should use python3" [Undecided,In progress]17:13
balloonsfcole90, you can see the tweaks needed.. ahayzen did most of the others. you change debian packaging and cmake yes17:14
ahayzenballoons, hehe i sense more helper and py3 conversions looming :)17:14
* balloons files bugs for changes for docviewer17:15
ahayzenthanks balloons :)17:15
fcole90balloons, ahayzen: ok then I will take inspiration from music app :)17:15
mihirnik90: balloons  again it failed :(17:19
mihirballoons: it crashed before adding guest17:19
fcole90balloons, ahayzen: thank you very much :)17:20
mihirnik90: can you try to merge from trunk , that wait MR has been merged17:20
ahayzenfcole90, no problem ping us if you need any more help :)17:21
fcole90ahayzen: I will, thanks :)17:21
nik90mihir: your branch passed17:22
mihirnik90: yoooo17:22
nik90mihir: my branch is failing due to a genuine failure which I fixed and pushed17:22
* mihir got frustrated with Jenkins rando failure 17:22
mihirnik90: got it , you changed the contactpopover object name17:23
nik90well more like replaced NewEntryField with Textfield since the guest features requires it17:24
balloonsmihir, sorry I will be investigating the calendar app failures more in a sec.. the re-runs should help diagnose if it's a specific test or something random17:28
* balloons looks at re-run17:28
mihirballoons: sure, thanks :)17:29
balloonsmihir, that said, we don't need to merge that.. ohh, lol17:29
balloonsdoesn't hurt anything persay :-) but clearly that wasn't the problem as it still failed17:29
balloonsmihir, so where is the failure?17:30
mihirballoons: hahah , weird :|17:30
mihirballoons: on which MR you re-run the Jenkins ?17:31
balloonsmihir, I didn't; I thought you did.. you said it failed again17:31
balloonsor if I did, I don't remember :-)17:31
mihirballoons: sorry, got confused, it must be re-runing now , there was some AP change so lets wait for now17:32
balloonsmihir, which mp?17:32
mihirrunninng*17:32
mihirballoons: https://code.launchpad.net/~ubuntu-calendar-dev/ubuntu-calendar-app/show-dayname/+merge/23471117:32
mihirbut nik90 did you merge with the trunk , may be it might fail , as it has merged my wait branch17:33
mihirnik90: ahhh ,it apporves finally :D17:33
nik90:)17:33
mihirballoons: it approved this time :D17:33
nik90mihir: can it be top-approved?17:34
* mihir top-approves nik90 MR17:34
mihirnik90: done :)17:34
nik90thnx17:34
balloonsnik90,  on https://code.launchpad.net/~ubuntu-calendar-dev/ubuntu-calendar-app/show-dayname/+merge/234711, why the change to ubuntuuitoolkit.TextField?17:34
balloonsNewEventEntryField should inherit from it yes?17:35
nik90balloons: because I changed the component from the custom component to a regular Textfield.17:35
balloonsnik90, ohh lovely!17:35
mihirballoons: nik90  changed it17:35
nik90balloons: no NewEventEntryField now derives a Label with custom colors17:35
balloonsawesome17:35
balloonscarry on :-)17:35
mihir:D17:36
=== chihchun_afk is now known as chihchun
Letozaf_balloons, hi19:15
balloonsLetozaf_, hello19:16
Letozaf_balloons, howzit ?19:16
balloonsLetozaf_, not bad, yourself?19:17
Letozaf_balloons, I'm fine thnaks :)19:17
Letozaf_balloons, I wanted to ask you about that StyledItem I wrote you about19:17
Letozaf_balloons, is there something in ubuntuuitoolkit or do I have to get it using objectNames ?19:18
balloonsLetozaf_, right.. can you share the branch again19:23
Letozaf_balloons, I had created an emulators.py file in my branch and this is wrong :( so probably what you will get is something not useful19:24
Letozaf_balloons, I pulled the "original" one again19:25
Letozaf_balloons, and I am looking at that one19:25
Letozaf_balloons, but, I do not find this SytledItem in the qml file but just the Actions of the StyledItem19:26
Letozaf_balloons, let me share the screenshot19:26
Letozaf_balloons,      https://dl.dropboxusercontent.com/u/10083981/Screenshot%20from%202014-09-14%2021%3A43%3A34.png19:27
Letozaf_balloons, and also the head button: https://dl.dropboxusercontent.com/u/10083981/Screenshot%20from%202014-09-15%2021%3A33%3A08.png19:35
Letozaf_balloons, is there something in the ubuntuitoolkit or shall I get it with objectNames19:36
balloonssorry, I'm branching now Letozaf_ just a moment while I have a look19:37
Letozaf_balloons, hope you are not getting my branch :P19:37
balloonsLetozaf_, lol, why not?19:38
Letozaf_balloons, I had created a emulators.py file, that's not ok19:38
Letozaf_balloons, so it's wrong19:38
balloonsLetozaf_, it's no worries. I meant we switched to just putting it in __init__.py but it matters not19:39
Letozaf_balloons, I am writing the tests all over again19:39
balloonsand switching is as easy as copying it19:39
balloonsyour branch looks fine19:40
balloonsso the header is indeed a helper19:40
balloonsyou can use myheader = self.main_view.get_header() then myheader.click_action_button()19:41
Letozaf_balloons, it's missing the emulators.py file, I forgot to add it19:41
balloonsLetozaf_, ahh.. I see the the overflow button stuff too is handled by the helper19:42
balloonsI didn't understand your question until now19:42
balloonsso if you want to click any button in the header, do it like I showed above19:42
Letozaf_balloons, I used the myheader = self.main_view.get_header() then myheader.click_action_button() but it did not work,but let me try again19:42
Letozaf_balloons, since I had created the emulators.py... I will get back to you when I have re-written the tests without the emulators.py19:43
Letozaf_Letozaf_, just wanted to be sure this is the right way19:44
balloonsLetozaf_, well you should extrapolate on what I wrote :-)19:45
Letozaf_balloons, yes19:46
balloonsLetozaf_, for example, here's something from calendar.. a helper method to click the new event button19:46
balloonshttp://paste.ubuntu.com/8352753/19:47
Letozaf_balloons, thanks19:47
Letozaf_balloons, this is the error I used to get and still get: http://paste.ubuntu.com/8352769/19:52
balloonsLetozaf_, you are obviously running an up to date version of utopic right?19:54
Letozaf_balloons, yes apart today's updates...19:55
Letozaf_balloons, let me update, never know19:55
balloonsLetozaf_, lol, no that's fine19:55
balloonsLetozaf_, it would be helpful for you to have a full checkin of something19:55
balloonsand then we can work on this specific test together19:56
balloonsLetozaf_, I do remember rss reader seemed really funny, and didn't want to "just work" like it should19:56
balloonsso I suspect there might need to be some qml changes19:56
balloonsbut I'm not sure19:56
=== chihchun is now known as chihchun_afk
Letozaf_balloons, the fact is that the first test I started to work on gave me  this problem so I am stuck at the beginning of  shorts_app.tests.test_rssreader.TestMainWindow.test_add_feed_to_new_topic test19:58
Letozaf_balloons, the first action of the test is to click on the header button19:58
Letozaf_balloons, as I understand header = self.get_header() gives me a 'list' object19:59
balloonsyea, that doesn't seem quite right does it?20:01
Letozaf_balloons, yep but then it does not find the action button, let me check better20:02
Letozaf_balloons, that's weired... everything looks ok, the code and what I find using autopilot vis but it fails, I do not understand why20:18
Letozaf_balloons, everything looks in place20:19
balloonsLetozaf_, ;-(20:19
balloonssorry it's causing grief20:20
Letozaf_balloons, no problem, I just wish I could find out why :P20:20
balloonsdo you want to continue with the other changes while I investigate?20:20
Letozaf_balloons, yes as I cannot find out why on my own :)20:20
Letozaf_balloons, I know it's not the right way, but as I needed to click on that header button and select add feeds to continue with the tests (you need to add a feed to carry out a test)  I used this: http://paste.ubuntu.com/8353070/20:49
Letozaf_balloons, when the header issue will be fixed I will cancel this20:49
balloonsLetozaf_, that's totally fine20:50
Letozaf_balloons, ok thanks :-)20:51
Letozaf_balloons, buona notte :)  ci vediamo domani20:51
=== salem_ is now known as _salem
fcole90balloons: Can I ask some help? :) I have done some modifications but jenkins still calls python2 :/21:44
balloonsfcole90, shoot21:47
fcole90balloons: I have changed the debian files according to the modifications of the music app, but I'm wondering if there is any other call to python2 that I'm missing :)21:49
fcole90balloons: http://bazaar.launchpad.net/~fcole90/ubuntu-docviewer-app/ubuntu-docviewer-app/revision/4121:50
balloonsfcole90, I'm not sure I understand those chanes21:51
balloonswhy did you remove all that stuff?21:52
fcole90balloons: also, I have searched but cannot understand the meaning of "usr/lib/*/dist-packages/ubuntu_docviewer_app/*", doesn't it need a space after '*'? :)21:52
balloonsboth those files look good21:52
balloons?21:52
balloonsfcole90, no the /usr/lib/*/ is fine21:52
fcole90balloons: Both music app and calendar app changed that, so I was thinking that it was better to use the new path :)21:53
balloonsfcole90, it's going to be specific to the app I think, but I was saying for the moment it's not the reason tests don't work21:55
balloonstests/autopilot/CMakeLists.txt needs to be python221:56
balloonserr well.. just python is fine21:56
balloonsfcole90, then also set the build-depends in debian control to just 'python'21:56
balloonsthat should do it21:56
fcole90balloons: I was understanding the opposite, that we where going to switch to python3 :)21:58
balloonsfcole90, we can't without changing jenkins21:58
balloonsfor the moment, I assumed you wanted to just get things working21:58
balloonsfirst things first21:58
fcole90balloons: ah ok then, I will fix this before ;)21:59
fcole90balloons: so bug https://bugs.launchpad.net/ubuntu-docviewer-app/+bug/1366007 is still open, the other apps just made a workaround? :)21:59
ubot5Launchpad bug 1366007 in Ubuntu Document Viewer App "Debian and cmake packaging should use python3" [Low,In progress]21:59
balloonsfcole90, we can do the conversion, but it's fine to run as python2 for now22:07
balloonslots of changes to the tests will need to happn22:07
fcole90balloons: Ok, then, thank you very much for all your help :)22:08
dakerpopey: ping22:16
popeydaker: hey!22:26
dakerpopey: PM for a few mins ?22:27
popeysure22:27
=== aaron1 is now known as ahoneybun
=== kalikiana_ is now known as kalikiana

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!