=== chriadam|away is now known as chriadam [08:00] good morning === chriadam is now known as chriadam|away [09:19] Good morning all; happy Single Tasking Day! :-D === teknico_ is now known as teknico [09:37] dpm: o/ [09:37] hey mzanetti, morning! [09:38] dpm: can you give this a shot when you have some time? lp:~mzanetti/reminders-app/tablet [09:38] dpm: It's not complete, but I'd like a quick discussion about it [09:38] once you've seen it [09:38] mzanetti, 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 device [09:38] desktop is fine [09:39] I didn't test it on a real tablet yet either [09:39] ok, give me 5 mins and I'll give it a go [09:39] * dpm is looking forward to seeing tablet mode [09:40] don't get too excited yet [09:41] zsombi: hey good morning [09:41] zsombi: I got a small question about Ubuntu Layouts [09:41] nik90: good afternoon :) [09:41] nik90: gpo ahead [09:41] zsombi:timezones yes [09:41] ;) [09:42] zsombi: can I define a Flickable inside a conditional layout instead of a row or a column? [09:42] nik90: everything should be possible [09:42] nik90: we don't have samples for that, but Fliockable is yet another Item, right? [09:42] Flickable I mean [09:42] zsombi: I am getting an issue where the page header doesn't detect that the convergent layout is using the flickable [09:42] zsombi: as a result the page header doesn't hide when scrolling up [09:43] zsombi: that's why I thought perhaps I was doing something wrong [09:43] nik90: yep, the page header is an issue... it does not detect runtime changes [09:43] same issue here :) [09:43] zsombi: but yes Flickable is an item [09:43] zsombi: so If I set the flickable property of the page manually, then hopefully that should fix the issue? [09:43] nik90: it's a problem of page header unfortunately [09:44] nik90: yep, that shoudl do the job [09:44] mzanetti: nice to know :) .. if one of us has a solution to it, then let the other person know. [09:44] zsombi: thnx.. [09:44] I just started yesterday evening with the tablet mode for reminders [09:44] the content itself works quite well [09:44] but header and toolbar get messed up badly when unsing 2 pagestacks next to each other [09:45] mzanetti: nik90: you guys should have started the whole app with that in mind ;) :P [09:45] mzanetti: I started in the weekend but had to stop because of a critical crash when switching layouts.. [09:45] mzanetti, ack [09:45] mik90: I'm investigating the crash there [09:45] mzanetti: Florian figured out that using a Dialer widget caused it [09:45] zsombi: well... its just about not pushing everything to the same pagstack [09:45] mzanetti: ;) [09:45] zsombi: thnx [09:46] but yeah... need to figure how to deal with panel and header [09:46] nik90: 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] seems the last push()/pop() to any pagestack updates them [09:47] dpm: so yeah... that's what I wanted to discuss. How do other apps deal with this? [09:47] mzanetti: yes, because push/pop updates the flickable property so header can adjust its logic accordingly [09:48] zsombi: 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 other [09:48] mzanetti: not really [09:48] mzanetty that messes up the header for good [09:48] mzanetti: ^ [09:48] yeah... so what do you suggest? [09:49] mzanetti: you'd need a two'column pagestack kinda setup? [09:49] mzanetti, 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] zsombi: yeah [09:49] zsombi: let me show you a few screenies.... gimme a minute [09:50] mzanetti: 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/null [09:50] standard view phone: http://i.imgur.com/thyV6T4.png [09:50] after clicking on a note on the phone: http://i.imgur.com/1FTesPj.png [09:51] standard view tablet: http://i.imgur.com/TsHrtMF.png [09:51] clicking on a note on the tablet: http://i.imgur.com/nUlHE5j.png [09:51] now... as you can see there's an edit button for the note, which would push the EditPage on top of the note view page [09:51] mzanetti: yes, this is what I had for my PageStack [09:52] but at the same time, the tabs in the header allow you to switch to notebooks [09:52] which in turn would push pages to the left hand side [09:52] so yeah. not entirely sure how to deal with it right now [09:52] zsombi: are there any chances your 2 column pagestack will make it to trunk at some point? [09:53] mzanetti: it depends on how UX will pick it up... they're pretty busy with the whole UI reorg atm [09:53] heh... true. why am I doing this before the UI reorg actually? [09:53] mzanetti: but hey, the standard view phone is a Tabs, right? [09:53] zsombi: yeah, it is [09:54] mzanetti: same as the standard tablet, right? [09:54] mzanetti: and Notes is a PageStack [09:54] zsombi: yeah... I just make the pagestack non-anchors-fill just like the uitk gallery [09:54] Notes tab I mean [09:54] and put a second pagestack next to the other [09:55] and do some conditional pushes. the rest is identical between phone and tablet [09:55] mzanetti: ok, is the PageStack depth 2? [09:55] zsombi: 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] yeah, that's when the depth == 2 [09:56] mzaneti: brilliant! [09:56] or also, if you press the search button in the toolbar it pushes a searchPage to the left pagestack [09:56] mzanetti: then, in Phone layout you use PageStack, on tablet layout you use two columns, no need for pagestack [09:57] its not /that/ easy I'm afraid [09:57] mzanetti: the search will end up in the header, so forget about pushing a page [09:57] mzanetti: why? [09:57] cause both stacks need panels for example [09:57] mzanetti: what panels? [09:57] or well, toolactions, wherever they might be [09:57] left one needs: Add note for example [09:57] mzanetti: the tools will land on the header [09:58] right one needs: edit note, delete note, etc [09:58] mzanetti: you can update those tools based on what you have active [09:59] mzanetti: and yes, the problem in here is that the whole header-toolbar-search is changing, and everything will be upside down... [09:59] zsombi: 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? [10:01] mzanetti: ..... damn it... it would be really good to have that two-column PageStack mode solved by now... [10:01] oh yeah, it would :D [10:01] why are you chatting around here? get to work :P [10:01] mzanetti: :P I'm trying to help you out... [10:02] I know. just kidding [10:02] mzanetti: here's the MR: https://code.launchpad.net/~zsombi/ubuntu-ui-toolkit/two-pane-pagestack/+merge/172288 [10:03] zsombi: oh man... this is *exactly* what I'm doing... if its on the left side, pushing, if its on the right side replacing [10:03] mzanetti: so this setup was supporting depth >2 too [10:03] mzanetti: ;) [10:04] mzanetti: 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 page [10:05] mzanetti: so it all depended whether the pushOrReplace() or push() was used [10:05] ah... ok... then its a bit different... more like a column-view in a file manager [10:05] mzanetti: it all depends how you use it [10:06] mzanetti: "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] mzanettI: even I don't remember exactly the functionality :D [10:07] mzanetti: so PushOrReplace() did all 4 U [10:07] zsombi: 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/diTyYFa [10:07] right... although not exactly how I'd need it according to this description [10:08] I 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 one [10:08] zsombi: Although If I replace those label ItemLayouts with Rectangles, I see that they are placed correctly [10:08] zsombi: https://code.launchpad.net/~nik90/ubuntu-clock-app/converge-clock-tab [10:08] mzanetti: if the active page was on the left side, the pushOrReplace() did show up the page on the right pane [10:09] mzanetti: then every consecutive pushOrReplace on the left page was doing a replace :) [10:09] mzanetti: the right-to-left (and vice versa) comes into force when depth > 2 [10:09] nik90: the anchors.verticalCenter will anchor the Label, not the text within [10:10] ok. would need to try it... not sure if its really exactly what I'd need... [10:10] but anyways, still leaves the issue with the header and the toolbar I guess [10:10] nik90: you should use Label.verticalAlignment [10:11] mzanetti: this stuff was working with those pretty well [10:11] zsombi: will give that a shot nnow [10:11] mzanetti: and I think we should get this branch up alive again [10:11] ideally I'd have 2 headers [10:11] mzanetti: why would you? [10:11] one for the tabs, and one for the content title for the right side [10:12] mzanetti: ah, yeah, that was also discussed, but that is pretty ugly... and misleading [10:12] zsombi: that did the trick...I didn't have to do this before [10:12] nik90: beware of this with Labets ;) [10:12] Labels [10:14] zsombi: :) [10:20] dpm: what do you think is better http://imgur.com/AYpB1Ir or http://imgur.com/diTyYFa === zsombi1 is now known as zsombi [10:32] nik90, both look good, but I personally find http://imgur.com/diTyYFa better. I find the shadow on the other one a bit too deep [10:32] popey, can you have a go at lp:~mzanetti/reminders-app/tablet on the N10? [10:33] dpm: yeah I reverted it to the one that you liked before you said it...It needs more work to make it look better.. [10:38] nik90, cool. Did you work around that crash? I've not been following the conversation... [10:40] dpm: no the crash is still there, however I believe Kaleo and zsombi are on it. [10:40] dpm: I am meanwhile trying to implement the phone and tablet views for all tabs. [10:40] awesome [10:40] dpm: when the crash is resolved and pushed to phone, then I will propose a MR [10:42] kalikiana: I saw a boat load of fixes land in the u1db trunk...nice that you got someone to review the MP finally.. [10:42] kalikiana: is there a planned landing slot for them? [10:43] nik90: no idea yet, still waiting to find out how the fancy new process works, and then to see how to apply it to u1db-qt [10:43] I'll know more later today [10:44] kalikiana: nice..I already got most of the fixes from the PPA on my 13.10 :) [10:46] dpm: ok [10:53] (12.40.04) nik90: dpm: no the crash is still there, however I believe Kaleo and zsombi are on it. [10:54] nik90: dpm: yes, we are on it [10:54] oSoMoN: 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 advice [10:56] oSoMoN: 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 displaying [10:57] oSoMoN: 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 viewer [10:57] nerochiaro, and the SDK thumbnailer can’t be fed with live data, it understands only files on disk? [10:58] oSoMoN: 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] oSoMoN: it saves them on disk based on the md5 of the original image path [11:01] dpm: rotation is broken on nexus 10. [11:02] oSoMoN: 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 anyway [11:02] nerochiaro, how about saving modifications to the original images to a separate file on disk, and use that one to generate the thumbnail [11:02] ? [11:03] oSoMoN: that might work, but it's one extra disk write of the full image that we're doing just to generate a thumbnail [11:04] nerochiaro, 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 fly [11:06] dpm: mzanetti http://imgur.com/t5xxEpB,528Nscu,2GjsSPb,fseUpjH ⍨ [11:06] popey: oh [11:08] * popey updates tablet [11:14] popey: hmm... works here :/ [11:14] fine here now I updated [11:14] oSoMoN: 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 don [11:14] e that way, otherwise the original is taken, and all remaining operations are applied to it again in order minus the one being undone [11:15] oSoMoN: does it make sense ? [11:16] nerochiaro, doesn’t sound like a good idea, worded like this :) [11:17] oSoMoN: well, it's the best that can be done if there are operations which are lossy, like crop or enhance [11:17] nerochiaro, I guess that really depends on what operations we want to optimize [11:17] or rather, what use-cases we want to optimize [11:17] oSoMoN: well at the moment we only have 3: rotation, crop and enhance [11:18] oSoMoN: but one would expect to be a lot more [11:19] nerochiaro, what I meant is: I think it makes sense to ensure that viewing images in their current state is the fastest operation [11:19] argh [11:19] scratch "operation" [11:19] fastest use case [11:19] and it’s probably acceptable to have some overhead when applying new operations, or undoing them [11:20] nerochiaro, what are the other options we have, re- generating thumbnails? [11:20] oSoMoN: 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 gallery [11:21] oSoMoN: 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 viewer [11:21] oSoMoN: the thumbs for browsing will still not be 100% accurate though [11:21] oSoMoN: so if that's important too we can't really switch [11:21] nerochiaro, and is modifying the SDK thumbnailer to accept live data an option? [11:22] oSoMoN: not that I know of [11:22] oSoMoN: we can add that ability I guess, but it will be completely custom for us, not part of the freedesktop standard in any way [11:24] doesn’t sound like the best idea [11:24] dpm: mzanetti http://imgur.com/tpUFHdc,w3KLulY,ATWS6NG,VZstxmj,GH7B1cM,1TYThPp,UQfIhG6,nrlcgJ1,1yJ2BGx better [11:24] nice [11:25] oSoMoN: 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 it [11:25] popey: so yeah. the question what to do with the toolbar still persists [11:26] oSoMoN: if we were doing that too, it would solve the issue I think [11:26] oSoMoN: as we will regen the thumbs only when we finish with editing [11:27] oSoMoN: but it requires (a) a decision to actually do it and (b) design input on how to ask for save or discard [11:28] nerochiaro, I’d tend to think that if we’re not asking the user to save/discard, it’s because it was designed this way [11:28] nik90: ok, I think I found the problem causing Layouts crash [11:28] oSoMoN: i wouldn't be surprised if a persistent undo/redo stack is something we didn't ask for and yorba decided to put n [11:28] in [11:29] nerochiaro, I don’t know, but in any case you should check with design first [11:30] oSoMoN: 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:49] nik90: ping [11:54] mzanetti, https://code.launchpad.net/~mzanetti/reminders-app/fix-attaching-images/+merge/206816/comments/485207 [11:55] rpadovani: nice, thanks [12:03] ybon: pong [12:03] zsombi: oh nice :) [12:03] zsombi: is it a simple fix? [12:03] nik90: 3 lines of code :) [12:04] zsombi: lol [12:04] nik90: 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] just in case I've a bad one [12:04] zsombi: in the SDK or the clock app? [12:04] I'm still unable to run them, and that's annoying :( [12:04] nik90: in Layouts [12:04] nik90: SDK [12:08] omg, my irc client didnt send my last few messages..lets try agian [12:08] zsombi: ok...convergence is something I need for the MWC. [12:08] zsombi: can we expect a fix to land on the phone (not just the ppa) by friday? [12:08] ybon: http://paste.ubuntu.com/6954086/ [12:08] nik90: I got that, I'm already preparing the MR [12:09] nik90: more than sure [12:09] zsombi: thnx! you rock! [12:09] nik90: thanks :) [12:09] so my autopilot-desktop seems out of date [12:10] I've two versions in my cache, let's take the good one :) [12:10] ybon: :) [12:11] ah: [12:11] The following packages have unmet dependencies: [12:11] autopilot-desktop : Depends: gir1.2-glib-2.0 (>= 1.39.0-0ubuntu1) but 1.38.0-0ubuntu1 is to be installed [12:11] ybon: which version of ubuntu are you running? [12:11] 13.10? [12:11] sorry I may have asked this before [12:11] yep [12:13] nik90: do you where you get the gir1 lib from? [12:13] ybon: btw, the entire QA is on a sprint...they are generally online around 19:00 UTC [12:13] ybon: let me check [12:14] ybon: I have 1.38.0-0ubuntu1 installed [12:14] oh, like mine so [12:14] humm [12:14] ybon: what are the autopilot ppa you have? [12:14] do you have more than one? [12:15] humm [12:16] does the 0 sized ones can be of any conflict here: http://paste.ubuntu.com/6954120/ ? [12:17] no idea [12:17] when you removed the other 2 ppas, do you purge them? [12:18] humm, I've just --remove I think [12:18] let me remove those [12:18] I think you should do a ppa-purge on the ppa to remove all other dependencies it upgraded completely [12:18] right [12:25] can 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 name [12:25] did you do sudo ppa-purge ppa:name-of-ppa ? [12:26] yup [12:26] and without ppa: and with -p option also [12:33] I think they are purged, at least they do not appears in y-ppa-manager [12:44] Has someone experience about having an own update mechanism and leftover files? [12:45] I'll tried to add some of the self-updated files to /var/lib/dpkg/info/.list and they will removed correctly. Is that the way to go? To Update the listfile? [12:50] dholbach, 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 transition [12:52] wellsb, great - let me know how it goes === dholbach_ is now known as dholbach === gatox is now known as gatox_brb [13:13] dholbach, 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:16] mzanetti: Do you know why a label takes so much width? http://imgur.com/OyOIsaB [13:16] mzanetti: I set the label width: contentWidth [13:18] mzanetti: nevermind I fixed it [13:21] :) [13:42] nik90: sorry, me again ;) Can you show me the autopilot-desktop details you have? http://paste.ubuntu.com/6954457/ [13:42] I don't understand at this point how you have autopilot-desktop 1.4 BUT gir1 1.38 [13:43] nik90: the MP that fixes the layouts crash: https://code.launchpad.net/~zsombi/ubuntu-ui-toolkit/layouts-fix1280359/+merge/206930 [13:43] ybon: http://paste.ubuntu.com/6954464/ [13:43] or in other worlds why my version of autopitlo-desktop want gir1 > 1.39 [13:43] thanks :) [13:43] nik90, zsombi got your fix :) https://code.launchpad.net/~zsombi/ubuntu-ui-toolkit/layouts-fix1280359/+merge/206930 [13:44] Kaleo: yup he just told me :) [13:44] Kaleo, zsombi: Will test it immediately..I finally figured out to build SDK source branches and test stuff out :) [13:44] nik90, nice ) [13:44] ybon: np [13:44] ah, we don't have the same exact version [13:44] 1.4+14.04.20131125bzr397saucy0 vs 1.4+14.04.20131125bzr411saucy0 [13:44] zsombi: it is funny how the solution is few lines but the test cases are huge :) [13:45] ybon: I told you I am running the autopilot experimentat PPA(trunk) while you are using the 1.4 PPA [13:45] aaaahh, damn, missed this point, sorry [13:45] ybon: mine gets updated every few days :) [13:45] let's upgrade once again, so :) [13:46] ybon: ensure you first purge your 1.4 ppa properly [13:46] nik90: yes :) and there were more than 3 lines this time, as there was an other issue with items removed during re-layouting [13:46] yep :) [13:46] randomcpp: hey ... its been a long time..how is it going? [13:48] rhââ, still not the same: 1.4+14.04.20131125-1experimental409~ubuntu13.10.1 [13:48] I'm studying for exams :S how are you? [13:48] nik90: this one ppa:autopilot/experimental ? [13:49] maybe in march I will be able to work again on my projects, I hope [13:49] zsombi: it works! No more crashes yay :) [13:49] randomcpp: good luck with your exams.. I am doing good [13:49] nik90: überüawesomness :D [13:49] randomcpp: looking forward to saucy bacon new features [13:49] ybon: yeah [13:50] ehh, I mean über-awesomness [13:50] ybon: https://launchpad.net/~autopilot/+archive/experimental [13:50] I have some planned, I will develop them as soon as possible when I'll have time :) [13:50] I promise [13:50] randomcpp: I also got time to publish my own app as well :) [13:51] nik90: are we sure autopilot-desktop comes from this ppa? [13:51] Kaleo: :) (15.49.20) nik90: zsombi: it works! No more crashes yay :) [13:51] nik90, if you have suggestions just post them on saucybacon issue tracker on github [13:51] nik90, nice :) [13:51] randomcpp: definitely like old times :) === gatox_brb is now known as gatox [13:51] yeah :) [13:51] :) [13:52] ybon: it seems not [13:52] ybon: I am not sure if autopilot and autopilot-desktop are the same thing [13:52] me neither :) [13:52] but my tests are failing and it's the only package I have different from yours [13:53] so I need to test updating it to see if I can finally run autopilot tests, and push MRs :) [13:54] ybon: just continuing proposing MRs. Jenkins will test your branches until then [13:54] ah, right [13:54] ybon: 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 code [13:54] well, let's try upgrading anyway, just in case :) [13:54] ybon: upgrading to 14.04? [13:54] no no :) [13:55] just trying to upgrade autopilot-desktop [13:55] ybon: ah ok [13:55] I'm more comfortable if I can run my tests :) [13:55] so 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:58] ybon: ok..you can then later ask in the autopilot channel around 19:00 UTC to get help if it still doesn't work. [13:59] I will be in my plane for Berlin at this time ;) [13:59] but I will in the incoming days :) [13:59] and thanks for your help :) [14:00] np [14:01] ap-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.4 [14:02] (not event https://launchpad.net/~autopilot/+archive/ppa ) [14:07] dpm or rpadovani: https://code.launchpad.net/~mzanetti/reminders-app/fix-compile-warnings/+merge/206938 [14:08] looking at it now... [14:10] network and compilation race :D [14:12] mzanetti, confirm, no warning, good work :-) [14:14] mzanetti, why do you change int i in unsigned int i? It's a question to learn, not a doubt on your code ;-) [14:14] rpadovani: because size() from stdlib returns a unsigned int [14:14] rpadovani: and the compiler was complaining that I compared int and unsigned int [14:15] mzanetti: am I right in saying you said you'd file a bug about the multi-pagestack header issue? [14:15] rpadovani: which might cause an issue if we have more than 2 and a half million notes [14:15] mzanetti, ahhh, ok, thanks :-) [14:18] mzanetti, reviewed and commented [14:22] dpm: :D [14:22] :) [14:22] dpm: I'm happy to explain stuff to you if you are interested [14:22] just ask [14:24] mzanetti, 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:25] dpm: btw. the bit at line 9 is still exactly the same code as before... [14:25] dpm: I just put a #ifdef in between to suppress a unused parameter warning [14:26] dpm: because the parameter "policy" is only used when HAVE_SCHED_GET_PRIORITY_MAX || HAVE_SCHED_GET_PRIORITY_MIN [14:26] so without those its: static int toPthreadPriority(POLICY, PRIORITY priority) { [14:26] and with those its: static int toPthreadPriority(POLICY policy, PRIORITY priority) { [14:26] mzanetti, gotcha, thanks. Yeah, I wasn't sure when or why those defines would be defined [14:27] seb128, 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/206945 [14:27] zsombi, great, I'm going to test in a bit, thanks! [14:27] Kaleo: ^ also, could you pick it up for review? [14:29] * zsombi brb [14:29] zsombi, awesome! [14:30] zsombi, nice!! [14:30] seb128, kenvandine, sorry for missing it [14:31] seb128, kenvandine: yep, sorry guys, it slipped our eyes during the sprint :( [14:33] Kaleo, zsombi: np, thanks for fixing it! === gatox is now known as gatox_lunch === gatox_lunch is now known as gatox [16:45] nik90: ping [16:45] balloons: pong [16:46] nik90: so still looking into things today. Any changes from your end? I'm assuming something changed that broke clock at this point [16:46] balloons: I did not dig into it today...I am busy today with convergence stuff.. [16:47] nik90: no worries, just checking in and letting you know what was up [16:47] balloons: I have a feeling EDS could be a potential culprit since I noticed lots of console output when you delete alarms [16:47] balloons: however I am not sure why it causes trunk to fail since there the alarm tests are skipped [16:47] nik90: I see alot of EDS noise too.. is there a rev pre-EDS to try? [16:48] balloons: well that is like rev 200 or something..pre-alarms period [16:48] I don't believe EDS is the reason the app fails to launch however [16:48] balloons: +1 [16:48] but there seems to be an issue with it too :-) [16:48] nik90: ok, I'll go run a rev from there just to see :)) [16:48] balloons: 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 you [16:49] balloons: let me know how your investigation goes :) [16:49] nik90: yep.. again, trunk failing, and pre-u1db failing, etc.. It can't be clock :=-) [16:50] balloons: I guess then as far as I know the packages that changed in the past week (which affects clock) are EDS and ui-toolkit [16:51] balloons: can you try an older ui-toolkit release? [16:51] balloons: let's leave EDS to the end [16:51] nik90: I cannot, but yes I agree we need to rollback dependencies until we find out what's up [16:52] balloons: wait you mean in the archive itself? or just locally in your computer? [16:52] nik90: on the jenkins box specifically, as we can't reproduce locally [16:52] balloons: just a heads up, I need the latest ui-toolkit for my upcoming critical patches :P [16:52] nik90: yes, textfield emulator :-) [16:52] balloons: well that and the performance and convergence patches for MWC [16:53] nik90: I think part of the trouble for rolling back jenkins is it's used by more than clock [16:53] balloons: isn't jenkins just a normal desktop running 14.04? [16:53] balloons: it is really weird that we cannot reproduce the failures on our laptops [17:05] hmm really weird I don't see much of popey these days...I need a test bot popey :) [17:36] WOW, how many awesome partenership! [17:36] http://insights.ubuntu.com/news/growing-app-ecosystem-for-ubuntu-phones/ [17:44] neat [17:47] dpm: popey: can you please test lp:~mzanetti/reminders-app/tablet and tell me what you think? [17:47] popey: I've been playing with another way of working around our current issues. [17:47] imo quite successful, but with downsides. [17:48] it'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] but if you're ok with the direction in general I can fix that [17:50] teaser screenshot: http://i.imgur.com/tlptJiN.png [17:53] mzanetti: ok [17:56] mzanetti, stepping out for a bit, will try to be around later [17:56] ok === dpm is now known as dpm-afk [17:57] popey: hey, could you test something for me on the n10? [18:00] m-b-o: sure [18:01] popey: can you test the hourly scrolling in lp:~martin-borho/ubuntu-weather-app/conditional-weather ? [18:01] ya [18:02] m-b-o: just testing another app first, one moment [18:02] ping alex-abreu, are you still here? [18:02] elopio, in a meeting [18:02] mzanetti: s/Not note/No Note/ [18:02] elopio, still have you rbranch to review [18:03] popey: heh, thanks [18:03] alex-abreu, ah, yes that. [18:03] alex-abreu, and I'm looking at your autopilot fixes for the webapps, but they have a conflict. [18:03] mzanetti: it looks janky.. lemme get a picture [18:03] popey: yeah, the edit page, right? [18:03] no, initial page with no note open [18:03] alex-abreu, continue with your meeting. Sorry to interrupt. [18:05] mzanetti: http://imgur.com/QS36zmR [18:05] popey: right... the side stage handle is another issue in unity8, that's not the reminders app [18:05] popey: if you mean the header, I could probably put an empty header there for the initial page [18:06] popey: 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] mzanetti: I think it would be nice to show a vertical divider between the two columns [18:07] nik90: +1 [18:07] nik90: is there already some existing patterns other apps use? [18:07] mzanetti: I use a component written by iBelieve where he takes the ListItem.ThinDivider and rotates it 90 degrees. [18:08] nik90: ack. will do that [18:23] m-b-o: yes, that fixes hourly [18:23] (which looks awesome with the animations btw) [18:24] popey: thanks! And the values are getting updated in all the shapes? [18:24] yes [18:24] cool :) [18:24] nic one! [18:25] popey: can you make screenhsots on the n10? [18:25] sadly not [18:25] only shaky-cam photos [18:25] How many days are shown in the right sidebar? [18:25] 9.5 ☻ [18:26] but you can scroll to 10.0? [18:26] yes [18:26] without getiing bounced back or so? [18:27] yes [18:27] its fine [18:27] you either see the top 9.5 or the bottom 9.5 [18:27] i take it we can't make those 10% smaller to fit them all in? [18:28] for MWC? [18:28] in general, there other tablets as well... :) [18:28] yeah, not tried on my nexus 7, but that resolution is all over the place [18:29] how do you get the app on the device, btw? [18:30] with some pain [18:30] hehe [18:30] http://pastebin.ubuntu.com/6949462/ [18:31] adb shell.. click list | grep weather, click unregister com.ubuntu.weather x.x, pkcon install-local com.ubuntu.weather....click [18:33] cmake ☹ [18:34] at this stage in development where would help be most appreciated? e.g. bug triage, bug reporting, coding, designing... [18:35] popey: yes... and what does "click-buddy --dir . --provision" ? [18:36] m-b-o: pass [18:36] mrgoodcat: you offering your services? ☻ [18:37] recently 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 start [18:41] mrgoodcat: what's your specific area of interest? [18:43] nothing 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 talent [18:52] m-b-o: "click-buddy --dir . --provision" gets the device ready to run autopilot tests [18:52] nik90: ah, ok. but doesn't install it? [18:52] m-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:53] m-b-o: click-buddy installs it on the device [18:53] nik90: you the api_key [18:53] m-b-o: ah [19:44] nik90: still working on getting things fixed with clock landing [19:46] balloons: I just heard from someone else that they also had a similar dbus error while running AP tests on the 14.04 desktop [19:47] balloons: it was the sudoku app developer [19:47] nik90: ohh, any more news on that? [19:47] nik90: and you are on saucy yes? [19:47] balloons: that's right, I am on saucy [20:36] hi there === dac is now known as Guest96699 [20:37] is anyone here? [20:38] Guest96699, lot of people :-) [20:39] hehe, I see. [20:40] I have a question regarding Qt Creator and HTML5 app. [20:41] Guest96699, so, please, do your question :-) If someone knows the answer will reply [20:43] Well, 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. === dpm-afk is now known as dpm [20:46] nik90, 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 working [20:46] dpm: I got it working..m-b-o showed me how to do it [20:46] great [20:46] dpm: I just wanted to peek at the convergence look [20:46] dpm: btw the critical convergence bug has been fixed [20:47] cool! [20:47] dpm: I finished clock, stopwatch convergence... [20:47] dpm: working on alarm and timer now [20:49] nik90, wohoo! excellent [20:50] dpm would you have a look at the conditonal-weatehr branch? the right side, the hieght of the list especially [20:50] hi m-b-o, sure, checking it out now... [20:56] m-b-o, looks good! A couple of comments: [20:57] - 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] yes [20:58] cool [20:59] m-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? [21:01] dpm: I'm using the "ListItem.Subtitled", don't know if it's possible [21:01] I'll try [21:02] m-b-o, ah, I see [21:05] dpm: I guess not [21:05] not with Subtitled [21:05] I guess [21:11] m-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:12] popey said 9.5 are visible on n10, n7 we don't know [21:13] dpm: propably setting width and height in ubuntu-weather-app to 128 and 80 shoudl give us a hint [21:13] ubuntu-weather-app.qml [21:13] ah, good idea [21:35] dpm: neokore will be ready with the left side tomorrow. too late? [21:37] m-b-o, it depends on the time tomorrow, I'll need to ask cwayne [21:38] (I'm doing that now) [21:39] dpm: 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] both are awesome..just wondering [21:41] nik90, 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] nice..yeah I noticed that some are requestion scopes and lenses..pretty cool that they see the benefit of scopes in the unity dash [21:42] or for Cordova it's doing platform work to ensure Ubuntu is a supported platform by upstream [21:42] s/requestion/request [21:42] indeed :) [21:46] dpm: Im not available tomorrow evening, so perhaps you could copy conditional-weather and raul will merge in your branch? [21:49] m-b-o, yes [21:50] so tomorrow evening would be okay to get the merge from raul? [21:50] m-b-o, yes, but if possible as early in the evening as possible [21:50] m-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:51] dpm: already pushed! ;) [21:51] awesome, pulling :) [21:52] dpm: I'll tell raul [21:52] ok, cool [21:53] m-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] (looking good, btw) [21:53] I think not, these ListItems are relatively static. [21:54] ok, then no worries [21:54] let 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] popey: thnx for the new clock app icon :) [21:55] to see if we can get Pressure shown instead of UV index, so that we're consistent with OWM [21:56] dpm: there is pressure for TWC? [21:57] that's what I wanted to check, I can't tell off the top of my head, but I want to see if it's possible [21:59] m-b-o, it's possible using the Observation command [22:00] dpm: but not for days (or hours) AFAIk [22:01] oh, I see, it's instant barometric pressure, bummer [22:01] ☻ nik90 [22:05] dpm: we should consolidate the availabel data on occasion. Sunset/Sunrise shoudl be availabel for TWC and OWM for the current day for example [22:05] also feelsLike for TWC and so on [22:08] m-b-o, +1 [22:09] m-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:10] I'm going to step out in a minute [22:10] dpm: I'm too! :) [22:10] dpm: raul knows hwo we will merge [22:10] dpm: I think it'll be okay for MWC. :) [22:11] dpm: will be around tomorrow until 16utc, so if there are some changes needed, ping me! [22:11] m-b-o, awesome, thanks a lot for all the effort, this is going to be extremely helpful to showcase the tablet at MWC! [22:12] I'm eager to learn what will be announced! [22:12] (if) [22:13] :-) [22:14] good n8 [22:15] good night m-b-o and everyone! [22:43] anyone else got problems installing the ubuntu-sdk ? [22:47] For anyone interested details here: https://bugs.launchpad.net/ubuntu-qtcreator-plugins/+bug/1281814 [22:47] Launchpad bug 1281814 in [obsolete] Ubuntu QtCreator Plugins "Ubuntu SDK does not install on clean 12.04 LTS (64bit)" [Undecided,New] [22:54] rpadovani: hey, wanna test some reminders goodness? https://code.launchpad.net/~mzanetti/reminders-app/tablet/+merge/207062 [22:56] mzanetti, yes, sure :-) There is a texconflict in EditNotePage ;-) [22:56] interesting.. will fix [22:58] rpadovani: pushed [22:59] k [23:12] nik90: branches should be landing again [23:13] balloons: yay! [23:13] balloons: What was the fix? [23:13] nik90: qt5organizer5-eds causing issues [23:14] balloons: 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] balloons: so would my alarm tests pass considering they need EDS to work? [23:14] nik90: it's what we thought might be the case, but it's odd [23:14] nik90: that package is not a dependency of clock atm.. should it be? it seems to be being used by it [23:15] balloons: it is not a direct dependency of clock [23:15] balloons: clock uses Alarms API (from SDK) -> SDK should have EDS as dependency [23:16] nik90: I guess we will see. We can have them use the old version if needed, which I assumed they were going to rollback , but [23:17] balloons: but how old are you rolling back to? [23:17] balloons: since there were some recent fixes to EDS that the Alarms API needs [23:18] balloons: either way, let try to merge my Alarms API then next [23:19] mzanetti, I find only one error, great work :-) [23:19] But also two strange behavior [23:19] https://code.launchpad.net/~mzanetti/reminders-app/tablet/+merge/207062/comments/485656 [23:19] nik90: right, just try and let me know [23:20] balloons: can you approve https://code.launchpad.net/~nik90/ubuntu-clock-app/fix-alarm-tests/+merge/205154 then [23:21] elopio: if you agree as well, pls approve the above MP [23:21] ^^ [23:26] balloons, ping [23:26] iBelieve: pong [23:27] rpadovani: ok, will fix [23:27] hey 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:28] balloons, they're working when I run them locally. The merge request is: https://code.launchpad.net/~mdspencer/ubuntu-filemanager-app/better-desktop-support/+merge/198165 [23:28] rpadovani: where is the height smaller than the window? at the bottom or the top? [23:30] mzanetti, at the bottom. I'm not able to take a screenshot because when I press stamp the toolbar goes up [23:30] mhhh, I can use scrot [23:31] strange... doesn't happen here. a screenshot would be helpful, yes [23:33] mzanetti, http://people.ubuntu.com/~rpadovani/touch/tabletReminder.jpg [23:34] rpadovani: I think this is the edit panel but you're missing the icons [23:34] mzanetti, I am a bit ashamed, you are right [23:35] no problem [23:35] admittedly it doesn't really look good [23:35] nik90, I'm here! [23:35] I'm sorry, I'm going to the backscroll and I'll try to help you. [23:36] mzanetti, maybe with icon is better :-) [23:36] *icons [23:36] iBelieve: I see the issue [23:36] iBelieve: so things went to cmake, and you need to add new stuff to the CMakelist [23:36] rpadovani: http://i.imgur.com/1IxcliC.png [23:36] iBelieve: did you add pathbar? [23:37] mzanetti, yap, it is better, but maybe design input is useful [23:37] iBelieve: so in CMakeList, look for the install line where it's adding the qml files, and include pathbar there also [23:37] balloons, 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] balloons, thanks for finding the problem! [23:37] elopio: can you trigger jenkins to run the tests on https://code.launchpad.net/~nik90/ubuntu-clock-app/fix-alarm-tests/+merge/205154 [23:37] elopio: I like to know if the tests are passing now [23:37] iBelieve: no worries. The way clock is setup avoids the need for this [23:38] balloons, does it use subdirectories and wildcards? [23:38] iBelieve: so it's worth considering arranging things in that directory layout.. it's odd how include work [23:38] iBelieve: the imports are "foldername" and it seems to get the includes correct that way [23:39] nik90, I'm trying... [23:39] elopio: thnx [23:39] balloons, okay. I'll rearrange the files after this branch gets merged. [23:39] iBelieve: yea, I'd be curious on feedback from tweaking the layout and includes [23:39] I'm not master at this, but I liked the layout [23:40] iBelieve: take a look at the clock root folder.. that should give you an idea [23:40] iBelieve: however it is going to be a huge task since you need to modify the includes on all the qml files [23:40] iBelieve: one of the reasons, why I did the organization of the clock app folder (in april 2013) way early in the development [23:41] * balloons listens [23:41] nik90, I don't have an account on that jenkins, I'm trying to get one. But balloons rebuild it, it's running. [23:41] nik90, 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 hard [23:42] iBelieve: that should be okay as well [23:44] iBelieve: after you do that, you *should* be able to drop the install line where everything is specified [23:44] and imports should *just work* === chriadam|away is now known as chriadam [23:44] balloons, that would be very nice to have [23:54] mzanetti, a couple of questions about camera: [23:54] - Adding support to switch camera requests a lot of code, is it necessary? [23:55] because it is an 'advance' feature and requests C++. I can do it, but I think it too much for a photo in reminder app [23:56] - In camera app there is support for rotation of icons if you rotate the device, do I implement it? [23:56] balloons, nik90: just refactored the files into the new layout. I only had to change a few QML imports. Nice! [23:57] iBelieve: wow.. that simple eh? [23:57] mzanetti, switch camera = swap between front and retro camera [23:57] balloons, yep. The question is, will the CMakeList file work right... [23:57] * balloons believes [23:58] rpadovani: no need for rotation [23:58] rpadovani: hmm... switching camera would be quite useful I think. but not required for a first step [23:58] mzanetti, ok, good, I have only a couple of issues, hope to fix for tomorrow afternoon :-) [23:59] balloons, :)