mterrykdub, poke about libhybris00:10
kdubhello mterry i did run it just now on my nexus 7, seems to work when you force the fallback mode00:11
mterrykdub, awesome!  So no objection to a saucy upload?00:11
kdubwell, not from me... but I'm not sure what the hybris landing requirements are exactly :)00:12
kdubthe n7 hwc hal doesnt hybrisize well, but the rest of the driver looked ok00:16
=== _salem is now known as salem_
=== salem_ is now known as _salem
=== mmrazik is now known as mmrazik|afk
=== mmrazik|afk is now known as mmrazik
=== alan_g is now known as alan_g|afk
=== alan_g|afk is now known as alan_g
mzanettigreyback: hey08:15
greybackmzanetti: hey ho08:16
mzanettigreyback: is your branch to be reviewed or you still fighting the notes-app?08:16
greybackmzanetti: still fighting08:16
mzanettigreyback: ok. ping me when you want it reviewed08:16
greybackmzanetti: will do08:16
nic-doffaySaviq, as soon as my sdk branch lands I'll have the orientation unity branch up for review.08:24
nic-doffayIt's been approved so it should be shortly.08:24
Saviqnic-doffay, ok cool, did the performance improve when you applied those tweaks?08:25
nic-doffaySaviq, sadly not.08:25
Saviqnic-doffay, means there's probably more to be done08:25
nic-doffaySaviq, agreed.08:25
nic-doffayThe review atm is a work in progress.08:25
Saviqnic-doffay, k08:26
nic-doffayIf you wouldn't mind just leaving your comments then I'll get to them.08:26
seb128MacSlow, hey08:32
MacSlowseb128, hey there08:37
seb128MacSlow, hey, how are you?08:38
seb128MacSlow, did you see https://code.launchpad.net/~larsu/notify-osd/fix-1189281/+merge/168550 ? can you review/approve it if you are happy with it? that fixes the segfault in saucy08:38
MacSlowseb128, looking at Jenkins' console output...08:42
seb128MacSlow, the builds are fine, it has coverity errors but that's not new and it's complaining about the missing commit message08:44
seb128MacSlow, I've tested that patch locally and confirmed it fixes the segfault if that's worth anything08:44
MacSlowseb128, the change itself looks ok... I just wonder if it passes "make check"08:45
seb128MacSlow, it does locally here08:48
Saviqnic-doffay, please don't self-approve08:49
sil2100larsu: ping08:49
Saviqnic-doffay, especially since CI failed08:49
nic-doffaySaviq, k08:50
nic-doffaySaviq, who else should I get to review the branch?08:52
MacSlowseb128, on my Saucy machine it does not... looking into it08:53
seb128MacSlow, what is wrong?08:55
MacSlowseb128, make check just hangs there after the second line of "make check-TESTS"08:57
seb128MacSlow, it took like 30s here and I started getting notifications08:57
MacSlowseb128, that doesn't happen on raring08:58
seb128MacSlow, does it happen on saucy before lars' patch?08:58
MacSlowseb128, just trying08:58
Saviqmzanetti, btw, did you read michi's answer? it's a lot of what I would write08:59
mzanettiSaviq: half way through. got distracted but will continue reading now08:59
Saviqmzanetti, k08:59
Saviqmzanetti, I just have some things to add re: potential style exception for obviously-Qt code and some QML things that I will send to discuss on the ML09:00
MacSlowseb128, ah... well then... this 30 sec period happens with and without Lars' patch on saucy... still odd09:01
MacSlowseb128, but I'm ok with the patch09:02
seb128MacSlow, thanks, can you ack it with a commit message then please? ;-)09:02
MacSlowseb128, what about the coverity-failure?09:05
seb128MacSlow, those are not new either, we should fix it but not block the segfault fix on that imho09:05
MacSlowseb128, I'll set the status to "Approved" manually to speed things up09:16
seb128MacSlow, I just did it09:16
seb128MacSlow, you have to set it manually anyway, jenkins doesn't do that for you09:17
nic-doffaySaviq, that branch should be landing now. Bug in launchpad was causing jenkins to fail.09:29
Saviqnic-doffay, yes, saw it09:30
greybackmzanetti: hey, any tips on reproducing a segv that jenkins gets, on your local machine? In theory a raring pbuilder would be enough, no?09:40
mzanettigreyback: yes... probably ptest is the easiest way09:41
greybackmzanetti: ok thanks09:41
mzanettigreyback: when executing qmluitests?09:41
greybackmzanetti: yeah09:41
sil2100bregma, dednick: hi!09:41
greybackmzanetti: my StageManager tests segv, apparantly before even managing to run one test. I'm not sure why09:42
sil2100bregma, dednick: can you take a look at the recent unity failures in jenkins?09:42
sil2100It's over the threshold right now...09:42
=== alan_g is now known as alan_g|tea
nic-doffaySaviq, in the mean time any idea why you think the orientation is so slow? So I can work on making it quicker.09:43
sil2100Trevinho: ^09:46
dednicksil2100: yo09:46
Saviqnic-doffay, as I said, everything is being laid out again on every frame09:46
dednicksil2100: can you post a link?09:46
dednicksil2100: bregma probably wont be on for awhile09:47
Saviqnic-doffay, because the width/height changes09:47
sil2100dednick: pasted09:47
sil2100Ok, I go lie down now for sure09:47
Saviqnic-doffay, so we need to be smart and do some tricks09:48
dednicksil2100: looks pretty bawked across the board there.09:48
Saviqnic-doffay, I'd need a deeper look, but first of all: nothing that's not on screen (e.g. the dash when there's an app focused or the phone is locked) should animate09:50
Saviqnic-doffay, that's why you added the property on OrientationHelper09:50
Saviqnic-doffay, so that you can do some smart things while rotating09:50
Saviqnic-doffay, I don't have ready-made answers, though09:51
Saviqnic-doffay, it's probably going to be a combination of changing anchors, layering and visibility that will be the solution09:52
Saviqnic-doffay, for example, the panel and launcher should probably not even be inside the Helper09:52
dednicksil2100: dash failures seem to be caused by 'super' not showing dash. Same with hud not shoing with 'alt'.09:52
Saviqnic-doffay, and instead they should go offscreen and hide, wait for the rotation to finish, and come back on screen then (the launcher should stay out, too)09:53
Saviqnic-doffay, but we won't get there in a single go, most probably09:53
dednicksil2100: but have no idea if that's a random failure of keymappings or a new saucy issue.09:53
Saviqnic-doffay, we'll go with something initially and then expand on it later09:53
nic-doffaySaviq, wouldn't making it invisible accomplish the same thing as putting them off screen?09:55
Saviqnic-doffay, not visually09:55
Saviqnic-doffay, IMO the best visual experience would be to slide them out09:56
Saviqnic-doffay, and then make invisible09:56
nic-doffaySaviq, ah right, I assumed it was just for speed.09:56
nic-doffaySaviq, from the left?09:56
nic-doffaySaviq, make them invisible then slide them out after?09:56
Saviqnic-doffay, then rotate/re-anchor when rotating = false, make visible and slide the panel back in09:57
Saviqnic-doffay, if you make them invisible first then they'd just disappear in a single frame, we need an animation09:57
nic-doffaySaviq, yeah. So basically: Step 1 - trigger slide out animation on panel and launcher. Step 2 - rotate/re-anchor when panel and launcher have finished animation. Step 3 - trigger slide in animation on panel and launcher once rotation is done.09:59
Saviqnic-doffay, 2 - rotate/re-anchor when OrientationHelper.rotating becomes false09:59
Saviqnic-doffay, 3 - only slide the panel in, not the launcher09:59
Saviqnic-doffay, even if it was on screen, we don't want to bring it back straight away10:00
nic-doffaySaviq, ah I see you mean rotate/re-anchor the panel and the launcher, correct?10:00
=== greyback is now known as greyback|shops
nic-doffaySaviq, I'm going to start this now. Are there existing animations I should be using?10:00
Saviqnic-doffay, they're not fast enough, probably10:01
Saviqnic-doffay, actually that made me look10:01
Saviqnic-doffay, OrientationHelper doesn't expose the current angle10:01
Saviqnic-doffay, http://bazaar.launchpad.net/~ubuntu-sdk-team/ubuntu-ui-toolkit/trunk/view/head:/modules/Ubuntu/Components/OrientationHelper.qml#L6010:02
nic-doffaySaviq, isn't that what it's doing on line 60?10:02
Saviqnic-doffay, but it's not exposed10:02
Saviqnic-doffay, the __10:02
nic-doffaySaviq, ah I see.10:03
Saviqnic-doffay, you could directly alk to Screen10:03
Saviqnic-doffay, but I think it's better to make sure you're in sync with the OrientationHelper10:03
nic-doffaySaviq, I agree, the int is there already, might as well use it.10:03
nic-doffaySo I guess this needs another modification.10:03
Saviqnic-doffay, yeah, I un-approved https://code.launchpad.net/~nicolas-doffay/ubuntu-ui-toolkit/orientation-helper-anim-alias/+merge/16848910:04
Saviqnic-doffay, please include it there10:04
nic-doffaySaviq, cool.10:04
=== alan_g|tea is now known as alan_g
=== om26er is now known as om26er|away
* greyback going out for lunch, bbiab11:00
dednickSaviq: ping.11:07
Saviqdednick, pong11:07
dednickSaviq: https://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests/1236/console. Is this because haven't got a indicator-client package in debian/control ?11:07
Saviqdednick, in debian/*install, rather11:09
Saviqdednick, we need indicators-client for autopilot tests, do we?11:10
dednickSaviq: yes. at the moment anyway.11:10
Saviqdednick, we should probably package it separately11:10
Saviqdednick, so we don't install it (nor the /usr/share/indicators files) with unity811:11
dednickSaviq: i thought it was, apparently i'm doing it wrong though11:11
Saviqdednick, a completely new package, on which unity8-autopilot depends is probably best11:11
Saviqdednick, other than that you need to install plugins/IndicatorsClient somewhere11:12
Saviqdednick, although it should be "modules" if there's qml files in there11:12
Saviqdednick, plugin == C++ only, import == QML only, module == C++ and QML11:12
dednickSaviq: ic11:13
=== mmrazik is now known as mmrazik|lunch
Saviqdednick, there's the fake-env package that we might want to use11:16
Saviqdednick, but also, we should probably not install anything temporary in /usr/share, we should set XDG_DATA_DIR instead to some custom prefix where we need it11:17
dednickSaviq: fake-env?11:17
Saviqdednick, there's a unity8-fake-env package11:17
Saviqdednick, that includes the LightDM mocks, for example, and which unity8-autopilot depends on11:18
=== MacSlow is now known as MacSlow|lunch
nic-doffaySaviq, is it safe to move the entire overlay out the orientationHelper?11:20
Saviqnic-doffay, what overlay?11:21
Saviqnic-doffay, but anyway, yes, the OHelper is just a rectangle that rotates with a transition11:21
Saviqnic-doffay, so I see no reason why not11:21
nic-doffaySaviq,  just to clarify the Item with the overlay id in Shell.qml.11:23
nic-doffayNone of the components in there are meant to rotate?11:24
Saviqnic-doffay, good question11:24
nic-doffayjust want to check if I'm missing something.11:24
Saviqnic-doffay, I think the only question there is the hud11:24
Saviqnic-doffay, but I think we should be fine with dismissing the hud when you rotate11:24
Saviqat least until someone tells us otherwise11:24
nic-doffaySaviq, ok great.11:24
Saviqnic-doffay, there should be no reason to animate all the Stages, too11:25
Saviqnic-doffay, right now the SideStage app doesn't move anyway11:26
Saviqnic-doffay, uh it's tricky on the tablet...11:27
Saviqnic-doffay, since the apps don't move, just rotate in-place11:27
Saviqnic-doffay, so we suddenly have a BottomStage...11:27
Saviqor TopStage, depending on how you look at it11:27
Saviqnic-doffay, and that's something I don't have an answer for atm11:29
=== s1aden is now known as sladen
nic-doffaySaviq, I'll see what it looks like.11:30
=== mmrazik|lunch is now known as mmrazik
=== mzanetti is now known as mzanetti|lunch
nic-doffaySaviq, I've moved what I currently can out the Orientation Helper. I'm going to start working on what we discussed earlier then we can have a further look at what more could be done.11:54
nic-doffaySound reasonable?11:54
Saviqnic-doffay, sure11:54
dednickSaviq: so with the IndicatorsClient module, does that need to be in a separate source folder then? ie. not in plugins, or is it just the install folder that should be modules/IndicatorsClient?12:08
Saviqdednick, separate source, please12:09
dednickSaviq: ok12:09
TrevinhoCimi: I think I finally found what was wrong in libbamf...12:15
TrevinhoCimi: can you give me the bug #?12:16
=== MacSlow|lunch is now known as MacSlow
Saviqpaulliu, what's the difference between (< 7.80) and (<< 7.80)?12:32
paulliuSaviq: < is obsolete. And < means <= IIRC. Better to only use <= or <<12:33
Saviqpaulliu, good to know, thanks12:33
larsusil2100: hey12:41
=== mzanetti|lunch is now known as mzanetti
dandradermzanetti, ping12:46
mzanettihi dandrader12:47
dandraderwould you have time to review this? https://code.launchpad.net/~dandrader/unity/8_directionalDragMinSpeed/+merge/16813612:47
=== _salem is now known as salem_
mzanettidandrader: ok. doing that now12:47
dandradermzanetti, thanks!12:54
=== alan_g is now known as alan_g|lunch
=== boiko_ is now known as boiko
=== jhodapp is now known as jhodapp|afk
Saviqmzanetti, standup13:32
Saviqdednick, nic-doffay, standup13:33
Cimilp:ubuntu-settings-components dednick13:37
greybacksorry, something gone wrong on my machine, trying to re-join13:38
mzanettiCimi: btw... was my review on the calendar tests helpful? Or do you want me to explain something more in depth?13:43
Cimimzanetti, I have a question for you on the first test13:44
mzanettiCimi: hit me13:44
Cimimzanetti, the others got changes13:44
Cimimzanetti, I don't know what to check for the compressed state13:44
Cimimzanetti, the dimension isn't really 6 times13:44
Cimithere's padding etc13:44
mzanettiCimi: ah, I see...13:45
mzanettiCimi: is it too complicated to quickly sum up the padding in the test code?13:45
mzanettiCimi: something like compare(newHeight, "oldHeight * 6 + spacing * 5")13:45
Cimiold height contains padding13:46
Cimiso it's not that immediate13:46
mzanettiCimi: sure... I don't have the code here, but you get the idea...13:46
Cimiunless you change code just for that13:46
mzanettiCimi: if thats not feasible, I'd probably go for something like this:13:47
Cimiwhich is then useless imho, at this point you don't test at all13:47
mzanettiCimi: verify(newHeight > oldHeight * 5)13:47
mzanettiCimi: so the test verifies that it actually expads... if there are some elements in the expanding area that change visible to true while expanded you could check that too in the test13:48
Cimiah ok13:48
Cimiwill have it a go13:48
=== alan_g|lunch is now known as alan_g
mzanettiwhats the minimum battery level to flash the phone?13:49
sil2100larsu: heya13:50
sil2100larsu: so, we have some HUD bugs that are blocking releases, and tedg was working on those yesterday - he put up 2 merges that need reviewing13:51
sil2100larsu: did you take a look at those already?13:51
dednickCimi: have you integrated those components into any of the indicators yet?13:53
larsusil2100: no, I don't usually review hud stuff - I'm not very familiar with the code13:53
Cimidednick, nope13:53
Cimidednick, it's not my task13:53
Cimiat the moment13:53
Ciminow I am looking into theming13:53
dednickCimi: sure.13:53
larsusil2100: is there noone else that could look at them?13:53
dednickCimi: what you need me to review? pretty much everything there?13:55
Cimidednick, yeah13:55
Cimidednick, it's more an easy review, like having a look and tell me if you don't like things13:55
sil2100larsu: hmm, not sure, let me ask pete-woods if he could take a look13:55
sil2100pete-woods: ping13:55
dednickCimi: ok. you had the design guys go over it yet? or is that after me?13:56
sil2100dednick: ping14:10
dandraderSaviq, any estimates as to when unity8 will work normally (with all dashes) in saucy?14:11
Saviqdandrader, https://code.launchpad.net/~unity-team/unity/8.new-libunity/+merge/16773314:11
Saviqdandrader, so, this or early next week14:11
sil2100dednick: so, regarding those unity failures - is that the case for only one test-machine?14:12
sil2100dednick: or was that happening on both?14:12
dednicksil2100: looks like it was just the intel machine14:14
dednicknot sure what was happening on the ati box14:14
sil2100Still a bit too much, since it's 19 failures on ati14:18
sil2100Intel had 2214:18
sil2100dednick: from what I see, unity seems a bit broken14:19
sil2100Trevinho, andyrock: ping14:19
Trevinhosil2100: pong14:19
sil2100Are you very busy? ;)14:20
sil2100Trevinho, andyrock: since I would need some volunteers to join a quest of regression hunting14:20
pete-woodslarsu: hi, what's the change you want looking at?14:22
larsupete-woods: I don't know... sil2100 asked me :)14:22
pete-woodsoh, okay!14:23
pete-woodsping sil2100!14:23
Trevinhosil2100: mh, then?14:23
sil2100pete-woods: hi!14:24
pete-woodssil2100: I understand you want a MR looking at or something like that?14:24
sil2100Trevinho: since the latest unity test run returned many failures, some seem to be related to hud issues, some to keybindings...14:24
sil2100pete-woods: yes ;) Since I think tedg made some hud fixes14:25
dandradermterry, a leftover from yesterday's merge proposal: https://code.launchpad.net/~dandrader/unity/8_stageDDACleanup/+merge/16867614:25
sil2100pete-woods: I remember you were familiar with the code, yes?14:25
pete-woodssil2100: somewhat!14:25
didrocksfginther: around?14:25
sil2100pete-woods: I see that one of the merges got reviewed and approved already, but the other one:14:25
fgintherdidrocks, morning14:25
didrocksfginther: hey!14:25
didrocksfginther: do you have a minute for https://code.launchpad.net/~didrocks/cupstream2distro-config/python-upa-rename/+merge/168677?14:26
fgintherdidrocks, certainly14:26
didrocksfginther: I think you need to redeploy something on your side as well :)14:26
tedgpete-woods, Wellark is looking at some, but there's a few.14:26
sil2100tedg: hi! Do the two merges fix the issues with 'no results returned' I poked you about yesterday?14:26
pete-woodssil2100: looks like Wellark beat me to that one14:26
tedgsil2100, The bamf-focus-fix one was the core issue, the others were things I found along the way.14:26
tedgStupid BAMF changes.14:27
* tedg is really excited to drop dbus-glib there14:27
tedgShould be *much* fewer BAMF issues in the future.14:27
sil2100tedg: \o/ so, once the bamf fix gets merged, you think it's safe to rebuild the hud stack and check the test results again?14:28
fgintherdidrocks, thanks for the heads up, I'll deploy on our side when the merge completes14:30
didrocksthanks fginther :)14:30
tedgsil2100, Yeah, since there's a few queued to land I'd wait for all of them though :-)14:31
* didrocks is happy that tests finds real regressions :)14:31
mzanettidandrader: hey, reading through the diff the right edge (Stage) should already use that minSpeed thing, right?14:33
dandradermzanetti, yes14:34
sil2100didrocks: those fixes should also resolve some unity problems, although there's still one issue in unity that just irritates me14:34
mzanettidandrader: on the device, if I press the right edge without moving, it behaves weird14:34
mzanettidandrader: it starts moving the app, then it releases the app and grabs it again, resulting in weird flickering14:35
dandradermzanetti, I'll check14:35
didrockssil2100: ok, good hunt! do we have stuff that we should publish (apart from QA where I'm running the build with the renamed component)14:36
=== jhodapp|afk is now known as jhodapp
sil2100didrocks: are we still blocked with apps on the copyright issues?14:39
didrockssil2100: no, everything's merged (same for mediaplayer in the media stack)14:41
didrockssil2100: I removed this morning the "force manual publishing"14:41
sil2100didrocks: ok, so I  think we can publish both apps and media, since they're passing nicely anyway - there have only been AP-related issues with those14:42
mzanettimterry: btw. I replied to your comments on the MR regarding the lockscreens14:42
didrockssil2100: so the stack deps on hud/indicators are not blocking and the packaging changes are good?14:42
sil2100didrocks: indicators, well, I need to check the vids again, since first it was an jenkins issue and now just some u-g-m things failed14:42
didrockssil2100: maybe you want me to have a second look on the packaging changes?14:42
mterrymzanetti, yup, will look again14:42
sil2100didrocks: I'll re-check the packages, one moment14:43
mzanettimterry: I don't really see why I should make it complicated and copy/paste the whole test suite just to use the single user backend it the full backend would fit exactly my needs14:43
mterrymzanetti, well...  I guess I didn't see why there was a has-pin user in full/ anyway.  Since you already have single-full/.  Not a problem per se, just didn't seem necessary, especially since for tablet testing with full/, we wouldn't be showing a PIN pad anyway.  I don't know what you mean by copy/pasting a whole test suite?14:46
sil2100didrocks: I only checked gallery-app now, but I already see a small copyright-packaging problem already14:46
sil2100didrocks: the .cmake files in cmake/, some have a different license than GPL-314:46
sil2100didrocks: as they're copyrighted by cmake guys and GPL-214:47
mterrymzanetti, ah I see.  You mean for the passphrase vs pin14:47
sil2100More of a nit-pick, but still14:47
didrockssil2100: we don't really care about build systems14:47
mzanettimterry: I run all the test functions twice (by using _data() functions)14:47
didrockssil2100: see my email from this morning on the new components :)14:47
mzanettimterry: and if I would need to run it with different LD_LIBRARY_PATHs I would need to rip it apart14:47
mzanetticopy/pasting the test functions14:47
* sil2100 now read that14:48
sil2100Oh, ok14:48
mterrymzanetti, so no tests that cover the Shell.qml bit?  It would be good to have an autopilot test maybe?14:48
mzanettimterry: definitely an autopilot thing, however, I don't think that code will stay in shell.qml for long, does it?14:49
sil2100Since Mirv was pointing out the cmake/ files so I was fixing those as asked14:49
sil2100didrocks: nevermind that then ;)14:49
mterrymzanetti, it will move to something similar, a GreeterShell.qml or whatever14:49
didrocksok, gallery-app sounds good to me14:49
mzanettimterry: I would suggest once we move the stuff away from unity8 into a real lightdm greeter we get together and write autopilot integration tests for that setup together14:50
didrockssil2100: so you can publish the app stack if there is no issue with the stack dep (just tell me so that I can NEW the apps right away)14:50
mzanettimterry: writing tests with all the shell objectNames etc would need to be replaced anyways in the next step14:50
mterrymzanetti, it's not so bad.  https://code.launchpad.net/~mterry/unity/8.split/+merge/168688 keeps most of the greeter tests intact14:51
mterrymzanetti, the journey for autopilot won't change, we'll just launch a different executable before testing a given test14:52
sil2100didrocks: ok, deps look good, I'll publish14:52
mzanettimterry: sure, but its not in Shell.qml any more?14:52
sil2100didrocks: will re-check media and publish as well14:52
mterrymzanetti, the greeter stuff is pulled into a separate GreeterShell.qml, at least in my branch14:53
didrockssil2100: hum, let me recheck the packaging diff (normally only people with upload rights should ack, but we'll get that fixed for you and Mirv by the end of cycle ;))14:53
sil2100didrocks: I'm still a bit inexperienced in that, some things I miss, but slowly getting a hang of it14:54
didrockssil2100: it's just that packaging diff should be check with people with upload rights (it's the contract with the release team)14:55
sil2100didrocks: give me a sign if it's green and I publish then ;)14:56
didrockssil2100: just checked it, the packaging diff looks good to me :)14:56
didrocks(but feel free to look at it as well and raise questions if any ;))14:56
sil2100Sickness calls, brb!14:56
nic-doffaySaviq, at the moment the OrientationHelper's states are called whenever the rotation angle changes. This is a core part of the OrientationHelper. Any ideas on how I should get these state changes to trigger after the launcher and the panel, both of which are located in Shell.qml?15:00
sil2100Ok, publishing!15:00
nic-doffayI mean the animating bit is easy, but there's a lot of code here that has been done without all of this in mind.15:00
mmrazikdidrocks: quick question regarding adding stuff to daily release -- I'm going to add https://launchpad.net/libusermetrics to the indicators stack (with daily_release: False). Is there any process to make sure you notice new projects and can (eventually) daily_release: True-them?15:00
Saviqnic-doffay, you mean to delay the OrientationHelper's rotation?15:02
nic-doffaySaviq, yeah15:03
didrocksmmrazik: no process, but this week, I'll get everyone looking at what's in the config with daily_release: False15:03
didrocksmmrazik: for the future, pings are enough :)15:03
mmrazikdidrocks: ok15:03
didrocksthanks mmrazik :)15:03
nic-doffaySaviq, it's tricky because it's a part of the SDK and technically it shouldn't be delayed.15:03
nic-doffayimo at least15:03
Saviqnic-doffay, it could have a configurable delay, though15:05
=== alan_g is now known as alan_g|tea
Saviqnic-doffay, 0 by default15:05
nic-doffaySaviq, that's true. How do you delay states though?15:05
nic-doffayAt the moment whenever the orientationAngle changes it toStrings to the state.15:05
Saviqnic-doffay, you don't, you add a PauseAnimation in the transition15:06
Saviqnic-doffay, but15:06
sil2100didrocks: apps and media stacks are being published, noticed a failure in the QA stack - should I restart it? Is it because of the renaming? (I see it's missing python-ubuntu-platform-api in the prepare step)15:06
Saviqnic-doffay, delaying would mean that the app's rotation gets out of sync with the shell rotation15:06
nic-doffaySaviq, exactly.15:06
Saviqnic-doffay, but then if we're talking that the panel should go away anyway15:06
nic-doffaySaviq, I'm mainly worried about the panel going away at the correct time.15:07
nic-doffayNot while the rotation is happening.15:07
nic-doffayWhich is the case atm.15:07
didrockssil2100: oh weird, I redeployed with that15:09
didrocks/var/log/upstart/otto-setup.log: E: Package 'python-ubuntu-platform-api' has no installation candidate15:11
didrockssil2100: seems different?15:11
didrocksah, it's armhf only15:11
didrockssil2100: I guess for now, I need to remove it15:12
didrocksfrom the packages to install15:12
dandraderSaviq, is run_on_device working fine for you on a freshly-flashed device(with phablet-flash and no /home/phabler/shell dir)?15:12
Saviqdandrader, checking15:13
=== alan_g|tea is now known as alan_g
didrockssil2100: not sure if autopilot-touch is armhf only. If not, that can block the migration to -proposed15:14
didrockssil2100: let's keep an eye on it15:14
dandraderSaviq, I get this http://paste.ubuntu.com/5755165/15:14
sil2100didrocks: ah, hm, indeed15:14
didrocksrestarted with "foo"15:14
Saviqdandrader, never saw that15:16
dandraderSaviq, so all worked fine for you now?15:16
Saviqdandrader, just checking, didn't flash yet, just dropped "shell"15:16
Saviqdandrader, but it went past that error for me15:17
Saviqdandrader, /me flashes15:17
Saviqnic-doffay, problem is that even if you delay the OHelper in the shell, the app won't know about it15:19
Saviqnic-doffay, so it will rotate regardless15:19
nic-doffaySaviq, yeah. I'm not sure how to get around all of this.15:19
Saviqnic-doffay, no way around it now, we can't talk to the app about this yet15:19
dandraderSaviq, where that build.ninja comes from?15:20
nic-doffaySaviq, right for the time being I'll just make the hide animation very quick.15:20
Saviqdandrader, ninja is the build system we switched to15:20
Saviqnic-doffay, yup, SnapDuration15:20
nic-doffayAlso, who is an expert on the panel?15:20
Saviqnic-doffay, dednick15:20
nic-doffaySaviq, where is SnapDuration?15:20
dandraderSaviq, I mean that file the error is complaining about15:20
Saviqnic-doffay, UbuntuAnimation.SnapDuration15:21
nic-doffaySaviq, cool ta15:21
Saviqnic-doffay, from Ubuntu.Components15:21
Saviqdandrader, it should create it...15:21
Saviqdandrader, cmake -G Ninja should15:21
nic-doffaySaviq, one more thing. I get loads of reference errors when using UbuntuAnimation.15:22
dandraderSaviq, ah, that's likely the problem. cmake is not ther. run_on_device --setup doesn't install it15:22
nic-doffayIt's probably because it's C++15:22
Saviqdandrader, it does, but must've failed somehow15:22
Saviqnic-doffay, what do you mean?15:23
Saviqnic-doffay, it's there, doesn't matter if it's C++ or not15:23
nic-doffaySaviq, with UbuntuComponents included it gives me a reference error.15:23
nic-doffayWhenever I UbuntuAnimation. anything.15:23
nic-doffayIt works however.15:23
nic-doffaywhich  is perplexing15:23
Saviqnic-doffay, can you paste the exact error?15:23
Saviqnic-doffay, and your toolkit version?15:24
nic-doffaySaviq, https://pastebin.canonical.com/92548/15:24
dandraderSaviq, got it: "E: Unable to find a source package for unity8"15:24
dandraderfrom run_on_device --setup15:24
nic-doffaySaviq, where can I find the toolkit version?15:24
Saviqnic-doffay, apt-cache policy qtdeclarative5-ubuntu-ui-toolkit-plugin15:25
Saviqdandrader, the most recent run_on_device should use mk-build-deps15:25
Saviqdandrader, which just builds a package depending on the build deps from debian/control15:25
dandraderSaviq, hmm, ok. gonna rebase my branch then15:25
nic-doffaySaviq, 0.1.46daily13.06.05.1bzr538saucy015:25
Saviqnic-doffay, yeah, the newest one15:26
nic-doffaySaviq, I'll ignore it for now.15:26
Saviqnic-doffay, `qmlplugindump Ubuntu.Components 0.1 | grep -i animation` ?15:27
Saviqnic-doffay, and ignore the font complaints15:27
Saviqdandrader, so yeah, rebasing will help15:27
nic-doffaySaviq, looks ok to me: https://pastebin.canonical.com/92551/15:28
Saviqnic-doffay, yeah15:28
Saviqnic-doffay, not sure what's happening, we'll see over the review15:28
dandraderSaviq, yeah. that did it. sorry for the noise15:29
Saviqdandrader, cheers15:29
nic-doffaySaviq, agreed. Should have some additional stuff for commentary soon too.15:29
Saviqdandrader, it should be much more robust no15:29
nic-doffaySaviq, I think there's already been a bit of a speed up moving some components out the OH15:29
Saviqnic-doffay, yeah, there should be15:30
dandradermzanetti, when you tried out my MP, was it merged with trunk of did you try out the branch directly?15:30
mzanettidandrader: directly your branch15:30
dandradermzanetti, ok15:31
Saviqdandrader, mzanetti I think to be consistent, QML uses "velocity" usually15:32
Saviqdandrader, mzanetti, and it's in px/s15:32
Saviqor whatever the coordinate system is15:33
Saviqmzanetti, dandrader e.g. http://qt-project.org/doc/qt-5.0/qtquick/qml-qtquick2-flickable.html#maximumFlickVelocity-prop15:33
dandraderSaviq, hmm, so I can safely say that "spacial unit used by QML coordinate system" === "pixels"15:34
Saviqdandrader, yes15:34
Saviqdandrader, it's a good enough assumption15:35
nic-doffaySaviq, just to double check, the panel itself needs to hide, not just the dropdown on rotation?15:42
Saviqnic-doffay, yes, the whole panel15:43
dednickSaviq: why would the panel hide? it's always visible.15:43
Saviqdednick, only during rotation15:43
Saviqdednick, so that we don't have to try and sync with the app15:43
mzanettimterry: added autopilot tests for lockscreens15:46
* mterry hugs mzanetti 15:46
mterrymzanetti, hopefully we can get this thing in today15:47
mzanettimterry: I created a new testfile just for lockscreen tests and created 2 szenarios: one for pin, one for passphrase. just testing on phone form factor15:47
mterrymzanetti, why was "self.assertThat(pinentryField.text, Eventually(Equals("432")))" necessary?  It's not done in the success mode15:50
mzanettimterry: I just want to make sure that it actually types, and upon recognition of the wrong password it clears the field again15:55
mzanettimterry: without that line there would be the theoretical chance that it's not even typing and hence the test still passing15:55
mterrymzanetti, ah...  fair I guess15:55
mterrymzanetti, approved!  Thanks man15:56
mzanettimterry: cool :)15:56
tedgsil2100, FYI, those branches landed.15:58
tedgdidrocks, Could we have, on hud failures (as an example), just call "ubuntu-bug hud" or something like that.  So we'd get a bug with all the log files.15:59
tedgdidrocks, Then we could use the apport hooks for that stuff.15:59
cyphermoxsil2100: didrocks: does python-ubuntu-platform-api really belong in the QA stack?16:00
didrockscyphermox: yeah, it's done and conceived for being used only for autopilot16:00
didrockstedg: define the "failure" part? :)16:01
didrockstedg: you mean, autopilot tests failing?16:01
tedgdidrocks, Yeah16:01
didrocks(if so, it's more an autopilot option I guess)16:01
tedgdidrocks, So who do I harass?  thomi?  :-)16:03
didrockstedg: yeah, or veebers16:03
didrockstedg: I would recommend both :p16:04
* tedg does jujitsu and gets veebers to harass thomi16:04
nic-doffaySaviq, one more glaring issue. The panel leaves a gaping space that's quite visible upon rotation. Any ideas on what I should do to get around this?16:08
mzanettidandrader: I rebased your branch on trunk and its working like a charm now16:10
mzanettidandrader: and wow! feels perfect! great job16:10
Saviqnic-doffay, I don't think we can do anything about this, the app will fill that space anyway, when we implement that in qtubuntu16:11
nic-doffaySaviq, ok. There's one thing left now. The launcher and the panel still have to rotate.16:12
nic-doffayObviously this isn't happening since I moved them out the OH16:12
nic-doffayIt's a similar issue to what we spoke about earlier. Since everything happens depending on the rotating variable.16:12
Saviqnic-doffay, yeah, you need to have states for them that change their anchors and rotation16:12
nic-doffaySet the states if they aren't rotating Saviq?16:13
Saviqnic-doffay, yes, after rotating changes back to false, change the anchors and rotation accordingly, without transitions16:13
Saviqnic-doffay, and slide back in16:13
seb128mhr3, not on #ubuntu-touch?16:14
nic-doffaySaviq, should these states live in Shell.qml or the individual Launcher/Panel.qml ?16:14
Saviqnic-doffay, Shell, as they don't know about the rotation, really (and shouldn't)16:14
mhr3seb128, another channel i need to monitor :)16:15
seb128mhr3, yeah ;-)16:15
=== dandrader_ is now known as dandrader|lunch
=== om26er_ is now known as om26er
mzanettidandrader|lunch: ok, I'm through with the review. left 2 more comments. rest looks good16:34
Cantidewhere can one find information on the state of unity 8 for the desktop? or is it not yet at a stage which is presentable to the public?16:41
sil2100tedg: hmmm, there still seem to be some issues with HUD17:00
sil2100But different ones it seems17:00
sil2100tedg: I need to go, but could you take a look in your free time? I'll paste the link17:02
tedgFree time?  :-)17:02
sil2100Ok ok, that was a bad joke17:02
sil2100Sorry about that :<17:02
tedgI think these are Trevinho's bugs.17:03
tedgThey're all BAMF warnings.17:04
tedgunity <unknown>:0 Unable to fetch children: No such interface 'org.ayatana.bamf.view' on object at path /org/ayatana/bamf/application6280146217:04
Trevinhotedg: yep... Not really a bug actually.. .It can happen when apps are quickly opened/closed... Also if I'm looking to avoid this as well17:04
=== mmrazik is now known as mmrazik|afk
=== alan_g is now known as alan_g|life
tedgTrevinho, Would it happen if we ask for the view before the app is realized?  Guessing something like that is happening in the autopilot testing.17:06
Trevinhotedg: mh it could be but it's unlikely I think...17:08
Trevinhotedg: generally that happens when the app is closed and we try to get something that is not really on bus17:09
tedgTrevinho, Seems like it should notify that it is removed from the bus, no?  Then libbamf returns a critical?17:11
Trevinhotedg: yes, that happens, but it seems there are some problems on the library that prevents to check this locally... I'll look into that soon, but it shouldn't change the result of that call btw17:12
tedgTrevinho, I think the only time we're doing that is on startup in HUD.17:13
tedgTrevinho, Basically getting the list of applications, and then the list of windows.17:13
tedgTrevinho, So that's why I was thinking it was a race at startup.17:13
Trevinhotedg: couldn't be that an app that you cached is actually closed?17:14
tedgTrevinho, It could be, but unlikely in the autopilot test because there's only one app.17:14
mhr3Saviq, ping?17:16
mhr3Saviq, sorry for the late pings recently...17:16
mhr3Saviq, any recommended reading for tomorrows meeting?17:16
mhr3Saviq, as in a doc that lists all the objects and their properties available to QML?17:17
=== mmrazik|afk is now known as mmrazik
=== dandrader|lunch is now known as dandrader
olli_bregma, thanks for the update on the 13.10 BP18:06
=== olli_ is now known as olli
* bregma is whittlin' down his to-do list18:09
=== AlanChicken is now known as AlanBell
=== mmrazik is now known as mmrazik|afk
ollibregma, ping19:21
bregmaolli, pong19:21
=== dandrader is now known as dandrader|afk
ollibregma, sorry...20:08
olliI was chatting with pmcgowan about u8 on 13.1020:08
ollihe suggested to add movie player & the music player app to the list20:09
olliand was anticipating some issues around files vs content picking etc20:09
ollimind adding the first to the BP (my write rights are gone:)20:09
olliand for the 2nd (files)... just wanted to mention it, wasn't sure if this was on your radar20:10
=== dandrader|afk is now known as dandrader
bregmaolli, done20:24
ollibregma, thx!20:24
Esokrateshi, are there already plans for dynamic workspace support in unity-next? how concrete are potential plans?20:24
Esokratesis design input by the community welcome, or is the design restricted (and there are already exact plans)?20:26
=== racarr_ is now known as racarr
=== naee is now known as eean
=== salem_ is now known as _salem

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