/srv/irclogs.ubuntu.com/2015/09/04/#ubuntu-unity.txt

Saviqmorning07:07
Saviqmzanetti, fyi, bug #1491566 is getting more and more heat07:08
ubot5bug 1491566 in unity8 (Ubuntu) "Greeter and edges not responsive after an incoming SMS" [Critical,Confirmed] https://launchpad.net/bugs/149156607:08
Saviqmzanetti, also fyi, mir and u-s-c are there in silo 14 now, too07:40
Saviqthere's just one thing I don't necessarily understand there07:41
Saviqmir in wily is 0.15.1, in overlay it's 0.14.107:41
Saviqso not really a no-change rebuild, is it...07:42
tsdgeosSaviq: overlay (or vivid) has some 0.13 bits that end up in the image, i wonder if that's because some rebuilds are needed or because we actually want to ship those 0.13 bits to support "old" click apps08:07
Saviqtsdgeos, you mean the image has some 0.13 mir bits?08:08
tsdgeosSaviq: yep08:08
tsdgeosii  libmirclient8:armhf                                  0.13.3+15.04.20150617-0ubuntu1                        armhf        Display server for Ubuntu - client library08:08
tsdgeosii  libmircommon4:armhf                                  0.13.3+15.04.20150617-0ubuntu1                        armhf        Display server for Ubuntu - shared library08:08
Saviqhttps://launchpad.net/ubuntu/+source/mir vivid has 0.1208:09
Saviqhmm interesting what pulls that in08:09
* Saviq checks08:09
tsdgeosgdk308:09
tsdgeosand libubuntu-platform-hardware-api2:armhf08:09
tsdgeoswhich is also not used by anyone and could be dropped08:09
Saviqlooks like mir 0.14 was release without rebuilding08:10
tsdgeosunless we want it there for retro compatibility08:10
anpokbut platform-api was rebuilt with 0.1508:10
anpoksorry08:10
anpok0.1408:10
anpokbut I guess libubuntu-platform-hardware-api2 is listed somewhere - i.e. app support?08:10
anpokwhich version of gdk3?08:11
anpokvivid+overlay should have 3.14 and that was part of the 0.14 landing iirc08:11
Saviqanpok, http://pastebin.ubuntu.com/12271399/08:13
mzanettihey08:13
Saviqanpok, yeah, libgtk-3-0 deps on libmirclient8 (>= 0.12.1+15.04.20150324)08:14
Saviqbut that looks fine, >=08:15
Saviqah wait, 8, which means previous ABI08:15
Saviqbut apparently there's more like that08:16
Saviqsomehow mir-test-tools, mir-utils 0.14 also depend on libmirclient8 :?08:17
Saviqit looks like a bunch of things were not rebuilt for 0.1408:19
anpokhmm the gtk 3.14 that was built with mir-0.14 is not in the stable phone ppa08:23
Saviqah wait, qtubuntu-android is fine, the version in the overlay ppa deps on libmirclient908:26
Saviqhttp://i.imgur.com/999gfHq.png08:27
Saviqbut regardless of gtk, ubuntu-touch pulls it in, too08:28
anpokguess there might be apps using the api version 2 of platform api08:29
Saviqanpok, yeah, but shouldn't platform 2 be rebuilt against new mir? or has there been incompatibilities?08:30
anpoki wonder where the gtk3 package was lost.. we did spend some time porting and testing that on vivid08:30
anpokhm I dont know exactly.. it might leak mir parts08:31
anpok-might leak +leaks08:32
Saviquhm08:32
cimitsdgeos, how do you want to split the work for the filters?08:37
cimiI suppose I can do those components like the previews, shouldn't be too hard08:39
tsdgeoscimi: can you start trying to do the option selector component while i finish the mock backend for it and when we're both done we can hook it up in a test?08:40
cimiok08:43
cimitsdgeos, you have a junk branch I can rebase?08:47
cimitsdgeos, or we do overlay for now?08:47
tsdgeoscimi: yes08:49
tsdgeoscimi: https://code.launchpad.net/~aacid/unity8/filters08:49
=== maclin1 is now known as maclin
Saviq@unity please don't (re)target MPs to lp:unity8/overlay, we're getting rid of that asap08:56
tsdgeoscimi: just realized for that branch you'll need a unity-api that doesn't exist outside my computer, will junk push it somewhere08:56
cimiSaviq, what's the plan? clearing up the overlay review list then merge back?08:57
Saviqcimi, there's a silo merging trunk (just .po changes) into overlay, that's gonna land on overlay along with a few branches (silo 14)08:58
Saviqcimi, then we push that to trunk and delete overlay, so anything remaining there will have to be resubmitted against lp:unity8 again08:58
Saviqcimi, unfortunately there's no easy transition08:58
cimiSaviq, finishing approving all overlay branches then merge back no?08:59
tsdgeoscimi: you can use lp:~aacid/unity-api/ignore_filters_15.0408:59
Saviqcimi, and what about all the new branches in between/08:59
Saviqand how long do we keep overlay going?08:59
cimiSaviq, we try to approve quickly :P :D08:59
Saviqcimi, it's easy enough to just resubmit the remaining ones against trunk09:00
Saviqyou won't even have to rebase, as the history will be the same09:00
Saviqmzanetti, ok, silo 14 looks sane now10:27
Saviqmzanetti, but since it's an upgrade of mir into vivid, we'll need to test Mir, too, or employ someone from there to do it10:27
Saviqhmm but wait, we need qtubuntu and platform-api in there now, too, then10:28
mzanetti:)10:28
mzanettisee you on monday then, I guess10:28
mzanettijust make sure you don't pull in the kernel10:29
Saviqd'oh10:30
greyback__Saviq: only if mirclient api changes...O hope10:30
* Saviq checks10:30
Saviqhmm libmirclient9, so hopefully that's fine10:31
Saviqmirserver got a bump10:32
greyback__should only impact qtmir10:33
Saviqand mirplatform10:35
Saviqso need to rebuilt qtmir for sure10:36
* Saviq kicks10:38
tsdgeoscimi: how did the listview for comments end?10:43
cimitsdgeos, it sounds to me we will need a redesign anyway10:43
tsdgeoswhy?10:43
cimitsdgeos, something like collapsed by default10:44
cimitsdgeos, are we really going to show 1000+ comments in a single listview?10:44
tsdgeoswhy not?10:44
tsdgeoswhat does it matter if they are collapesed or not?10:44
cimitsdgeos, if our issue is loading, we don't have a problem is they are collapsed by default10:45
tsdgeoshow do you collapse 1000 comments?10:46
tsdgeosi don't even understand what you mean by collapsed comment10:46
tsdgeosdo you mean not showing them at all10:47
tsdgeos?10:47
tsdgeosand have a "show comments"?10:47
Saviqgreyback__, mzanetti, any ideas to investigate bug #1491566 ? I only have mako and krillin, and have not seen that issue10:47
ubot5bug 1491566 in unity8 (Ubuntu) "Greeter and edges not responsive after an incoming SMS" [Critical,Confirmed] https://launchpad.net/bugs/149156610:47
mzanettiSaviq, spend a while trying to repro yday... haven't managed10:48
Saviq*spent10:48
=== greyback__ is now known as greyback
mzanettiOh, how I missed that10:48
* Saviq no has time machine to spend time yesterday10:48
Saviq:D10:48
greybackso nice to have captain pedantic back10:48
cimitsdgeos, yeah10:49
Saviqmzanetti, that just means you didn't fulfil all your responsibilities :P10:49
cimitsdgeos, showing like 5 last comments by default10:49
mzanettiSaviq, hah... fair enough10:49
cimitsdgeos, having a button with "show more"10:49
cimietc10:49
tsdgeoscimi: that's a poor man solution10:49
tsdgeosi can't make a list view10:49
tsdgeosso i'll just give you 5 items10:49
cimitsdgeos, I think what we have now is a poor man solution10:50
tsdgeoscimi: right, that's why we need a listview instead of a row10:50
cimitsdgeos, showing the user an endless list of comments because we didn't plan to limit them10:50
cimiI don't think people care of reading 1000 reviews, the most recent ones are enough10:51
cimiI was wondering if there was a design for that, or we just ignored the problem10:51
cimitsdgeos, anyway I did some work for limiting the delegate range, and the issue I had was calculating the limits when the listview is placed on screen10:52
cimithe first value ever I mean10:52
tsdgeosok, i disagree with you that we should just drop this optimization and wait for a design change that makes us not care about the performance10:53
tsdgeosbut we have work to do on filters10:53
tsdgeosi'll open a bug10:53
cimitsdgeos, it's not dropped, it's on hold10:55
cimitsdgeos, I have some local code, but couldn't figure out how to setup the initial displaymarginbeginning/end10:56
Saviqmzanetti, greyback, ideas for data to collect off of a phone that went into the no-input state could be useful, I initially thought stacktrace, but unity isn't hung, so not useful10:56
cimitsdgeos, looks like Component.onCompleted is not the signal I want10:56
Saviqmzanetti, greyback, right now I'm suspecting either Qt's input machinery gone mad, or u-s-c unfocusing the session, only two that make sense to me?10:57
cimiI want the signal that is called when the Item is actually placed on screen in its final position10:57
mzanettiSaviq, debug prints, IMO10:57
Saviqyeah, we need a debug mode for that10:58
Saviqand many more debug calls in our code10:58
cimionly solution for that might be onYChanged but from outside the preview, which implies adding more API through the loaders just for this use case, and I wasn't happy with that10:58
mzanettimaybe even something that's auto-enabled in *-proposed images and disabled in stable ones10:58
greybackSaviq: it is that *nothing* in unity8 is getting events, or just edges?10:58
Saviqgreyback, nothing10:58
greybackcategory logging ftw10:58
mzanettigreyback, the problem with that is, when the issue happens it's not enabled10:59
Saviqyeah we need to have a discussion about that platform-wide10:59
greybackmzanetti: sure, but with a small dbus interface we could enable at runtime10:59
mzanettifair enough...10:59
Saviqmight still be too late ;)10:59
mzanettiindeed10:59
SaviqI was surprised to see how much android logs all the time10:59
mzanettiyeah... IMO *-proposed should have debug prints enabled10:59
mzanettiit makes sense... really11:00
Saviqeven stable should11:00
Saviqsome in-memory circular buffer or something11:00
Saviqthat gets read when an issue is reported11:01
Saviqotherwise we're too late11:01
Saviqdandrader, we're discussing bug #1491566, any ideas for debugging that?11:03
ubot5bug 1491566 in unity8 (Ubuntu) "Shell not responsive after an incoming SMS" [Critical,Confirmed] https://launchpad.net/bugs/149156611:03
Saviqat least we should prepare a debug-enabled silo that people that can reproduce can install it11:03
greybackSaviq: I'd agree with you, either unity8's surface looses focus in USC, else Qt's input processing gets stuck in a bad state.11:04
greybackwe've had to do filtering to fix up Qt's inputting with Mir's dirty input, perhaps something slips through still and breaks it11:05
dandraderwell, at least on N9 times logging was hampering overall performance11:14
dandraderiirc11:15
Saviqsure, we'll need to measure the impact, but maybe we can hold an in-memory log of, say, a megabyte or something, one that's quite verbose, and grab its contents when a problem occurs11:18
Saviqsomething like that should not impact performance much11:18
Saviqwith no I/O involved11:18
Saviqgreyback, dandrader, could one of you please start a couple of MPs adding some debug logging to unity8 and u-s-c where you think we'd get most info on where the input goes, please?11:19
dandraderI think we could enable mir input logging in usc and unity8 (ie, qtmir)11:19
Saviqplease please11:19
dandraderand qtEventFeeder logging in qtmir as well11:20
dandraderthat should be enough to find out what's happening11:20
Saviqdandrader, sounds like you just volunteered?11:20
dandraderSaviq, it's not a matter of MPing, we just have to set the environment variables11:20
Saviqdandrader, oh even better, can you describe what needs to be done in the bug report?11:21
greybackdandrader: will some be very noisy though?11:21
dandradergreyback, it will11:21
tsdgeosSaviq: mzanetti: cimi: in GenericScopeView we have this showPageHeader property that noone uses and that complicates the code a bit by introducing a Loader, can i kill it?11:21
cimitsdgeos, let me ready why we have that11:22
cimiread11:22
greybackSaviq: I fear we can realistically only debug log on every input event, which is very noisy11:22
dandraderSaviq, ok, I will try out on a device and then post the step-by-step instructions11:22
cimitsdgeos, who wrote that?11:22
cimiand why we did?11:23
dandradergreyback, the idea is that whoever wants to reproduce this bug will enable the input logging. so I don't see the problem11:23
Saviqyeah, ↑11:23
Saviqsomeone who saw this on his device(s011:23
greybackah in that case, the env vars are exactly what that's for11:24
Saviq+Shift11:24
Saviqit should be made into a runtime switch11:24
Saviqenv is so 1980s11:24
greyback+111:24
greybackoh like totallay11:25
tsdgeoscimi: i did, because we used it another life :D11:35
tsdgeosfood!11:35
cimitsdgeos, well, then go for it :)11:45
=== JMulholland_ is now known as JMulholland
=== MacSlow is now known as MacSlow|lunch
=== alan_g is now known as alan_g|lunch
=== alan_g|lunch is now known as alan_g
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
cimitsdgeos, for the filter loader, something around this? http://paste.ubuntu.com/12273227/13:53
cimitsdgeos, similar to the preview widgets basically13:54
tsdgeoscimi: yaeh, leave out the triggered for now, not sure how we'll handle that13:55
tsdgeosand the expanded13:55
cimitsdgeos, no expanded?13:55
cimitsdgeos, I thought we would have some expanded filters like the calendar and such13:55
tsdgeoscimi: as far as i understand you either shown them or not13:56
tsdgeosthere's no shown but unexpanded status we have in the preview widgets13:56
tsdgeosso no need to tell the inner widget if it's expanded, no?13:58
cimimaybe :)13:59
cimitsdgeos, or we can leave it like showPageHeader for another life :)13:59
tsdgeosjust don't add it for now14:00
tsdgeoswe'll add it if we need14:00
=== MacSlow|lunch is now known as MacSlow
mterryWhat are the current instructions for pocket desktop?  rc+proposed + Silo 0 + external hardware?14:04
mterrySaviq, ^14:04
Saviqmterry, silo0 isn't in a good state afaik, and it won't get better14:04
Saviqgreyback, ↑?14:04
greybackmterry: best instructions are on the last pages of https://docs.google.com/document/d/1EtDf3MXVrTaW3xfPNUALYmKTNmTHxBAAtTmgnpRX-E4/edit#14:05
Saviqmzanetti, why you not in #ubuntu-mir? ;)14:06
mzanettiSaviq, dunno... didn't feel the urge :D14:07
mzanettisometimes my bounces forgets all the channels and I have ro rejoin14:07
Saviqdumb bouncer14:07
mzanettinow I'm in #ubuntu-mir?  :D14:07
Saviqmust've been a back-street boxer before and the brainz not working that well anymore14:07
mterryWhoops, got disconnected...  will repeat last messages just in case.  Ignore if you already saw them  :)14:15
mterry<mterry> greyback, ok, thanks.  That still suggests silo 0, so I'm uncertain how testable that is.  But it might get me somewhere14:15
mterry Saviq, actually, do you happen to know which bits of silo0 aren't landed yet?  maybe I can just build those myself14:15
greybackmterry: why are you trying this?14:15
greybackit's just kept alive for demos, we're working on landing the bits in it properly14:16
mterrygreyback, I wanted to help pick off some bugs, but just wanted to recreate setup first14:17
=== charles_ is now known as charles
=== dandrader_ is now known as dandrader
dandraderSaviq, your script tells me lp:unity8/overlay has a bogus tag14:47
dandraderSaviq, do you confirm that?14:47
Saviqdandrader, yeah I saw that, it decides it's bogus locally14:47
Saviqdandrader, but it's fine remotely14:47
Saviqand well, it's really bzr that's saying that14:48
Saviqdandrader, trunk has the same14:48
dandraderSaviq, so I should just ignore it, I guess?14:48
Saviqdandrader, yeah14:48
dandradertsdgeos, got an easy one for you https://code.launchpad.net/~dandrader/unity8/stabilizeShellTestAgain/+merge/27018814:55
tsdgeoscool, will do on monday, early EOD for me today, going to the mountains for the weekend14:57
tsdgeoswith forecast of rain14:57
tsdgeos:/14:57
dandradertsdgeos, hmm, montains. sounds great14:57
dandraderminus the wheather forecast, naturally :)14:57
tsdgeosoh well it's 50% chance rain14:58
tsdgeoshoope we're lucky14:58
* tsdgeos waves14:58
Saviqltinkl, is it on purpose that reboot button is grey now?15:00
ltinklSaviq: what branch are you running?15:03
ltinklSaviq: but generall yes, it was a request from design IIRC15:03
Saviqltinkl, I'm testing silo 14 which has your global shortcuts branch in it15:04
=== JMulholland_ is now known as JMulholland
dandradermzanetti, ping15:27
mzanettidandrader, hey15:27
dandradermzanetti, so greyback wants me to move the cursor image provider (and probably some other related stuff will have to go along with it, will see while I'm at it) from qtmir to unity815:28
dandradermzanetti, should it go to Unity.Utils or own module?15:28
dandraderI'm not sure what's the best or preferred option15:29
mzanettidandrader, not sure either, but I would probably go with Utils... I feel we're having a bit too many plugins already15:29
mzanettidandrader, had a discussion about this a while back with some people but we didn't really reach a conclusion15:29
dandradermzanetti, which raises the questions: what's worse? a big module or several smaller ones? (I don't have the answer)15:30
* greyback likes lots of little modules. Is modular :)15:30
mzanettigreyback, I am a bit concerned about loading times etc, but in general yes, modular is good15:30
mzanettigreyback, but then seeing that we have a bunch of plugins that only have like 2 lines of actual code ...15:30
mzanettidandrader, you might want a MouseUtils? I assume we'll need those edgepush areas etc soon15:31
mzanettialthough they could go to Gestures too probably...15:32
greybackI can't say without having some numbers on the topic. the searching the filesystem and symbol resolution does have a little cost, dunno if it's much in the grand scheme of things tho15:32
mzanettigreyback, it's not much... but once we have 100 plugins it's 100 times not much15:32
greybackwhich still may not be much ;)15:33
greybackI have no idea of the perf impact15:34
mzanettidandrader, actually, use it's own module... makes it easier to reuse without pulling unity specific utils15:34
mzanettidandrader, I just came by this: http://doc.qt.io/qt-5/qml-qtquick-mousearea.html#cursorShape-prop15:38
mzanettidandrader, will this be supported?15:38
dandradermzanetti, yes15:38
mzanettiawesomes15:38
dandradermzanetti, but unfortunately not enough for us15:38
mzanettireally? what's missing?15:38
dandradermzanetti, Qt's enumeration doesn't contain all the 8 different shapes we use for window resize15:38
mzanettilol15:38
mzanettihad no idea it's that many15:39
greybackwe can always define extra enums on top15:39
mzanettican we extend?15:39
mzanettidoesn't sound hard to patch in, and we're in a better position of upstreaming things lately15:39
dandradermzanetti, so had to add a Mir.cursorName API for that. but the two coexist fine15:39
dandradermzanetti, thing is, Qt is all about multiplatform15:39
mzanettisure...15:40
dandradermzanetti, so I don't know if all those extra cursors make sense in other platoforms15:40
mzanettibut that doesn't mean only supporting the smalles common set15:40
dandradermzanetti, might be Xcursor specific15:40
dandradermzanetti, right15:41
dandradergreyback, about the extra enums, I looked at this possibility, but looking at Qt source code, on how it handles the cursorShape value, it doesn't seem possible15:44
greyback:(15:44
greybackI guessed it was just a number in the end15:44
mzanettigreyback, it's not that easy to register them for QML etc15:45
greybackmzanetti: they're still just numbers to qml tho, no?15:45
mzanettigreyback, right... you sure can use integers, but that's not really nice... if extended it really should be additional Qt.CursorShapeBlabla things15:46
mzanettino?15:47
greybackwell if we're extending, we should use our own namespace. Unity.CursorSomething ?15:47
mzanettinot if extending upstream... otherwise yes, but I thought that's what daniel does15:48
greybacktrue15:50
=== dandrader is now known as dandrader|afk
mzanettigotta go15:53
mzanettisee you all on monday15:53
mzanettio/15:53
=== alan_g is now known as alan_g|EOW
=== dandrader|afk is now known as dandrader
dandradergreyback, still around?18:40
mterryAre bluetooth and mako supposed to be friends?  My BT keyboard isn't working so hot18:46
=== dandrader is now known as dandrader|afk
=== cwayne is now known as cwayne-afk
=== dandrader|afk is now known as dandrader

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