/srv/irclogs.ubuntu.com/2013/05/17/#ubuntu-unity.txt

=== marlinc_ is now known as marlinc
=== kgunn is now known as Guest55148
=== _salem is now known as salem_
=== salem_ is now known as _salem
=== _salem is now known as salem_
=== salem_ is now known as _salem
=== _salem is now known as salem_
=== salem_ is now known as _salem
jibeldidrocks, when I run autopilot on Saucy with Otto, I often get http://paste.ubuntu.com/5673206/ for nearly all the tests, do you know what it means ?06:36
jibeland in dbus.log http://paste.ubuntu.com/5673216/06:36
didrocksjibel: sounds like zeitgeist is crashing, but mhr3 is on holidays…06:39
didrocksor can't create its database06:39
didrocksactivity.sqlite doesn't exist I guess?06:39
=== mamad is now known as boynux
boynux\q07:30
boynuxDoes anybody knows how can I contribute to UnityNext project?08:41
Saviqboynux, hey08:44
Saviqboynux, it's a slightly difficult time as everything is so fluid right now08:44
boynuxhi08:44
Saviqand we definitely need to get better in enabling you guys to do so08:45
Saviqbut we have a list of TODOs and FIXMEs some of which are bite-size08:45
Saviqhttps://docs.google.com/a/canonical.com/spreadsheet/ccc?key=0AoGvOYxwuvpFdEJ5dURFb3Y0cnlKeEcxc0piNDZrWXc#gid=008:45
boynuxso, is there any list or irc channel just to have an idea how to start?08:45
SaviqIRC is here08:45
boynuxok, thanks08:46
Saviqand we don't have a specific ML, and ubuntu-phone is a good enough place (as you've noticed)08:46
boynuxyes, I understand. and is there any possibility to take some of the tasks in that google doc?08:48
Saviqboynux, there's an "owner / assigned" column08:48
Saviqboynux, just drop yourself in there08:48
boynuxok, fine.08:49
boynuxbut there is no time frame for tasks!08:49
Saviqboynux, the "Difficulty" should be an indication08:50
Saviqboynux, if by time frame you mean how time consuming a task is08:51
Saviqboynux, there's also the set of blueprints you can reach from https://blueprints.launchpad.net/ubuntu/+spec/client-1303-unity-ui-iteration-008:51
=== gusch is now known as gusch|away
Saviqboynux, where, if you find a TODO item (even if it's assigned to someone) that you'd like to work on, ping me about it08:52
boynuxSaviq, thanks for all information.08:53
boynuxok!08:53
boynuxand the code base is the one from bzr? right?08:53
boynuxalso can you tel me how to send out patches afterward?08:53
Saviqboynux, lp:unity/phablet08:53
Saviqboynux, https://dev.launchpad.net/UsingMergeProposals is our process08:54
boynuxSaviq, OK, I think now I have enough info to to digest for the moment :) thank you.08:58
Saviqboynux, cheers, feel free to ping me or just hang out here if I'm unavailable08:58
boynuxSaviq, sure, thank you.08:59
Saviqbiab09:14
tsdgeosgreyback: oh man, can't get the lvwph test to pass once i move from the broken tryCompare to the non broken one :/09:15
tsdgeosi mean, they pass locally09:15
tsdgeosbut not in jenkins :D09:15
greybacktsdgeos: I sympathize. One fail in particular, or several?09:15
tsdgeosseveral :/09:15
greybackpoint me to a branch, I might be able to help09:16
tsdgeos lp:~aacid/unity/fix_tryCompare09:16
tsdgeoslatest failures http://10.97.2.10:8080/job/unity-phablet-qmluitests/913/09:17
tsdgeosafter billion of different permutations :/09:17
greybacktsdgeos: weird thing I found while reading source to qtquickcontrols, specifically the ScrollView code, it always adds a non-zero arguments to the flick call09:31
greybackwith a comment "flick() does not work with 0 yVelocity"09:31
greybacktsdgeos: also I'm not convinced the mouseFlick function always works well09:32
tsdgeosgreyback: we need it09:32
tsdgeosflick doesn't want to flick up09:33
tsdgeoswhen you're up09:33
tsdgeosso either we kill that test09:33
tsdgeosor09:33
greybackI see09:33
greybacktsdgeos: I think we need a better solution for testing Flickables then. I don't think we can reliably manipulate them via JS09:36
greybackso I would vote for disabling the tests which we suspect are unreliable, and look for a better solution. Probably C++ based09:36
greybacktsdgeos: just a random thought, what GU size is set on jenkins?09:37
tsdgeosnot sure09:37
tsdgeosmzanetti: ?09:37
mzanettinone09:38
mzanettiwhich defaults to 10 afaik09:38
mzanettigreyback: tsdgeos ^09:38
tsdgeosmay be09:38
dednickTO ALL: i've just accidentily overwritten lp:unity/phablet. :( please noone pull latest until we get it resolved09:39
greybacktsdgeos: yep that caused a fail09:39
dednickanyone know how to get the latest good revision back?09:39
* mzanetti wonders how that would happen09:39
mzanettidednick: depends on what you did09:39
tsdgeosi have 675, anyone has anything newer?09:39
mzanettisame here09:40
dednickmzanetti: i pushed with overwrite instead of pull.09:40
greybackme too09:40
mzanettidednick: can you bzr uncommit && bzr revert && push?09:40
tsdgeoswe can ask jenkins what's the last branch it autolanded09:41
* tsdgeos checks09:41
dednickmzanetti: no. the version is a few days old.09:41
mzanettilet me pull09:41
tsdgeosso https://code.launchpad.net/~macslow/unity/phablet-notification-renderer/+merge/155512 is the last thing merged09:41
tsdgeosso 675 is fine as "last" rev09:41
tsdgeoswonder if when http://s-jenkins:8080/job/unity-phablet-autolanding/327/console autolands09:41
tsdgeoswhat will happen :D09:42
mzanettiif we're luck it causes a merge conflict and fails09:42
mzanettiotherwise... hmm. need to remerge I'd say09:42
tsdgeosthen we just need to push overwrite 675 again and accept that change to autoland again09:42
tsdgeosshould not be that "bad"09:42
mzanettino... no biggie right now... but really interesting that this can happen...09:43
mzanettiso the revisions 655 - 675 are gone forever? there's no history or anything left any more?09:44
greybackdidrocks: can this be locked down? We had an accidental push directly to trunk09:44
tsdgeosmzanetti: well,  you just push --overwrite and they'll be back09:44
greybackindeed09:44
mzanettisure... but if we wouldn't have a local copy we'd be quite screwed?09:45
greybackprobably yes :)09:45
tsdgeospossibly09:45
mzanettiwow...09:45
greybackwhich is why I thought bzr was locked down that we couldn't directly push, only the tarmac/jenkins could09:45
mzanettiso... who will push 675?09:46
tsdgeosmzanetti: i'd wait until the autolanding thing finishes09:46
mzanettitsdgeos: why? if your fast enough it won't be noticed by the job09:46
tsdgeosah it finished09:46
dednickbut the 675 MP doesnt have the full history does it?09:46
tsdgeosand it merged09:46
tsdgeos:S09:47
greybacklet's just push overwrite, and re-propose the merge09:47
mzanettiack. I'm on re-proposing the merge... you guys push in the meantime09:47
greybackbut I'd like didrocks ' opinion before proceeding, just in case there's something subtle09:48
mzanettioh wait09:48
mzanettihttps://code.launchpad.net/~saviq/unity/phablet.release-176/+merge/16419209:48
mzanettithis was merged too09:48
mzanetti67609:48
tsdgeosyou sure?09:48
tsdgeosdoes that skip unity-phablet-autolanding ?09:48
mzanettiit says in the MP at least09:48
dednickplus that one has full history.09:48
tsdgeoscan't see it's job09:49
didrocksgreyback: I'm debugging stuff in a hangout?09:49
greybackdidrocks: ah sorry, when you have time let us know09:49
didrocksgreyback: is it urgent?09:49
mzanettitsdgeos: I think there is some special case for changelog-only changes09:50
tsdgeosi see09:50
mzanettitsdgeos: might not go through regular autolanding09:50
tsdgeosso we need to push 675, and then reapprove that one and the https://code.launchpad.net/~mzanetti/unity/phablet-tease-launcher/+merge/163906 one09:50
tsdgeossounds like a plan?09:50
greybackdidrocks: kinda. We accidentally bzr push --overwrite to our trunk. We want to restore it back to a last known revision by doing the same, and then re-proposing any merges that happened in the mean time. See any problem with that?09:50
mzanettiwhy not just pushing this: lp:~saviq/unity/phablet.release-176 ?09:50
tsdgeosmzanetti: that works too09:51
greybackmzanetti: if that did indeed merge, then yeah09:51
didrocksgreyback: no, sounds good to me :)09:51
didrocksthat will still work even with dailies09:51
greybackdidrocks: ok cool, thanks. We should chat later to see if we might want to lock down us pushing directly to trunk09:52
tsdgeosok, so who pusehs  lp:~saviq/unity/phablet.release-176 ?09:52
didrocksyep :)09:52
mzanettitsdgeos: feel free09:52
tsdgeosok, doing, give me 5 min09:52
mzanettioh... this one failed to merge: https://code.launchpad.net/~mzanetti/unity/phablet-tease-launcher/+merge/16390609:53
tsdgeosyep09:53
tsdgeosthat's "good"09:53
mzanettiso we just push Saviq's branch and set that one to Approved again and we're fine09:53
tsdgeosi guess :D09:53
greybackyep, agreed09:53
* dednick purges push --overwrite from .bash_history09:54
mzanettihehe09:54
mzanettibetter, yes :D09:55
mzanettitsdgeos: you better do so too afterwards :D09:55
tsdgeoshow does one do that?09:56
mzanettidednick: ?09:56
tsdgeosok, we're all set09:56
tsdgeoshttps://code.launchpad.net/~unity-team/unity/phablet09:56
tsdgeos676 is back09:56
tsdgeosand i just reapproved https://code.launchpad.net/~mzanetti/unity/phablet-tease-launcher/+merge/16390609:56
dednicktsdgeos: edit ~/.bash_history09:56
mzanettitsdgeos: cheers09:56
mzanettidednick: huh? isn't that a binary file?09:57
dednickthanks all!09:57
dednicknope09:57
mzanettiwow... its not in ubuntu09:57
mzanetticrazy09:57
tsdgeosmzanetti: never been one afair09:57
mzanettiall the distros I've used before stored the history in some binary/encrypted format09:58
tsdgeosreally?09:58
tsdgeosinteresting09:58
mzanettinow I'm confused... just checked my Archlinux based TV09:58
mzanettialso plaintext09:58
mzanettibut I'm 100% sure I've seen that encrypted already09:58
mzanettidednick: first time my highlight filter on "all:" was useful :D10:01
mzanettiusually it only produces false positives on dh_install: pastes etc10:01
dednickmzanetti: ahha10:01
Saviqoh boys, oh boys10:12
mzanettihaha10:13
mzanettitsdgeos: what exactly is the reason why we need to depend on the fonts?10:17
tsdgeosmzanetti: it's expplained in the commit log ;-)10:17
mzanettinot really...10:17
mzanettiit says otherwise a test fails10:17
mzanettibut I would like to understand why10:17
tsdgeos"install the ubuntu font since we use it and otherwise the test10:18
tsdgeosdoes not pass because font sizing is different"10:18
tsdgeosfont is different10:18
mzanettisure... but why would a slightly larger/smaller font mess up the test?10:18
tsdgeospageHeader label is different10:18
tsdgeosso instead of "active"10:18
tsdgeosit goes into "narrowActive"10:18
tsdgeosbecause there's not enough space10:19
mzanettioh... I see10:19
tsdgeosand the size comparison fails10:19
Saviqtsdgeos, so the test should be made better10:19
Saviqtsdgeos, i.e. grow the window until it goes into "active"10:19
tsdgeosSaviq: well, tell whoever made the test10:19
Saviq;)10:19
tsdgeosanyway we should depend on the font10:20
tsdgeossince we use it10:20
tsdgeoswhat's the point otherwise10:20
Saviqtrue10:20
Saviqthe theme should depend on it, btw10:20
mzanettiyeah... fine with that... Just wanted to understand what exactly is happening10:21
Saviqmzanetti, gimme a sec10:22
mzanetti?10:22
mzanettinot sure for what... but sure... take your time10:23
Saviqmzanetti, for tsdgeos's branch10:23
mzanettiah10:23
Saviqmzanetti, but go for it, we're explicitly using Ubuntu as the font family in a few places10:23
Saviqwhich we shouldn't, but that's another story10:23
Saviqmzanetti, remember the bug about buttons going through transparent when hovering? you did report it didn't you?10:27
Saviqyeah10:27
mzanettiSaviq: when, clicking, not howering10:27
Saviqmzanetti, yea10:27
Saviqhowering10:27
Saviqhmm10:27
mzanetti:D10:28
Saviqhow how, I'm howering10:28
nic-doffaypete-woods, some changes with the colours in the circle.10:28
* mzanetti needs to actually pay attention to the spell checker when talking to Saviq10:28
nic-doffaythe outer circles I mean.10:28
mzanettiSaviq: you need that bug or did you find it?10:29
Saviqmzanetti, I _am_ the spell checker10:29
Saviqmzanetti, #111507110:29
pete-woodsnic-doffay: have these changes been pushed?10:32
nic-doffayNo pete-woods busy talking about them still.10:32
=== gusch|away is now known as gusch
pete-woodsokay10:32
didrocksgreyback: ok, available now :)10:33
greybackdidrocks: all's good, we're back to normal.10:34
greybackSaviq: would you consider locking down bzr a little to stop us pushing directly to trunk?10:34
didrocksgreat :)10:34
Saviqgreyback, not sure it's worth it, it's the first time this happened, and with bzr there's loads of copies lying around10:35
Saviqgreyback, it's not like it was the end of the world, just some panicking10:35
Saviqgreyback, and if we actually lock it down we might end up in a bigger conundrum when there's noone around that can push to it10:37
Saviqgreyback, unless it becomes a recommendation that trunks are owned by pspmteam or something10:38
tsdgeosSaviq: actually, we might just limit overwrite pushes, if that's possible10:38
greybackSaviq: well's it handy for crisis prevention, we're just lucky there's lots of copies around. But yes it ties us down a little10:38
Saviqtsdgeos, I don't think it ise10:39
Saviq-e10:39
tsdgeos:-/10:39
tsdgeosthat'd be handy10:39
Saviqtsdgeos, you either have write access or you don't10:39
Saviqwe'd just have to change the owner to a team that we're not all member of10:39
tsdgeossure, but does bazaar has hooks?10:39
tsdgeosi mean a pre-commit hook might reject overwrite pushes10:39
tsdgeosin my theorical mind10:40
tsdgeosnot saying that's possible at all10:40
Saviqtsdgeos, would have to be manually installed, I think10:40
Saviqtsdgeos, and then again, not worth it, IMO10:40
tsdgeosok10:40
SaviqI myself have 5 copies lying around every other time10:40
Saviqand backed up daily10:40
greybackhttp://doc.bazaar.canonical.com/latest/en/user-reference/configuration-help.html#append-revisions-only10:40
Saviqas my whole home dir10:40
Saviqgreyback, read the doc, that would not help with overwriting10:41
Saviqreally guys, not the end of the world :D10:41
dednickSaviq: there seems to be a dependency break in the new qtdeclarative5-ubuntu-ui-toolkit-plugin. it's requiring qt5.0.2 while we only have 5.0.1 in raring11:09
Saviqdednick, yeah, 5.0.2 is in ppa:canonical-qt5-edgers/qt5-proper11:09
dednickSaviq: ah11:09
dednickthanks11:09
=== MacSlow is now known as MacSlow|lunch
=== mmrazik is now known as mmrazik|afk
pete-woodsnic-doffay: just pulled some new changes from you regarding the dot sizes, and they look a bit screwed now12:09
nic-doffayTry it on the phone pete-woods12:09
nic-doffayNo idea what it looks like on the tablet sadly pete-woods12:09
nic-doffayBasically it looks screwed on the desktop but great on the phone.12:10
pete-woodsnic-doffay: is that because of the screen resolution or something?12:10
nic-doffayNo idea why.12:10
nic-doffaypete-woods, that would be my first guess. Can't say for sure though.12:10
pete-woodsnic-doffay: it's gotta be screen resolution - looks okay on the tablet too12:12
nic-doffaypete-woods, ok great.12:12
dandraderSaviq, what's the reasoning behind using "#include <QtQuick/QQuickItem>" instead of "#include <QQuickItem>"?12:12
Saviqdandrader, explicit12:12
nic-doffaypete-woods, as long as it looks cool on the tablet too :)12:12
Saviqdandrader, and/or consistent12:12
pete-woodsnic-doffay: it's bad if we have resolution-dependent code, though12:13
Saviqdandrader, obviously it doesn't matter that much 'cause qt5_add_modules adds the include dir anyway12:13
pete-woodsremember this is going to be the desktop login screen, too, with unity next12:14
pete-woodsnic-doffay: also, the arrow icon is too low resolution, it looks pretty pixelated on the tablet12:14
dandraderSaviq,  Qt documentation also omits the module dir in its include. I mean, what problem can arise from omitting the module directory? I doubt there can be a conflict (otherwise Qt docs would tell you to put the module dir when you include smthing). But ok, I'll do it as it seems to be the project's coding style (consistency) although I still don't see the point in it12:16
nic-doffaypete-woods,  going to commit a change, can you let me know if it makes a diff?12:16
pete-woodsof course :)12:16
kgunnpete-woods: nic-doffay isn't it because the assetts are 1:1 on the phone...but scaling on desktop ?12:16
nic-doffaypete-woods, pushed.12:16
nic-doffaykgunn, nope. Something else that I didn't think through properly :P12:17
Saviqnic-doffay, pete-woods, kgunn, we just need to provide multiple @px versions of those assets, probably12:17
nic-doffaySaviq, it looks fine as long as it's being scaled down.12:18
Saviqnic-doffay, it should never be scaled up :)12:18
nic-doffaySo we just need to provide one large asset.12:18
kgunnnic-doffay: actually no12:18
kgunnnic-doffay: cause then you spend processing time :)12:18
Saviqkgunn, that's fine12:18
pete-woodsSaviq: I think it's more than that - the relative positions of the inner dots seems to be affected by the resolution12:18
nic-doffaykgunn, what's the largest px?12:18
Saviqnic-doffay, 3012:18
Saviqkgunn, we're going to rescale once and cache12:19
Saviqkgunn, we can even prescale them on the servers somewhere12:19
kgunnSaviq: that's cool12:19
pete-woodsnic-doffay: at desktop resolution, the dots are too close to the centre of the circle12:19
Saviqkgunn, so the workflow is: provide @30 and only provide others if scaling down doesn't work12:20
nic-doffaypete-woods, yeah do we have to account for this though?12:20
kgunnSaviq: nic-doffay it'll certainly work for now, and oem12:20
kgunn's can tweak for 1:112:20
kgunnwhen that day comes12:20
Saviqyup12:21
pete-woodsnic-doffay: I don't think that the icon sizes are so important - as Saviq says, we can provide correctly sized ones, it's more the positioning that I think should be fixed12:23
nic-doffaypete-woods, you mean for the desktop?12:23
pete-woodswell, generally - there must be some mistake if the dots are going in the wrong place12:23
pete-woodsas it should just be some proportion of the size of the circle12:24
Saviqnic-doffay, pete-woods it's a regression, too, it was fine on desktop before12:25
nic-doffaySaviq, yeah but it wasn't on the phone.12:25
nic-doffaySaviq, the phone looks perfect now, but the desktop is screwed.12:26
Saviqnic-doffay, that's weird, 'cause if you go GRID_UNIT_PX=16 on the desktop12:27
kgunnSaviq: just curious...do you know if scaling is it relying on gpu to scale ? or cpu sw ?12:27
Saviqnic-doffay, you should get exactly what you would get on the phone12:27
Saviqkgunn, GPU right now, AFAIK, they're not cached yet12:27
nic-doffaySaviq, yeah not entirely sure what is influencing the drastic differences though. At the moment I scale the object by the radius of the circle.12:27
Saviqkgunn, when they're going to be prescaled for caching, CPU, and then nothing ;)12:27
kgunnSaviq: so potentially you wouldn't get the same thing on phone as desktop12:27
nic-doffaySorry let me be more specific I scale the dot by the radius of the centre circle.12:28
Saviqnic-doffay, scale it?12:28
Saviqnic-doffay, it should always be the same size, no?12:28
Saviqin GUs I mean12:28
nic-doffaySaviq, if that's the case then we'll need those multiple assets.12:28
Saviqnic-doffay, I think you're missing something here12:29
Saviqnic-doffay, the size of the dot should be static in GUs, right?12:29
Saviqnic-doffay, i.e. 1x1 GU12:29
Saviqnic-doffay, that will mean 10x10px on desktop12:29
Saviq16x16px on phone12:29
Saviq20x20px on tablet12:29
Saviqand QML will take care of that scaling regardless of what you give it as your asset12:30
Saviqif there's only a @30.png asset12:30
Saviqit will get scaled down12:30
=== MacSlow|lunch is now known as MacSlow
Saviqyou just don't worry about it12:30
Saviqyour asset will be 30x3012:30
Saviqbecause it's 1x1 gu @ 30 px per gu12:30
Saviqnic-doffay, you could provide prescaled @10.png @16.png @20.png, but that's unnecessary12:31
Saviqnic-doffay, as a dot @30 will scale down just fine12:31
Saviqas will the arrow12:31
Saviqnic-doffay, what's more they get cached after being scaled down12:32
nic-doffaySaviq, so I shouldn't perform any scaling on the image at all?12:32
didrocksMirv: did you see the sdk issue?12:32
Saviqnic-doffay, it should just be "width: units.gu(1); height: units.gu(1)"12:33
nic-doffaySaviq, got it.12:33
Saviqnic-doffay, nothing else12:33
didrocksMirv: on the daily release12:33
Saviqkgunn, to clarify - IIRC they are currently cached for a single run, just not pre-cached and stored12:33
kgunnSaviq: got it....i was following....so really, once nic-doffay goes pure gu's it should be cached by the engine12:34
Saviqyes12:34
Saviqre-scaled on each run12:34
nic-doffaySaviq, improvement already, nice one!12:35
kgunnright-o, i follow12:35
nic-doffayJust have to fiddle around with the positioning maths a bit to compensate.12:35
Saviqkgunn, and the end goal is that it should be pre-cached and installed in the image on a per-device basis and ad-hoc for stuff that's added to the device later12:37
Saviqbut still kept in a cache so that it's only re-scaled when needed12:37
kgunnSaviq: sounds good...oem's will care about launch times, that'll help12:38
Saviqyup12:38
Mirvdidrocks: yes, briefly, but I then had more urgent issues. there's a version number conflict from somewhere.12:40
nic-doffaySaviq, kgunn, pete-woods fixed.12:40
nic-doffayPosition and size are good on desktop & phone.12:41
nic-doffayAssuming tablet will be fine too.12:41
nic-doffaypushing now...12:41
kgunnnic-doffay: woohooo12:41
didrocksMirv: it means that a publish happened, but then, the merge back branch either wasn't merged or was reverted12:41
Saviqnic-doffay, thought you already pushed, didn't look fixed ;D12:41
didrocksMirv: you will fix it before your EOD? so that we don't block next daily?12:41
Mirvdidrocks: I think I see the issue, now fixing12:41
didrocksMirv: thanks :)12:41
didrocksMirv: you can look at the publish12:41
didrocksrev 1612:42
didrocksit referts to https://code.launchpad.net/~ps-jenkins/ubuntu-ui-toolkit/latestsnapshot-0.1.46daily13.05.14ubuntu.unity.next-0ubuntu1/+merge/16369612:42
nic-doffaySaviq, that was a small experiment to see what the pointer looked like on the tablet ^_^12:43
=== mmrazik|afk is now known as mmrazik
Mirvdidrocks: there was actually erronous changelog manual change in the trunk, so reverting that..12:43
didrocksMirv: ah ok ;)12:45
pete-woodsnic-doffay: nearly there12:45
Saviqnic-doffay, you can just go « ./run -- -fullscreen » to see how stuff looks12:45
didrocksMirv: at least, the daily release is defensive in that regard, so that we don't overwrite :)12:45
pete-woodsyou've now got the arrow as being the same size as all the dots again12:45
Saviqnic-doffay, and « GRID_UNIT_PX=16 ./run » to see the phone in full-resolution on your desktop12:46
pete-woodsnic-doffay: the dots seem to be the same size all the time now - shouldn't their size be relative to the size of the circle somehow?12:47
pete-woodsi.e. if I run it in tablet mode, the dots look tiny12:48
pete-woods(but are the same size as in phone mode)12:48
Saviqpete-woods, look fine here12:48
* Saviq gets a few shots12:48
Saviqpete-woods, apart from the arrow being too small, it looks fine12:48
pete-woodsSaviq: I'm just running it fullscreen with no overriding resolution set, and the dots are really small12:49
Saviqactually they look too big on the phone ;)12:50
pete-woodslooks good on the actual tablet, though12:50
=== greyback is now known as greyback|lunch
Saviqpete-woods, nic-doffay, so... I assumed the infographics would be static size in GUs12:54
Saviqwhich it doesn't seem to be the case now12:55
Saviqand I based my "dots should always be 1x1 GU" on that assumption12:55
pete-woodsSaviq: I don't really know about GU's or anything, but I think that the dots should be some proportion of the size of the infographic circle12:56
Saviqpete-woods, just treat GUs as a kind of "big pixels"12:56
Saviqpete-woods, so yeah, they should definitely be a proportion of the big circle12:56
Saviqpete-woods, but I assumed that the big circle would always be 50x50 GUs or something12:57
Saviqwhich isn't the case now12:57
Saviqthe big circle scales according to the available size12:57
Saviqand I don't know whether that's the designed behaviour or not12:57
pete-woodsI thought it was the behaviour we wanted?12:59
SaviqI dunno, design would need to say13:00
Saviqpete-woods, btw, just noticed something13:00
pete-woods?13:01
Saviqpete-woods, when switching between "Guest" and "Lola Chang"13:01
Saviqpete-woods, the circle shows up for a split second13:01
pete-woodsSaviq: yes, that's because the lightdm is being told that lois is selected, briefly13:02
pete-woodsI don't know what the API can do about that13:02
Saviqpete-woods, probably nothing, we need to make sure that's not the case in the UI13:03
pete-woodsit feels like the UI should only tell the API the user has changed once the selection has settled for some kind of grace period13:03
Saviqyup13:03
pete-woodsis there some proper way of doing that in QML world?13:04
Saviqpete-woods, still hiccups when switching users, did you have time to investigate that?13:04
pete-woodsI changed it to use a signal13:04
pete-woodsand then subscribed to that signal13:04
pete-woodsI've not spent any time performance tuning the inside of the mock implementation13:04
Saviqthat's fine13:05
Saviqjust asking13:05
pete-woodsit does seem to stutter, though, yes13:05
pete-woodspossibly the animations are starting too early13:05
pete-woodsand it's loading in a background picture at the same time as triggering the animations13:06
Saviqpete-woods, the background picture is loaded async13:06
pete-woodsloading images still seems to hurt the shell quite a bit (presumably some sort of pre-loader type thing hasn't been implemented yet)13:06
pete-woodsfor example with the video previews, you get a delay each time you look at one you've not seen before13:07
pete-woodsprobably not related to this, though13:07
Saviqpete-woods, it doesn't happen when there's no infographics, so ;)13:07
pete-woodsindeed :)13:07
pete-woodsit's strange how it's exactly halfway between each user it stutters each time13:08
pete-woodswe need to be triggered at the same time as the shell background changes, that seems to be doing the right thing13:10
Saviqpete-woods, background doesn't change between "empty-name" and "Anna Olson" here13:10
Saviqpete-woods, still stutters13:11
Saviqpete-woods, anyway, the signal thing must not've helped 'cause it's just created a synchronous signal-slot connection, so it will still wait for the slot to finish13:11
Saviqpete-woods, but anyway, it might not be that, even13:12
Saviqpete-woods, it might be just the creation of Infographics13:12
pete-woodsSaviq: yeah, quite possibly13:13
Saviqpete-woods, there's like 100 objects created at that point13:13
pete-woodsyep13:13
pete-woodsSaviq: I was planning for the real implementation for it to produce a minimal set of change notifications13:16
pete-woodsso it could update the circles as necessary13:17
=== boiko_ is now known as boiko
Saviqyup13:17
tedgbregma, So I'm realizing from the Unity8 session we didn't decide what the name of the session would be.13:31
tedgbregma, Do you have thoughts on that?  "ubuntu-touch" ?13:31
bregmahow about a nice simple Unity 8?13:31
bregma'cos most people don;t have a touchscreen, so they;d be sonfused13:32
bregmalike my typing13:32
kgunnbregma: +113:32
kgunnon the name not your typing :)13:32
mzanettidednick_: standup13:33
dednick_mzanetti: on my way13:33
dednick_as soon as i can get mumble to work that is13:33
mzanetti:)13:33
pete-woodsSaviq: I've changed the timing of the animation for the infographic - it's only triggered on the proper signal for when we've stopped moving now - so hopefully the stutter isn't (as) visible :)13:38
Saviqpete-woods, cheers :)13:38
Saviqpete-woods, yeah, much better13:40
pete-woods:)13:40
=== greyback|lunch is now known as greyback
tedgbregma, Don't think it can have a space, no?  So "unity8" ?13:43
tedgbregma, I don't care about the human readable name... humans are messy.13:44
tedgFYI, the current session is "ubuntu"13:44
kgunnpaulliu: just curious, your crash on the people lens test...is it on desktop or phone ?13:46
paulliukgunn: on desktop.13:47
kgunnpaulliu: have you tried it on phone ?13:47
paulliukgunn: That's unit test.13:47
kgunni know it shouldn't matter13:47
kgunnpaulliu: ah!13:47
paulliukgunn: No, didn't tried that on phone. But make testLensView also very strange.13:48
paulliukgunn: It doesn't get the fake lens right now.13:48
paulliukgunn: I'm still tracing.13:48
paulliukgunn: the fake Unity plugins are currently strange.13:48
=== HENDRIK_ is now known as Guest43341
dednick_didrocks: ping14:06
didrocksdednick_: pong14:07
dednick_didrocks: hi. so unity smart-scopes is landing right? but seems to be failing?14:08
Saviqdandrader, I assigned you a WI to allow dismissing close mode in running apps when tapping outside of app14:11
dandraderSaviq, ok. by "app" you mean "app thumbnail"?14:12
Saviqdandrader, yes14:12
dandraderSaviq, and is the work item in a blueprint?14:12
Saviqdandrader, yeah, just put it in the dash blueprint14:12
Saviqdandrader, (that's where I assigned it to you)14:12
dandraderSaviq, ah, this one is ready to go now: https://code.launchpad.net/~dandrader/unity/phablet_touchEventsInQmlTests/+merge/16424414:13
Saviqdandrader, k, will try and go through it today14:13
didrocksdednick_: right, sync with pstolowski, only half of the things landed first…14:17
didrocksdednick_: where everything needed to be landed at the same time14:17
tsdgeosguys, when someone has some free time14:20
tsdgeoscan you visit https://bugs.launchpad.net/touch-preview-images/+bug/1180511 and check that14:21
ubot5Launchpad bug 1180511 in touch-preview-images "Cancel network connect on password request crashes shell" [Medium,Confirmed]14:21
tsdgeosa) it crashes14:21
tsdgeosb) installing the beta-ppa packages no longer crashes14:21
tsdgeosand comment on the bug?14:21
tsdgeosMirv wants at least two people confirming it14:21
kgunngreyback: just doing the weekly...when you say "hacks" to make it all work...you mean in the qt plugin ?14:26
=== dandrader is now known as dandrader|afk
tedgTrevinho, andyrock, so what do we do with this icon_free() thing.  I was trying to be helpful, but it seems like it's turning into something crazy :-)14:27
Trevinhotedg: Oh, I didn't read your comments (if you did) yet... But in my tests I was getting crashes14:28
greybackkgunn: yep, one hack in particular forcing gles2, but qts code uses gles1. In fact the driver reports EGL version 1.4, so I understand why Qt does what it does.14:28
tedgTrevinho, Are you on raring or saucy?14:28
Trevinhotedg: raring...14:28
tedgI think this is related to the new GTK in Saucy14:28
Trevinhotedg: ah, ok...14:28
* tedg looks for the downgrade button14:29
tedgIs there a way to remove the deprecation warning for a function?14:29
tedgIt seems like going back to image_info_free() and ignoring the deprecation for now seems sane.14:29
Trevinhotedg: mh, I don't think it's possible just for one14:30
tedgI bet if there is a gcc option for it, bregma would know it.  :-)14:30
Trevinhotedg: probably using an #ifdef is the best thing here14:30
bregma-Wno-deprecated ??14:31
Trevinhomore than #ifdef, the gtk macro for version14:31
Trevinhobregma: yes, but that's general14:31
Trevinhobregma: and I think we want it for most of code14:32
bregmawell, considering they've probably annotated the code with __attribute__((deprecated)),  you're probably out of luck14:33
bregmabut deprecated just means it will be removed at some unspecified point in the future, so it should be OK to disable the warning in productionmode14:33
tedgYeah, we're just compiling with -Werror14:34
tedgSo it "breaks" the compile.14:34
tedgIn general, I think that's a good thing, but it'd be nice to handle it for this one case.14:34
* tedg wants __attribute__((i_know_that_its_deprecated))14:36
bregmait's generally a bad idea to have -Werror during packaging builds, because it causes much pain due to minor change in dependencies with little to no benefit, but that's my experience14:36
=== _salem is now known as salem_
=== dandrader|afk is now known as dandrader
nic-doffaySaviq, I tried scaling the component down with some maths. It doesn't look good, I think we're going to opt for the multiple asset approach.14:47
nic-doffayIt was the correct size, but it didn't look good.14:47
Saviqnic-doffay, sure, that might happen (as I said we're not pre-scaling them in HQ yet)14:51
Saviqnic-doffay, and we'll be able to revert to a single asset when that happens14:51
Saviqmzanetti, I just noticed a small bug on the launcher hint on a tablet... it doesn't hint when you press anywhere around the user list14:55
Saviqmzanetti, but does, when you press to the right of the user list...14:56
Saviqmzanetti, nic-doffay, about the blur, doesn't the stock Qt blur work?14:59
nic-doffaySaviq, too slow.14:59
Saviqnic-doffay, for a screenshot, even?14:59
mzanettiSaviq: yes. the List is on top... need to check with design what to do there15:00
mzanettiSaviq: re bur: no. it works fine on the desktop but is very slow on the phone15:00
mzanettiSaviq: the FastBlur doesn't work on the phone at all15:00
mzanettiSaviq: and the GaussianBlur is slow as hell ( >1s for one frame)15:01
Saviqmzanetti, uh15:01
mzanettiyep. exactly my reaction15:01
mzanettiSaviq: also. the GaussianBlur needs _a lot_ of iterations to achieve that much blurring.15:02
Saviqmzanetti, what about the recursive one?15:02
mzanettiSaviq: so if we downscale first and blur with 1/4 iterations it should already help15:02
Saviqmzanetti, k15:02
mzanettiSaviq: which recursive?15:02
Saviqmzanetti, http://qt-project.org/doc/qt-5.0/qtgraphicaleffects/qml-qtgraphicaleffects1-recursiveblur.html15:02
mzanettioh... missed that one15:03
Saviq"takes more time"15:03
mzanettiah ok15:03
mzanettiwell if it does so with less iterations... will give it a shot to learn the difference15:03
Saviqk15:03
nic-doffaySaviq, could you pull from the infographic branch and test it on tablet?15:18
nic-doffayIt looks small on desktop but fine on the phone.15:19
Saviqnic-doffay, screenshot coming right up15:21
Saviqnic-doffay, http://ubuntuone.com/5g8LsQ5CiBztSJKXbJGHxP15:21
nic-doffaySaviq, link doesn't seem to be workin15:21
Saviqhum15:21
Saviqnic-doffay, http://ubuntuone.com/5RY0CzR13IOe53TRAf6R3N15:22
Saviqnic-doffay, dots too big it seems15:22
nic-doffaySaviq, yeah sorted!15:22
Saviqnic-doffay, you can always go `GRID_UNIT_PX=20 ./run` after setting "tablet" to true in Shell.qml15:23
Saviqnic-doffay, that should get you the exact same result15:23
nic-doffaySaviq, awesome.15:23
dednick_mzanetti: ping15:30
=== dednick_ is now known as dednick
mzanettidednick: pong15:33
dednickmzanetti: hey. trying to figure out autopilot for UnityNext. anything special you need to get it working?15:34
mzanettidednick: hmm. no, not really. install all depenedencies of qml-phone-shell-autopilot and then run "make autopilot" in the builddir15:35
=== slangase` is now known as slangasek
Trevinhotedg: this should work everywhere https://code.launchpad.net/~3v1n0/unity/gtk-wrapper-icon-info/+merge/16443415:37
nic-doffaymzanetti, so I have a data model for the Infographics test. Should I just compare strings of the label when an animation has changed?15:38
mzanettiyeah. but more important are the bubbles radius etc15:39
=== gusch is now known as gusch|away
tedgTrevinho, You've got conflicts :-)15:45
Trevinhotedg: yes, I'm fixing them :)15:45
Trevinhotedg: but the idea is that...15:45
tedgSure, makes sense to me.15:46
tedgGrabbing the branch, I'll build it on Saucy.15:46
* tedg will have results in 3 days15:47
Trevinhotedg: thanks, builds here, but crashes of course :P15:47
=== mmrazik is now known as mmrazik|afk
=== boynux is now known as boynux|away
=== gusch|away is now known as gusch
=== dandrader is now known as dandrader|lunch
tedgTrevinho, So that branch crashes for me, but it seems to be in Nux, not anything Image Info related.  Perhaps?16:32
* greyback eow16:34
greybackgood weekend guys!16:34
Trevinhotedg: have you installed all the textures in proper path? It has changed..16:35
tedgTrevinho, I installed the packages...16:36
tedghttp://paste.ubuntu.com/5674559/16:36
tedgThough that certainly looks like the issue.16:37
Trevinhotedg: mhmh16:37
tedgTrevinho, Which package has the textures?16:37
Trevinhotedg: unity-common should16:37
Trevinhotedg: the crash is weird though16:37
Trevinhotedg: new textures should be in /usr/share/unity/icons16:38
tedgThis is the contents of my unity-common: http://paste.ubuntu.com/5674563/16:38
tedgIt seems like they're in /usr/share/unity/6/ ?16:39
Trevinhotedg: that was before the 100 scopes landed...16:39
tedgIt's your branch man ;-)16:39
tedgDo I need to repull it?16:39
tedgI must have grabbed it before you merged with trunk.16:40
chilukMirv for the compiz package in https://launchpad.net/~unity-team/+archive/sru/+packages that you uploaded can you tell me if a fix for https://bugs.launchpad.net/compiz/+bug/763148 is included?16:40
ubot5Launchpad bug 763148 in Compiz 0.9.9 "Adding/Removing an external monitor causes open windows to move to another workspace" [Medium,Triaged]16:41
* tedg is building again16:41
tedgOh my, I now need libunity 7, which requires a new libdee16:45
Trevinhotedg: yes, and new nux16:49
tedgOh, my, I'm not at that point yet.  Don't spoil the fun!16:49
tedg:-)16:49
Trevinhoehehe16:49
Trevinhotedg: well, not much changed on it, but there's a version mismatch since the daily ppa16:49
* Trevinho loved the times of the staging ppa for these cases16:50
Trevinhousing jhbuild is a solution, but also a waste of time for most of situations... I always prefer to build the minimum binaries  can...16:50
tedgYeah, jhbuild I don't find to be that useful.16:51
tedgbregma, Yes, I think it has regressed.16:52
tedgbregma, But I don't think it should block daily releases.16:52
bregmaagreed16:52
bregmaTrevinho, was there any plan for a library transition with all these 100-scopes changes?  I didn't see one....16:53
bschaefershould we create a new bug to get it down 'formally'16:53
Trevinhobregma: I don't know.. andyrock ^16:54
bregmabschaefer, https://bugs.launchpad.net/hud/+bug/118090316:54
ubot5Launchpad bug 1180903 in Unity "HUD command activation takes a long time when results are updating" [Undecided,New]16:54
bschaeferbregma, awesome then16:57
Trevinhotedg: fyi I resolved the conflicts in https://code.launchpad.net/~3v1n0/unity/gtk-wrapper-icon-info/+merge/164434 and added some more cleanups16:58
bschaeferbregma, I thought that bug would be marked as fixed when sil2100_ merged branch16:59
tedgTrevinho, Cool, I'm building the deps now, then I'll test.17:02
=== gatox is now known as gatox_lunch
bregmabschaefer, his merge commit message does not close he bug, so the merge will only fix the AP failures and unblock the daily releases17:07
bregmathe actual regression still needs to be fixed17:07
bregmaat which point we may want to back out his timout changes17:07
bschaeferbregma, right, which makes sense17:09
sil2100_bregma: that's right17:17
=== dandrader|lunch is now known as dandrader
=== sil2100_ is now known as sil2100
sil2100hmmm17:39
=== gatox_lunch is now known as gatox
=== jhodapp is now known as jhodapp|bbiab
=== mhall119_ is now known as mhall119
=== dandrader is now known as dandrader|afk
tedgTrevinho, So this looks good.  Do we have to wait for Jenkins to top approve?18:47
tedgJenkins said that it started 3 hours ago :-)18:49
kgunnmterry: ping18:54
Trevinhotedg: not strictly needed, we it will run everything before landing anyway18:55
tedgTrevinho, okay, then I'll top approve.18:57
=== dandrader|afk is now known as dandrader
=== jhodapp|bbiab is now known as jhodapp
sil2100fginther: ping20:49
fginthersil2100, what's up?20:50
sil2100fginther: hello Francis :)20:50
sil2100fginther: not sure who to ping regarding that, but some of the *scope* branches in CI add the prevalidation PPA, but I would need them to add the daily-build-next PPA instead now20:51
sil2100For instance, like in this MR20:51
sil2100https://code.launchpad.net/~submarine/unity-scope-audacious/bump-unity-deps/+merge/16436420:51
fginthersil2100, should they be in their own head/scopes.cfg stack now?20:52
sil2100fginther: oh, wait, hm... actually I added them to the head/unity.cfg stack20:53
sil2100fginther: so we should have a separate stack for those?20:53
* sil2100 checks the docs20:53
fginthersil2100, no, adding to the unity stack may be the right answer20:53
sil2100fginther: since I added them to unity stack already, the branch landed and the stack is redeployed20:54
fginthersil2100, but we need to be notified to redeploy the upsteam jobs20:54
fginthers/upstream jobs/upstream merger jobs/20:54
fginthersil2100, until then, the ci/autolanding will still build against the old stuff20:55
sil2100fginther: since we're doing the switch now, with all the bits-and-pieces ready, I think we'll need the scopes branches start using upstream20:56
sil2100fginther: the scopes are now in the cu2d config and redeployed20:56
fginthersil2100, what file did you say they are in?20:57
sil2100head/unity.cfg20:58
* fginther forgot to bzr pull20:59
fginthersil2100, sorry, it makes sense now20:59
fginthersil2100, I should just need to redeploy and it will switch to the -next ppa21:00
fginthersil2100, should be done in a minute21:01
sil2100fginther: thanks!21:01
fginthersil2100, we also need to restart the jenkins service, so it might be a while before the new jobs pick up branches21:01
sil2100fginther: ok, so clicking /rebuild will not work?21:02
fginthersil2100, no, since the ppa's changed the hook parameter will be wrong21:04
fginthersil2100, you can do a rebuild, but you would need to patch in the right name21:04
sil2100fginther: if not, the CI will pick up those branches and re-run by itself in some time, yes?21:08
fginthersil2100, the CI would only automatically run again if there was a new revision21:08
sil2100fginther: in the case of the scopes branches, there won't be new commits for now21:09
fginthersil2100, I'll scan those jobs for failures and restart any MPs I see when jenkins is back online21:09
fginthersil2100, shouldn't be too much of an issue21:10
fginthersil2100, gotta go21:10
fgintherhave a good weekend :-)21:10
sil2100Have a good weekend as well, thanks!21:14
sil2100Oh, btw!21:14
sil2100fginther: will this get done now?21:14
sil2100I mean, will that happen automatically?21:15
sil2100kenvandine: approved!21:17
=== salem_ is now known as _salem

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