/srv/irclogs.ubuntu.com/2014/02/18/#ubuntu-app-devel.txt

=== chriadam|away is now known as chriadam
dholbachgood morning08:00
=== chriadam is now known as chriadam|away
JamesTaitGood morning all; happy Single Tasking Day! :-D09:19
=== teknico_ is now known as teknico
mzanettidpm: o/09:37
dpmhey mzanetti, morning!09:37
mzanettidpm: can you give this a shot when you have some time? lp:~mzanetti/reminders-app/tablet09:38
mzanettidpm: It's not complete, but I'd like a quick discussion about it09:38
mzanettionce you've seen it09:38
dpmmzanetti, sure, I don't have a tablet, so I'll be testing it on the desktop. popey has borrowed a N10 and can probably test it on a real device09:38
mzanettidesktop is fine09:38
mzanettiI didn't test it on a real tablet yet either09:39
dpmok, give me 5 mins and I'll give it a go09:39
* dpm is looking forward to seeing tablet mode09:39
mzanettidon't get too excited yet09:40
nik90zsombi: hey good morning09:41
nik90zsombi: I got a small question about Ubuntu Layouts09:41
zsombinik90: good afternoon :)09:41
zsombinik90: gpo ahead09:41
nik90zsombi:timezones yes09:41
zsombi;)09:41
nik90zsombi: can I define a Flickable inside a conditional layout instead of a row or a column?09:42
zsombinik90: everything should be possible09:42
zsombinik90: we don't have samples for that, but Fliockable is yet another Item, right?09:42
zsombiFlickable I mean09:42
nik90zsombi: I am getting an issue where the page header doesn't detect that the convergent layout is using the flickable09:42
nik90zsombi: as a result the page header doesn't hide when scrolling up09:42
nik90zsombi: that's why I thought perhaps I was doing something wrong09:43
zsombinik90: yep, the page header is an issue... it does not detect runtime changes09:43
mzanettisame issue here :)09:43
nik90zsombi: but yes Flickable is an item09:43
nik90zsombi: so If I set the flickable property of the page manually, then hopefully that should fix the issue?09:43
zsombinik90: it's a problem of page header unfortunately09:43
zsombinik90: yep, that shoudl do the job09:44
nik90mzanetti: nice to know :) .. if one of us has a solution to it, then let the other person know.09:44
nik90zsombi: thnx..09:44
mzanettiI just started yesterday evening with the tablet mode for reminders09:44
mzanettithe content itself works quite well09:44
mzanettibut header and toolbar get messed up badly when unsing 2 pagestacks next to each other09:44
zsombimzanetti: nik90: you guys should have started the whole app with that in mind ;) :P09:45
nik90mzanetti: I started in the weekend but had to stop because of a critical crash when switching layouts..09:45
dpmmzanetti, ack09:45
zsombimik90: I'm investigating the crash there09:45
nik90mzanetti: Florian figured out that using a Dialer widget caused it09:45
mzanettizsombi: well... its just about not pushing everything to the same pagstack09:45
zsombimzanetti: ;)09:45
nik90zsombi: thnx09:45
mzanettibut yeah... need to figure how to deal with panel and header09:46
zsombinik90: it's a setup, where the item laid out contains a loader. Dialer is one of those, but Picker is such as well...09:46
mzanettiseems the last push()/pop() to any pagestack updates them09:46
mzanettidpm: so yeah... that's what I wanted to discuss. How do other apps deal with this?09:47
zsombimzanetti: yes, because push/pop updates the flickable property so header can adjust its logic accordingly09:47
mzanettizsombi: yeah... makes sense if you have only one pagestack. makes me wonder if I'm actually supposed/allowed to have 2 of them next to each other09:48
zsombimzanetti: not really09:48
zsombimzanetty that messes up the header for good09:48
zsombimzanetti: ^09:48
mzanettiyeah... so what do you suggest?09:48
zsombimzanetti: you'd need a two'column pagestack kinda setup?09:49
dpmmzanetti, so far we've only got weather and clock that we are also looking to get tablet modes for MWC for, so this is also relatively new to everyone, but as zsombi is here as the master of layouts, he can give us some advice :)09:49
mzanettizsombi: yeah09:49
mzanettizsombi: let me show you a few screenies.... gimme a minute09:49
zsombimzanetti: hehe :) I have a branch which turns PageStack into a two-column layout automatically using Layouts, but as ckpringle went away, that stuff got forgotten onto /dev/null09:50
mzanettistandard view phone: http://i.imgur.com/thyV6T4.png09:50
mzanettiafter clicking on a note on the phone: http://i.imgur.com/1FTesPj.png09:50
mzanettistandard view tablet: http://i.imgur.com/TsHrtMF.png09:51
mzanetticlicking on a note on the tablet: http://i.imgur.com/nUlHE5j.png09:51
mzanettinow... as you can see there's an edit button for the note, which would push the EditPage on top of the note view page09:51
zsombimzanetti: yes, this is what I had for my PageStack09:51
mzanettibut at the same time, the tabs in the header allow you to switch to notebooks09:52
mzanettiwhich in turn would push pages to the left hand side09:52
mzanettiso yeah. not entirely sure how to deal with it right now09:52
mzanettizsombi: are there any chances your 2 column pagestack will make it to trunk at some point?09:52
zsombimzanetti: it depends on how UX will pick it up... they're pretty busy with the whole UI reorg atm09:53
mzanettiheh... true. why am I doing this before the UI reorg actually?09:53
zsombimzanetti: but hey, the standard view phone is a Tabs, right?09:53
mzanettizsombi: yeah, it is09:53
zsombimzanetti: same as the standard tablet, right?09:54
zsombimzanetti: and Notes is a PageStack09:54
mzanettizsombi: yeah... I just make the pagestack non-anchors-fill just like the uitk gallery09:54
zsombiNotes tab I mean09:54
mzanettiand put a second pagestack next to the other09:54
mzanettiand do some conditional pushes. the rest is identical between phone and tablet09:55
zsombimzanetti: ok, is the PageStack depth 2?09:55
mzanettizsombi: if you go to Notebooks, you'll get a list of notebooks, you enter one and it pushes a page with the notes in there.09:55
mzanettiyeah, that's when the depth == 209:55
zsombimzaneti: brilliant!09:56
mzanettior also, if you press the search button in the toolbar it pushes a searchPage to the left pagestack09:56
zsombimzanetti: then, in Phone layout you use PageStack, on tablet layout you use two columns, no need for pagestack09:56
mzanettiits not /that/ easy I'm afraid09:57
zsombimzanetti: the search will end up in the header, so forget about pushing a page09:57
zsombimzanetti: why?09:57
mzanetticause both stacks need panels for example09:57
zsombimzanetti: what panels?09:57
mzanettior well, toolactions, wherever they might be09:57
mzanettileft one needs: Add note for example09:57
zsombimzanetti: the tools will land on the header09:57
mzanettiright one needs: edit note, delete note, etc09:58
zsombimzanetti: you can update those tools based on what you have active09:58
zsombimzanetti: and yes, the problem in here is that the whole header-toolbar-search is changing, and everything will be upside down...09:59
mzanettizsombi: ah, so I'd need to drop the toolbars from all the pages and only have one toolbar in the MainView, updating contents as I need it?09:59
zsombimzanetti: ..... damn it... it would be really good to have that two-column PageStack mode solved by now...10:01
mzanettioh yeah, it would :D10:01
mzanettiwhy are you chatting around here? get to work :P10:01
zsombimzanetti: :P I'm trying to help you out...10:01
mzanettiI know. just kidding10:02
zsombimzanetti: here's the MR: https://code.launchpad.net/~zsombi/ubuntu-ui-toolkit/two-pane-pagestack/+merge/17228810:02
mzanettizsombi: oh man... this is *exactly* what I'm doing... if its on the left side, pushing, if its on the right side replacing10:03
zsombimzanetti: so this setup was supporting depth >2 too10:03
zsombimzanetti: ;)10:03
zsombimzanetti: this was doing the way that if push() was used on the right pane, the right pane was moved to the left pane, and the right one was updated with the new pushed page10:04
zsombimzanetti: so it all depended whether the pushOrReplace() or push() was used10:05
mzanettiah... ok... then its a bit different... more like a column-view in a file manager10:05
zsombimzanetti: it all depends how you use it10:05
zsombimzanetti: "When pages are pushed from the current page (right pane) the current page will be moved into the left pane and the  new one will be placed into the right one. This results in a push operation. When a page is pushed from a left pane page, the current page (from the right pane) will be replaced with the  page that is just pushed."10:06
zsombimzanettI: even I don't remember exactly the functionality :D10:06
zsombimzanetti: so PushOrReplace() did all 4 U10:07
nik90zsombi: I have a small issue with labels not being positioned correctly despite using anchors.horizontalcenter to place them properly. As result I get, phone->http://imgur.com/03FmgaI , tablet->http://imgur.com/diTyYFa10:07
mzanettiright... although not exactly how I'd need it according to this description10:07
mzanettiI also have the use case that I have depth 1 on the left side and depth 2 on the right side. and then need to push to the left side without changing the right one10:08
nik90zsombi: Although If I replace those label ItemLayouts with Rectangles, I see that they are placed correctly10:08
nik90zsombi: https://code.launchpad.net/~nik90/ubuntu-clock-app/converge-clock-tab10:08
zsombimzanetti: if the active page was on the left side, the pushOrReplace() did show up the page on the right pane10:08
zsombimzanetti: then every consecutive pushOrReplace on the left page was doing a replace :)10:09
zsombimzanetti: the right-to-left (and vice versa) comes into force when depth > 210:09
zsombinik90: the anchors.verticalCenter will anchor the Label, not the text within10:09
mzanettiok. would need to try it... not sure if its really  exactly what I'd need...10:10
mzanettibut anyways, still leaves the issue with the header and the toolbar I guess10:10
zsombinik90: you should use Label.verticalAlignment10:10
zsombimzanetti: this stuff was working with those pretty well10:11
nik90zsombi: will give that a shot nnow10:11
zsombimzanetti: and I think we should get this branch up alive again10:11
mzanettiideally I'd have 2 headers10:11
zsombimzanetti: why would you?10:11
mzanettione for the tabs, and one for the content title for the right side10:11
zsombimzanetti: ah, yeah, that was also discussed, but that is pretty ugly... and misleading10:12
nik90zsombi: that did the trick...I didn't have to do this before10:12
zsombinik90: beware of this with Labets ;)10:12
zsombiLabels10:12
nik90zsombi: :)10:14
nik90dpm: what do you think is better http://imgur.com/AYpB1Ir or http://imgur.com/diTyYFa10:20
=== zsombi1 is now known as zsombi
dpmnik90, both look good, but I personally find http://imgur.com/diTyYFa better. I find the shadow on the other one a bit too deep10:32
dpmpopey, can you have a go at lp:~mzanetti/reminders-app/tablet on the N10?10:32
nik90dpm: yeah I reverted it to the one that you liked before you said it...It needs more work to make it look better..10:33
dpmnik90, cool. Did you work around that crash? I've not been following the conversation...10:38
nik90dpm: no the crash is still there, however I believe Kaleo and zsombi are on it.10:40
nik90dpm: I am meanwhile trying to implement the phone and tablet views for all tabs.10:40
dpmawesome10:40
nik90dpm: when the crash is resolved and pushed to phone, then I will propose a MR10:40
nik90kalikiana: I saw a boat load of fixes land in the u1db trunk...nice that you got someone to review the MP finally..10:42
nik90kalikiana: is there a planned landing slot for them?10:42
kalikiananik90: no idea yet, still waiting to find out how the fancy new process works, and then to see how to apply it to u1db-qt10:43
kalikianaI'll know more later today10:43
nik90kalikiana: nice..I already got most of the fixes from the PPA on my 13.10 :)10:44
popeydpm: ok10:46
zsombi(12.40.04) nik90: dpm: no the crash is still there, however I believe Kaleo and zsombi are on it.10:53
zsombinik90: dpm: yes, we are on it10:54
nerochiarooSoMoN: do you have a minute to talk about gallery-app and thumbnails ? I'm trying to figure out the best way to approach the problem I'm having and I can use some advice10:54
nerochiarooSoMoN: essentially when you edit the picture the gallery doesn't save the changes on the image on disk, but keeps them in an internal database and applied them to the image on the fly. So if I use the SDK thumbnailer, it will use the images on disk which are different from what the gallery is actually displaying10:56
nerochiarooSoMoN: and the thumbnails won't match, which is not a huge deal when browsing the pictures, but it's a problem when scrolling through them in the photo viewer10:57
oSoMoNnerochiaro, and the SDK thumbnailer can’t be fed with live data, it understands only files on disk?10:57
nerochiarooSoMoN: yes, you pass it a path and it generates the thumbnail for it (or returns it if it's already generated and fresh)10:58
nerochiarooSoMoN: it saves them on disk based on the md5 of the original image path10:58
popeydpm: rotation is broken on nexus 10.11:01
nerochiarooSoMoN: i wasn't expecting it to do something like this, i thought it would save changes to the image immediately (losslessly when possible). i think this whole undo/redo stack is problematic as it is implemented now also for integrating the gallery with content hub, as it will have to "flatten" the edit stack to disk when exporting images via the hub anyway11:02
oSoMoNnerochiaro, how about saving modifications to the original images to a separate file on disk, and use that one to generate the thumbnail11:02
oSoMoN?11:02
nerochiarooSoMoN: that might work, but it's one extra disk write of the full image that we're doing just to generate a thumbnail11:03
oSoMoNnerochiaro, well, not just to generate the thumbnail, the image can then be used to be displayed in the app, instead of having to apply all the modifications on the fly11:04
popeydpm: mzanetti http://imgur.com/t5xxEpB,528Nscu,2GjsSPb,fseUpjH11:06
mzanettipopey: oh11:06
* popey updates tablet11:08
mzanettipopey: hmm... works here :/11:14
popeyfine here now I updated11:14
nerochiarooSoMoN: so basically you think that the undo/redo stack should work by keeping around: (1) a copy of the original image (2) the edited version of the file (3) the stack of operations that were applied on it. Then at each operation added to the stack, the file is modified. And when undoing from the stack if the last operation can be rolled back losslessly over the edited image (e.g. rotation for jpegs) then it's don11:14
nerochiaroe that way, otherwise the original is taken, and all remaining operations are applied to it again in order minus the one being undone11:14
nerochiarooSoMoN: does it make sense ?11:15
oSoMoNnerochiaro, doesn’t sound like a good idea, worded like this :)11:16
nerochiarooSoMoN: well, it's the best that can be done if there are operations which are lossy, like crop or enhance11:17
oSoMoNnerochiaro, I guess that really depends on what operations we want to optimize11:17
oSoMoNor rather, what use-cases we want to optimize11:17
nerochiarooSoMoN: well at the moment we only have 3: rotation, crop and enhance11:17
nerochiarooSoMoN: but one would expect to be a lot more11:18
oSoMoNnerochiaro, what I meant is: I think it makes sense to ensure that viewing images in their current state is the fastest operation11:19
oSoMoNargh11:19
oSoMoNscratch "operation"11:19
oSoMoNfastest use case11:19
oSoMoNand it’s probably acceptable to have some overhead when applying new operations, or undoing them11:19
oSoMoNnerochiaro, what are the other options we have, re- generating thumbnails?11:20
nerochiarooSoMoN: i was specifically asked to use the SDK thumbnailer for gallery, that's why I ended up in this situation. before we were using an internal thumbnailer for gallery11:20
nerochiarooSoMoN: i think what we can do if we want the easy way out is to use the SDK thumbnailer for browsing and keep the internal for the photo viewer11:21
nerochiarooSoMoN: the thumbs for browsing will still not be 100% accurate though11:21
nerochiarooSoMoN: so if that's important too we can't really switch11:21
oSoMoNnerochiaro, and is modifying the SDK thumbnailer to accept live data an option?11:21
nerochiarooSoMoN: not that I know of11:22
nerochiarooSoMoN: we can add that ability I guess, but it will be completely custom for us, not part of the freedesktop standard in any way11:22
oSoMoNdoesn’t sound like the best idea11:24
popeydpm: mzanetti http://imgur.com/tpUFHdc,w3KLulY,ATWS6NG,VZstxmj,GH7B1cM,1TYThPp,UQfIhG6,nrlcgJ1,1yJ2BGx  better11:24
mzanettinice11:24
nerochiarooSoMoN: other editors seem to keep undo/redo history in memory only while you're editing a picture and they ask you to save when you browse away from it11:25
mzanettipopey: so yeah. the question what to do with the toolbar still persists11:25
nerochiarooSoMoN: if we were doing that too, it would solve the issue I think11:26
nerochiarooSoMoN: as we will regen the thumbs only when we finish with editing11:26
nerochiarooSoMoN: but it requires (a) a decision to actually do it and (b) design input on how to ask for save or discard11:27
oSoMoNnerochiaro, I’d tend to think that if we’re not asking the user to save/discard, it’s because it was designed this way11:28
zsombinik90: ok, I think I found the problem causing Layouts crash11:28
nerochiarooSoMoN: i wouldn't be surprised if a persistent undo/redo stack is something we didn't ask for and yorba decided to put n11:28
nerochiaroin11:28
oSoMoNnerochiaro, I don’t know, but in any case you should check with design first11:29
nerochiarooSoMoN: right. so while I do that and put this work on hold, is there any other priority that i should take care of, or should i go back and hack on gallery and camera bugs from the high priority list ?11:30
ybonnik90: ping11:49
rpadovanimzanetti, https://code.launchpad.net/~mzanetti/reminders-app/fix-attaching-images/+merge/206816/comments/48520711:54
mzanettirpadovani: nice, thanks11:55
nik90ybon: pong12:03
nik90zsombi: oh nice :)12:03
nik90zsombi: is it a simple fix?12:03
zsombinik90: 3 lines of code :)12:03
nik90zsombi: lol12:04
ybonnik90: as autopilot tests are working for you, can you just give me the version of the libs you have: http://paste.ubuntu.com/6954036/12:04
ybonjust in case I've a bad one12:04
nik90zsombi: in the SDK or the clock app?12:04
ybonI'm still unable to run them, and that's annoying :(12:04
zsombinik90: in Layouts12:04
zsombinik90: SDK12:04
nik90omg, my irc client didnt send my last few messages..lets try agian12:08
nik90zsombi: ok...convergence is something I need for the MWC.12:08
nik90zsombi: can we expect a fix to land on the phone (not just the ppa) by friday?12:08
nik90ybon: http://paste.ubuntu.com/6954086/12:08
zsombinik90: I got that, I'm already preparing the MR12:08
zsombinik90: more than sure12:09
nik90zsombi: thnx! you rock!12:09
ybonnik90: thanks :)12:09
ybonso my autopilot-desktop seems out of date12:09
ybonI've two versions in my cache, let's take the good one :)12:10
nik90ybon: :)12:10
ybonah:12:11
ybonThe following packages have unmet dependencies:12:11
ybon autopilot-desktop : Depends: gir1.2-glib-2.0 (>= 1.39.0-0ubuntu1) but 1.38.0-0ubuntu1 is to be installed12:11
nik90ybon: which version of ubuntu are you running?12:11
nik9013.10?12:11
nik90sorry I may have asked this before12:11
ybonyep12:11
ybonnik90: do you where you get the gir1 lib from?12:13
nik90ybon: btw, the entire QA is on a sprint...they are generally online around 19:00 UTC12:13
nik90ybon: let me check12:13
nik90ybon: I have 1.38.0-0ubuntu1 installed12:14
ybonoh, like mine so12:14
ybonhumm12:14
nik90ybon: what are the autopilot ppa you have?12:14
nik90do you have more than one?12:14
ybonhumm12:15
ybondoes the 0 sized ones can be of any conflict here: http://paste.ubuntu.com/6954120/ ?12:16
nik90no idea12:17
nik90when you removed the other 2 ppas, do you purge them?12:17
ybonhumm, I've just --remove I think12:18
ybonlet me remove those12:18
nik90I think you should do a ppa-purge on the ppa to remove all other dependencies it upgraded completely12:18
ybonright12:18
yboncan reach to get ppa-purge to get anything (could not find package...), not sure they are really out or I'm missing the syntax for the name12:25
nik90did you do sudo ppa-purge ppa:name-of-ppa ?12:25
ybonyup12:26
ybonand without ppa: and with -p option also12:26
ybonI think they are purged, at least they do not appears in y-ppa-manager12:33
imapadosHas someone experience about having an own update mechanism and leftover files?12:44
imapadosI'll tried to add some of the self-updated files to /var/lib/dpkg/info/<package>.list and they will removed correctly. Is that the way to go? To Update the listfile?12:45
wellsbdholbach, shouldn't the QtWebKit test be a warning, not something that prevents approval (application 41)?  I'll look into whether module module Ubuntu.Components.Extras.Browser UbuntuWebView will work in my situation.  If the experimental features work, I should be able to make the transition12:50
dholbach_wellsb, great - let me know how it goes12:52
=== dholbach_ is now known as dholbach
=== gatox is now known as gatox_brb
wellsbdholbach, I'm sure this is probably well documented somewhere, but I've missed a lot of the recent developments.  How can I execute the click test you run before I upload to the store so I can verify there are no errors or warnings and not waste your time?13:13
nik90mzanetti: Do you know why a label takes so much width? http://imgur.com/OyOIsaB13:16
nik90mzanetti: I set the label width: contentWidth13:16
nik90mzanetti: nevermind I fixed it13:18
mzanetti:)13:21
ybonnik90: sorry, me again ;) Can you show me the autopilot-desktop details you have? http://paste.ubuntu.com/6954457/13:42
ybonI don't understand at this point how you have autopilot-desktop 1.4 BUT gir1 1.3813:42
zsombinik90: the MP that fixes the layouts crash: https://code.launchpad.net/~zsombi/ubuntu-ui-toolkit/layouts-fix1280359/+merge/20693013:43
nik90ybon: http://paste.ubuntu.com/6954464/13:43
ybonor in other worlds why my version of autopitlo-desktop want gir1 > 1.3913:43
ybonthanks :)13:43
Kaleonik90, zsombi got your fix :) https://code.launchpad.net/~zsombi/ubuntu-ui-toolkit/layouts-fix1280359/+merge/20693013:43
nik90Kaleo: yup he just told me :)13:44
nik90Kaleo, zsombi: Will test it immediately..I finally figured out to build SDK source branches and test stuff out  :)13:44
Kaleonik90, nice )13:44
nik90ybon: np13:44
ybonah, we don't have the same exact version13:44
ybon1.4+14.04.20131125bzr397saucy0 vs 1.4+14.04.20131125bzr411saucy013:44
nik90zsombi: it is funny how the solution is few lines but the test cases are huge :)13:44
nik90ybon: I told you I am running the autopilot experimentat PPA(trunk) while you are using the 1.4 PPA13:45
ybonaaaahh, damn, missed this point, sorry13:45
nik90ybon: mine gets updated every few days :)13:45
ybonlet's upgrade once again, so :)13:45
nik90ybon: ensure you first purge your 1.4 ppa properly13:46
zsombinik90: yes :) and there were more than 3 lines this time, as there was an other issue with items removed during re-layouting13:46
ybonyep :)13:46
nik90randomcpp: hey ... its been a long time..how is it going?13:46
ybonrhââ, still not the same: 1.4+14.04.20131125-1experimental409~ubuntu13.10.113:48
randomcppI'm studying for exams :S how are you?13:48
ybonnik90: this one ppa:autopilot/experimental ?13:48
randomcppmaybe in march I will be able to work again on my projects, I hope13:49
nik90zsombi: it works! No more crashes yay :)13:49
nik90randomcpp: good luck with your exams.. I am doing good13:49
zsombinik90: überüawesomness :D13:49
nik90randomcpp: looking forward to saucy bacon new features13:49
nik90ybon: yeah13:49
zsombiehh, I mean über-awesomness13:50
nik90ybon: https://launchpad.net/~autopilot/+archive/experimental13:50
randomcppI have some planned, I will develop them as soon as possible when I'll have time :)13:50
randomcppI promise13:50
nik90randomcpp: I also got time to publish my own app as well :)13:50
ybonnik90: are we sure autopilot-desktop comes from this ppa?13:51
zsombiKaleo: :) (15.49.20) nik90: zsombi: it works! No more crashes yay :)13:51
randomcppnik90, if you have suggestions just post them on saucybacon issue tracker on github13:51
randomcppnik90, nice :)13:51
nik90randomcpp: definitely like old times :)13:51
=== gatox_brb is now known as gatox
randomcppyeah :)13:51
Kaleo:)13:51
nik90ybon: it seems not13:52
nik90ybon: I am not sure if autopilot and autopilot-desktop are the same thing13:52
ybonme neither :)13:52
ybonbut my tests are failing and it's the only package I have different from yours13:52
ybonso I need to test updating it to see if I can finally run autopilot tests, and push MRs :)13:53
nik90ybon: just continuing proposing MRs. Jenkins will test your branches until then13:54
ybonah, right13:54
nik90ybon: the only drawback of this situation is you are unable to run your tests, but for now don't let that stop your main work which is the calendar code13:54
ybonwell, let's try upgrading anyway, just in case :)13:54
nik90ybon: upgrading to 14.04?13:54
ybonno no :)13:54
ybonjust trying to upgrade autopilot-desktop13:55
nik90ybon: ah ok13:55
ybonI'm more comfortable if I can run my tests :)13:55
ybonso now that i've a possible explanation, let's try to solve, and if it's still not work, I will follow your advice :)13:55
nik90ybon: ok..you can then later ask in the autopilot channel around 19:00 UTC to get help if it still doesn't work.13:58
ybonI will be in my plane for Berlin at this time ;)13:59
ybonbut I will in the incoming days :)13:59
ybonand thanks for your help :)13:59
nik90np14:00
ybonap-desktop weems here https://launchpad.net/ubuntu/+source/autopilot in the description, but then not in the ppa list https://launchpad.net/~autopilot/+archive/1.414:01
ybon(not event https://launchpad.net/~autopilot/+archive/ppa )14:02
mzanettidpm or rpadovani: https://code.launchpad.net/~mzanetti/reminders-app/fix-compile-warnings/+merge/20693814:07
dpmlooking at it now...14:08
rpadovaninetwork and compilation race :D14:10
rpadovanimzanetti, confirm, no warning, good work :-)14:12
rpadovanimzanetti, why do you change int i in unsigned int i? It's a question to learn, not a doubt on your code ;-)14:14
mzanettirpadovani: because size() from stdlib returns a unsigned int14:14
mzanettirpadovani: and the compiler was complaining that I compared int and unsigned int14:14
popeymzanetti: am I right in saying you said you'd file a bug about the multi-pagestack header issue?14:15
mzanettirpadovani: which might cause an issue if we have more than 2 and a half million notes14:15
rpadovanimzanetti, ahhh, ok, thanks :-)14:15
dpmmzanetti, reviewed and commented14:18
mzanettidpm: :D14:22
dpm:)14:22
mzanettidpm: I'm happy to explain stuff to you if you are interested14:22
mzanettijust ask14:22
dpmmzanetti, that's fine. I am interested indeed (it's been quite a while since I programmed in C and a bit in C++, and sometimes I miss it a bit, perhaps in a masochistic way :), but I might not have time today.14:24
mzanettidpm: btw. the bit at line 9 is still exactly the same code as before...14:25
mzanettidpm: I just put a #ifdef in between to suppress a unused parameter warning14:25
mzanettidpm: because the parameter "policy" is only used when HAVE_SCHED_GET_PRIORITY_MAX || HAVE_SCHED_GET_PRIORITY_MIN14:26
mzanettiso without those its: static int toPthreadPriority(POLICY, PRIORITY priority) {14:26
mzanettiand with those its: static int toPthreadPriority(POLICY policy, PRIORITY priority) {14:26
dpmmzanetti, gotcha, thanks. Yeah, I wasn't sure when or why those defines would be defined14:26
zsombiseb128, kenvandine: hi guys, I have a fix for the ItemSelector regression, here you go for testing: https://code.launchpad.net/~zsombi/ubuntu-ui-toolkit/itemselector-fix1275861/+merge/20694514:27
seb128zsombi, great, I'm going to test in a bit, thanks!14:27
zsombiKaleo: ^ also, could you pick it up for review?14:27
* zsombi brb14:29
kenvandinezsombi, awesome!14:29
Kaleozsombi, nice!!14:30
Kaleoseb128, kenvandine, sorry for missing it14:30
zsombiseb128, kenvandine: yep, sorry guys, it slipped our eyes during the sprint :(14:31
seb128Kaleo, zsombi: np, thanks for fixing it!14:33
=== gatox is now known as gatox_lunch
=== gatox_lunch is now known as gatox
balloonsnik90: ping16:45
nik90balloons: pong16:45
balloonsnik90: so still looking into things today. Any changes from your end? I'm assuming something changed that broke clock at this point16:46
nik90balloons: I did not dig into it today...I am busy today with convergence stuff..16:46
balloonsnik90: no worries, just checking in and letting you know what was up16:47
nik90balloons: I have a feeling EDS could be a potential culprit since I noticed lots of console output when you delete alarms16:47
nik90balloons: however I am not sure why it causes trunk to fail since there the alarm tests are skipped16:47
balloonsnik90: I see alot of EDS noise too.. is there a rev pre-EDS to try?16:47
nik90balloons: well that is like rev 200 or something..pre-alarms period16:48
balloonsI don't believe EDS is the reason the app fails to launch however16:48
nik90balloons: +116:48
balloonsbut there seems to be an issue with it too :-)16:48
balloonsnik90: ok, I'll go run a rev from there just to see :))16:48
nik90balloons: at this point, my guess would be u1db since that was the main change in the backend, but you tested the rev before the u1db transition and it still failed for you16:48
nik90balloons: let me know how your investigation goes :)16:49
balloonsnik90: yep.. again, trunk failing, and pre-u1db failing, etc.. It can't be clock :=-)16:49
nik90balloons: I guess then as far as I know the packages that changed in the past week (which affects clock) are EDS and ui-toolkit16:50
nik90balloons: can you try an older ui-toolkit release?16:51
nik90balloons: let's leave EDS to the end16:51
balloonsnik90: I cannot, but yes I agree we need to rollback dependencies until we find out what's up16:51
nik90balloons: wait you mean in the archive itself? or just locally in your computer?16:52
balloonsnik90: on the jenkins box specifically, as we can't reproduce locally16:52
nik90balloons: just a heads up, I need the latest ui-toolkit for my upcoming critical patches :P16:52
balloonsnik90: yes, textfield emulator :-)16:52
nik90balloons: well that and the performance and convergence patches for MWC16:52
balloonsnik90: I think part of the trouble for rolling back jenkins is it's used by more than clock16:53
nik90balloons: isn't jenkins just a normal desktop running 14.04?16:53
nik90balloons: it is really weird that we cannot reproduce the failures on our laptops16:53
nik90hmm really weird I don't see much of popey these days...I need a test bot popey :)17:05
rpadovaniWOW, how many awesome partenership!17:36
rpadovanihttp://insights.ubuntu.com/news/growing-app-ecosystem-for-ubuntu-phones/17:36
kalikiananeat17:44
mzanettidpm: popey: can you please test lp:~mzanetti/reminders-app/tablet and tell me what you think?17:47
mzanettipopey: I've been playing with another way of working around our current issues.17:47
mzanettiimo quite successful, but with downsides.17:47
mzanettiit's not polished yet, it'll break if you try to save a note in editpage and the editpage needs to be adapted to make the cheating less visible.17:48
mzanettibut if you're ok with the direction in general I can fix that17:48
mzanettiteaser screenshot: http://i.imgur.com/tlptJiN.png17:50
popeymzanetti: ok17:53
dpmmzanetti, stepping out for a bit, will try to be around later17:56
mzanettiok17:56
=== dpm is now known as dpm-afk
m-b-opopey: hey, could you test something for me on the n10?17:57
popeym-b-o: sure18:00
m-b-opopey: can you test the hourly scrolling in lp:~martin-borho/ubuntu-weather-app/conditional-weather ?18:01
popeyya18:01
popeym-b-o: just testing another app first, one moment18:02
elopioping alex-abreu, are you still here?18:02
alex-abreuelopio, in a meeting18:02
popeymzanetti: s/Not note/No Note/18:02
alex-abreuelopio, still have you rbranch to review18:02
mzanettipopey: heh, thanks18:03
elopioalex-abreu, ah, yes that.18:03
elopioalex-abreu, and I'm looking at your autopilot fixes for the webapps, but they have a conflict.18:03
popeymzanetti: it looks janky.. lemme get a picture18:03
mzanettipopey: yeah, the edit page, right?18:03
popeyno, initial page with no note open18:03
elopioalex-abreu, continue with your meeting. Sorry to interrupt.18:03
popeymzanetti: http://imgur.com/QS36zmR18:05
mzanettipopey: right... the side stage handle is another issue in unity8, that's not the reminders app18:05
mzanettipopey: if you mean the header, I could probably put an empty header there for the initial page18:05
mzanettipopey: but what I'm really interested in, is if the behavior of the headers and toolbars is ok this way (if I'll polish a little more)18:06
nik90mzanetti: I think it would be nice to show a vertical divider between the two columns18:06
mzanettinik90: +118:07
mzanettinik90: is there already some existing patterns other apps use?18:07
nik90mzanetti: I use a component written by iBelieve where he takes the ListItem.ThinDivider and rotates it 90 degrees.18:07
mzanettinik90: ack. will do that18:08
popeym-b-o: yes, that fixes hourly18:23
popey(which looks awesome with the animations btw)18:23
m-b-opopey: thanks! And the values are getting updated in all the shapes?18:24
popeyyes18:24
m-b-ocool :)18:24
popeynic one!18:24
m-b-opopey: can you make screenhsots on the n10?18:25
popeysadly not18:25
popeyonly shaky-cam photos18:25
m-b-oHow many days are shown in the right sidebar?18:25
popey9.5 ☻18:25
m-b-obut you can scroll to 10.0?18:26
popeyyes18:26
m-b-owithout getiing bounced back or so?18:26
popeyyes18:27
popeyits fine18:27
popeyyou either see the top 9.5 or the bottom 9.518:27
popeyi take it we can't make those 10% smaller to fit them all in?18:27
m-b-ofor MWC?18:28
m-b-oin general, there other tablets as well... :)18:28
popeyyeah, not tried on my nexus 7, but that resolution is all over the place18:28
m-b-ohow do you get the app on the device, btw?18:29
popeywith some pain18:30
m-b-ohehe18:30
popeyhttp://pastebin.ubuntu.com/6949462/18:30
popeyadb shell.. click list | grep weather, click unregister com.ubuntu.weather x.x, pkcon install-local com.ubuntu.weather....click18:31
popeycmake ☹18:33
mrgoodcatat this stage in development where would help be most appreciated? e.g. bug triage, bug reporting, coding, designing...18:34
m-b-opopey: yes... and what does "click-buddy --dir . --provision"  ?18:35
popeym-b-o: pass18:36
popeymrgoodcat: you offering your services? ☻18:36
mrgoodcatrecently i've been trying to get more involved in the community in general. just joined michigan loco and a local coding group here as well. since touch apps seem to be the focus right now it seemed a good place to start18:37
popeymrgoodcat: what's your specific area of interest?18:41
mrgoodcatnothing in particular. thats why i asked where help would be appreciated. i have coded in other languages but never qml (although i pick up languages fairly quickly) but i wouldn't limit myself to just coding since there doesn't seem to be a shortage of coding talent18:43
nik90m-b-o:  "click-buddy --dir . --provision" gets the device ready to run autopilot tests18:52
m-b-onik90: ah, ok. but doesn't install it?18:52
nik90m-b-o: btw I am unable to run your branch properly for some reason..as It asks me for my location which I then choose, but it doesn't get past the loading indicator :/18:52
nik90m-b-o: click-buddy installs it on the device18:53
m-b-onik90: you the api_key18:53
nik90m-b-o: ah18:53
balloonsnik90: still working on getting things fixed with clock landing19:44
nik90balloons: I just heard from someone else that they also had a similar dbus error while running AP tests on the 14.04 desktop19:46
nik90balloons: it was the sudoku app developer19:47
balloonsnik90: ohh, any more news on that?19:47
balloonsnik90: and you are on saucy yes?19:47
nik90balloons: that's right, I am on saucy19:47
dachi there20:36
=== dac is now known as Guest96699
Guest96699is anyone here?20:37
rpadovaniGuest96699, lot of people :-)20:38
Guest96699hehe, I see.20:39
Guest96699I have a question regarding Qt Creator and HTML5 app.20:40
rpadovaniGuest96699, so, please, do your question :-) If someone knows the answer will reply20:41
Guest96699Well, I created a html5 app project (Hello world). But when I start it, I can't click on Hello, I mean, nothing happens. When I run index.html on chrome, I can click on Hello and go to second page but then BACK button does not work.20:43
=== dpm-afk is now known as dpm
dpmnik90, did you get sorted with the Weather app's API key? If not, just let me know and I can explain how to get it working20:46
nik90dpm: I got it working..m-b-o showed me how to do it20:46
dpmgreat20:46
nik90dpm: I just wanted to peek at the convergence look20:46
nik90dpm: btw the critical convergence bug has been fixed20:46
dpmcool!20:47
nik90dpm: I finished clock, stopwatch convergence...20:47
nik90dpm: working on alarm and timer now20:47
dpmnik90, wohoo! excellent20:49
m-b-odpm would you have a look at the conditonal-weatehr branch? the right side, the hieght of the list especially20:50
dpmhi m-b-o, sure, checking it out now...20:50
dpmm-b-o, looks good! A couple of comments:20:56
dpm- Would it be possible to add some padding to the left of the list, so that the left edge of the icons aligns with the "Forecast" label?20:57
m-b-oyes20:57
dpmcool20:58
dpmm-b-o, also still regarding padding: - could we add some padding to the bottom of each list item, so that both the top and bottom padding surrounding the icon is the same?20:59
m-b-odpm: I'm using the  "ListItem.Subtitled", don't know if it's possible21:01
m-b-oI'll try21:01
dpmm-b-o, ah, I see21:02
m-b-odpm: I guess not21:05
m-b-onot with Subtitled21:05
m-b-oI guess21:05
dpmm-b-o, no worries, thanks for looking. Regarding the list height, do you know if the list on the right panel is fully shown on a N7 and on a N10? I think you were asking around earlier on...21:11
m-b-opopey said 9.5 are visible on n10, n7 we don't know21:12
m-b-odpm: propably setting width and height in ubuntu-weather-app to 128 and 80 shoudl give us a hint21:13
m-b-oubuntu-weather-app.qml21:13
dpmah, good idea21:13
m-b-odpm: neokore will be ready with the left side tomorrow. too late?21:35
dpmm-b-o, it depends on the time tomorrow, I'll need to ask cwayne21:37
dpm(I'm doing that now)21:38
nik90dpm: regarding http://insights.ubuntu.com/news/growing-app-ecosystem-for-ubuntu-phones/, are all these done in collaboration with the community? Or these companies are announcing that they are building it for ubuntu touch..21:39
nik90both are awesome..just wondering21:39
dpmnik90, it depends on the ISV. For TWC for example, it's direct engagement with them + community work = Ubuntu Weather app + TWC backend. For the others in that list, it's mostly direct engagement and Canonical creating webapps for them.21:41
nik90nice..yeah I noticed that some are requestion scopes and lenses..pretty cool that they see the benefit of scopes in the unity dash21:41
dpmor for Cordova it's doing platform work to ensure Ubuntu is a supported platform by upstream21:42
nik90s/requestion/request21:42
dpmindeed :)21:42
m-b-odpm: Im not available tomorrow evening, so perhaps you could copy conditional-weather and raul will merge in your branch?21:46
dpmm-b-o, yes21:49
m-b-oso tomorrow evening would be okay to get the merge from raul?21:50
dpmm-b-o, yes, but if possible as early in the evening as possible21:50
dpmm-b-o, for that left padding I mentioned earlier on, would you like me to file a bug, or is this something you're already looking into?21:50
m-b-odpm: already pushed! ;)21:51
dpmawesome, pulling :)21:51
m-b-odpm: I'll tell raul21:52
dpmok, cool21:52
dpmm-b-o, is it possible to add the padding to the lines separating each list item, or is this something that you cannot change?21:53
dpm(looking good, btw)21:53
m-b-oI think not, these ListItems are relatively static.21:53
dpmok, then no worries21:54
dpmlet me have a look at the API docs, there was another thing I wanted to ask you about the data we're showing for TWC...21:54
nik90popey: thnx for the new clock app icon :)21:54
dpmto see if we can get Pressure shown instead of UV index, so that we're consistent with OWM21:55
m-b-odpm: there is pressure for TWC?21:56
dpmthat's what I wanted to check, I can't tell off the top of my head, but I want to see if it's possible21:57
dpmm-b-o, it's possible using the Observation command21:59
m-b-odpm: but not for days (or hours) AFAIk22:00
dpmoh, I see, it's instant barometric pressure, bummer22:01
popey☻ nik9022:01
m-b-odpm: we should consolidate the availabel data on occasion. Sunset/Sunrise shoudl be availabel for TWC and OWM for the current day for example22:05
m-b-oalso feelsLike for TWC and so on22:05
dpmm-b-o, +122:08
dpmm-b-o, ok, so if I coordinate with neokore the landing of the left side panel, are you're happy with the status of the branch as it is now?22:09
dpmI'm going to step out in a minute22:10
m-b-odpm: I'm too! :)22:10
m-b-odpm: raul knows hwo we will merge22:10
m-b-odpm: I think it'll be okay for MWC. :)22:10
m-b-odpm: will be around tomorrow until 16utc, so if there are some changes needed, ping me!22:11
dpmm-b-o, awesome, thanks a lot for all the effort, this is going to be extremely helpful to showcase the tablet at MWC!22:11
m-b-oI'm eager to learn what will be announced!22:12
m-b-o(if)22:12
dpm:-)22:13
m-b-ogood n822:14
dpmgood night m-b-o and everyone!22:15
wittybearanyone else got problems installing the ubuntu-sdk ?22:43
wittybearFor anyone interested details here: https://bugs.launchpad.net/ubuntu-qtcreator-plugins/+bug/128181422:47
ubot2`Launchpad bug 1281814 in [obsolete] Ubuntu QtCreator Plugins "Ubuntu SDK does not install on clean 12.04 LTS (64bit)" [Undecided,New]22:47
mzanettirpadovani: hey, wanna test some reminders goodness? https://code.launchpad.net/~mzanetti/reminders-app/tablet/+merge/20706222:54
rpadovanimzanetti, yes, sure :-) There is a texconflict in EditNotePage ;-)22:56
mzanettiinteresting.. will fix22:56
mzanettirpadovani: pushed22:58
rpadovanik22:59
balloonsnik90: branches should be landing again23:12
nik90balloons: yay!23:13
nik90balloons: What was the fix?23:13
balloonsnik90: qt5organizer5-eds causing issues23:13
nik90balloons: I just read fginther's comments in an MP "I think the tests are working now after removing qtorganizer5-eds from the test environment. Re-approving to retest."23:14
nik90balloons: so would my alarm tests pass considering they need EDS to work?23:14
balloonsnik90: it's what we thought might be the case, but it's odd23:14
balloonsnik90: that package is not a dependency of clock atm.. should it be? it seems to be being used by it23:14
nik90balloons: it is not a direct dependency of clock23:15
nik90balloons: clock uses Alarms API (from SDK) -> SDK should have EDS as dependency23:15
balloonsnik90: I guess we will see. We can have them use the old version if needed, which I assumed they were going to rollback , but23:16
nik90balloons: but how old are you rolling back to?23:17
nik90balloons: since there were some recent fixes to EDS that the Alarms API needs23:17
nik90balloons: either way, let try to merge my Alarms API then next23:18
rpadovanimzanetti, I find only one error, great work :-)23:19
rpadovaniBut also two strange behavior23:19
rpadovanihttps://code.launchpad.net/~mzanetti/reminders-app/tablet/+merge/207062/comments/48565623:19
balloonsnik90: right, just try and let me know23:19
nik90balloons: can you approve https://code.launchpad.net/~nik90/ubuntu-clock-app/fix-alarm-tests/+merge/205154 then23:20
nik90elopio: if you agree as well, pls approve the above MP23:21
nik90^^23:21
iBelieveballoons, ping23:26
balloonsiBelieve: pong23:26
mzanettirpadovani: ok, will fix23:27
iBelievehey balloons, all the autopilot tests crashed on my merge request and I'm wondering if you can help figure out why: http://91.189.93.70:8080/job/generic-mediumtests-trusty/1386/?23:27
iBelieveballoons, they're working when I run them locally. The merge request is: https://code.launchpad.net/~mdspencer/ubuntu-filemanager-app/better-desktop-support/+merge/19816523:28
mzanettirpadovani: where is the height smaller than the window? at the bottom or the top?23:28
rpadovanimzanetti, at the bottom. I'm not able to take a screenshot because when I press stamp the toolbar goes up23:30
rpadovanimhhh, I can use scrot23:30
mzanettistrange... doesn't happen here. a screenshot would be helpful, yes23:31
rpadovanimzanetti, http://people.ubuntu.com/~rpadovani/touch/tabletReminder.jpg23:33
mzanettirpadovani: I think this is the edit panel but you're missing the icons23:34
rpadovanimzanetti, I am a bit ashamed, you are right23:34
mzanettino problem23:35
mzanettiadmittedly it doesn't really look good23:35
elopionik90, I'm here!23:35
elopioI'm sorry, I'm going to the backscroll and I'll try to help you.23:35
rpadovanimzanetti, maybe with icon is better :-)23:36
rpadovani*icons23:36
balloonsiBelieve: I see the issue23:36
balloonsiBelieve: so things went to cmake, and you need to add new stuff to the CMakelist23:36
mzanettirpadovani: http://i.imgur.com/1IxcliC.png23:36
balloonsiBelieve: did you add pathbar?23:36
rpadovanimzanetti, yap, it is better, but maybe design input is useful23:37
balloonsiBelieve: so in CMakeList, look for the install line where it's adding the qml files, and include pathbar there also23:37
iBelieveballoons, ahh, so that's the problem. I merged from master and all I did was change the CMakeList file to rename a file I changed.23:37
iBelieveballoons, thanks for finding the problem!23:37
nik90elopio: can you trigger jenkins to run the tests on https://code.launchpad.net/~nik90/ubuntu-clock-app/fix-alarm-tests/+merge/20515423:37
nik90elopio: I like to know if the tests are passing now23:37
balloonsiBelieve: no worries. The way clock is setup avoids the need for this23:37
iBelieveballoons, does it use subdirectories and wildcards?23:38
balloonsiBelieve: so it's worth considering arranging things in that directory layout.. it's odd how include work23:38
balloonsiBelieve: the imports are "foldername" and it seems to get the includes correct that way23:38
elopionik90, I'm trying...23:39
nik90elopio: thnx23:39
iBelieveballoons, okay. I'll rearrange the files after this branch gets merged.23:39
balloonsiBelieve: yea, I'd be curious on feedback from tweaking the layout and includes23:39
balloonsI'm not master at this, but I liked the layout23:39
nik90iBelieve: take a look at the clock root folder.. that should give you an idea23:40
nik90iBelieve: however it is going to be a huge task since you need to modify the includes on all the qml files23:40
nik90iBelieve: one of the reasons, why I did the organization of the clock app folder (in april 2013) way early in the development23:40
* balloons listens23:41
elopionik90, I don't have an account on that jenkins, I'm trying to get one. But balloons rebuild it, it's running.23:41
iBelievenik90, what I was thinking about was having "components/" for non-page-level stuff, and "ui/" for pages, dialogs, and sheets. That way it shouldn't be too hard23:41
nik90iBelieve: that should be okay as well23:42
balloonsiBelieve: after you do that, you *should* be able to drop the install line where everything is specified23:44
balloonsand imports should *just work*23:44
=== chriadam|away is now known as chriadam
iBelieveballoons, that would be very nice to have23:44
rpadovanimzanetti, a couple of questions about camera:23:54
rpadovani- Adding support to switch camera requests a lot of code, is it necessary?23:54
rpadovanibecause it is an 'advance' feature and requests C++. I can do it, but I think it too much for a photo in reminder app23:55
rpadovani- In camera app there is support for rotation of icons if you rotate the device, do I implement it?23:56
iBelieveballoons, nik90: just refactored the files into the new layout. I only had to change a few QML imports. Nice!23:56
balloonsiBelieve: wow.. that simple eh?23:57
rpadovanimzanetti, switch camera = swap between front and retro camera23:57
iBelieveballoons, yep. The question is, will the CMakeList file work right...23:57
* balloons believes23:57
mzanettirpadovani: no need for rotation23:58
mzanettirpadovani: hmm... switching camera would be quite useful I think. but not required for a first step23:58
rpadovanimzanetti, ok, good, I have only a couple of issues, hope to fix for tomorrow afternoon :-)23:58
iBelieveballoons, :)23:59

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