/srv/irclogs.ubuntu.com/2014/12/01/#ubuntu-unity.txt

tsdgeosSaviq: are we going to land stuff soon ? https://code.launchpad.net/~unity-team/unity8/trunk/+activereviews is getting crowded again09:21
Saviqtsdgeos, yeah, I'll prep a silo today09:21
tsdgeos:)09:21
=== vrruiz_ is now known as rvr
Saviqpstolowski, hey, you had a silo for bottom edge in dash didn't you?09:53
Saviqdamn mterry has all the tags again09:54
mzanetti:)09:54
Saviqpstolowski, hmm or was it kgunn's silo09:55
Saviqin any case, what's the status there from your PoV?09:55
pstolowskiSaviq, I had last week, but had to pull that feature off for a moment from that silo10:08
Saviqpstolowski, ok10:08
pstolowskiSaviq, because of the issues with search (and we had to unblock and land other stuff)10:08
Saviqpstolowski, I'm landing unity8 soon, but without the bottom menu then10:08
pstolowskiSaviq, we're waiting for green light fro search changes from design today. when we get it, i'll request a new silo10:09
pstolowskiSaviq, ok, sure10:09
Saviqpstolowski, yup, I should be hopefully done by then10:09
=== dandrader is now known as dandrader|afk
=== greyback_ is now known as greyback
tsdgeosCimi: https://code.launchpad.net/~cimi/unity8/fix-1368778-2/+merge/242942 ?11:10
tsdgeosmzanetti: why are you marked as reviewer in https://code.launchpad.net/~mzanetti/unity8/spread-opacity-changes/+merge/242915 ?11:10
=== _salem is now known as salem_
tsdgeosMacSlow: how is https://code.launchpad.net/~macslow/unity8/swipe-dismiss-snap-decisions/+merge/233347 going?11:10
mzanettitsdgeos: wrong click in launchpad... I wanted to "request another review" but accidentally clicked on "claim review"11:11
tsdgeosmzanetti: ah, ok11:11
tsdgeosi'll top approve then11:11
mzanettitsdgeos: cool, thanks11:11
MacSlowtsdgeos, almost...I'll poke you when it's ready11:13
tsdgeosMacSlow: cool :)11:13
tsdgeosSaviq: why didn't CI run in https://code.launchpad.net/~nick-dedekind/unity8/lp1385331.led/+merge/241417 ?11:14
Saviqtsdgeos, good queston11:14
Saviq+i11:15
* Saviq needs to talk to fginther11:15
tsdgeosSaviq: if you have the vpn setup can you trigger it?11:15
Saviqtsdgeos, yeah doing11:15
tsdgeostx11:15
Saviqtsdgeos, ah, think I found the reason...11:15
Saviqtsdgeos, jobs are blocked for 2 days now11:15
tsdgeosoh11:15
Saviqon autopilot11:16
tsdgeosall to the red!11:16
Saviqyeah11:16
tsdgeosthat's bad :/11:16
=== dandrader|afk is now known as dandrader
tsdgeosSaviq: so what do you want to do with https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1384776 ? The root cause is fixed in Qt but we can still make it "better" (i.e. less dbus handling) with https://code.launchpad.net/~aacid/unity8/networkingstatus in our side11:29
ubot5Launchpad bug 1384776 in unity8 (Ubuntu) "Occasional hang in unity 8 dash on the phone" [High,In progress]11:29
tsdgeoswant to keep it open or just mark it as invalid?11:29
tsdgeossince it isn't really an unity8 bug?11:29
Saviqtsdgeos, invalidated11:29
facundobatistaHolas11:29
tsdgeosSaviq: oki11:30
tsdgeosSaviq: hola hola11:30
Saviqtsdgeos, ?11:31
tsdgeoser11:31
tsdgeosfacundobatista: hola hola11:31
tsdgeosSaviq: wrong mind autocompletion11:31
Saviqtsdgeos, "hola hola" is ~"waaaait a minute" in PL ;D11:31
facundobatistatsdgeos, :)11:31
tsdgeosSaviq: really?11:31
tsdgeoslol11:31
Saviqtsdgeos, yeah, it's a sound you'd make when someone would get ahead of themselves11:32
Saviqnot a real word I think11:32
Saviqactually it is, means "stop doing that!" according to the dictionary11:33
alan_ggreyback: as promised I've been hacking at qtmir - would you take a look at https://code.launchpad.net/~alan-griffiths/qtmir/new-migrate-to-mir-Server-API/+merge/243177 when you have the time?11:36
greybackalan_g: certainly11:37
mzanettiSaviq: hey, seeing you unassign RTM targets from yourself, should I stop assigning them to you?11:38
Saviqmzanetti, yeah, someone else might be landing after all11:39
mzanettiok11:39
Saviqmzanetti, I'll be leaving them New until we know when to land, Triaged when they have a milestone into RTM, and assign them, and make In Progress, when I actually want to silo them11:39
mzanettiok11:39
Cimitsdgeos, fixed11:43
tsdgeosCimi: ok11:44
mzanettidandrader: what's the ETA for the rotation stuff?11:45
dandradermzanetti, I guess late next week.11:50
dandradermzanetti, code is pretty much done. but the last 3% can be slow. like coordinating changes with SDK11:52
tsdgeosCimi: why all that commenting code?11:52
Cimitsdgeos, because mzanetti says we will need something similar for desktop11:52
mzanettidandrader: well, I hoped that desktop stuff would have landed a week ago... by now it's so many branches chained that I stopped working on it because it's becoming a pain11:52
mzanettiSaviq: is there something that prevents us from landing the desktop stuff?11:53
Cimitsdgeos, we have option of making the timer do nothing (but still running it), or comment code11:53
Saviqmzanetti, they're not reviewed? ;)11:53
mzanettiSaviq: well, the first one is, by gerry11:53
* greyback gonna have a review day, so can look at them11:54
Saviqmzanetti, not acked though https://code.launchpad.net/~mzanetti/unity8/desktop-stage/+merge/24214011:54
greybackmzanetti: ^^11:54
mzanettigreyback: cool11:54
Saviqmzanetti, one thing I'd add there - a minimum GU size for desktop stage11:55
Saviqmzanetti, not sure we want desktop stage on phone even if you enable it...11:55
greybackbut but convergence!11:56
Saviqmaybe we do, after all The Hypothesis Generator™ will take it into account11:56
mzanettiSaviq: yeah... but that's the kind of stuff that will change a lot...11:56
mzanettiexactly the HG will come into place11:56
mzanettifor now I'd say lets keep disruptive changes in Shell.qml as small as possible11:56
Saviqkk11:56
mzanetti(imho)11:56
Saviqmzanetti, you know, though, we'll need this all to become a single stage in the end? for nice transitions between modes? ;)11:57
mzanettiI was thinking about that yeah...11:57
mzanettireally not sure how to do that yet11:58
Saviqseb128, I just grepped through /usr for Ubuntu.Connectivity, looks like you guys are using it but not depending on it11:58
Saviqand we just dropped our dep11:58
=== MacSlow is now known as MacSlow|lunch
Saviqmzanetti, unity8.application_lifecycle.tests.test_application_lifecycle.ApplicationLifecycleTests.test_app_moves_from_unfocused_to_focused fails with your spread changes (I expect the spread threshold change, as it's getting into spread when it shouldn't)12:06
Saviqsilo 3 for reference12:06
mzanettiack, will check12:07
Saviqtsdgeos, small one for you https://code.launchpad.net/~saviq/ubuntu-system-settings/add-connectivity-dep/+merge/24327312:09
Saviqtsdgeos, but also, unity8.shell.tests.test_emulators.GenericScopeViewEmulatorTestCase.test_open_preview fails12:10
Saviqtsdgeos, it seems with all your branches it takes a really long time for the dash to load the fake scopes12:10
Saviqmzanetti, tsdgeos, it's all in silo 3 for reference12:10
* Saviq is running qml tests on it then12:10
Saviqhttp://people.canonical.com/~platform/citrain_dashboard/#?distro=ubuntu&q=landing-00312:11
mzanettitsdgeos: sorry, vesar just found an issue with this: https://code.launchpad.net/~mzanetti/unity8/spread-opacity-changes/+merge/24291512:11
tsdgeosSaviq: that's autopilot, right?12:11
mzanettitsdgeos: I fixed it already, but requires another approval I guess12:11
Saviqtsdgeos, yes12:11
tsdgeosSaviq: doesn't make sense it takes more time, but i'll check12:11
Saviqtsdgeos, yeah, I know, but I can see it sit there for a really long time12:12
tsdgeosok12:14
tsdgeosmzanetti: pushed?12:15
mzanettitsdgeos: I'd say yes12:15
tsdgeosmzanetti: don't se it in the mr page12:15
mzanettitsdgeos: rev 145212:15
mzanettitsdgeos: diff looks good too12:16
mzanettinot sure why it doesn't show up in the comments history12:16
Saviqtsdgeos, looks like the failure first showed up in https://code.launchpad.net/~aacid/unity8/photoscopeimprovements/+merge/23983412:16
Saviqtsdgeos, CI phones are dead until someone goes there and reflashes them, so for another few hours still12:16
tsdgeosok12:16
Saviqtsdgeos, yeah, same failure I got https://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-vivid-mako/334/testReport/junit/unity8.shell.tests.test_emulators/GenericScopeViewEmulatorTestCase/test_open_preview_Nexus_4_/12:17
tsdgeosoki, will check12:17
Saviqwe need more reliable AP tests, both of the tests I just saw fail failed in CI, too :?12:21
Saviq/food12:21
tsdgeosSaviq: approved https://code.launchpad.net/~saviq/ubuntu-system-settings/add-connectivity-dep/+merge/24327312:21
Saviqtsdgeos, tx12:21
SaviqCimi, hey, could you help dandrader and greyback testing shell rotation today?12:35
Saviqdandrader, greyback, what device btw?12:35
greybackSaviq: phone for first pass12:35
greybackthen tablet after, N7, then N10 is possible12:36
Saviqgreyback, maybe we can parallelize then12:38
greybacksure, if anyone else available12:39
SaviqMacSlow|lunch, when you swallow, could you help the guys test shell rotation ↑12:41
Saviqhttps://wiki.ubuntu.com/Unity8/FullShellRotation12:41
MacSlow|lunchSaviq, noted12:42
seb128Saviq, hey, could be, can you open a bug about that?12:44
Saviqseb128, already building in my silo, want a bug nevertheless?12:45
seb128Saviq, no, it was for tracking, you have a mp coming that works too ;-)12:45
Saviqseb128, https://code.launchpad.net/~saviq/ubuntu-system-settings/add-connectivity-dep/+merge/24327312:45
seb128Saviq, why did you guys stop depending on it? Is it buggy/should we stop using it as well?12:46
Saviqtsdgeos, ↑12:47
tsdgeoswe did and then we did not12:47
Saviqseb128, ah no, we just didn't need it12:47
tsdgeoslunch calling, i'll give a longer answer later :D12:47
* Saviq conflates it with QNAM stil12:47
Saviql12:47
Saviqmzanetti, testPhoneStage fails for me here, too12:51
mzanettireally12:52
Saviqin test_enterSpread12:52
Saviqand test_selectAppFromSpread12:52
mzanettistrange... I did test/fix that12:52
mzanettiwill check again12:52
CimiSaviq, sure12:52
Saviqmzanetti, it passed fine in jenkins, at least a while ago12:53
Saviqmzanetti, but it might be some interaction with the other MPs12:54
Saviqalthough nothing seems related12:54
* Saviq tries trunk + reversible12:55
Saviqmzanetti, yeah in ↑I get 4 failures12:57
Saviq5 from the silo12:57
mzanettiSaviq: ap test fixed. looking into qmltests now12:58
Saviqtx12:58
mzanettiTotals: 25 passed, 0 failed, 0 skipped12:59
mzanettiyeah, fails after merging13:00
Saviqmzanetti, aha, lp:~dandrader/unity8/fixTestCaseTouchFlick impacts that, too13:01
Saviqgot 6 fails now13:01
Saviqwith yours + ↑13:01
Saviqneed food13:01
=== MacSlow|lunch is now known as MacSlow
mzanettiSaviq: should all work again now13:23
Saviqmzanetti, okies13:43
tsdgeosseb128: Saviq: so we were using it for something, we don't need it anymore since we changed the feature, so it's gone, *but* it will be back once https://code.launchpad.net/~aacid/unity8/networkingstatus/+merge/239694 is merged13:51
seb128tsdgeos, ok, good to know, thanks13:51
Saviqwell, back but in a different way13:51
Saviqnot the qml module13:51
Saviqbut the Qt lib13:51
tsdgeosseb128: so no, it's not "bad", it's just that we're not using it for what we used to be because we don't do that anymore13:52
tsdgeosSaviq: true13:52
seb128k13:52
=== dandrader is now known as dandrader|afk
larsutsdgeos: is this the thing using the new nm property that desrt added?14:07
tsdgeoslarsu: i don't know14:08
larsutsdgeos: you proposed the branch, no?14:08
tsdgeoslarsu: yes, i am not the coder of libconnectivity-qt1-dev, just the user14:08
mterrymzanetti, hey, remember when you were looking at that branch that loaded the greeter async, and I had some weird problems with the infographic that were solved by an empty Connections object?  Where did we leave that?  Did you end up having any thoughts of where I should look?14:09
mzanettimterry: not really, sorry... I do agree something weird is going on, but couldn't figure what14:09
larsutsdgeos: hm, seems to use com.ubuntu.connectivity1...14:10
mterrymzanetti, I guess I should play with timing (which was what originally exposed the problem) and see if I can make it go away14:10
tsdgeosSaviq: is there a branch with all those branches in silo3?14:10
mzanettimterry: yeah... there are different ways of making it go away, still a bit unsatisfying that we've no idea what's going on14:11
larsupete-woods: what supplied com.ubuntu.connectivity1?14:11
larsusorry, *supplies14:11
Saviqtsdgeos, not a branch, but you can take the tarball from https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/landing-003/+sourcepub/4596048/+listing-archive-extra14:11
Saviqmterry, you got tags all over the place again14:11
mterrySaviq, ugh do you have a branch list?14:12
Saviqmterry, ./strip-tags.py `grep lp: | cut -d" " -f1`14:12
Saviqmterry, and paste the whole of https://code.launchpad.net/unity8/+activereviews14:13
Saviqmterry, and Ctrl+D14:13
mterryhah14:13
mzanettistill doesn't work for me :D14:13
Saviqmterry, but really, you must have them locally somewhere, just make sure you run the script on all your local checkouts too14:13
mzanettilook at that. it does work now14:15
mterrySaviq, yup.  Just sometimes it seems like the remotes ones hold on to tags even after I've cleaned things locally14:15
mterrySo I need to make sure I do a clean sweep of both at the same time14:15
Saviqmterry, they *always* do hang to the tags14:22
Saviqmterry, you have to run the strip tags script on lp:~foo14:22
=== jgdxx is now known as jgdx
pete-woodslarsu: I think that's the connectivity API14:26
pete-woodslarsu: maintained by Wellark14:26
larsupete-woods: no, that just uses it. Looks like indicator-network (of all places?) supplies it14:26
larsupete-woods: I wonder why it's not using the property desrt added to network-manager14:27
pete-woodslarsu: maybe it should? maybe Wellark doesn't know about this new property? maybe it's not what he needed? (I have no context here)14:27
larsupete-woods: me neither. Trying to find out right now. Just pinged you because you wrote connectivity-api, but that seems to be the qt wrapper only14:28
larsupete-woods: I thought we talked to the relevant people in DC, but apparently it didn't trickly through or something14:29
pete-woodslarsu: I didn't write connectivity API. I generated some Qt bindings that the indictor secret-agent uses14:29
pete-woodsperhaps they are also used for connectivity API now14:29
larsupete-woods: ah, might be. Saw your name on launchpad on some page associated with it. Sorry :)14:30
pete-woodslarsu: yeah, I created the project on LP for Antti, as he was v. busy at that point14:30
larsu:)14:31
pete-woods:)14:31
mterrySaviq, fyi, all my branches are clean of tags now14:51
tedglarsu, I think that long term we won't be able to use that as we want to move applications off of talking to the system bus.14:55
tedglarsu, When we loose fine grained control of dbus with kdbus we won't be able to do confinement of specific attributes.14:56
larsutedg: it still talks to the bus via an API and we don't lose anything with kdbus, it's just that the service does the enforcing now instead of the daemon14:59
tedglarsu, No, the service does the enforcing not the security module… which is designed to enforce policy.15:03
tedglarsu, It does it via the daemon, but it can't in the future as there's not enough information available.15:04
larsutedg: what information is missing? The kernel supplies all the necessary bits, no?15:04
tedglarsu, Nope, the headers were removed as "redundant"15:05
larsutedg: I don't think that's true, but I also don't care enough to have this argument now15:05
=== karni is now known as karni-afk
tsdgeosSaviq: i can reproduce the problem, trying to find out if i can see why is it so much slower in autopilot or at least just make it pass15:26
=== dandrader|afk is now known as dandrader
=== alan_g is now known as alan_g|tea
attente_mzanetti: hi15:46
attente_mzanetti: how did you implement this (window positioning)? https://www.youtube.com/watch?v=MXHulRlq10s, is it using the mir client api?15:47
mzanettiattente_: it's using that api through qtmir15:49
mzanettiso in the end it's just moving QtQuick items around15:49
attente_mzanetti: how does qtmir tell mir where to place the surfaces? is there a way?15:52
attente_mzanetti: also, could you point me to the branch where your window management code is?15:52
tsdgeosSaviq: i pushed a change that makes the test pass15:53
tsdgeosi'll now revert to trunk and see if it's really slower or just more async (and that's why i need wait_select_single instead of select_single)15:53
=== karni-afk is now known as karni
mzanettiattente_: this is the branch in unity8: https://code.launchpad.net/~mzanetti/unity8/desktop-stage15:57
mzanettiattente_: and this is the qtmir code, the one that does the interaction with mir: https://code.launchpad.net/~mir-team/qtmir/trunk15:57
attente_mzanetti: thanks! i'll take a look15:59
mzanettiattente_: so what happens is that qtmir maps the client buffer to a QQuickItem16:02
mzanettiattente_: based on the position and size of that QQuickItem in the shell, qtmir tells Mir where to draw the buffer16:02
mzanettimaybe even draws it on its own, not exactly sure16:02
attente_mzanetti: but there isn't any client api for specifying the position of the surface is there?16:03
mzanettiattente_: ah, hmm... not sure if we support that yet. definitely not up to the unity8 level yet. it's on our todo16:05
mzanettiattente_: the plan is to use the toolkit's apis. so in Qt you would resize your QApplicationWindow (or what it's called)16:05
attente_mzanetti: oh. forgot to mention, we're need to implement this in the gdk-mir backend :)16:06
mzanettiattente_: right... so I don't know much about gdk, but I assume there is some sort of MainWindow class there too16:07
mzanettigreyback: do you know about the state of the client positioning/sizing api?16:08
attente_mzanetti: right, but we can't implement it at all if we don't have a way to position surfaces in the client api, no?16:08
mzanettiattente_: good question. I don't know about the state of that in Mir. I'm mostly looking at it from the shell perspective. You might want to ask in #ubuntu-mir about the client side apis16:09
greybackattente_: Mir will not allow clients to position themselves in general. But it will allow clients to request position & dimensions for a surface on creation, but that's not implemented yet unfortunately16:09
greybackit will be up to the shell to decide whether to do what the client asks or not.16:10
attente_greyback: so if a gtk app asks to set the position of the window, we have to request it from the shell?16:11
greybackattente_: you can ask for x:100, y:200. The shell might honor it, it might not, depending on its policy (e.g. on a phone, those coordinates would not be respected)16:12
mzanettiI guess you still request it from Mir, that one will forward it to the shell and the shell then does it (or not)16:12
greybackattente_: for menus, shell is very likely to respect the position16:12
greybackbut that position will be relative to the parent surface16:12
=== alan_g|tea is now known as alan_g
dandradermzanetti, I'm purging that ApplicationManager.ExecFlags in the shellRotation branches as it's no longer needed (as we now have separate Phone and Tablet stages)17:21
mzanettidandrader: +117:21
mzanettidandrader: I think qtmir shouldn't on which stage some surface is17:22
mzanettidandrader: I think qtmir shouldn't know on which stage some surface is17:22
mzanettior at least it shouldn't care about it17:22
mzanettimaybe for convenience some flag is to be stored in there, but it shouldn't actually do anything with it17:22
mzanettidandrader: greyback is purging focus handling from there currently, you might want to sync with him as he probably killed that flag too17:23
dandradergreyback, hmmm, do you plan to land that branch before shellRotation?17:24
greybackdandrader: after17:24
greybackdandrader: if you're making changes that aren't essential to shell rotation, please keep them in separate branches to minimize diffs17:25
dandradergreyback, there are quite a few cases like this in the shellRotation branches17:28
greybackdandrader: ok well use your judgement17:28
dandradergreyback, I would like to put them all in separate MPs17:28
dandradergreyback, but the work involved in creating and testing them all is just too much17:28
dandradertoo time consuming17:28
=== dandrader is now known as dandrader|afk
greybackdandrader: sure I see that. But just consider the reviewers17:31
=== greyback is now known as greyback|afk
=== dandrader|afk is now known as dandrader
=== alan_g is now known as alan_g|EOD
cwayneblargh, after installing fglrx my cursor is huge -- changing /org/gnome/desktop/interface/cursor-size doesnt seem to stick..19:08
cwayneanyone have any idea why setting a setting in dconf-editor wouldn't stick across reboots?19:45
cwayneTrevinho: ping19:58
cwaynebregma: also ping20:03
davmor2cwayne: it hates you?20:10
davmor2cwayne: check your resolution in the catalyst admin app20:11
=== broder__ is now known as broder
bregmacwayne, no idea, dconf is dconf20:11
cwayneso there's something weird re: scaling and my cursor20:12
cwayneif im in chrome/pidgin its one size, then as soon as i go to the launcher, cursor gets huge20:13
cwaynedriving me completely insane20:13
Saviqmterry, all your local checkouts, too? :)21:16
mterrySaviq, yes  :)21:16
Saviqmterry, coolz21:18
=== Pici is now known as Guest74903
=== Zhenech_ is now known as Zhenech
=== Trevinho_ is now known as Trevinho
=== robru_ is now known as robru
=== salem_ is now known as _salem

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