=== alan_g is now known as alan_g|afk === alan_g|afk is now known as alan_g [08:14] hey sil2100, how are you? [08:29] didrocks: hi! Sleepy ;) [08:36] didrocks: I re-triggered all the stupid powerpc builds that failed [08:36] ;/ [08:37] sil2100: do we know why? it seems that there is something that can't be installed (some arch:all, arch:any, but can't figure out what): https://launchpadlibrarian.net/140580074/buildlog_ubuntu-raring-powerpc.unity-lens-files_7.1.0daily13.05.24ubuntu.unity.next-0ubuntu1_FAILEDTOBUILD.txt.gz [09:15] Wellark: ping ;) [09:39] didrocks: I need to look into that again, since it happens too frequently [09:40] And why only powerpc? Mystery [09:40] yep :/ [09:44] Saviq, need your expertise for an issue. [09:44] nic-doffay, hit me [09:45] Saviq, bit of a complicated scenario. [09:45] Saviq, https://code.launchpad.net/~unity-team/unity/infographics-with-lightdm [09:45] there's the branch. [09:45] If you look in Infographics.qml [09:46] You'll see the onDataChanged signal and onDataAboutToChange signal. [09:47] Saviq, let me know if you can't find it. [09:47] nic-doffay, I'm there [09:49] The current issue is that the onDataAboutToChange and onDataChanged signal are fired off too quickly. [09:49] So the reverse animation affects the new set of data. [09:49] (As does the reveal animation) [09:50] You can run it and see what I mean (I've commented out the reveal animation to make it more obvious). [09:50] nic-doffay, I do understand what you mean [09:50] nic-doffay, you need to "cache" the old model before swapping it if the new one [09:51] nic-doffay, pete-woods1 there's two possible solutions I can see [09:51] Saviq, ok. Then perform the animation on the cached one. [09:51] nic-doffay, pete-woods1, either expose both the "current" and "next" models [09:52] nic-doffay, and then you switch the model after you've finished the reverse animation [09:52] nic-doffay, and then start the reveal animation [09:52] nic-doffay, pete-woods1, or you signal the "I'm ready to get the new model" after you've finished the reverse animation [09:53] nic-doffay, and wait for the backend to tell you "the model has been updated" before starting the new animation [09:53] s/new/reveal/ [09:53] Saviq, that's what I suggested, but pete-woods1 feels the backend should only communicate to the UI. I think that's a valid point too. [09:53] what I said is I can't interrogate the UI's state [09:54] nic-doffay, can't interrogate, that's fine [09:54] nic-doffay, but you can _tell it_ [09:54] nic-doffay, via a signal / method exposed on the model or somewhere [09:54] nic-doffay, that you're now ready to take the new data [09:54] nic-doffay, pete-woods1, that approach seems potentially easier [09:55] Saviq, yeah. Just a signal on the QML to fire's when the animation has stopped. [09:55] nic-doffay, pete-woods1, but it might come with a delay between the reverse and reveal animations [09:55] you have to check [09:55] Saviq, what sort of delay are you thinking? [09:55] nic-doffay, the time needed for the backend to replace the data [09:56] and to transport it to the UI [09:56] and potentially create new circles, etc [09:56] pete-woods1, that would need to happen regardless of which mode we go for [09:56] I think that's probably the best time for the delay, though [09:56] as the thing should be invisible at that point [09:56] pete-woods1, of course, but if you can prepare the model _while_ the reverse animation is happening [09:57] pete-woods1, the delay could be lower [09:57] true [09:57] pete-woods1, but that's premature optimization [09:57] indeedy [09:58] Saviq, I can come up with something else in between the delay to make up for it. [09:58] pete-woods1, nic-doffay, so I'd go for "backend says there's going to be a new data set soon; UI goes for reverse animation; UI signals that it's now ready for new data; backend fills the model with new data; backend signals that data is ready; UI goes for the reveal animation" [09:58] Saviq, sounds good. [09:58] and see how it does [09:58] pete-woods1, on what object do you want the signals in the QML? [10:02] nic-doffay: the signals would be present on the main infographic model [10:02] you would be saying "ready to change" or something like that at the end of the fade out animation [10:03] I'll add the signal to the model now, but it won't do anything, then at least you can get the UI to call it at the right time [10:03] then I'll tweak the model so it behaves how you want [10:04] pete-woods1, cool sounds good [10:09] * Saviq biab [10:09] Saviq, ta for the suggestions. [10:19] didrocks: hm, cyphermox already moved unity-gtk-module to the indicators stack, right? [10:19] didrocks: do you know if he redeployed the unity stack after the change? [10:20] didrocks: since the unity parameters in the generic check job seem to have unity-gtk-module in it [10:20] Would suck to have unity-gtk-module released 2 times ;) [10:21] sil2100: from jenkins, it seems he did that :) [10:21] sil2100: I don't see the module, at least, he removed the job [10:23] didrocks: then I wonder why ps-generic-* is still testpackages unity-autopilot unity-gtk-module-autopilot, and the invalid tests: unity unity-gtk-module unity-gtk-module-common libunity-gtk3-parser0 [10:23] Crazy stuff [10:25] sil2100: yeah, weird (and no time to look at that TBH) [10:27] mzanetti, Saviq: I've fixed the issue from the comments on the MR (https://code.launchpad.net/~macslow/unity/phablet-snap-decision-action-expansion/+merge/165370/comments/366256), but can't get around a wait(100) for waiting on the collapse-animation to complete, thus the before-after height-test passes... any ideas? The expansion-animation doesn't seem to need this for its before-after test. === mmrazik is now known as mmrazik|lunch [10:33] MacSlow: hmm... not really sure what happens. but usually an animation animates some property and you should be able to use a tryCompare() to wait for that property to be finished animating [10:33] mzanetti, ok... trying tryCompare() again [10:34] MacSlow: otherwise, looking at the MR, seems there are a bit too many actionSpy.clear()s in it [10:34] MacSlow: rest looks quite ok to me [10:40] mzanetti, tryCompare() just sits there... that's not working [10:41] MacSlow: should I have a look at it or do you want to go with that wait(100)? [10:41] mzanetti, well I know a wait(100) will not pass the review :) [10:41] ok... let me check === MacSlow is now known as MacSlow|lunch [11:08] MacSlow|lunch: the tryCompare works for me. I've commented on the MR... === mzanetti is now known as mzanetti|lunch [11:24] sil2100: pong === alan_g is now known as alan_g|lunch === mzanetti|lunch is now known as mzanetti [12:10] Wellark: hi! Are you working on the HUD backend? [12:16] mzanetti, I'm reviewing lp:~dandrader/unity/phablet_edgeDragInLauncher... I hope you'll simplify Launcher.qml a lot ;) [12:17] Saviq: yes. I expect the revealing and hiding to happen by daniel's code and all the rest will just go away [12:17] mzanetti, the amount of states alone is scary [12:17] Saviq: no more states (maybe just a closed, opened and hint) [12:18] at max [12:18] mzanetti, cool :) [12:19] anyone knows whats the state of mobile data connection is? friends of mine are just waiting for that before flashing their phones :D [12:22] rsalveti, ^? === mmrazik|lunch is now known as mmrazik === _salem is now known as salem_ [12:33] sil2100: should you restart the apps stack? [12:34] didrocks: will do, just checking the exact failure [12:34] Eh [12:35] thanks! [12:38] mzanetti, when working on the N10 greeter, I recommend switching to adb over TCP - it doesn't drop the connection when you lock the screen - so it doesn't destroy tablet-services [12:39] mzanetti, I'll have a small tweak to run_on_device to help with that [12:39] Saviq: afaik our run_on_device always works through adb forwarding [12:39] mzanetti, that's not even about that [12:39] Saviq: https://code.launchpad.net/~paulliu/unity/i18n/+merge/165160' [12:40] Saviq: https://code.launchpad.net/~paulliu/unity/i18n/+merge/165160 [12:40] mzanetti, the important thing is that adb itself goes through TCP and not USB [12:40] Saviq: oh, interesting [12:40] Saviq: BTW, the changes to debian/* is caused by wrap-and-sort. Do we want that in a seperated branch? [12:40] paulliu, yes please [12:40] Saviq: ok. [12:41] mzanetti, whether it does forwarding or not is a different issue (and only an issue when you have two devices at the same time - and that I want to enable in run_on_device) [12:41] Saviq: Please review that branch. And I'll prepare a wrap-and-sort for debian/* later. [12:41] mzanetti, I can now go run_on_device for both phone and tablet at the same time [12:42] Saviq: cool. iirc there is some command where you can register to the HUP signal. We might be able to use that too so when you unplug during a run_on_device it still restores the *-services file [12:42] mzanetti, right [12:51] mzanetti, https://code.launchpad.net/~saviq/unity/phablet.run-on-device-remote/+merge/165606 [12:52] mzanetti, with that you can export TARGET_IP and TARGET_SSH_PORT in your shell for run_on_device to go over TCP [12:52] mzanetti, and http://stackoverflow.com/questions/2604727/how-can-i-connect-to-android-with-adb-over-tcp for enabling adb over tcp [12:53] mzanetti, if you want it to persist across reboots (and even flashes, it seems), `adb setprop persist.adb.tcp.port 5555` === MacSlow|lunch is now known as MacSlow [12:54] mzanetti, I also recommend installing ninja-build on the device to improve build time (especially no-build-needed time) === alan_g|lunch is now known as alan_g [12:55] Saviq: so we should probably pack that into build -s [12:55] mzanetti, indeed, actually let me drop that into the same branch [12:57] mzanetti, actually not for build -s as that would mean people have to s/make/ninja/ in their minds [12:57] isn't that transparent? [12:57] mzanetti, not if you `make test` or similar [13:02] paulliu, the pl.po file got wrong encoding [13:02] Saviq: TARGET_IP=${TARGET_IP-127.0.0.1} [13:02] Saviq: haven't seen that notation before. what exactly does that do? [13:02] mzanetti, that's the default value if unset [13:02] mzanetti, so if $TARGET_IP is unset (not empty, but _unset_) [13:03] mzanetti, it will get set to 127.0.0.1 [13:03] cool. useful stuff :) [13:03] paulliu, `wget -O po/pl.po http://pastebin.ubuntu.com/5694011/plain/` should help [13:04] ugh [13:04] if only pastebin wasn't stupid [13:04] and wouldn't require SSO for plaintext download [13:05] Saviq: ok..wait [13:05] paulliu, will email you the file in a sec [13:08] paulliu, you've got it in email, should be fine if you save it as is [13:08] ok [13:09] paulliu, "66 + * wrap-and-sort the debian/* files." is still there in the changelog [13:12] Saviq: ok. I'll remove it. [13:17] paulliu, also, http://pastebin.ubuntu.com/5696932/ happens [13:18] paulliu, you're calling the bindTextDomain too quickly [13:20] Saviq: ah.. that's it. [13:20] mzanetti, my Launcher modifications have landed. You can now rebase and propose yours [13:20] Saviq: Let me fix it. [13:21] dandrader: thanks [13:23] paulliu, do you think it would be possible to make it work without "make install"? [13:23] paulliu, currently we need to make install just to make gettext pick the locales up from the installation dir [13:23] mzanetti: Saviq: we were able to do our first 3g data connection yesterday, should be landing next week [13:23] rsalveti, awesome :) [13:25] rsalveti: \o/ [13:25] paulliu, and if you actually run from builddir/install (LD_LIBRARY_PATH=../unity_build/build/lib ./builddir/install/bin/qml-phone-shell) it won't pick the translations up, 'cause isRunningInstalled() is true in that case [13:25] paulliu, isRunningInstalled() return true when running from CMAKE_INSTALL_PREFIX [13:27] Saviq: yeah..let me see if I can fix that. Because we are using the default cmake gettext support. The mo files are only put into correct dir structure on install. [13:27] paulliu, yeah, let's leave it for now [13:28] paulliu, we'll just note that you need to `make install` before the translations get picked up for now [13:28] paulliu, and later we'll look for a solution for that [13:28] Saviq: ok. [13:29] paulliu, you should update the .pot and .po now that you've added more translatable strings ;) [13:29] Saviq: yes.. [13:29] Saviq: I'll add notes and refresh the pot/po files together. wait. [13:30] paulliu, I'll send you an updated pl.po in a sec [13:34] dandrader: d'oh... the merge is conflicting quite a lot already [13:35] * dandrader is glad he got to merge his work first :) [13:35] mzanetti, http://www.nelson-haha.com/ [13:36] paulliu, sent you an update [13:36] Saviq: ok. [13:36] dandrader: not funny [13:37] :P [13:37] dandrader: http://nooooooooooooooo.com/ [13:38] :) [13:38] sil2100: didrocks: I couldn't redeploy unity, it was running at the time [13:44] cyphermox: ok, mind doing it now? [13:44] sil2100: don't touch the apps stack [13:44] o> [13:44] dednick_, nic-doffay standup? [13:45] didrocks: sucks, one test failed in apps [13:46] didrocks: that's what I'm doing [13:46] sil2100: yeah, we're trying otto on it [13:46] cyphermox: thanks! [13:48] didrocks: done. [13:48] should I rerun unity as well? [13:49] I'd double-check indicators too, to pick up from tomorrow [13:52] I meant yesterday of course [13:52] mzanetti, regarding my ColorImageProvider, one problem with your suggestion is that we use CrossFadeImage for the wallpaper. I suppose I could add color support directly to that Component, but I was leery of modifying those [13:53] mterry: ohhh... right... didn't think of that [13:53] mterry: hmm... the ColorImageProvider still seems funky to me :D [13:54] mterry: FYI: the CrossfadeImage is only used in the greeter. so breaking it just breaks our stuff [13:54] Saviq: whats your opinion? [13:55] Saviq, this is in reference to https://code.launchpad.net/~mterry/unity/phablet-greeter-features/+merge/165479 [13:55] Saviq: mterry created a ColorImageProvider which just paints a rectangle with the given color. I said he should use regular Rectangles in QML for that [13:55] Saviq, and it's ColorImageProvider [13:55] mzanetti, mterry yeah I know what's this about [13:55] :) [13:56] mterry it does feel like it'd be slower than a Rectangle [13:56] mterry, so yeah, I'd probably go for tweaking CrossFadeImage [13:56] OK! [13:57] mterry, so that it would support "fade to color" apart from "fade to another image" [13:57] mterry, I think it makes sense to be there [13:57] mzanetti, ^ [13:57] yep [13:57] I agree [13:59] Saviq: Just pushed.. https://code.launchpad.net/~paulliu/unity/i18n/+merge/165160 [14:00] paulliu, cheers [14:00] dandrader: can you please shortly explain what your change actually changes? [14:01] dandrader: should there be a change in behavior or should I not notice anything different? [14:01] mzanetti, 1- replaced DraggingArea with DirectionalDragArea [14:01] ok, seen that [14:02] mzanetti, 2- removed the behavior where you can drag and already launch an application once you release [14:02] dandrader: ok. that was removed in my branch too [14:02] mzanetti, so you no longer see that toll-tip balloon [14:02] ah... you deleted the LauncherLabel.qml? [14:03] because that was a bit of a weird conflict [14:03] mzanetti, yes [14:03] mzanetti, as it's no longer used [14:03] ok... explains things [14:03] dandrader: I think we still need it later.. anyways. no problem [14:03] mzanetti, that was my initial thought as well [14:04] dandrader: it will change a lot... I agree. so probably just deleting current version is the most sensible thing... I agree [14:04] mzanetti, well, it's still there in history :) [14:06] ok... so as of now Launcher.qml is my domain. any confllicting changes will be overwritten :D === mmrazik is now known as mmrazik|afk [14:07] Saviq: ah... forgot to mention in the standup: before you use time to find the cause - I did it already: https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1183727 [14:07] Launchpad bug 1183727 in Ubuntu UI Toolkit "[UbuntuShape] using an image with properties asynchronous and sourceSize set causes artifacts when resizing" [High,New] [14:12] mzanetti, not seeing in the shell ;) [14:13] Saviq: no? [14:13] kgunn, here's the CODING update https://code.launchpad.net/~saviq/unity/phablet.run-on-device-remote/+merge/165606 [14:13] mzanetti, what exactly should I be looking at / doing? [14:13] mzanetti, ah or do you mean when I resize the window on my desktop? [14:13] Saviq: cool, i'll take care [14:13] Saviq: yes [14:13] mzanetti, yeah, seeing it [14:14] mzanetti, I broke the autolanding in here https://code.launchpad.net/~saviq/unity/phablet.run-on-device-remote/+merge/165606 [14:15] naughty boy [14:17] Saviq: is that CODING file generated to html or anything? [14:18] mzanetti, kgunn does the generating to html ;) [14:18] mzanetti, but it's ~ ReST [14:18] or whatever the shortcut is for restructured text [14:18] Saviq: "TL;DR" ?? [14:18] mzanetti, Too Long; Didn't Read [14:18] mzanetti, your geek foo is not good [14:19] hehe [14:19] Saviq: the geek-foo dict is TL:DR [14:19] mzanetti, http://www.urbandictionary.com/define.php?term=tl%3Bdr [14:19] :P [14:20] mzanetti, http://en.wikipedia.org/wiki/Wikipedia:Too_long;_didn%27t_read [14:20] mzanetti, http://en.wiktionary.org/wiki/TL;DR [14:20] mzanetti, really? :P [14:20] mzanetti: i had to look it up the other day [14:20] kgunn, you're excused, you're a manager ;P [14:20] :)) [14:21] you know... I get that in a chat and look it up. I see that in a file in the code tree and think this will generate some fancy html tag in the docs [14:23] mzanetti, looking at your branch atm. [14:24] Must I uncomment everything that's commented? [14:24] in Lockscreen. [14:24] nic-doffay: no... [14:24] mzanetti, what needs to be uncommented? [14:24] nic-doffay: there should be your shader somewhere... use the one in Shell.qml [14:24] mzanetti, getting rid of that there for now. [14:24] Adding it somewhere else. === alan_g is now known as alan_g|tea [14:24] mainly wondering about this mzanetti layer.enabled: root.required && ! greeter.shown && shell.applicationManager.mainStageFocusedApplication [14:25] nic-doffay: let me check... I think I pused some stuff today morning that removes all the shaders === alan_g|tea is now known as alan_g [14:40] dandrader, re autopilot - sounds like that's expected? a Pointer shouldn't do Touch [14:40] dandrader, so it probably sent mouse events anyway? [14:40] Saviq, "The Pointer class is a wrapper that unifies the API of the Mouse and Touch classes, which can be helpful if you want to write a test that can use either a mouse of a touch device." [14:41] dandrader, bad naming it seems ;) === dandrader is now known as dandrader|afk === dandrader|afk is now known as dandrader [15:13] bregma, any idea which version of unity will work with Nux 4.0? [15:14] nic-doffay, Nux 4.0 is the current version of Nux (well, 4.0.2), which is trunk Uniy (Unity 7) [15:14] bregma, ok ta === salem_ is now known as _salem [16:04] Saviq: ping [16:04] sil2100, pong === _salem is now known as salem_ [16:29] dandrader, we were 2 tests away from 300 and we're back to 296 because of your merge :P [16:29] dandrader, not nice, not nice at all! [16:30] Saviq, the Launcher change? yeah :) [16:32] dandrader: hey. I'm having a strange issue with mouse events using the DirectionalDraggingArea. do you have a minute? [16:32] mzanetti, yes [16:33] dandrader: I have a draggingarea and on top of it the launcher now [16:33] dandrader: that works quite well most of the time [16:34] dandrader: but sometimes, even tho I click on the launcher, the DirectionalDraggingArea underneath recognizes some touches [16:34] dandrader: status goes to Undecided and distanceChanged is emitted [16:35] dandrader: but the launchers flickable should handle it of course which is on top of the DraggingArea [16:36] could you think of something causing this? [16:36] mzanetti, yes [16:37] mzanetti, a touch event is sent first. it doesn't get handled by anyone Qt synthesizes an equivalent mouse event and sends it [16:37] s/it doesn't/if it doesn't [16:37] DraggingArea handles mouse events but DirectionalDragArea handles touch events [16:37] dandrader: oh. I might have been unclear. This is on the desktop. so I suppose we only have mouse clicks [16:38] mzanetti, no, mouse events are converted to touch events by MouseTouchAdaptor [16:38] mzanetti, so that we have the same code path as if we were running on the device [16:38] dandrader: ah, ok. makes sense [16:39] mzanetti, try it with ./run --nomousetouch [16:39] dandrader: so, still it works most of the time. then I flick the launcher up and down a couple of times and suddendly the DirectionalDraggingArea gets the events [16:39] DirectionalDragAreas won't work though... [16:41] mzanetti, you could add some qDebugs() here and there to confirm my theory [16:42] dandrader: sure [16:42] but I don't really see why that would cause this behavior [16:43] mzanetti, or using something that handles touches instead of a DraggingArea, but I'm not sure if it would be quick to try it out === salem_ is now known as _salem [16:48] mzanetti, I updated the branch, btw [16:48] mterry: ok. I'll review soon [16:49] mzanetti, I'd be happy to help talk you through changes you'll need for the PIN stuff today? unless you're close to EOD [16:52] mterry: hmm... actually I am. Would like to finish that current task I'm on. But after the review today I have some idea on what to do next. will probably ask some smaller questions on monday === _salem is now known as salem_ [16:53] mzanetti, OK. Monday is a national holiday in US. But I'll try to idle on IRC and check it now and then [16:53] mzanetti, but bottom line is that all the current PAM interaction is inside LoginList.qml, which isn't even used in phone mode [16:53] sil2100: around? [16:54] mzanetti, so you'll have to write the PAM interaction in phone mode. But you should be able to ape what's in LoginList [16:54] mterry: yes. my branch uses the singleton from within a new qml file (Lockscreen.qml) [16:54] mterry: yes. thats what I thought. great [16:54] mzanetti, OK. My new branch exercises PAM workflows pretty well, so you should be able to see examples for whatever you need === dandrader is now known as dandrader|lunch [16:55] mzanetti, there are features in PAM that you may decide not to support this first go-through. For example, info messages... Not sure if the design mockups took such things into account [16:56] mterry: ah.. what I didn't see yet. is there a nice way to fire the showPrompt signal manually for testing already? [16:56] didrocks: yep [16:56] didrocks: what's up? :) [16:57] mzanetti, that's done inside the mock liblightdm object, triggered from an authenticate() call usually [16:57] mterry: no. I don't think there are info messages. at least not in the current designs [16:57] mzanetti, (well, there could be info messages from PAM, but we don't have a place to show them -- unlikely on our phone though) [16:57] mzanetti, see tests/mocks/LightDM/full/GreeterPrivate.cpp for examples of showPrompt [16:58] sil2100: did you get in touch with osomon about the webbrowser failures? [16:58] mterry: ok. sounds good. I think I'll figure the rest [16:58] thanks a lot [16:58] mzanetti, sure, and like I said, I'll try to idle on IRC on Monday [16:58] May end up doing some work that day after all [16:59] mterry: I'll try to keep it to a minimum [16:59] mzanetti, have a good day! will stop pinging you so you can go :) [17:00] mterry: thanks. have a nice weekend! [17:00] Enjoy the weekend everyone! [17:02] didrocks: got disconnected from the internet... [17:02] 18:58:09 didrocks | sil2100: did you get in touch with osomon about the webbrowser failures? [17:03] didrocks: yes, we're chatting this very moment [17:03] sil2100: excellent! :) [17:03] thanks :) [17:05] sil2100, hey, also I got around the ABI for nux/2.0 (with help from bregma) [17:05] ABI break [17:05] ! [17:05] bschaefer: oh! [17:05] sil2100, using a union :) [17:05] bschaefer: since I poked the SRU team with a question about the ABI breakage, but no answer yet [17:05] sil2100, well now no worries! Thanks for poking them! [17:06] Colin probably is busy [17:06] sil2100, but when the time comes to actually SRU there is a nux and unity branch that need to go in at the same time === alan_g is now known as alan_g|EOW [17:06] * bschaefer just wants to give enough of a heads up [17:08] but really, the nux branch doesn't really depend on the unity one...its just the unity one enables XIM support ;0 [17:08] :)* [17:09] ;) [17:09] Awesome! It would be great to get that to 2.0 [17:10] yup! I've a ppa, that i need to actual fix for when you don't have an IM active [17:13] I'll be testing that on my old laptop on Monday then ;) [17:14] But for now, I disappear [17:14] See you around! [17:14] sil2100, cya! === dandrader|lunch is now known as dandrader === dandrader is now known as dandrader|afk === dandrader|afk is now known as dandrader === dandrader_ is now known as dandrader === salem_ is now known as _salem