/srv/irclogs.ubuntu.com/2014/11/10/#ubuntu-unity.txt

=== jhodapp|sick is now known as jhodapp
tsdgeosSaviq: what do you think of https://code.launchpad.net/~aacid/unity8/dierfv ?08:34
Saviqtsdgeos, surprise, did we not use it in the end?08:34
tsdgeosSaviq: we were using this only for the screenshots thing08:35
tsdgeosin the old apps scope08:35
MacSlowtsdgeos, hm... that is odd... the "if"-issue08:35
tsdgeosMacSlow: it's what jenkins says :D08:35
Saviqtsdgeos, ah, Flow...08:35
tsdgeosSaviq: yep08:35
MacSlowtsdgeos, jenkins is my personal enemy ;)08:35
Saviqtsdgeos, now I remember, I even got to that conclusion when I first saw your MP, still confusing this with Journals ;)08:36
Saviqtsdgeos, yeah, +1 of course08:36
tsdgeosSaviq: he he08:36
tsdgeosSaviq: ok :)08:36
tsdgeosMacSlow: you can not reproduce locally?08:43
MacSlowtsdgeos, just trying... but thunderbird is messing up my system, thus everytihng is dog-slow08:43
MacSlowand exiting thunderbird takes its time too :)08:44
=== duflu_ is now known as duflu
* MacSlow just can't get the hang of mutt08:44
Saviqseb128, do you have plans to land ubuntu-system-settings into rtm? or should I talk to someone else?08:58
seb128Saviq, ken has been handling the rtm landing, I can do one but better to sync up with him first08:58
Saviqseb128, ok, will do08:58
seb128Saviq, plans we have yes, there are a stack of fixes that should go to rtm or ota108:58
Saviqyup08:59
* Saviq preps unity8 alone then08:59
seb128what fixes do you lan?08:59
seb128d08:59
MacSlowtsdgeos, I've fixed the if-issue... but still have to iron out a part of the dismiss-test (sync/confirmation notifications were not available when I initially worked on the swipe-to-dismiss branch)09:08
tsdgeosok09:09
MacSlowtsdgeos, I'll mark my branch as "wip" for the time being.09:09
Saviqseb128, the accepted ones from the recent unity8 release https://launchpad.net/ubuntu/+source/unity8/8.01+15.04.20141107.2-0ubuntu109:09
seb128Saviq, hey, could you look at the current comment on https://bugs.launchpad.net/ubuntu/+source/ubuntu-system-settings/+bug/1390643 and comment on what you think is the best approach between those suggested?09:34
ubot5Ubuntu bug 1390643 in ubuntu-system-settings (Ubuntu) "Orientation lock switch doesn't notice gsettings changes" [High,Confirmed]09:34
Saviqseb128, https://bugs.launchpad.net/ubuntu/+source/ubuntu-system-settings/+bug/1390643/comments/309:40
ubot5Ubuntu bug 1390643 in ubuntu-system-settings (Ubuntu) "Orientation lock switch doesn't notice gsettings changes" [High,Confirmed]09:40
seb128Saviq, thanks09:40
=== Malsasa_ is now known as Malsasa
mzanettiSaviq: hey, what happened to the fact that we only lock the screen when waking up, as opposed to locking it before going to sleep?10:05
mzanettiiirc you once said there's a branch fixing it10:06
Saviqmzanetti, there were some approaches, but none that we implemented yet10:07
Saviqmzanetti, not a problem on krillin, so lower prio10:08
mzanettiah ok10:08
mzanettiinteresting... why isn't it a problem on krillin? display waking up slower?10:09
Saviqmzanetti, yup10:09
mzanettiSaviq: may I assign the RTM ones to you? given you do the cherry-picks... https://bugs.launchpad.net/ubuntu-rtm/+source/qtmir/+bug/135155910:13
ubot5Ubuntu bug 1351559 in unity8 (Ubuntu RTM) "Apps in spread are not anti-aliased" [High,In progress]10:13
Saviqmzanetti, yeah, I should've done that10:14
mzanettino prob10:14
seb128Saviq, if you use a Binding{} in the other way around too, don't you create a binding loop?10:33
Saviqseb128, kinda, but it's quickly interrupted by if (old != new) in all the setters10:34
seb128Saviq, well, it makes a warning be printed on stdout10:35
Saviqseb128, so even if you get a warning, it's benign10:35
seb128or stderr10:35
seb128I don't like getting warnings ;-)10:35
Saviqseb128, yes, that's why we need a BidirectionalBinding { } ;)10:35
seb128when is that coming? ;-)10:36
mzanettiSaviq: is there something like that in the works yet?10:53
Saviqmzanetti, not really, we were considering it with dednick at some point10:54
Saviqseb128, patches welcome ;)10:54
mzanettiwell, so far I have decent results with Binding {} from backend to ui, and using ui's onClicked handler to change the backend value10:55
mzanettithat's what I usually do in my apps10:55
mzanettibut I agree QML should make it somewhat easier10:55
dednickSaviq, seb128, mzanetti: there was something for that in the ubuntu-settings-components check sync branch, but seb didnt want a special component.10:56
seb128dednick, I didn't say I don't want a special component, but rather than it should be in the uitk10:57
mzanetti-110:57
dednickhttps://code.launchpad.net/~nick-dedekind/ubuntu-settings-components/lp1336715.check.sync10:57
mzanettishould be upstream Qt10:57
seb128mzanetti, that's "toolkit" for me10:57
seb128so +1 for qt directly10:57
mzanettiseb128: you said uitk :D10:57
seb128I didn't know if it makes sense for qt10:58
seb128but the more upstream/mainstream it can be, the better10:58
dednickhm. not sure if it makes sense for qt10:58
mzanettiI'd say yes... some sort of UiBinding {} component10:58
dednickhttp://bazaar.launchpad.net/~nick-dedekind/ubuntu-settings-components/lp1336715.check.sync/view/head:/Ubuntu/Settings/Components/signalbinder.h10:58
Saviqmzanetti, problem is you might need to do some reconciliation, especially in the case of floats and such, like which value should you decide on if both sides changed, what if one side is slow to apply etc.10:58
mzanettiright...10:59
mzanettistill I guess there could be something useful... if you need very special stuff you can do it the old way10:59
mzanettibut yeah... there's a danger of making the mistake the uitk does... trying to be too clever and in the end not being useful for anyone10:59
Saviqtotally11:00
* mzanetti thinks of some listitems11:00
dednicki could propose a branch for the uitk which includes the signal binder. i think the implmentations of the switch/check are to specific for uitk.11:02
facundobatistaHola11:08
Saviqmzanetti, hah, splash screens are not antialiased ;)11:49
mzanettimeh12:02
=== MacSlow is now known as MacSlow|lunch
=== dandrader is now known as dandrader|afk
Saviqmzanetti, hum, is it right that music app seems to be exempt from lifecycle still?13:04
mzanettiyes13:05
mzanettistill no playing music from apps in ubuntu :/13:05
=== dandrader|afk is now known as dandrader
Saviqah, playlist progression :|13:11
Saviqis what we're missing13:11
Saviqmzanetti, and yes, we know your feeling on our lifecycle ;P13:11
mzanettiit wouldn't be so bad if I would see any progress on the background stuff13:14
=== alan_g is now known as alan_g|lunch
mzanettidandrader: ping13:27
dandradermzanetti, pong13:27
mzanettidandrader: hey, having some weird issue with the EdgeDragArea. maybe it makes sense to you:13:27
mzanettidandrader: so I start dragging at the right edge and move the finger all across the screen to wards left13:28
mzanetti2 grid units *before* reaching the left edge, dragging goes to false13:28
mzanetti(not sure about the 2 grid units - its some half centimeter)13:29
dandradermzanetti, enable debug logs for TouchRegistry and DDA and then show me the log13:29
mzanettidandrader: that debug output is really useful :D13:31
mzanettiproblem solved13:31
dandradermzanetti, so, what was it?13:31
mzanettienabled went to false13:31
mzanettiso it makes sense13:31
dandradermzanetti, :D13:31
mzanettikinda hard to figure otherwise13:31
mzanettibut seems you did a good job with that debug stuff13:31
dandraderglad you liked it13:32
=== MacSlow|lunch is now known as MacSlow
=== Guest6771 is now known as balloons
mzanettidandrader: hey, this report came in. do you think this could be anything related to missing input? https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/139114914:20
ubot5Ubuntu bug 1391149 in unity8 (Ubuntu) "greeter not reacting to swipes" [Undecided,New]14:20
mzanettidandrader: if you think no, let's assign it to mterry, otherwise could you have a look?14:21
dandradermzanetti, looks like it could be related to the TouchGate in Greeter.qml. But I would rather keep working on shell rotation as I'm on a tight schedule14:26
mzanettidandrader: sure... just wanted to have some first assessment to know where to push it to14:26
dandradermzanetti, I will add a comment there14:26
mzanettithanks14:26
tsdgeospstolowski:14:53
tsdgeosping14:53
tsdgeosunping14:54
tedgSaviq, So it looks like qtmir is raising sigstop after the mir connection is setup, but it seems like that should be later, no?15:40
tedgSaviq, Like after notifications interfaces and other stuff U8 provides are setup?15:41
Saviqtedg, well, as I commented I think it'd be better, if possible, if the indicators tried to reconnect15:42
Saviqtedg, depends on the definition of unity8 'started'15:42
tedgSaviq, The problem is that it's not a "reconnect" issue for notification, but the bigger issue is pulse. QtMir is initializing Mir quicker than Pulse can get setup.15:43
Saviqand it's kinda tricky because we'd have to explicitly wait for all the services we know we're waiting on to say "yeah, we good, go!"15:43
tedgSaviq, We can't depend on pulse because the upstart job is an touch-specific thing :-/15:43
* tedg hates on the ubuntu-touch-session package a bit15:44
tedgSaviq, Confused? You mean plugins or services?15:44
Saviqtedg, well, so us pushing the sigstop further could just mean we're hiding the other issue?15:44
Saviqtedg, plugins == services15:44
Saviqtedg, like the notifications service is a plugin15:44
Saviqeverything is a plugin, really15:44
tedgSaviq, Ah, okay. That's confusing to me. Not sure why things shipped together need dynamic loading :-)15:44
tedgSaviq, So I was thinking just an idle function, when the mainloop gets idle then raise the SIGSTOP.15:45
Saviqoh I'm afraid that's gonna be far too late15:45
tedgWhy?15:46
tedgIt seems like that's when things have finished their work.15:46
=== dandrader is now known as dandrader|lunch
tedgKeep in mind that it should happen faster if we stop the CPU from context switching to indicators and the such by removing them from competing for CPU.15:46
=== alan_g|lunch is now known as alan_g
Saviqtedg, well, that's counter to the fact that ogra wants to start everything in parallel, as early as possible ;)15:49
Saviqtedg, because that would mean the UI is set up, the dash started and stopped drawing, things like that15:49
Saviqtedg, so you'd actually be able to start using your phone and only then would indicators show up (or well, they wouldn't at all, if you used the phone enough)15:50
Saviqtedg, as for "why plugins", all kinds of reasons15:50
tedgGenerally the goal isn't to start everything at once.15:50
tedgYou just want as many things as you have cores.15:51
Saviqkinda difficult to encode in upstart events15:52
tedgIt seems to me that there has to be a point where the Unity8 mainloop goes idle.15:52
tedgGenerally things like phases come out of the setup of the Upstart events.15:53
tedgNot officially, but causally. So for instance, we don't limit the indicators to a number, but start them after so that they end up staying out of the way.15:53
Saviqtedg, unity8's main loop (i.e. the one on the first thread) is actually Mir, not the Qt one, where all the dbus things happen15:54
tedgAh, okay. So would it be better to put a signal from the U8 application object when it thinks that it's done loading plugins?15:55
tedgOr do they all delay their inits?15:56
Saviqtedg, it's not just about loading, we'd need each "interesting" bit to report when they're ready16:01
Saviqtedg, as you're interested when it actually registers the name on dbus, not earlier than that16:01
tedgSaviq, Sure, that's why I was asking if the init was delayed. Sounds like it is.16:01
tedgI really feel like we should be more honest in the Unity8 startup reporting to Upstart.16:03
Saviqtedg, so if pulse is the real issue, how would delaying "unity8 started" even help? by chance?16:03
tedgBut it is looking to me like this isn't an "rtm sized" fix.16:03
Saviqtedg, I think we just need to be more granular actually16:03
Saviqtedg, each service should emit its own 'foo-started' signal16:03
tedgSaviq, Well it's just that indicators are getting started exactly at the same time as pulse.16:03
tedgSaviq, Almost any delay would help.16:04
Saviqtedg, well, yeah, but that's by chance, meaning that you could just as well put "sleep 2" in there...16:04
tedgThinking we might have to override in ubuntu-touch-session until that pulse job can be pushed upstream.16:04
tedgSure, but trying to fix the problem more than put a sleep in.16:05
tedgI think the "fixish" is just to override indicator sound to be "start on indicators-start and started pulseaudio"16:05
tedgBut that can only be in touch, not desktop.16:05
Saviqtedg, sounds like a good idea to add an .override then?16:06
tedgSaviq, Yeah, I don't like that solution, but looking like the best right now.16:07
tedgSaviq, I would still like to move the indicators signal into post-start though if that's cool with you.16:07
tedgSaviq, It doesn't fix the bug, but seems more correct.16:07
Saviqtedg, sure16:07
MacSlowCimi, what about https://code.launchpad.net/~macslow/unity8/icon-clipping-fix-1378417/+merge/241097 ... did you review it yet?17:10
CimiMacSlow, looked like a no brainer17:11
MacSlowCimi, sure... but you've not touched it :)17:11
CimiMacSlow, if you cannot think of any possibly wrong cases, let's approve17:11
CimiMacSlow, yeah sorry...17:11
MacSlowCimi, np... that's why I asked :)17:11
=== dandrader|lunch is now known as dandrader
MacSlowCimi, thx17:33
=== alan_g is now known as alan_g|EOD

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