[08:35] <mzanetti> moin moin
[08:48] <mzanetti> Saviq: can you log in to jenkins? doesn't work any more for me
[08:49] <Saviq> mzanetti, s-jenkins?
[08:49] <mzanetti> I think it happens to all jenkins instances that use SSO for me
[08:49] <Saviq> mzanetti, works fine here
[08:49] <Saviq> mzanetti, try logging out of login.ubuntu.com and start from scratch?
[08:49]  * mzanetti tries
[08:50] <mzanetti> Saviq: nope. always get a connection reset
[08:50] <Saviq> mzanetti, different browser?
[08:53] <mzanetti> same thing... chrome says "No data received"
[08:53] <mzanetti> Error code: ERR_EMPTY_RESPONSE
[08:55] <Saviq> mzanetti, sounds like you need #is
[08:59] <mzanetti> Saviq: can you please trigger some rebuilds on my MPs, now that jenkins should pass again
[08:59] <Saviq> mzanetti, will o
[08:59] <Saviq> do
[08:59] <mzanetti> thanks
[09:09] <Saviq> mzanetti, btw, https://code.launchpad.net/~om26er/unity8/show_password_on_label_tap/+merge/240767 has no tags now
[09:09] <Saviq> +bad
[10:04] <larsu> Saviq: this s/StatusIcon/Icon branch doesn't stop haunting me :/
[10:04] <larsu> Saviq: did DefaultIndicatorWidget.qml get removed?
[10:05] <Saviq> larsu, yeah, the indicators got reworked
[10:07] <larsu> k, thanks
[10:22] <dandrader> mzanetti, ping
[10:27] <mzanetti> dandrader: hey
[10:27] <tsdgeos> Cimi: Saviq: what's missing for https://code.launchpad.net/~aacid/unity8/photoscopeimprovements/+merge/239834 approval?
[10:27] <Cimi> tsdgeos, moar testing
[10:28] <tsdgeos> ok
[10:28] <Cimi> tsdgeos, I asked someone to test week ago
[10:28] <dandrader> mzanetti, I need your help fixing a PhoneStage bug in the shellRotaion branch. Can you help me out?
[10:28] <mzanetti> dandrader: sure
[10:28] <Cimi> tsdgeos, since I was having issues on krillin
[10:28] <mzanetti> dandrader: btw, I had a read through your changes today
[10:29] <Cimi> tsdgeos, no one had time/did
[10:29] <dandrader> mzanetti, it's bug #1 here: https://docs.google.com/a/canonical.com/spreadsheets/d/140Icn5zcZwMvg1SONrwRKXYip-Pie7jtbEARpWwgxfw/edit#gid=0
[10:29] <mzanetti> in progress? I thought this was fixed
[10:29] <dandrader> mzanetti, I found it quite trick to fix it
[10:30] <mzanetti> dandrader: so what I wanted to ask, if we couldn't drop those properties you added to the PhoneStage API
[10:30] <mzanetti> dandrader: they won't work out with the desktop stage
[10:30] <mzanetti> which I'm gonna start working on this week
[10:30] <dandrader> mzanetti, I originally fixed it by removing the dash visibility optimizations I had in place but that, in its turn, caused the bug in row 16
[10:31] <mzanetti> yeah... I'm afraid that's the if() you dropped
[10:31] <dandrader> mzanetti, because I did that visibility optimization so long ago that I forgot what I did it for :D
[10:31] <mzanetti> but need to check...
[10:31] <dandrader> mzanetti, the problem with the optmization code is, as I debuged, that when a new app is added, but the newcomer *and* dash have index[10:32] <dandrader> mzanetti, before the newcomer is finally assigned index[10:32] <mzanetti> dandrader: actually the newcomer should be ApplicationManager.count at first
[10:32] <dandrader> mzanetti, and then you get an invisible dash for a frame
[10:32] <dandrader> mzanetti, ah, good, so you know how to fix it then :D
[10:32] <mzanetti> I guess I'll figure it
[10:33] <dandrader> mzanetti, well, that was what the console.log()s I had in place told me.
[10:33] <dandrader> mzanetti, thanks!
[10:33] <mzanetti> dandrader: so again, those public properties you added in the PhoneStage api
[10:33] <mzanetti> they won't work for non-spread stages
[10:33] <mzanetti> so we should find a way to do things without those
[10:34] <dandrader> what's a "non-spread" stage?
[10:34] <dandrader> mzanetti, ^
[10:34] <mzanetti> a desktop
[10:34] <mzanetti> for example
[10:34] <mzanetti> or a tv
[10:35] <mzanetti> actually, for tv I guess they'd work because that's still only fullscreen apps (assumingly)
[10:35] <mzanetti> but for desktop it won't. And we're starting to work on the desktop *now*
[10:36] <mzanetti> dandrader: can you point me to the ppa with the rotation stuff?
[10:36] <dandrader> mzanetti, https://launchpad.net/~unity-team/+archive/ubuntu/demo-stuff
[10:36] <mzanetti> thanks
[10:36] <dandrader> mzanetti, branch is lp:~unity-team/unity8/shellRotation
[10:37] <dandrader> mzanetti, I'll update that spreadsheet to have that bug in row 2 assigned to you and to be "in progress"
[10:38] <mzanetti> ok
[11:19] <facundobatista> Hola
[11:23] <mzanetti> dandrader|afk: there's a bit more broken actually... if you have an app running and start another one, the first one will just disappear
[11:24] <mzanetti> sometimes the dash disappears too
[11:43] <dandrader> mzanetti, right... can you fix it? :-D
[11:44] <mzanetti> dandrader: I wonder how you broke it though... probably you changed the ordering in qtmir... will have a look, yeah
[11:44] <mzanetti> going for a run now. bbiab
[12:04] <dandrader> dednick, ping
[12:21] <dednick> dandrader: howdy
[12:21] <dandrader> dednick, hi. busy with rtm bugs?
[12:22] <dednick> dandrader: nope
[12:22] <dandrader> dednick, great. :)  I need your help with the indicators in lp:~unity-team/unity8/shellRotation
[12:22] <dandrader> dednick, so I removed that Flickable.contentX binding
[12:23] <dandrader> dednick, but that caused the bug in row 15 https://docs.google.com/a/canonical.com/spreadsheets/d/140Icn5zcZwMvg1SONrwRKXYip-Pie7jtbEARpWwgxfw/edit#gid=0
[12:23] <dandrader> dednick, so we have to find a way to implement this feature without fiddling with contentX
[12:24] <dednick> dandrader: woops
[12:25] <dandrader> dednick, maybe tracking contenX and adding a left marging accordingly
[12:25] <dandrader> I don't know. But there must be a way :)
[12:26] <dednick> dandrader: ya. let me take a look
[12:27] <dandrader> dednick, thanks!
[12:34] <dednick> dandrader: does tryOrientedShell work now?
[12:35] <dandrader> dednick, it does
[12:35] <dandrader> dednick, I use it all the time
[12:35] <dednick> dandrader: hm. not working for me. press the buttons and nothing happens
[12:35] <tsdgeos> MacSlow|lunch: any lucj in tracking down https://code.launchpad.net/~macslow/unity8/swipe-dismiss-snap-decisions/+merge/233347 ?
[12:36] <dednick> dandrader: are you using lp:~unity-team or lp:~dandrader as latest?
[12:36] <dandrader> dednick, you also need unity-api from the PPA
[12:36] <dandrader> dednick, the lp:~dandrader one has my WIP work
[12:36] <dednick> dandrader: ya. i have it. how about qtubuntu and all?
[12:36] <dandrader> dednick, no need for qtubuntu
[12:37] <dandrader> dednick, do you get any error messages printed on the terminal?
[12:37] <dandrader> dednick, are you on vivid?
[12:38] <dednick> dandrader: on vivid. no messages
[12:38] <tsdgeos> larsu: broken merge in https://code.launchpad.net/~larsu/unity8/stop-using-statusicon/+merge/234502
[12:39] <dandrader> dednick, dash does not rotate. so you have to launch an app that supports rotation
[12:39] <dandrader> dednick, like twitter or facebook
[12:39] <dednick> dandrader: but the indicators should?
[12:39] <dandrader> dednick, ?
[12:39] <dednick> dandrader: ah. do the indicators only rotate when the app does.
[12:39] <dednick> sorry, i'm stupid
[12:40] <dandrader> dednick, it's the whole Shell.qml that rotates if the focused app supports multiple orientations
[12:40] <dednick> dandrader: right :)
[12:41] <dednick>  was thinking everything always rotates if you push the button.
[12:41] <dandrader> the button just updates the fake device orientation
[12:44] <tsdgeos> Saviq: any idea why there's no CI job of https://code.launchpad.net/~larsu/unity8/stop-using-statusicon/+merge/234502 running?
[12:46] <dandrader> dednick, the indicators resize bug in doesn't show up with "make tryOrientedShell" though
[12:47] <dandrader> dednick, but it does show up with "make tryPanel" when you resize the qmlscene window
[12:48] <dandrader> dednick,  well, not with current shellRotation as it has the fix, but just as a heads up :)
[12:50] <Saviq> tsdgeos, it should run soon
[12:56] <mzanetti> dandrader: is this the qtmir related branch? https://code.launchpad.net/~dandrader/qtmir/supportedOrientations
[12:56] <dandrader> mzanetti, yes
[12:57] <mzanetti> dandrader: I'm proposing them as WIP merges to see diffs
[12:59] <greyback> +1. I think we can start reviewing the lower level bits
[13:00] <mzanetti> the qtmir diff is much smaller than i expected... like that :)
[13:17] <MacSlow> tsdgeos, not yet
[13:51] <Cimi> are we sure -m works on the desktop?
[13:51] <Cimi> cannot reveal the launcher
[13:53] <Saviq> Cimi, we are sure
[13:54] <Cimi> Saviq, ./run.sh -m and you can reveal the launcher?
[13:54] <MacSlow> tsdgeos, mzanetti: I think I know what's causing the issue with the qmltest-failure of swipe-dismiss branch for notifications.
[13:55] <mzanetti> nice!
[13:55] <Saviq> Cimi, sec
[13:55] <Cimi> mzanetti, https://code.launchpad.net/~mzanetti/unity8/rounded-bfb/+merge/241931
[13:57] <Saviq> Cimi, "-m, --nomousetouch Run without -mousetouch argument."
[13:57] <mzanetti> Cimi: thanks
[13:57] <Cimi> Saviq, both with and without -m I cannot reveal launcher
[13:57] <Cimi> vivid
[13:58] <Saviq> Cimi, wfm
[13:58] <Cimi> nfm
[13:58] <Saviq> mzanetti, mock apps are not antialiased either ;)
[14:00] <mzanetti> yeah well. there's lots of issue with mock apps
[14:01] <mzanetti> Cimi: fixed
[15:23] <Cimi> mzanetti, I'd increase the topMargin
[15:24] <mzanetti> Cimi: what topMargin?
[15:25] <mzanetti> Cimi: well, all those sizes are given to me by vesa, I wouldn't change them without checking back with him
[15:25] <Cimi> mzanetti, that topMargin is to paing the rounded corner outside the screen
[15:25] <Cimi> paint
[15:25] <Cimi> isn't it?
[15:26] <mzanetti> Cimi: yeah, it's to not have it rounded at the bottom
[15:26] <mzanetti> Cimi: note that top/bottom is inverted on the phone
[15:26] <mzanetti> Cimi: so if you see topMargin in here, it's actually bottom
[15:26] <Cimi> mzanetti, exactly
[15:27] <Cimi> mzanetti, if you try at 32GU
[15:27] <Cimi> mzanetti, you see there is 1px of radius
[15:27] <Cimi> that starts...
[15:27] <Cimi> mzanetti, I think is more than 1gu of radius
[15:28] <Cimi> mzanetti, maybe we can do 2gu
[15:39] <tsdgeos> Saviq: question
[15:39] <Saviq> tsdgeos, answer
[15:40] <tsdgeos> https://code.launchpad.net/~aacid/unity8/moreAsyncDash/+merge/241524 last ci job says "PASSED: Continuous integration, rev:1391"
[15:40] <tsdgeos> but that's really not possible
[15:40] <tsdgeos> and looking at https://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-vivid/86/consoleFull i see some 1397
[15:40] <tsdgeos> and some 1398
[15:40] <tsdgeos> is there a way to know what revision it is really?
[15:41] <Saviq> tsdgeos, I can't see 1397 in there?
[15:41] <tsdgeos> right sorry
[15:41] <tsdgeos> some 1398
[15:41] <tsdgeos> which is 1397 + the merge from trunk
[15:41] <Saviq> tsdgeos, hah, I think I know what happened
[15:41] <tsdgeos> in my random guess
[15:41] <Saviq> tsdgeos, you pushed while this job was running
[15:42] <tsdgeos> probably
[15:42] <Saviq> tsdgeos, so indeed it sounds like it's not valid there
[15:43] <Saviq> tsdgeos, but OTOH, that job isn't really using the installed packages is it...
[15:44] <Saviq> fginther, hey, tsdgeos found a "race" in the -ci jobs:
[15:45] <Saviq> hmm no
[15:45] <Saviq> tsdgeos, hmm so the problem really is that we don't *know* the revision in qmluitests do we?
[15:46] <tsdgeos> i don't know i
[15:46] <tsdgeos> mean
[15:46] <tsdgeos> according to https://code.launchpad.net/~aacid/unity8/moreAsyncDash/+merge/241524/comments/595561
[15:46] <tsdgeos> it's 1391
[15:46] <tsdgeos> but qmluitest can't succeed in 1391
[15:46] <tsdgeos> so it's not 1391 for sure
[15:47] <tsdgeos> i'd expect if that comment says 1391 qmluitests would run on 1391 too
[15:47] <tsdgeos> and not on "newest of branch"
[15:47] <Saviq> tsdgeos, yeah I think the qmluitests job doesn't actually take the rev into account
[15:47] <Saviq> fginther, ↑
[15:51] <Saviq> mzanetti, mterry, has design ACKed the more opaque black overlay in lockscreens?
[15:52] <mzanetti> Saviq: yes, requested actually
[15:52] <Saviq> the numbers/icons are quite stark now on top of it
[15:52] <Saviq> ok
[16:16] <tsdgeos> mzanetti: ping
[16:17] <mzanetti> tsdgeos: pong
[16:17] <tsdgeos> mzanetti: seen the comment i made in your MR?
[16:17] <mzanetti> nope, not yet
[16:17] <mzanetti> looking
[16:19] <mzanetti> tsdgeos: taht one? https://code.launchpad.net/~mzanetti/unity8/disable-swipetoclose-while-schnapping/+merge/241691
[16:19] <tsdgeos> yep
[16:19] <mzanetti> tsdgeos: it's described in the linked bug
[16:19] <mzanetti> tsdgeos: basically you go to spread, then tap on an app
[16:20] <tsdgeos> yeah i tried and failed
[16:20] <tsdgeos> need extra explanation i guess :D
[16:20] <mzanetti> tsdgeos: and while its animating to front, quickly press and hold again
[16:20] <mzanetti> tsdgeos: then you should be able to drag it up/down, even though it's in foreground
[16:20] <mzanetti> without that branch obviously
[16:22] <tsdgeos> ah
[16:22] <tsdgeos> i did that
[16:22] <tsdgeos> but not smart me did not realize that was wrong ^_^
[16:22] <mzanetti> :)
[17:37] <rpadovani> Hey all o/ The previewWidget documentation says  particular widget types is likely to change with different major versions of Unity - I'm looking for the documentation for action widget for Unity8, where can I find it?
[17:37] <rpadovani> http://developer.ubuntu.com/api/scopes/sdk-14.10/unity.scopes.PreviewWidget
[17:47] <rpadovani> I need to close the preview of a scope when I click on the action, there is a simple way?
[17:48] <mzanetti> alecu: do you know if there's a way to do that? ^
[17:53] <dandrader> mzanetti, ping
[17:53] <mzanetti> dandrader: pong
[17:54] <dandrader> mzanetti, when an app is launched is slides in from the right edge. what controls/defines that animation?
[17:54]  * mzanetti opens code
[17:55] <mzanetti> dandrader: there's a Behavor on x around line 330
[18:22] <alecu> mzanetti: rpadovani: I don't exactly recall how to close the preview of the scope, but surely there's a way....
[18:22] <alecu> I need to be afk for a couple of hours, will try to find it when I get back and post it here.
[18:25] <mzanetti> thanks alecu
[18:30] <rpadovani> thanks alecu
[19:57] <kgunn> lpotter: hey on that Qnam bug, i & some other folks are hesitant on pushing fwd since some of the unit tests fail
[19:57] <kgunn> i saw you say they pass for you locally on the device
[19:57] <lpotter> that because there's no network manager on the test machine
[19:57] <lpotter> and those tests require at least one configuration
[19:58] <lpotter> and since we block the generic plugin if the network manager plugin is found, there is none.
[19:58] <kgunn> lpotter: so Mirv actually add the net man to the test machine
[19:58] <kgunn> https://bugs.launchpad.net/savilerow/+bug/1357321/comments/87
[20:00] <kgunn> ...altho...
[20:00] <kgunn> not sure how many times he ran it
[20:00] <lpotter> hmmm... tests pass both on desktop and phone
[20:00] <kgunn> or if that is potentially a flaky test
[20:00] <lpotter> for me
[20:00] <kgunn> lpotter: the builders can be slooooowww
[20:01] <kgunn> lpotter: how long does it take to build/test on the device ?
[20:01] <lpotter> FAIL!  : tst_QNetworkConfiguration::comparison() 'configs.count()' returned FALSE. ()
[20:01] <lpotter> there's no good configurations
[20:02] <lpotter> building qt took quite a while. I usually just build the plugin and/or libQtNetwork
[20:08] <kgunn> lpotter: so are you saying it's actually the same prob (block generic plugin if network man plugin is found)
[20:08]  * kgunn is about to reveal how little he knows about all this :)
[20:10] <lpotter> well those tests are written in that they assume the generic plugin is loaded.. i.e. they dont actually test any plugins. and there's no working network manager on the test machines, but ya, if that patch that excludes generic plugin were taken out, they would pass
[20:15] <kgunn> lpotter: just for my education, patch that excludes the generic plugin is needed in order to avoid the connect/disconnect issue ?
[20:16] <lpotter> it's to make sure QNAM doesn't use the generic plugin as defaultConfiguration. It's either that patch, or do not ship the generic plugin
[20:24] <kgunn> lpotter: got it, architecturally, its a single plugin only
[20:33] <kgunn> lpotter: so is it right to just disable those tests? (that feels a bit naughty)
[21:11] <lpotter> we could certainly skip any specific tests that fail in that situation
[21:22] <kgunn> Mirv: ^