[06:16] moooarning [07:38] lpotter: hi, you still there? [07:38] tsdgeos: yeppers [07:38] lpotter: welcome! [07:39] thanks [07:39] just learn your starting today ^_^ [07:39] :) and I learned there's another brissie troll here [07:40] hope it wasn't too bad being a new starter when almost everyone else is still sleeping [07:40] chris? [07:42] not too bad. I'm used to trying to find things to do. [07:42] :) [07:44] discovered this weekend that NetworkManager backend for QtBearer is not working. Has had no love for quite a while [07:44] Saviq: so mzanetti was preparing a silo on thursday (friday was public holiday in germany) can you add https://code.launchpad.net/~aacid/unity8/wait_ap_test/+merge/237082 to it? it fixes a few ap failures here [07:45] Saviq: and https://code.launchpad.net/~aacid/qtmir/sessionAuthorizerWaitSignalListener/+merge/237064 a race condition in qtmir starting, but i'd like Gerry to have a look when he comes back (he was on holiday last week too, should be back today afair) [07:46] lpotter, could be related to bug #1357321? [07:46] bug 1357321 in qtubuntu (Ubuntu) "QNetworkAccessManager doesn't support roaming on Ubuntu" [Critical,In progress] https://launchpad.net/bugs/1357321 [07:47] tsdgeos, will do [07:48] lpotter: we have a canonical agreement for https://codereview.qt-project.org you want to probably sign up with your new @canonical.com address since it takes a while being a company agreement thing [07:48] Saviq: possibly. especially if the generic plugin is also being installed as well. [07:49] ahh ya.. [07:49] lpotter, Mathieu is cyphermox on IRC if you'd want to touch base on how far he'd gotten [07:50] lpotter, you can always ask the mup bot on irc.c.c for details like that (/msg mup poke Mathieu [07:50] ) [07:50] Saviq: and https://code.launchpad.net/~aacid/unity8/qt_strict_iterators is waiting for you too ;) [07:50] orly? [07:50] well i guess it's your decision if we want that or not :D [07:50] beign tech lead and stuff ;) [07:51] tsdgeos, +1 on compile-time errors, so ACK ;) [07:52] i mean it is not that without it things fail, but it'll be a bit slower, and it's not so hard to do right [07:52] so that's why i added it [07:52] tsdgeos, yup, looks good [07:53] tsdgeos, I'll leave the qtmir MP out for now [07:53] +1 [08:19] pstolowski: can you merge feeds? [08:20] tsdgeos, sure [08:22] tsdgeos, pushed [08:22] tx [08:25] tsdgeos: hey, do you know what happened with the indicators silo in the end? [08:26] mzanetti: 19? [08:26] can't remember the number [08:26] tsdgeos: well, we already had it merged to utopic [08:27] tsdgeos: and then design blocked its RTM landing because apparently it was not what they wanted [08:27] mzanetti: ah, and it had to be rtm'ed ? [08:27] no idea then [08:35] pstolowski: did you implement the child feed thing yet? [08:35] tsdgeos, almost, i've a bug to fix [08:35] tsdgeos, should be ready today [08:36] cool [08:42] welcome back Saviq! hope your movies were enjoyable and tear-free :) [08:42] Mirv, they were indeed! :D [08:44] Mirv, and thanks, glad to be back (that's what holidays are for, to appreciate what you have back home, are they not?) [08:44] like a dishwasher... a proper shower ;D [08:44] that they are for :) [08:52] pstolowski: ok, support is in on my side [08:54] tsdgeos, awesome, i also just fixed my bug [08:54] tsdgeos, is this the same branch of yours, or a new one? [08:54] pstolowski: same [08:55] o/ [08:55] greyback, elo o/ [08:55] Saviq, hey! :) [08:56] Saviq: remember what you do here again? [08:56] pstolowski, helo [08:56] greyback, yes, annoy people, never forgot [08:56] I'm only a week away and my user settings have been lost [08:58] greyback: i've a fix for you [08:59] greyback: https://code.launchpad.net/~aacid/qtmir/sessionAuthorizerWaitSignalListener/+merge/237064 [08:59] tsdgeos, fyi, this is mine: lp:~unity-api-team/unity-scopes-api/child-scopes-option and lp:~unity-api-team/unity-scopes-shell/display-child-feeds (2 branches, scopes api changes needed) [08:59] greyback: feel free to bin it, but the analysis of the problems wiht AP (no idea why it doesn't happen at all/more often with non AP) is there [09:00] pstolowski: oh, so can't use feeds anymore? [09:00] ok [09:00] tsdgeos, i can merge back to feeds [09:01] tsdgeos, actually, let me to it that way [09:01] pstolowski: i'd say it'll help, then i only need to find out how do i have unity-scopes-api to https://launchpad.net/~unity-team/+archive/ubuntu/phone-right-edge/ [09:01] mzanetti: Saviq: ↑ any quick recipe crash course? [09:02] tsdgeos, feeds updated [09:03] tsdgeos: ? [09:04] tsdgeos, https://code.launchpad.net/~unity-api-team/unity-scopes-api/child-scopes-option/+new-recipe [09:04] ah [09:04] tsdgeos, name it -overviewlist instead of daily [09:04] tsdgeos: yeah, just make sure to select unity-team as the owner [09:04] tsdgeos, ↑ that [09:04] Utopic as the series [09:04] and recipe: [09:04] tsdgeos: and for version I usually prepend 1: and append +{time} [09:05] # bzr-builder format 0.3 deb-version {debupstream}-{time}~{revno} [09:05] is what's used in the unity8 recipe [09:06] the *correct* thing to do would be to do: [09:06] lp:unity8 [09:06] ok [09:06] merge lp:~foo/unity8/foo [09:06] so that it's always based off of trunk [09:07] so did this [09:07] https://code.launchpad.net/~unity-ui-team/+recipe/unity-scopes-api-overviewlist [09:07] hope is not too horrible :D [09:07] https://code.launchpad.net/+help-code/recipe-syntax.html [09:07] tsdgeos, yup, looks fine [09:07] /food === pstolowski is now known as pstolowski|erran === vrruiz_ is now known as rvr [09:51] vesar: sorry things are releases so fast that i can't get the ppa to be in a state that you can actually use it, trying to [09:58] tsdgeos, thanks for trying. Let us know if anything we can do to help you.. [11:15] Holas [11:16] o/ [11:19] pstolowski|erran: ping [11:22] facundobatista: are you involved in the shope scope? [11:22] -e [11:23] tsdgeos, mmm... don't think so... it's a client scope, right? my doubt is if it would aggregate remote scopes like amazon or ebay [11:23] facundobatista: i mean the app store [11:23] man i'm bad with words today [11:23] tsdgeos, no, not at all [11:24] ok [11:24] tsdgeos, alecu is [11:25] alecu: you there? [11:29] dandrader: can you remerge master to lp:~dandrader/qtmir/UbuntuKeyboardInfoQMLSingleton and my list branch to lp:~dandrader/unity8/listOnBottomSwipe_touchOwnership ? [11:29] dandrader: good morning btw === pstolowski|erran is now known as pstolowski [11:30] tsdgeos, what master? [11:30] dandrader: trunk, lp:qtmir [11:31] tsdgeos, pong [11:31] pstolowski: i'm having a look at https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1343242 [11:31] Ubuntu bug 1343242 in unity8 (Ubuntu) "Departments break if going to a subdepartment of Store" [High,Triaged] [11:32] and at this moment the description of the bug looks much more a scopes bug than an unity8 one [11:32] i mean when going to the store with scope://com.canonical.scopes.clickstore?q=&dep=accessories [11:32] i get [11:32] ooh blank dash :| [11:32] navigation and currentNavigation to be "All" [11:32] not sure what i am supposed to do with that [11:32] oh, back...hmm [11:41] tsdgeos, the scope returns this: http://pastebin.ubuntu.com/8506792/ [11:42] tsdgeos, so perhaps there is a bug in scopes plugin if it says 'all' is the current [11:43] tsdgeos, rebased lp:~dandrader/qtmir/UbuntuKeyboardInfoQMLSingleton on top of latest lp:qtmir. btw, do you still "need information" there? [11:43] tsdgeos, i'll take a look at the plugin; but i've a feeling that even if it's fixed, it won't solve our problem [11:44] pstolowski: what is "our problem"? :D [11:44] that you can't go back? that it dooesn't list more siblings? [11:44] well of course it doesn't list more siblings, you just said it has 1 child :D [11:47] tsdgeos, yeah, siblings. but you're right, perhaps we will have to tell scopes to return siblings.. ugh [11:47] tsdgeos, but lp:~aacid/unity8/list_on_bottom_swipe is still "work in progress" [11:48] pstolowski: well i can't show siblings if i'm not told about them, and http://pastebin.ubuntu.com/8506792/ shows there is no siblings [11:48] dandrader: yeah it's not meant to be merged yet, it's part of the ppa because i'm trying to get design to have a look [11:48] but the world moves so fast i have not been able to find a time where the ppa has everything it needs to have yet [11:49] tsdgeos, I made this for kgunn last friday: lp:~dandrader/unity8/listOnBottomSwipe_touchOwnership [11:49] tsdgeos, so he could try out the two things together [11:50] dandrader: but that's weird since we already had a recipe that did that before, is it because there were confclits? [11:50] tsdgeos, yeah. big conflicts in the Dash [11:50] i see [11:50] tsdgeos, as both branches modify the EdgeDragArea in the Dash (that brings up dash overview) [11:51] right [11:52] tsdgeos: sessionAuthorizerWaitSignalListener approved, thanks! [11:52] :) [11:53] alecu, Cimi, how is it that the apps' titles are still center-aligned with https://code.launchpad.net/~unity-team/unity8/card-visual-tweaks/+merge/234332, even though they don't explicitly request centering? or do they? === MacSlow is now known as MacSlow|lunch [11:55] tsdgeos, is you ppa up-to-date with the branches from this morning? [11:56] pstolowski: not yet, need dandrader to merge stuff into lp:~dandrader/unity8/listOnBottomSwipe_touchOwnership [11:56] can you guys retrigger a build of https://code.launchpad.net/~unity-ui-team/+recipe/unity8-overviewlist once it's merged and pushed? [11:56] off to lunch now === alan_g is now known as alan_g|lunch [12:12] tsdgeos: pong [12:13] Saviq: hi! we finally agreed with design and Cimi to have centered titles when more than one line [12:16] alecu, well, yeah, but didn't we agree that they have to be explicitly requested by the scope? [12:18] alecu, I mean ["title"]["align"] !== "center" [12:23] hi thostr_, pstolowski, quick question: what's the LP project to file bugs against the 7digital scope? [12:23] facundobatista: ^ [12:24] dpm, ubuntu-rest-scopes [12:24] dpm, ubuntu-rest-scopes [12:24] :) [12:24] first! ;) [12:27] nice :) [12:27] thanks :) [12:32] Saviq: yes, I remember you wanted to make that change, but I could not remember the why. And we couldn't find a good reason to keep left-aligned the default for only titles, or titles with overflow, so we agreed to have them both centered, and updated the spec with that. [12:33] Saviq: if you still think lef-aligned should be the default for "titles (1 row)" and "titles (overflow to second line)", let's talk some more. [12:33] *left [12:34] tsdgeos, well, I merge the latest lp:~aacid/unity8/list_on_bottom_swipe/ onto lp:~dandrader/unity8/listOnBottomSwipe_touchOwnership but all it did was bring in the latest translations from lp:unity8 [12:34] s/merge/merged [12:34] alecu, the problem is that we impose them being centered on scopes that might not want it [12:34] alecu, anyway, I'll come back to it then [12:34] facundobatista, here you go bug 1377912 :) [12:34] bug 1377912 in Ubuntu Rest Scopes "[7digital] Collapse categories to make browsing easier" [Undecided,New] https://launchpad.net/bugs/1377912 [12:35] tsdgeos, because the only commit you had in lp:~aacid/unity8/list_on_bottom_swipe since the last time I merged it was syncing with trunk [12:40] dpm, that's how Design indicated us to do it [12:42] dpm, mmm... wait === alan_g|lunch is now known as alan_g [12:58] facundobatista, right, but it seems to me it'd be good for them to re-review it, as as it is now, it's a bit difficult to use. [12:59] facundobatista, another question: I've had this happen a few times, and now I can reproduce it: after having the 7digital scope returning some results, and then navigating back to another scope, if I then do the same search on the 7digital scope, I get no results at all [13:00] in fact, I don't get results for any search on that scope. Any ideas what could be going on? [13:00] dpm: facundobatista: design asked us to make some result lists in the app store shorter, and add that "see more" link. [13:00] dandrader: hmmm? bah not smart me had forgot to push [13:00] can you merge now? === MacSlow|lunch is now known as MacSlow [13:01] alecu, yeah, that'd be perfect for the 7digital results too [13:01] dpm, I was wrong, Design asked for the more/less button, so your bug is valid [13:02] tsdgeos, dandrader, btw there's no need to create a single branch to do a recipe build, you can base off of one branch and merge others in the recipe definition [13:02] Saviq: yeah we could do that [13:02] dpm, searches not returning results are tricky, we'd need to see particular cases in the logs (it'd be great if you get the phone ones) [13:03] Saviq: if we knew which one is going to land first :D [13:03] tsdgeos, done [13:03] dandrader: tx [13:03] tsdgeos, doesn't really matter, you can always change the recipe [13:03] facundobatista, which log would you need? [13:04] Saviq, there are merge conflics between the two and we don't know which one is landing firt, thus the separate branch [13:04] merging the two [13:04] ah ok, conflicts, understood [13:04] dpm, the smartscope proxy one [13:05] dpm, in ~/.cache/upstart/ , IIRC [13:06] question: I have the emulator running, how do I get inside it with "adb shell"? [13:07] facundobatista, ok, thanks. Now I seem to be getting results, though [13:07] facundobatista, exactly the same way [13:07] facundobatista, adb devices should show you "emulator:5554" or so [13:07] facundobatista, just run "adb shell" :) [13:07] facundobatista, it's a device like any other for adb [13:07] Saviq, "adb shell" tells me "device is online" [13:07] or if you've got both an emulator running and another device connected, you can specify the device id [13:08] facundobatista, what does "adb devices" say? [13:08] Saviq, emulator-5554 (which is ok) "offline" (which is not :p ) [13:08] facundobatista, right, that... === _salem is now known as salem_ === dandrader is now known as dandrader|afk === karni is now known as karni-lunch [13:40] tsdgeos, fyi, child feeds are shown correctly (but you need to manually modify one of the aggregator scopes .ini files to declare them) === pstolowski is now known as pstolowski|schoo [13:41] pstolowski|schoo: nice === dandrader|afk is now known as dandrader [13:59] mzanetti, kgunn, so in silo 19 I'm getting one AP failure: unity8.application_lifecycle.tests.test_application_lifecycle.ApplicationLifecycleTests.test_click_app_icon_on_dash_must_focus_it [13:59] that expected? [13:59] tsdgeos, ↑ [13:59] hmmm [14:00] is think that one fixed by https://code.launchpad.net/~aacid/unity8/wait_ap_test/+merge/237082 [14:00] not very smart me didn't write down which one that fixed ^_^ [14:01] Saviq: but you can try modyfying emulators/dash.py manually without repackaging stuff [14:01] after all is a two lines change [14:01] Saviq: yep, that's one of them [14:01] tsdgeos, that MP is included already [14:01] Saviq: ah [14:01] hmm... [14:02] then maybe its something else [14:02] i don't remember any lifecycle failing on my tries on silo 19 on friday [14:02] only the host of indicator ones [14:02] vesar: if you could install this ppa on a phone now and show it to designers that want to see the "scopes overview" replacement that'd be great https://launchpad.net/~unity-team/+archive/ubuntu/phone-right-edge [14:03] vesar: "soon" to make sure noone does a review of one of the affected projects :D [14:07] Saviq: is it consistent....i had some flaky ones last week where i ran 5 times, was about to give up...on 6th time got a pass [14:07] kgunn, didn't try 6 times yet, but failed 3 out of 3 [14:07] and why is unity8 taking so long to stop :| [14:08] Saviq: want me to look at it? [14:08] note that we still have a media-hub issue that is causing [14:08] dbus timeouts [14:08] specially on the unity8 stop phase [14:08] AlbertA2: aarrgg [14:08] i thot that landed ? [14:08] ah that [14:08] I'm trying to chase those demons away... [14:08] kgunn: apparently it's a different manifestation [14:08] hehe [14:08] kgunn: similar but different [14:08] mzanetti, what the cleanest way to check (in a tst_Shell test) that PhoneStage is showing the spread? [14:09] dandrader: not at all :P [14:10] hmm seems I just have too many apps in my launcher [14:10] dandrader: btw, test kraken all weekend, carried as my phone...i never saw or felt anything i thot was odd wrt touch input [14:10] kgunn, \o/ [14:10] dandrader: i only logged 2 bugs from the weekend, surely unrelated [14:10] https://bugs.launchpad.net/dialer-app/+bug/1377334 [14:10] Ubuntu bug 1377334 in dialer-app "cancel sync crash contacts" [Critical,New] [14:11] dandrader: so you can probably do that, but there is no clean way... you need to findChild() on the listview and check contentX or so [14:11] dandrader: or "phase" [14:11] https://bugs.launchpad.net/barajas/+bug/1377788 [14:11] Error: ubuntu bug 1377788 not found [14:11] dandrader: I figure we already have other tests that make the test suite phone specific [14:11] tsdgeos, Hmm I hope I'm not already late with it. [14:12] vesar: you should be fine :) [14:12] dandrader: why do you want to test that? for the kraken? [14:12] Saviq: ah, that's one of the things, our tests require kind of a clean slate, one of the tests was failing for me because i didn't have a favorited music scope [14:12] mzanetti, tsdgeos, kgunn, yeah, as you were, we need to fix the launcher emulator to deal with "the app is at the top of the launcher, far off screen" [14:12] mzanetti, yes. a regression test [14:12] dandrader: tsdgeos altho in that combined branch kraken+listoverview, i didn't have favoriting stars in the list like before [14:13] kgunn: you don't have favoritng stars? i just flashed it and they are there [14:13] tsdgeos: lemme update, i did it late friday [14:13] dandrader: well, the thing is this: no matter what I suggest you know, such a test will be phone specific, you might be able to make it work on tablet too, but at latest on the desktop its not gonna work out [14:14] mzanetti, I guess that would be it? tryCompare(spreadView, "phase", 2); [14:14] note: i have favorited stars, but not the empty ones, ...and the solid ones don't seem interactive [14:14] dandrader: yeah... but again, this works for tablet and phone, but not for desktop and tv [14:14] mzanetti, so you mean tests in tst_Shell should work on phone, tablet (not being worked on), desktop (not existent) and tv (not existent)? [14:15] dandrader: but then.. you might as well just add it, when we seriously start working on the destkop we need to rip tst_Shell in many pieces anyways I think [14:15] dandrader: sure, you can also now hardcode everything for the phone because that's the only thing currently worked on, and then start from scratch if something changes with the form factor :) [14:16] mzanetti, well, I can make the test bail out if it doesn't find PhoneStage.... [14:17] but not sure if that's good idea [14:18] I don't know either tbh... just add it, using what you said... we need to figure some bigger picture here with testing first [14:18] dandrader: ^ [14:18] kgunn: after update? [14:19] tsdgeos: no not after update...and now, my krillin decides he doesn't wanna let the shell start, devices shows nothing....great [14:20] kgunn: yeah that's one of the bad things of making adb not show by default, you may need to do the three button thing to get it to the boot loader and recover it from there if it doesn't come back === karni-lunch is now known as karni [14:21] tsdgeos: yeah, i flashed developer-mode...i mean it was working [14:25] Saviq: hmm... do we want to kill spurious tags on unity-api too? [14:31] mzanetti, I see you helping him with his OCD :D [14:31] Cimi: ? [14:32] mzanetti, for killing tags :) [14:32] mzanetti, when did we get them there? [14:33] I don't know. just filed a branch, worked through the checklist, saw them [14:33] Cimi: had to google OCD, but now I get it :D === pstolowski|schoo is now known as pstolowski === charles_ is now known as charles [15:22] kgunn, mzanetti, do you know what's the deal with "rtm" being in the rtm version number or not? [15:22] Saviq: yeah... [15:23] we ran into some issues with that. example: [15:23] I needed a dep on unity-schemas >= x.y.20140920 or so [15:23] all worked well until we tried to release to rtm [15:23] because it added ~rtm [15:24] tsdgeos, after installing the ppa is it just update + upgrade that's needed? [15:24] so the dep wasn't fulfilled any more. trainguards did some manual pushes without the ~rtm attached to get around it [15:25] vesar: yes, if fear it may be late if you're doing it now though https://launchpad.net/~unity-team/+archive/ubuntu/phone-right-edge/ lists some of the packages as outdated again :/ [15:26] mzanetti, ah ~... [15:27] kgunn, the latest release is building in rtm silo 9 [15:29] tsdgeos, well at least I got it working so that I get to the manage view nicely. [15:29] vesar: ok, then that would be good, can you show it to JMulholland and the rest? [15:29] Saviq: you bet, do you need test help ?...my krillin is f'd atm [15:29] tsdgeos, yeah I will. thanks. [15:29] thank you :) [15:30] kgunn, yeah, I'll be EOD'ing soon [15:30] kgunn, can you get it to recovery? [15:30] Saviq: yeah, but it still shows null in the adb devices list [15:31] gonna reboot my host, its the only thing i've not tried [15:31] dandrader: can you merge again qtmir and the bottomlist+touchownership thing? [15:31] kgunn, yeah [15:31] * kgunn gets desperate [15:31] don't worry [15:31] you can save it [15:31] tsdgeos, dandrader maybe push under ~unity-team? [15:31] kgunn: i had that [15:31] so it's fixable [15:31] don't reember how i got to flash it though [15:31] * Saviq didn't hear about any device bricked for real [15:32] tsdgeos, if you can get it into bootloader, you're fine [15:32] but if it doesn't show in adb phablet-flash won't like to flash it afair [15:32] with --bootstrap it will flash recovery and everything [15:32] or maybe you have to specify the device type [15:32] you do [15:32] --device=krillin is needed if you flash from recovery or fastboot [15:32] tsdgeos, Saviq, indeed. it would be better to move ~dandrader/unity8/listOnBottomSwipe_touchOwnership to unity-team and update the recipe accordingly. tsdgeos, do you wanna to it? :) [15:32] right [15:33] tell kgunn when he's back :D [15:33] will do [15:33] dandrader: ok, i will do [15:33] tsdgeos, thanks [15:33] tsdgeos, try booting with volume down ;) [15:33] there's kittens [15:36] tsdgeos: pray tell ? [15:36] kgunn: you need --bootstrap and --device=krillin is needed if you flash from recovery or fastboot [15:38] --bootstrap only from fastboot [15:38] --device=krillin from both indeed [15:38] kgunn, do you care about data you have on the phone? [15:40] dandrader: so you do the qtmir one still, no? [15:41] tsdgeos, lp:~dandrader/qtmir/UbuntuKeyboardInfoQMLSingleton ? [15:41] dandrader: yes [15:42] tsdgeos, oh, lp:qtmir has changed. You mean you want the above mentioned branch rebased on top of the new trunk [15:42] ? [15:42] dandrader: rebased or merged yes, so the ppa one has the "bigger" number and wins :D [15:42] ok [15:42] dandrader: also can you please merge unity8 trunk (there's also new contents) to your branch, it conflicts on some cmake file and i'm not sure what to do === mhall119_ is now known as mhall119 [15:46] tsdgeos, updated both [15:46] dandrader: awesomeness [15:46] tsdgeos, NB: added a new qmltest to touchOwnership tjat [15:47] that is currently failing (it's a regression test) [15:47] dandrader: good! [15:51] dandrader: Saviq: ~unity-team/unity8/listOnBottomSwipe_touchOwnership created and recipe update [15:51] d [15:51] t [15:51] x [15:52] tsdgeos, ok [15:59] greyback_, hey, does unity8 do anything with surfaces? Such as resize them under the hood yet? Getting a strange crash when doing software rendering, only in unity8 [15:59] if i run a mir server outside of unity8, it works fine... [16:00] bschaefer: it does surface resizing yeah [16:00] greyback_, hmm i see, how would i detect this and update with the correct surface size? [16:00] vs making the SDL surface incorrect vs the mir surface? [16:01] bschaefer: well unity8 fires a resize event to the application surface once it has drawn something [16:01] but that shouldn't make anything crash [16:01] greyback_, well its crashing as with software rendering i've to copy the pixels from the sdl surface to the mir surface [16:01] and something isn't lining up, ie the width/height are different [16:02] bschaefer: hmm, that's interesting [16:02] greyback_, i also talk directly to the mir server ... would that still allow me to get that event? [16:02] or do i need to be going through the umm sdk? [16:03] bschaefer: we do some funky stuff to deal with surface resizing in qtubuntu, lemme see [16:03] greyback_, i dont think i've checked...but does mir_surface->width get the correct properties? [16:03] or is it still what i have set manually? [16:04] bschaefer: http://bazaar.launchpad.net/~phablet-team/qtubuntu/trunk/view/head:/src/ubuntumirclient/window.cpp#L386 [16:04] bschaefer: mir_surface->width/height should be the correct dimensions of the surface you have to draw on [16:05] greyback_, let me double check those, as it seems my stride is messed up? [16:05] its been very annoying debugging this as it only happens with sdl1.2 in unity8 haha [16:05] bschaefer: yeah, think that method I sent you was to help Qt, not handy for you really [16:05] lovely [16:05] greyback_, yeeah hmm, so let me double check if i get the correct updated size with the mir_surface [16:06] greyback_, when does the resize happen? [16:06] I do know that once Mir gives you a buffer, it won't change [16:06] so it's only on buffer swap will you notice changes [16:06] i see, cool thanks! Ill do some more digging around [16:06] though you will also get an surface size change event [16:06] but it's safer to watch the buffer sizes IMO [16:06] yeah [16:07] i think its easy to update a SDL surface [16:07] soo if i can get the correct w/h i should be good [16:07] ack [16:08] if i get stuck ill be sure to email you! [16:09] welcome to === Estilanda_ is now known as Estilanda === dandrader_ is now known as dandrader|lunch [16:30] Saviq: ping! [16:59] sil2100, pong === alan_g is now known as alan_g|EOD [17:09] Saviq: hey, so... we've been wondering about unity8 cpu usage regarding some scopes [17:09] Saviq: we noticed that krillin + the custom tarball seems to have a much higher idle state in the dash [17:09] sil2100, we just landed lifecycle fixes [17:10] Saviq: it's probably something that we need to poke cwayne or thostr_, but we've been wondering about the lifecycle here [17:10] sil2100, which means dash will be suspended when unfocused [17:10] Oh [17:10] hah! [17:10] This might help then [17:10] sil2100, it's in rtm silo 9 [17:10] sil2100, but in any case I agree there's something we need to look at [17:10] sil2100, so that it doesn't use the CPU even when focused [17:11] unnecessarily, that is [17:11] Saviq: ok, let's check how this looks like and if it improves the performance during testing, if not we'll probably poke you again :) [17:11] Thanks [17:12] sure === gatox is now known as gatox_lunch === gatox_lunch is now known as gatox [19:11] greyback_, yup, confirmed it cuts the height by 39 pixels...now to figure out where that event comes in from.... [19:11] and hopefully sdl surface is pretty easy to update [19:12] bschaefer: atm there is always a resize event after the surface creation [19:12] bschaefer: I've an MR somewhere which fixes that, but not sure if it'll land for RTM or not [19:13] greyback_, awesome, well im working from proposed sooo hopefully is there :) [19:13] thanks! [19:13] so atm a fullscreen surface is allocated to the app, but once the client swaps the first buffer, is is resized [19:14] right, so i've to pay attention to that, mainly i just need to ensure that the surfaces are the same size before doing pixel copying [19:14] from sdl surface ---> mir region other wise itll seg fault on the memcpy [19:14] bschaefer: it's not in proposed yet either, but I hope to do so reasonable soon [19:14] oo, so atm no event? [19:14] atm there is an event [19:15] if there isn't, something is broken [19:15] greyback_, gotcha, though i talk with mir directly it might not be part of a mir event [19:15] i dont go through qt at all [19:15] greyback_, was it a mir system event or a qubuntu event? [19:16] bschaefer: is it a mir event [19:16] greyback_, sweet [19:16] thanks! [19:16] np [21:18] mterry_, Is anyone queued to review your greeter branch? [21:19] ted, no [21:19] mterry_, I think people should line up to review your branches :-) [21:19] ted, no one lines up for any reviews ever, man [21:19] mterry_, Is there anything I can do to help there? thostr would love the greeter stuff to land. [21:20] Saviq, kgunn: do you know anyone with spare review time? https://code.launchpad.net/~mterry/unity8/greeter-profiles/+merge/237155 is something ted is very interested in [21:20] mterry_, we'll get it reviewed tomorrow [21:21] Saviq, Thanks! [21:22] Saviq, If someone needs it, we've got it built in silo 13 [21:22] ted, yup, tx === salem_ is now known as _salem