[00:33] <robert_ancell> kgunn, no
[07:11] <tsdgeos> Saviq: why did you approve the height/animation thing?
[07:11] <tsdgeos> didn't we agree it didn't work good enough?
[07:35] <mzanetti> veebers: sorry... slept in.
[07:42] <Saviq> tsdgeos, didn't we agree that it's good enough for now, since we don't have better ideas yet?
[07:42] <tsdgeos> maybe
[07:42] <tsdgeos> i did have an idea yesternight
[07:42] <tsdgeos> it's not nice but it ought to work
[07:43] <tsdgeos> i'll give it a try later
[07:43] <tsdgeos> now busy trying to find out why pstolowski code makes stuff break
[07:44]  * pstolowski is keeping fingers crossed
[07:45] <tsdgeos> pstolowski: it's not cool, if i add all the mirrormodels code, then it works
[07:45] <pstolowski> ouch
[07:45] <tsdgeos> pstolowski: which means stuff works if i slow it down (because that's the only think mirrormodels do)
[07:45] <tsdgeos> which is not good
[07:45] <tsdgeos> :==/
[07:50] <pstolowski> tsdgeos: this is weird, coz I'm running unity8 (and able to reproduce) in virtualbox with no 3d acceleration, so things run very slow there
[07:50] <tsdgeos> it is
[07:51] <tsdgeos> want to give the patch a try and see if it seems to fix stuff for you too?
[08:10] <dandrader> Saviq, are we still scrambling to get that autopilot issue fixed?
[08:10] <Saviq> dandrader, which one?
[08:10] <dandrader> Saviq, the one that was getting in the way of having unity8-mir as the default
[08:11] <Saviq> dandrader, AFAIK it's all better now
[08:11] <Saviq> dandrader, thomi got it going last night
[08:11] <dandrader> Saviq, great
[08:12] <Saviq> MacSlow, two last comments on ext SDs https://code.launchpad.net/~macslow/unity8/extended-snap-decisions-part1/+merge/187312/comments/431919 https://code.launchpad.net/~macslow/unity8/extended-snap-decisions-part1/+merge/187312/comments/431924
[08:15] <Saviq> dandrader, just went through my email - and the output is https://bugs.launchpad.net/mir/+bug/1233944
[08:18] <dandrader> Saviq, bummer
[08:21] <Saviq> MacSlow, but can you confirm that's the case? that the menumodel goes away and that's why it contracts?
[08:22] <Saviq> MacSlow, we can work around it with a ShaderEffect hack, but probably not at this point
[08:26] <MacSlow> Saviq, I'll try to do the unexporting of actions and menu-model after the quit/close-handler to see if that changes anything...
[08:27] <Saviq> MacSlow, but you can see/reproduce?
[08:29] <MacSlow> Saviq, I think I see what you are refering too... the ext. snap-decision notification seems to "shrink upwards" before it actually disappears.
[08:29] <MacSlow> Saviq, that's what you mean?
[08:29] <Saviq> MacSlow, yup
[08:30] <MacSlow> Saviq, I just tried to do the unexport later... but that didn't change anything in this visual behaviour
[08:30] <Saviq> MacSlow, I wouldn't expect it to
[08:30] <Saviq> MacSlow, anyway - that's a glitch, let's do the more important things now
[08:31] <MacSlow> Saviq, indeed... those kind of things should be addressed in one go with the push on "spit&polish"
[08:31] <Saviq> MacSlow, the "simple" workaround I can think of is having a non-live ShaderEffect that we enable and update just before we unexport the actions
[08:31] <Saviq> MacSlow, this will create a "snapshot" of the notification while it disappears
[08:32] <MacSlow> Saviq, I hope there's a more generic solution to this... such "special case" handling doesn't sound right... we do too much of this already
[08:33] <MacSlow> Saviq, besides... my current headache revolves around the checkmark in the wifi-selection not working, because of the Loader.
[08:33] <Saviq> MacSlow, I don't think there is - not until we build the thing into unity-notifications, so that it will keep the structure while it's being removed
[08:34] <Saviq> MacSlow, right
[08:34] <Saviq> MacSlow, did you try/manage to reproduce in a small standalone QML file?
[08:35] <Saviq> MacSlow, or otherwise pinpointed the issue?
[08:36] <MacSlow> Saviq, yeah... the Loader-item used for the submenu used for the wifi-accesspoints causes the issue... OptionSelectorDelegate fails to set its internal listView correctly, thus there are several runtime-failures with null-references...
[08:37] <MacSlow> Saviq, I don't know how to approach this in a different way
[08:38] <Saviq> MacSlow, you mean that OptionSelectorDelegate isn't a direct delegate of OptionSelector's, but you have a Loader in the middle?
[08:38] <MacSlow> Saviq, fact is... wifi-selection does work... only the checkmark isn't showing...
[08:38] <Saviq> MacSlow, i.e. OptionSelector { delegate: Loader { sourceComponent: OptionSelectorDelegate { } } } more or less?
[08:39] <MacSlow> Saviq, still this is a nasty issue... not giving the user a visual clue what's currently selected.
[08:40] <MacSlow> Saviq, yes... http://bazaar.launchpad.net/~macslow/unity8/extended-snap-decisions-2/view/head:/Notifications/NotificationSubMenuItemFactory.qml
[08:40] <MacSlow> Saviq, and http://bazaar.launchpad.net/~macslow/unity8/extended-snap-decisions-2/view/head:/Notifications/NotificationMenuItemFactory.qml
[08:41] <Cimi> mzanetti, what?
[08:41] <Cimi> mzanetti, https://code.launchpad.net/~cimi/unity8/unity8.hud-2_hint-reveal-commit/+merge/187488/comments/431739
[08:41] <Cimi> I think I want to drop a bomb
[08:41] <MacSlow> Saviq, are there any QML-tricks to avoid this?
[08:42] <mzanetti> Cimi: haha... yeah. autopilot
[08:43] <Saviq> MacSlow, where is AccessPoint.qml?
[08:43] <Cimi> mzanetti, I did what you asked
[08:43] <Cimi> mzanetti, doesn't seem to work
[08:44] <MacSlow> Saviq, in NotificationSubMenuItemFactory.qml
[08:44] <MacSlow> Saviq, line 54
[08:44] <Saviq> MacSlow, no, that *uses* AccessPoint, where is that component defined?
[08:45] <MacSlow> Saviq, hold on... that's not the correct revision...
[08:45] <Saviq> MacSlow, anyway, you just need to make sure that OptionSelectorDelegate is passed the needed properties
[08:45] <Cimi> mzanetti, might be my fault, was damn working this morning
[08:46] <Cimi> mzanetti, maybe I did not update the test_hud.py on the phone and I thought was working
[08:46] <Cimi> mzanetti, anyway, you know how I can access __init.py__ class from TestHud?
[08:46] <mzanetti> Cimi: of course its your fault... it doesn't break on its own... that said, happens to me just as often. and I could start killing too when this sort of failures show up
[08:46] <Saviq> MacSlow, you need to bind the delegate's listView in the Loader
[08:48] <MacSlow> Saviq, listView of the OptionSelector is readonly... will that work?
[08:48] <mzanetti> Cimi: __init__.py defines UnityTestCase
[08:48] <Saviq> MacSlow, then it needs to be made non-readonly
[08:48] <Cimi> mzanetti, otherwise I revert to the previous working revicion
[08:48] <Saviq> MacSlow, in UITK
[08:48] <mzanetti> Cimi: test_hud.py inherits UnityTestCase
[08:48] <Cimi> mzanetti, __init__.py inside emulators
[08:48] <mzanetti> Cimi: no, that won't work
[08:48] <Saviq> MacSlow, but
[08:48] <Cimi> mzanetti, I tad you yesterday
[08:48] <Cimi> *told
[08:48] <Cimi> maybe we misunderstood?
[08:49] <Saviq> MacSlow, I don't think we should have that Loader at all
[08:49] <Cimi> mzanetti, what's a common place for hud.py and test_hud.py?
[08:49] <Saviq> MacSlow, we don't support any components other than OptionSelectorDelegate in an OptionSelector
[08:49] <Saviq> MacSlow, not dividers, not standarditems
[08:50] <MacSlow> Saviq, yes... what other way can I use to pull the info from the subMenu-model?
[08:50] <mzanetti> Cimi: I think the main __init__.py would work
[08:51] <Cimi> mzanetti, let me try
[08:51] <mzanetti> Cimi: in tests/autopilot/unity8/shell/__init__
[08:51] <Saviq> MacSlow, just include whatever's pulling that data in your OptionSelectorDelegate
[08:51] <Saviq> MacSlow, you shouldn't use the SubMenuItemFactory at all
[08:51] <Cimi> mzanetti, although doesn't have a class
[08:52] <mzanetti> Cimi: yeah... well, I'm really not a python expert either. you need to try or find someone who know python
[08:52] <Cimi> mzanetti, I'm trying
[08:52] <Cimi> mzanetti, nope
[08:52] <Cimi> mzanetti, let me revert to 330..
[08:52] <Saviq> MacSlow, everything you need is passed through to the delegate
[08:52] <Cimi> mzanetti, which btw was failing on jenkins
[08:53] <Saviq> MacSlow, menuModel, menuData, menuIndex - those are just properties
[08:53] <mzanetti> Saviq: do you know a solution to this?
[08:53] <nic-doffay> Saviq, regarding your most recent comments in the MP for the search indicator. I added a cross fade.
[08:53] <nic-doffay> Maybe it's too quick though, I'll experiment a bit more, otherwise I guess we'll have to check the search time.
[08:53] <dandrader> Saviq, how do I install a "devel-proposed" ubuntu-system image using phablet-flash?
[08:53] <Saviq> dandrader, --channel
[08:53] <Saviq> nic-doffay, k, will have a look, although when I tried a *real* cross fade it didn't look good, we rather need a sequential fade out + fade in
[08:54] <mzanetti> Cimi: oh... wait
[08:54] <mzanetti> Cimi: in test_hud.py we can access the emulators stuff
[08:54] <nic-doffay> Saviq, I'll try that now.
[08:54] <mzanetti> Cimi: with self.main_window.xxx
[08:54] <Saviq> nic-doffay, yeah, that's gonna be tricky without moving all that to states
[08:54] <Saviq> nic-doffay, and I started doing that yesterday but fooked something up
[08:54] <mzanetti> Cimi: so if you put the drag() in the hud emulator you should be able to use it in test_hud with self.main_window._drag()
[08:55] <Saviq> nic-doffay, let me get you a diff I started out wiht
[08:55] <Cimi> let's try
[08:55] <dandrader> ah, phablet-flash ubuntu-sytem --help mentions devel-proposed by I just didn't see it :/
[08:56] <Saviq> nic-doffay, something along the lines of http://paste.ubuntu.com/6183052/
[08:56] <Saviq> nic-doffay, but it didn't work right yet
[08:57] <Cimi> mzanetti, File "/usr/lib/python2.7/dist-packages/unity8/shell/tests/test_hud.py", line 99, in test_show_hud_appears
[08:57] <Cimi> self.main_window._drag(swipe_coords.start_x, swipe_coords.start_y, swipe_coords.start_x, swipe_coords.end_y)
[08:57] <Cimi> AttributeError: 'MainWindow' object has no attribute '_drag'
[08:57] <Cimi> mzanetti, come on let's revert and have this thing merged..
[08:57] <mzanetti> ok
[08:57] <Cimi> mzanetti, but was good to try
[08:57] <Cimi> mzanetti, we're not python experts
[08:59] <MacSlow> Saviq, I fixed it.
[08:59] <Saviq> MacSlow, see
[08:59] <MacSlow> Saviq, one more tiny bit of QML-wizardry learned today :)
[09:00] <Saviq> MacSlow, so you just flattened the SubMenuFactory?
[09:00] <Saviq> MacSlow, to be the OptionSelectorDelegate directly?
[09:00] <MacSlow> Saviq, yeah... pushing... one sec...
[09:03] <Saviq> mzanetti, Cimi, what's the deal there?
[09:03] <Cimi> Saviq, we saved jenkins from a bomb
[09:03] <mzanetti> Saviq: we need a custom drag() method in emulators and tests
[09:03] <mzanetti> Saviq: right now it's copied and I wanted Cimi to put it in some common place
[09:04] <mzanetti> Saviq: but we both failed to find one
[09:04] <Cimi> Saviq, I wanted it too but at this time I just want the tests to pass
[09:04] <MacSlow> Saviq, there you go http://bazaar.launchpad.net/~macslow/unity8/extended-snap-decisions-part2/view/head:/Notifications/NotificationMenuItemFactory.qml
[09:04] <Cimi> Saviq, I failed here http://bazaar.launchpad.net/~cimi/unity8/unity8.hud-2_hint-reveal-commit/revision/331
[09:05] <Saviq> mzanetti, Cimi you just need a small class DragMixin somewhere in __init__.py, import that and make the emulators inherit from it where needed
[09:05] <Saviq> mzanetti, Cimi one that will contain just the drag() method
[09:05] <Cimi> Saviq, yeah but I'm still learning python
[09:06] <Cimi> Saviq, and I have other tasks to do! :(
[09:06] <Cimi> Saviq, before the freeze
[09:06] <MacSlow> Saviq, once lp:~macslow/unity8/extended-snap-decisions-part1 made it into trunk I'll update the "part2" for frontend and backend and MR them, so we get the wifi-selection use-case in place too
[09:06] <Saviq> Cimi, sure
[09:06] <Saviq> MacSlow, yup, cool beans
[09:06] <Cimi> Saviq, was good to learn a bit but now I really want to move forward
[09:07] <Cimi> mzanetti, just tested again all hud tests with rev 330 and they pass
[09:07] <mzanetti> ok. lets see what jenkins thinks of it
[09:07] <Cimi> it's pushed as rev 332
[09:07] <Cimi> jenkins would better think he likes
[09:07] <mzanetti> lol
[09:07] <Cimi> I might become very upset
[09:08] <Saviq> mzanetti, Cimi something of the sort v
[09:08] <Saviq> http://pastebin.ubuntu.com/6183074/
[09:09] <mzanetti> Saviq: is the "Mixin" just some common thing to do or does it have a special meaning?
[09:10] <Saviq> mzanetti, just a name
[09:10] <mzanetti> ok
[09:10] <Saviq> mzanetti, you usually combine multiple mixins to get a union of multiple parent classes
[09:10] <Saviq> mzanetti, that's simply multi-inheritance
[09:11] <Saviq> dednick, minor thing here https://code.launchpad.net/~nick-dedekind/unity8/indicator-titles/+merge/186049/comments/431233
[09:11] <mzanetti> Saviq: yeah, got that. Just noticed that I see the name "Mixin" quite often so I was wondering if it actually _has_ to be named like that
[09:11] <Saviq> mzanetti, python is all about conventions :)
[09:12] <Cimi> Saviq, let me try
[09:12] <Cimi> Saviq, I'm not giving up
[09:12] <Saviq> dednick, also, any input on https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1215644 ?
[09:13] <dednick> Saviq: just upgrading phone now. I think it's fixed and released already.
[09:13] <Saviq> dednick, ah ok
[09:13] <Saviq> dednick, although I had a blue icon with no message this morning :D
[09:14] <dednick> Saviq: er. yeah. i dont seem to be seeing any messages at the moment :(
[09:14] <Saviq> dednick, yeah, no messages in the messaging menu
[09:14] <Saviq> dednick, I just got the notification, but nothing in the menu
[09:14] <Saviq> larsu, ↑
[09:16] <MacSlow> fginther, hey there...
[09:16] <Saviq> MacSlow, he's asleep, can I help?
[09:16] <Saviq> mzanetti, issue https://code.launchpad.net/~mzanetti/unity8/switching-previews/+merge/187327/comments/431952
[09:16] <MacSlow> Saviq, the autopilot-issue on jenkins seems fixed... what was the issue?
[09:16] <MacSlow> Saviq, just curious
[09:16] <Saviq> MacSlow, we knew the issue straight away
[09:16] <Saviq> MacSlow, it was multi-fold
[09:17] <mzanetti> Saviq: looking
[09:17] <Saviq> MacSlow, a) we started depending on libunity-core-6.0-8, which depended on libunity-protocol-private0, which required, but didn't depend on, a new libunity9
[09:18] <Saviq> MacSlow, so our two dependencies got upgraded but libunity9 didn't, and we were killed because of a missing gsettings key that was introduced recently
[09:19] <MacSlow> Saviq, ha... so basically one-line entry missing to cause all the trouble?! :)
[09:19] <Saviq> MacSlow, b) the machines (hardware, not VM) running the tests were not upgraded recently, but they do have unity7 installed, so the above occurred only on the hardware, as VMs are bare metal, and the new libunity9 was installed as usual
[09:20] <Saviq> MacSlow, c) when Francis switched us temporarily back to VMs, where the issue wasn't a problem - the jobs weren't compatible or something - and were failing to pick up the artifacts - so all the child jobs were SUCCESS, but the parent job still FAILED
[09:21] <Saviq> MacSlow, d) there was a huge build queue on jenkins yesterday, that only cleared up late in the evening, which meant that some of the -ci / -autolanding jobs before that took up to 4 hrs
[09:21] <Saviq> MacSlow, so yeah, *EVERYTHING* happened
[09:21] <MacSlow> Saviq, holy crap :)
[09:21] <mzanetti> Saviq: hmm... I do see an issue, but I'm not sure if its the same as you describe
[09:22] <dednick> Saviq: indicator-messages seems to be crashing
[09:22] <dednick> larsu: ^
[09:22] <tsdgeos> Saviq: this is the other height animation patch https://code.launchpad.net/~aacid/unity8/enable_height_animation_2/+merge/188785
[09:23] <tsdgeos> Saviq: it works better, but it's a bit nastier
[09:23] <Saviq> tsdgeos, k thanks
[09:23] <tsdgeos> i'll let you decide what we do :D
[09:23] <tsdgeos> i think this one makes more sense at this stage since the other doesn't really work
[09:23] <tsdgeos> but oh well
[09:24] <tsdgeos> i'm not happy with this one either :D
[09:24] <Cimi> Saviq, thanks, it works (cc mzanetti)
[09:25] <mzanetti>  ~o/
[09:25] <Saviq> MacSlow, I think that's a valid failure: http://s-jenkins:8080/job/generic-mediumtests-runner-maguro/1760/?
[09:25] <Saviq> MacSlow, due to moving the PinPad to components
[09:26] <Cimi> mzanetti, http://bazaar.launchpad.net/~cimi/unity8/unity8.hud-2_hint-reveal-commit/revision/333
[09:26] <mzanetti> Cimi: +1
[09:26] <Saviq> MacSlow, ah, why did you move PinLockscreen.qml at all?
[09:26] <Saviq> MacSlow, that should remain in Greeter, just the PinPad.qml should be moved
[09:27] <Saviq> MacSlow, that's probably my fault?
[09:27] <MacSlow> Saviq, you told me to
[09:27] <mzanetti> and PinPadButton.qml probably
[09:27] <MacSlow> Saviq, but the failing test pass here locally with that branch...
[09:27] <Saviq> MacSlow, you need to take me with a grain of salt ;)
[09:27] <Saviq> MacSlow, /me tries
[09:27] <MacSlow> Saviq, ok :)
[09:28] <MacSlow> Saviq, yup... both pass just fine
[09:29] <Saviq> MacSlow, fail here
[09:29] <Saviq> MacSlow, remove builddir/install
[09:29] <Saviq> MacSlow, make -C builddir install
[09:29] <Saviq> MacSlow, PYTHONPATH=tests/autopilot/ autopilot run unity8.shell.tests.test_lock_screen.TestLockscreen.test_can_unlock_pin_screen
[09:30] <Saviq> MacSlow, and make sure you don't have unity8 installed on your system - it will test that instead
[09:30] <Saviq> MacSlow, but anyway Greeter/PinLockscreen.qml: File not found
[09:30] <Cimi> Saviq, so next in priority
[09:30] <Saviq> MacSlow, that's a quite obvious error message ;)
[09:31] <Cimi> Saviq, welcome wizard or rendered for weather, dash plugins, or carousel fix, video renderer?
[09:33] <Saviq> Cimi, do the dash plugins renderer
[09:33] <Saviq> Cimi, should be simples
[09:33] <Saviq> mhr3, can you give Cimi a way to determine that a category is dash plugins?
[09:33] <Cimi> mhr3, I'm coming to the office in an hour and a bit
[09:34] <mhr3> Cimi, i'm not in today
[09:34] <Cimi> mhr3, ok thne
[09:34] <Cimi> mhr3, so my desk is free :P
[09:34] <Saviq> tsdgeos, aaargh ;(
[09:34] <dandrader> Saviq, do I still need this ppa on my desktop? http://ppa.launchpad.net/ubuntu-unity/next/ubuntu
[09:34] <mhr3> Cimi, yea, let's say that i knew that you want to come :)
[09:34] <Cimi> hah
[09:34] <Saviq> dandrader, *no*
[09:35] <Saviq> dandrader, that's dangerous :)
[09:35] <tsdgeos> Saviq: yeah :D
[09:35] <mhr3> Cimi, but yea, i need to expose one extra property for that to work
[09:35] <dandrader> hehehe, makes me wonder how I managed to survive so far
[09:36] <Saviq> tsdgeos, can we at least make the onDestruction less ugly?
[09:37] <tsdgeos> Saviq: ideas on how to do it?
[09:37] <Saviq> tsdgeos, like put some prop on RunningApplicationsGrid instead of looking at the source path?
[09:37] <tsdgeos> Saviq: well, it's wath onLoaded does
[09:37] <Saviq> tsdgeos, yeah, but I mean for the:
[09:37] <Saviq> if (source.toString().indexOf("Apps/RunningApplicationsGrid.qml") != -1) {
[09:37] <Saviq> tsdgeos, if RunningApplicationsGrid had a prop holding
[09:37] <Saviq> (results.firstModel.count + results.secondModel.count == 0)
[09:38] <Saviq> tsdgeos, we could use hasOwnProperty("blah")
[09:38] <Saviq> tsdgeos, and use that prop; else true;
[09:38] <larsu> dednick: my fixes from last Friday haven't been released yet
[09:38] <dednick> larsu: ah ok, i havent taken a look yet
[09:38] <Saviq> larsu, dednick, I'll build a package to test, then
[09:38] <mhr3> Saviq, or the renderer could use regular scope models and we could get rid of that special case :)
[09:39] <tsdgeos> Saviq: i'll see what i can do
[09:39] <Saviq> mhr3, got the scope for me yet?
[09:39] <mhr3> Saviq, it's been in the image for a few weeks now
[09:39] <larsu> dednick, Saviq: I'm pretty sure this fixes the issue. If not, let me know ;)
[09:39] <Saviq> mhr3, and working? with surfaceflinger? ;p
[09:39] <mhr3> question is: does it work? :)
[09:39] <larsu> (well, I'm sure you will)
[09:40] <dednick> larsu, Saviq: i need to test out some other odd behaviour i'm seeing as well. The icon still doesnt seem to be changing when recieving/removing items still. (looking at the data from the model here)
[09:40] <mhr3> pstolowski, were you able to test runningapps scope on the device?
[09:40] <mhr3> pstolowski, does it return anything now?
[09:40] <dednick> larsu: also, the sensitivity of the clear button is set to false.
[09:40] <pstolowski> mhr3: no, I plan to do today
[09:40] <Saviq> dednick, ok, I'll get you the package from i-m trunk
[09:40] <dednick> Saviq: ta
[09:41] <Cimi> guys, little guide to run mir on the phone?
[09:41] <Cimi> I promised kevin I'd have tested the new hud on the phone with mir
[09:42] <Saviq> Cimi, adb shell touch ~phablet/.display-mir; adb shell reboot
[09:42] <Cimi> Saviq, brilliant, thanks
[09:43] <larsu> dednick: I'll have a look (but I think I remember that working last Friday)
[09:45] <Saviq> larsu, if I have to install one more package on the host to just build a source package out of indicator-messages! you mark my words!
[09:46] <larsu> Saviq: huh?
[09:46] <Saviq> larsu, had to install dh-autoreconf, dh-translations, gobject-introspection
[09:46] <Saviq> larsu, just to build a source deb :P
[09:47] <larsu> Saviq: you need these for 80% of our desktop projects... also: learn about apt-get build-dep :P
[09:47] <Saviq> larsu, I don't want that
[09:47] <Saviq> larsu, am using sbuild to keep the host clean
[09:47] <larsu> well, you need to install all deps manually then
[09:48] <Saviq> larsu, didn't have to install anything for indicator-network, unity8, unity-notifications, mir, platform-api, unity-mir... and then some
[09:48] <dednick> larsu: it's easier to see what's going on using the indicators-client app on the phone. It gives you a textual representation of the unitymenumodel row data. Although the current release is not working. need trunk or even better: https://code.launchpad.net/~nick-dedekind/unity8/indicators-client.actionStateParser/+merge/188790
[09:49] <mzanetti> Saviq: follow up on yesterday night: this works: http://paste.kde.org/p8ce24ba6
[09:49] <mzanetti> notice line 8
[09:49] <Saviq> mzanetti, HUH!
[09:49] <mzanetti> removing that breaks it
[09:49] <larsu> Saviq: because you already had the stack for those installed...
[09:49] <Saviq> larsu, no, because everything happens *inside* sbuild there
[09:50] <Saviq> larsu, building the source package for those is just a case of putting stuff in the tarball and .dsc file
[09:51] <mzanetti> greyback: you around yet?
[09:51] <Saviq> larsu, for -messages, there's autoreconf, translations and stuff that happens during building of the source package, which happens outside of the chroot
[09:51] <greyback> mzanetti: yes
[09:51] <Cimi> Saviq, wow the hud button doesn't work on mir
[09:51] <mzanetti> Cimi: heh, yep, known issue. I'm on it
[09:51] <Saviq> Cimi, racarr was working on that yesterday
[09:51] <Cimi> ah ok
[09:51] <Saviq> in Mir that is
[09:52] <mzanetti> Cimi: yeah... racarr provides me an API to re-inject the mouse event's we're filtering
[09:52] <Saviq> mzanetti, although I can't say I get what the issue is - the fact that we want the input in both app and shell? isn't that the case everywhere?
[09:52] <Saviq> mzanetti, I don't get why that would be the case (re LazyImage)
[09:52] <Saviq> mzanetti, I'm not touching "height" anywhere there
[09:53] <Saviq> mzanetti, and anyway the tests *do* test the height
[09:53] <Saviq> mzanetti, or width, if scaleTo: "height"
[09:53] <pstolowski> greyback: ping
[09:53] <greyback> pstolowski: pong
[09:53] <pstolowski> greyback: morning!
[09:53] <mzanetti> Saviq: let me push, you can try with that use case if you want
[09:53] <greyback> pstolowski: yes :)
[09:53] <MacSlow> Saviq, so now it should be ok
[09:54] <pstolowski> greyback: is this https://lists.launchpad.net/ubuntu-phone/msg04088.html the only thing needed to get mir on the phone? how do I know if it's running (process name?)
[09:54] <Saviq> MacSlow, k thanks, and sorry for pushing you in the wrong direction ;)
[09:54] <Saviq> mzanetti, it's not about trying ;) I believe you that's your issue - just asking whether you have an idea why that happens?
[09:54] <mzanetti> Saviq: pushed.
[09:55] <mzanetti> Saviq: no, not really
[09:55] <Saviq> mzanetti, height: implicitHeight should be.... implicit, shouldn't it?
[09:55] <greyback> pstolowski: yes, that's all you do. To know it worked, does unity appear? Then check surfaceflinger is not running - then it must be Mir
[09:55] <Saviq> mzanetti, to where? :D
[09:55] <mzanetti> Saviq: I guess it should, yeah
[09:55] <mzanetti> Saviq: lp:~unity-team/unity8/fix-genericpreview
[09:55] <Saviq> mzanetti, k
[09:56] <Saviq> mzanetti, re: switching-previews, you really think it's related to the header? why?
[09:57] <mzanetti> Saviq: the only issue I can see is if you're opening a preview on an item which is partly covered by a header
[09:57] <Saviq> mzanetti, no, that's not it
[09:57] <mzanetti> Saviq: couldn't find any issue with the bottom
[09:57] <Saviq> mzanetti, expand "Installed" in Apps
[09:57] <larsu> Saviq: ah I see what you mean. That's the age-old decision that you don't need anything but a shell, make, and a compiler to build autotools-based packages
[09:57] <mhr3> btw everytime i try to build unity8 pkg i get a failure in launchermodeltest, known?
[09:57] <Saviq> mzanetti, open a preview on the last visible row, and go back to the beginning
[09:58] <Saviq> mhr3, no
[09:58] <mzanetti> oh...
[09:58] <Saviq> mzanetti, same with the other way
[09:58] <pstolowski> greyback: yeah, unity still works, I was just expecting performance drop on maguro, but can't see real diff... no surfaceflinger process running
[09:58] <Saviq> mzanetti, open a preview in the first row of the *last* category
[09:58] <Saviq> mzanetti, and go towards the end
[09:58] <Saviq> mzanetti, it will get confused
[09:58] <Saviq> mzanetti, but it's no surprise
[09:59] <mzanetti> Saviq: right... I got it
[09:59] <Saviq> mzanetti, if the ListView is open near the top - you can't scroll it enough to reach the bottom
[09:59] <Saviq> mzanetti, you need to move the gab
[09:59] <Saviq> gap
[09:59] <greyback> pstolowski: that's interesting, I must flash my maguro too to see. There was a perf drop on it, possibly fixed
[09:59] <mzanetti> Saviq: aye.have to go to a meeting now. will see if I can fix it afterwards
[09:59] <Saviq> mzanetti, o/
[10:00] <Saviq> larsu, huh... http://pastebin.ubuntu.com/6183192/
[10:01] <larsu> Saviq: oh wow, why didn't CI catch that?
[10:01] <Saviq> larsu, indeed
[10:02] <Saviq> MacSlow, hmm hmm hmm, why does notifications need to import Greeter?
[10:02] <Saviq> MacSlow, does it use PinLockscreen?
[10:02] <Saviq> MacSlow, and not just the PinPad?
[10:02] <MacSlow> Saviq, yes
[10:04] <Saviq> MacSlow, ok, I need a little bit more looks at that then...
[10:04] <dednick> larsu: i think the remove-all action is coming across with the incorrect action. it's showing up in unitymenumodel as "indicator.indicator.remove-all"
[10:05] <Saviq> MacSlow, feels like we should rename PinLockscreen... and yes, move it into Components, then...
[10:05] <larsu> dednick: give me a sec to fix that compile issue (and find out why CI didn't catch it)
[10:05] <MacSlow> Saviq, are you sure :)
[10:05] <Saviq> MacSlow, no - not yet ;d
[10:05] <dednick> larsu: which is why it's not sensitive i think. I'm getting the dednick log messages saying Enable remove-all.
[10:05] <Saviq> mzanetti, should the SIM PIN snap decision be using PinLockscreen directly?
[10:05] <dednick> larsu: ok
[10:05] <Saviq> MacSlow, will let you know when I am ;)
[10:05] <mzanetti> Saviq: yes
[10:05] <nic-doffay> Saviq, I switched up the search indicator to use states and transitions and sequential, however I'm not sure how effective it was.
[10:06] <Saviq> "dednick log messages"
[10:06] <larsu> the best kind of log mesages.
[10:06] <larsu> *messages
[10:06] <dednick> lol
[10:06] <Saviq> nic-doffay, k, will have a look (or mzanetti will, while I'm looking at his stuff)
[10:06] <dednick> *debug
[10:07] <Saviq> MacSlow, ok, let's leave as-is for now, if CI goes through, I'll approve, we can move around later
[10:08] <MacSlow> Saviq, ok
[10:09] <Saviq> larsu, seems it's built fine out of sbuild... *weird*
[10:10] <larsu> Saviq: did you override -Wall or something?
[10:10] <Saviq> larsu, sbuild might
[10:11] <larsu> Saviq: it definitely should be on though. I don't know why it isn't and this is a very legitimate bug
[10:11] <Saviq> larsu, yup
[10:12] <tsdgeos> mzanetti|otp: ping
[10:14] <tsdgeos> Saviq: https://code.launchpad.net/~aacid/unity8/enable_height_animation_2/+merge/188785 less ugly?
[10:18] <Saviq> tsdgeos, why do we need to compare the source?
[10:18] <Saviq> tsdgeos, can't we have a property *in* RunningApplicationsGrid
[10:18] <Saviq> tsdgeos, that GenericScopeView looks for/at?
[10:19] <tsdgeos> Saviq: isn't that the same that comparing the source?
[10:19] <dednick> larsu: fyi, indicator-messages r380 did in the remove-all action.
[10:19] <Saviq> tsdgeos, it's not, as we can rename the grid, we can add the same behavior to other renderers etc.
[10:19] <Saviq> tsdgeos, and it will work - now it's filename-specific
[10:19] <tsdgeos> totally yep
[10:20] <larsu> dednick: you're missing a verb there
[10:20] <Saviq> tsdgeos, and also, do the "firstModel.count + secondModel.count" *in* RunningApplicationsGrid
[10:20] <dednick> larsu: na, speakin' street yo.
[10:21] <larsu> dednick: hah :)
[10:21] <tsdgeos> Saviq: you're aware that this also needs the enableHeightBehavior in DashApps that is what makes it persistent over the destructions of RunningApplicationsGrid and that making it generic is not going to get us much really'
[10:21] <tsdgeos> i can do it
[10:21] <tsdgeos> just don't see how is going to help
[10:22] <Saviq> tsdgeos, it's going to help me sleep at night ;)
[10:22] <tsdgeos> ok
[10:22] <tsdgeos> that's a good reason enough
[10:23] <pstolowski> greyback: I've just tried to running-apps scope on the phone with mir, and it doesn't find com.canonical.Unity.WindowStack. is it available?
[10:24] <greyback> pstolowski: org.canonical.Unity.WindowStack
[10:24] <pstolowski> greyback: ah, thanks
[10:25] <mhr3> greyback, we're canonical.com, not canonical.org :)
[10:25] <pstolowski> :)
[10:25] <greyback> mhr3: patches welcome :P
[10:26] <larsu> Saviq: -Wall is set but I still don't get that error. Anyway, fix is here: https://code.launchpad.net/~larsu/indicator-messages/set-status-return-value/+merge/188803
[10:26] <Saviq> larsu, tricky :)
[10:26] <larsu> Saviq: ya... I get the warning if I run the preprocessor manually :-(
[10:26] <larsu> I'll try to figure this one out - but not now
[10:27] <Cimi> mzanetti|otp, Approve!
[10:28] <mhr3> Saviq, we need faster reviews, need to create 4th unity8 dir cause the other 3 are still not merged :P
[10:28] <Saviq> mhr3, use colocated branches
[10:29]  * mhr3 heads off to read bzr docs
[10:34] <mzanetti> tsdgeos: pong
[10:35] <mzanetti> Cimi: \o/
[10:37] <tsdgeos> mzanetti: http://paste.ubuntu.com/6183307/ otherwise we assert on debug builds
[10:37] <tsdgeos> mzanetti: me or you?
[10:37] <Saviq> dednick, http://people.canonical.com/~msawicz/indicator-messages/
[10:37] <dednick> Saviq: thanks
[10:37] <Saviq> tsdgeos, that mhr3's issue?
[10:37] <mzanetti> tsdgeos: oh... thanks for pointing it out
[10:38] <tsdgeos> Saviq: no clue what's mhr3 issue :D
[10:38] <tsdgeos> Saviq: where do i read about it?
[10:38] <Saviq> tsdgeos, mhr3 had some test failure in that test
[10:38] <Saviq> tsdgeos, or, a test for that
[10:38] <Saviq> I *think*
[10:38] <Saviq> but well, that was not for debug builds
[10:38] <tsdgeos> not sure if that has any real world impact
[10:38] <Saviq> as it was during package build
[10:38] <tsdgeos> debug build of qt i mean
[10:39] <tsdgeos> that has all the asserts enabled
[10:39] <Saviq> dednick, larsu, the icon did *something* on incoming text, but it did not get blue...
[10:40] <dednick> Saviq: yeah, it's still not working properly for me either.
[10:40] <Saviq> dednick, ok, you're on it so it'll be fixed in 5 I'm sure
[10:40] <dednick> putting some trace logging in now to figure it out.
[10:41] <dednick> lol. um, probably with another bug built in. but no, i dont understand much in libmessaging unfortunately.
[10:41] <dednick> the icon choosing is fairly complicated.
[10:42] <larsu> dednick: leave those bugs to me. No need to waste your time on them
[10:42] <larsu> dednick: as for the indicator.indicator.remove-all. Are you sure you're not adding one yourself along the way?
[10:42] <dednick> larsu: ok :)
[10:42] <larsu> I changed it in r380 to be consistent with all the other actions
[10:42] <larsu> i.e., in the menu, actions are referred to as indicator.<actionname>
[10:42] <dednick> larsu: you added an "indicator." onto the menu action in r380
[10:43] <Saviq> larsu, http://pastebin.ubuntu.com/6183316/ :D
[10:43] <larsu> dednick: yes. that's what I just said...
[10:43] <dednick> :) was busy typing :)
[10:44] <mhr3> Saviq, my issue is http://paste.ubuntu.com/6183319/ btw
[10:44] <dednick> larsu: erm, i'm just looking at the action role on the unitymenumodel.
[10:44] <mhr3> mzanetti,  ^^
[10:44] <mzanetti> tsdgeos: I'm actually wondering if it shouldn't be this: http://paste.ubuntu.com/6183320/
[10:44] <Saviq> mhr3, mzanetti I *think* we're interrogating the real AccountService there are we not?
[10:44] <larsu> Saviq: cool. As I said, I haven't yet managed to get these errors :(
[10:45] <larsu> Saviq: I'll fix them as soon as I've figured out why
[10:45] <mhr3> Saviq, ehm... don't do that?
[10:45] <tsdgeos> mzanetti: actually, yes
[10:45] <Saviq> mhr3, orly?
[10:45] <mhr3> yep :P
[10:45] <Saviq> mzanetti, ↑
[10:45] <mzanetti> yes...
[10:45] <tsdgeos> mzanetti: fixit! :D
[10:45] <Saviq> mhr3, gdbus call --system --dest org.freedesktop.Accounts --object-path /org/freedesktop/Accounts/User1000 --method org.freedesktop.DBus.Properties.Set com.canonical.unity.AccountsService launcher-items "<[{'defaults' : <true>}]>"
[10:46] <Saviq> mhr3, assuming you're UID1000
[10:47] <dednick> larsu: i still don't gel well with the namespace thing, so I dont know. the unitymenumodel gtk code seems to do some interesting things with namespaces when it comes to sections
[10:47] <mhr3> Saviq, but, but, i don't want to loose my laucher items
[10:47] <Saviq> mhr3, you won't *lose* them, you'll just reset them ;d
[10:48] <mhr3> ah, ok
[10:48] <mhr3> that makes me feel so much better
[10:52] <larsu> dednick: ah, it's indeed my fault: there's a namespace set in addition to the "indicator.remove-all" name. The question is: why does this work on the desktop...
[10:52] <om26er> bug 1234054
[10:52] <larsu> dednick: I'll remove it.
[10:52] <om26er> any takers/responders ?
[10:53] <Saviq> om26er, right
[10:53] <Saviq> om26er, it's the same reason - there's no scope behind it
[10:53] <om26er> Saviq, ok, understood.
[10:53] <om26er> Saviq, how about bug 1227595 ?
[10:54] <Saviq> dednick, ↑should be an easy fix
[10:54] <om26er> I just changed one line live: false to live:true on my phone and things are working fine
[10:54] <Saviq> om26er, yeah, exactly
[10:54] <Saviq> om26er, so, you proposing an MP?
[10:54] <om26er> that also fixed the problem for the sound slider
[10:54] <om26er> Saviq, sure.
[10:54] <Saviq> dednick, unping
[10:55] <Saviq> mhr3, do we have a plan for apps in home scope? will they come from a scope somewhere?
[10:56] <Saviq> dednick, reping https://code.launchpad.net/~nick-dedekind/unity8/indicator-titles/+merge/186049/comments/432038
[10:57] <mhr3> Saviq, step 1) we're going to get rid of the apps scope
[10:57] <mhr3> step 2) figure out if click scope is going to provide those results
[10:58] <larsu> dednick: https://code.launchpad.net/~larsu/indicator-messages/remove-stray-namespace/+merge/188809
[10:58] <dednick> larsu: btw, I had already put the trace logging in and built, so i just ran it up quick. Here's the output to help you on your way with the icon bug. http://pastebin.ubuntu.com/6183346/
[10:59] <larsu> dednick: ah cool thanks. So the icon doesn't change to "new" when it should? Or it doesn't change back to "normal"?
[10:59] <dednick> larsu: it changes to new, then straight back to normal
[11:00] <dednick> get a message_added, then a source_changed
[11:01] <larsu> okay, I'll look into it right after lunch
[11:02] <mzanetti> tsdgeos: https://code.launchpad.net/~mzanetti/unity8/fix-quicklist-inserting/+merge/188811
[11:07] <tsdgeos> Saviq: https://code.launchpad.net/~aacid/unity8/enable_height_animation_2/+merge/188785 better?
[11:07] <Saviq> tsdgeos, http://pastebin.ubuntu.com/6183391/
[11:08] <tsdgeos> Saviq: any more backtrace?
[11:08] <om26er> https://code.launchpad.net/~om26er/unity8/slider_live/+merge/188810
[11:08] <Saviq> tsdgeos, not atm, trying again
[11:08] <tsdgeos> Saviq: that doesn't help much, there's lots of calls to  ListViewWithPageHeader::ListItem::y
[11:08] <Saviq> tsdgeos, k, trying
[11:09] <tsdgeos> need to know where to see why the listitem was null
[11:09] <tsdgeos> since that should never happen
[11:10] <mzanetti> mhr3: https://code.launchpad.net/~mzanetti/unity8/dont-load-as-for-testing/+merge/188812
[11:11] <Saviq> tsdgeos, that's positionViewAtBeginning again
[11:11] <Saviq> tsdgeos, http://pastebin.ubuntu.com/6183404/
[11:12] <Saviq> tsdgeos, so the un-revert from yesterday...
[11:12] <tsdgeos> and why nobody could make it crash yesterday? :D
[11:12] <Saviq> tsdgeos, steps to repro: start shell, start app, swipe from left to get back to dash apps straight away
[11:12] <Saviq> tsdgeos, i.e. when app is launching
[11:13] <tsdgeos> i did that :/
[11:13] <tsdgeos> but yeah
[11:13] <tsdgeos> let's try again
[11:13] <Saviq> tsdgeos, 100% reproducible here
[11:13] <mhr3> mzanetti, sorry already did a reset, so test build wouldn't fail now... but the code looks ok
[11:14] <mzanetti> mhr3: yeah... this functionality has been in there already for the launcherbackendtest. but I don't manually instantiate the backend when doing the "frontend" tests.
[11:15] <om26er> my Phone is currently in this state. i.e. I try to change to switch to dash Home the app comes back automatically: http://videobin.org/+6xp/8lp.html
[11:16] <om26er> that video opens in firefox only
[11:16] <mzanetti> om26er: I've seen this too. can you report a bug please?
[11:16] <tsdgeos> Saviq: grrr, ok
[11:16] <om26er> mzanetti, I was wondering if there any logs that I could provide ?
[11:16] <Saviq> tsdgeos, got it?
[11:16] <tsdgeos> no, but i can see why the code may be wrong
[11:16] <tsdgeos> let me try to run it
[11:17] <Saviq> om26er, steps to reproduce would be best - logs are rarely useful for us
[11:17] <mzanetti> om26er: hmm. not sure if ~/.cache/upstart/unit8.log provides something useful in this case. you can still attach it you think there are some hints in there
[11:17] <om26er> Saviq, I am afraid there are no obvious steps but I think music-app was running in the background before it.
[11:17] <om26er> will try to come up with steps to reproduce the issue
[11:17] <Saviq> om26er, I know, as usual
[11:17] <om26er> mzanetti, ok
[11:18] <tsdgeos> Saviq: you start app from home or from app scope?
[11:18] <Saviq> tsdgeos, home
[11:19] <Saviq> tsdgeos, start from home, and while the white rect comes in, I drag from left
[11:19] <tsdgeos> nah
[11:19] <tsdgeos> all is fine here
[11:19] <tsdgeos> :(
[11:20] <tsdgeos> let me try with SF
[11:24] <tsdgeos> nothing :/
[11:26] <om26er> bug 1234108
[11:28] <Saviq> om26er, thanks
[11:29] <pstolowski> greyback: having an issue with winstack api; didn't we agree it would provide app names that map directly to .desktop files? it gives me "Browser" (app name), but the actual desktop file is webbrowser-app.desktop
[11:30] <mhr3> mzanetti, trying your previews branch on the phone
[11:30] <greyback> pstolowski: we did. That's a mistake
[11:30]  * mzanetti wonders if this is the beginning of a bugreport or just a FYI
[11:30] <greyback> thought I fixed that
[11:31] <mhr3> mzanetti, usually the lazyimage shows the "no-image" state as semi-transparent box, but sometimes it doesn't show anything
[11:31] <tsdgeos> Saviq: do you happen to have a loner backtrace of that crash?
[11:31] <mhr3> mzanetti, fixable?
[11:31] <Saviq> tsdgeos, longer?
[11:31] <Saviq> tsdgeos, will try and get more symbols
[11:31] <mzanetti> mhr3: can you define the "sometimes" a bit better?
[11:31] <tsdgeos> Saviq: want to know if it's directly coming from qml or reentring from the lvwph code
[11:32] <tsdgeos> not sure how it can happen yet
[11:32] <mhr3> mzanetti, i see it if i preview twitter
[11:32] <pstolowski> greyback: can you fix that soonish?
[11:32] <greyback> pstolowski: in progress...
[11:33] <mhr3> mzanetti, or rather... don't see it :)
[11:33] <pstolowski> greyback: thanks!
[11:33] <mzanetti> mhr3: hmm... I just saw that with trunk too
[11:33] <mzanetti> mhr3: will check
[11:34] <om26er> bug 1232381
[11:35] <om26er> currently looks like this: http://i.imgur.com/gWCvF7h.png
[11:35] <mhr3> mzanetti, also, movie previews don't have the lazy image?
[11:36] <tsdgeos> Saviq: if i can't make it crash do you think we can make some peer-debugging (i.e. i add some debugs and you tell me what you get)?
[11:36] <om26er> mhr3, is that ^ something to be fixed in the backend ? or unity8 needs to take care of that ?
[11:36] <Saviq> tsdgeos, sure
[11:36] <tsdgeos> ok, let me try to prepare a branch
[11:36] <mhr3> om26er, it's being worked on
[11:37] <om26er> mhr3, ack
[11:44] <davmor2> Hey guys I've been testing a n-m fix for 3g.  This has meant I've had 3g everytime I have booted up the phone.  Out of 10 reboot 2 have shown click packages for install and online scopes options on the apps scope is there any info I can get that would be useful to help with this?
[11:46] <tsdgeos> Saviq: lp:~aacid/unity8/positionAtBeginning_crash_debug
[11:47] <greyback_> pstolowski: I'll change org.canonical to com.canonical too, ok?
[11:48] <Saviq> tsdgeos, sec, just getting more symbols
[11:53] <mhr3> greyback_, yes pls
[11:55] <pstolowski> greyback_: +1
[11:58] <greyback_> pstolowski: https://code.launchpad.net/~gerboland/unity-mir/dbus-names-wrong/+merge/188820
[11:59] <Saviq> tsdgeos, http://pastebin.ubuntu.com/6183551/ - dunno where to get more
[12:00] <Saviq> tsdgeos, running your debug mode now
[12:03] <mhr3> Saviq, could you look at https://code.launchpad.net/~unity-team/unity8/scope-isactive/+merge/188731 once you have a sec?
[12:03] <Saviq> mhr3, yeah, I know I still need to fix the test there
[12:03] <mhr3> should be familiar :)
[12:03] <Saviq> and not yet sure how
[12:03] <mhr3> right
[12:05] <Saviq> tsdgeos, http://pastebin.ubuntu.com/6183566/
[12:10] <om26er> mzanetti, I added the steps to reproduce bug 1234108
[12:10] <om26er> please give it a try if you can
[12:10] <mzanetti> om26er: awesome, thanks, I'll check it out
[12:11] <om26er> we should probably have an autopilot test for that when the bug is fixed.
[12:11] <kgunn> dandrader: if you're not already, would you mind reading up & trying to help debug https://bugs.launchpad.net/mir/+bug/1233944
[12:12] <dandrader> kgunn, have been on that since I start work this morning. check my comments there
[12:12] <kgunn> awesome! dandrader ...love it when you guys are ahead of me
[12:23] <mzanetti> Saviq: yeah... I know about that thing in the carousel
[12:29] <karni> mzanetti: From comments I gather you are working on that fix-genericpreview branch? Looking forward to it.
[12:38] <mhr3> it's such an awesome branch :)
[12:46] <mzanetti> karni: yes, I am
[12:46] <karni> great
[12:57] <tsdgeos> Saviq: VisualDataModel::item: index out range 0 0            is really bad
[12:57] <tsdgeos> hmmm
[12:57] <tsdgeos> ah wai
[12:57] <tsdgeos> Saviq: do you have empty scopes?
[12:57] <paulliu> mzanetti: are you fixing the testing on fix-genericpreview branch? or I'll fix it?
[12:57] <Saviq> tsdgeos, dunno
[12:57] <tsdgeos> i mean scopes without items?
[12:57] <Saviq> tsdgeos, can't reach them
[12:58] <tsdgeos> that may very well be the case
[12:58] <Saviq> tsdgeos, ah well, yeah
[12:58] <Saviq> tsdgeos, music and video
[12:58] <tsdgeos> stupid me :D
[12:58] <mzanetti> paulliu: what's wrong with it?
[12:58]  * tsdgeos hits himself
[12:58] <mzanetti> Saviq: fighting with that carousel... it looks really bad when not forcing the currentHighlight to be centered
[12:58] <paulliu> mzanetti: the actions test should moved to test of GenericPreview. And apps/movie tests other stuff.
[12:59] <mzanetti> paulliu: I can fix that
[12:59] <paulliu> mzanetti: ok
[13:00] <Saviq> mzanetti, I thought we can get away with "select an item in $renderer; scroll vertically to match $gap (or move $gap if needed); move the pointer to $highlightedItem.center"
[13:03] <mhr3> Cimi, you'll want lp:~mhr3/unity8/expose-renderer-hint the dash plugins category will have rendererHint set to "toggled" (rendererId: "grid", contentType: "apps")
[13:05] <pstolowski> Saviq: ping
[13:05] <Saviq> pstolowski, pong
[13:07] <pstolowski> Saviq: hey! I'm testing and fixing some outstanding issues in running-apps scope (with mir's winstack api on the phone), so will probably be ready to land it soon; will you be able to implement rederer/ fix the Dash to actually use it?
[13:07] <Saviq> pstolowski, yeah, we should be good
[13:08] <pstolowski> Saviq: ok, great
[13:14] <pstolowski> greyback_: thanks
[13:16] <pstolowski> greyback_, Saviq : one more thing re running apps scope - the scope currently returns "image://screenshot/<appid>" for thumbanils; is this still correct? it doesn't produce any thumbnail at the moment, I'm guessing it's a renderer TODO?
[13:18] <Saviq> pstolowski, yeah, there's no image provider for it yet, I think
[13:19] <Saviq> pstolowski, it needs to use ApplicationImage
[13:19] <pstolowski> Saviq: is this on anybody's plate?
[13:19] <greyback_> Saviq: pstolowski there is an image provider in unity-mir. ApplicationImage is just a wrapper around it
[13:20] <Saviq> greyback_, ah, then is image://screenshot/appid good?
[13:20] <greyback_> Saviq: yes
[13:20] <Saviq> ok, then we'll need to debug
[13:20] <Saviq> pstolowski, can we enable the scope ~easily for testing?
[13:21] <pstolowski> Saviq: yes. just disable current application.scope mapping in ScopeDelegateMapper. And then you need the fix in Mir which greyback_ just implemented
[13:22] <greyback_> pstolowski: also please keep eye on the unity8 output. You might see messages from ApplicationScreenshotProvider that are relevant
[13:22] <tsdgeos> Saviq: https://code.launchpad.net/~aacid/unity8/empty_lvwph_crash_positionAtBeginning/+merge/188830
[13:22] <pstolowski> Saviq: the scope has already been present on the image for quite some time, just to really active
[13:22] <pstolowski> s/to/not/
[13:23] <Saviq> pstolowski, yeah, cool
[13:27] <Saviq> tsdgeos, happroved
[13:27] <pstolowski> greyback_: yeah, I think something is misbehaving http://pastebin.ubuntu.com/6183829/
[13:29] <greyback_> pstolowski: weird
[13:29] <pstolowski> greyback_: plus I copied webbrowser-app.desktop to Browser.desktop to circumvent the problem with winstack api giving me appnames, and this won't work with screenshots provider I guess
[13:30] <pstolowski> greyback_: the existing implementation of running apps in the shell uses some other way of obtaining screenshots?
[13:30] <greyback_> pstolowski: yeah, screenshot provider wants app id. You were accidentally getting the app name.
[13:30] <greyback_> pstolowski: same way
[13:31] <pstolowski> greyback_: does it, by any chance, expect .desktop extension?
[13:31] <greyback_> via this ApplicationImage component; http://bazaar.launchpad.net/~mir-team/unity-mir/trunk/view/head:/src/modules/Unity/Application/OSKController.qml
[13:31] <greyback_> pstolowski: nope, appId only
[13:32] <pstolowski> greyback_: hmm, so at least those apps from my pastebin should work
[13:32] <greyback_> pstolowski: sorry, bad link: http://bazaar.launchpad.net/~mir-team/unity-mir/trunk/view/head:/src/modules/Unity/Application/ApplicationImage.qml
[13:34] <pstolowski> greyback_: this is matches scope's code http://bazaar.launchpad.net/~unity-team/unity-lens-applications/trunk/view/head:/src/running-apps-search.vala#L163
[13:36] <greyback_> pstolowski: yeah, that log even gives a correct URI.
[13:37] <greyback_> pstolowski: I've one thing to do, then I'll be able to investigate.
[13:39] <greyback_> pstolowski: if you want, you could recompile unity-mir, adding debug to src/modules/Unity/Application/applicationscreenshotprovider.cpp - requestImage
[13:40] <tsdgeos> Mirv: ping
[13:41] <Saviq> nic-doffay, can you hear us?
[13:41] <Saviq> nic-doffay, your turn
[13:43] <dednick> Cimi: you in office tomorrow?
[13:43] <pstolowski> greyback_: trying
[13:43] <Cimi> dednick, nope
[13:43] <Cimi> dednick, friday?
[13:43] <dednick> Cimi: I've got to come in tomorrow.
[13:43] <dednick> Cimi: I could maybe on friday
[13:44] <Cimi> dednick, I have my ocado delivery tomorrow :)
[13:44] <dednick> lol
[13:53] <pete-woods> can anyone here point me to the unity window switching API? I believe we will need to use it to keep HUD working when we switch to Mir
[13:53] <pete-woods> I'm told this is a DBus API (and I'm guessing that Unity exports it, but I could be wrong)
[13:57] <Saviq> mhr3, can you drag the XDA screenshots with fix-screenshots all the way to the right?
[13:57] <mterry> pete-woods, try unity-mir?
[13:57] <Saviq> mzanetti, ↑↑?
[13:57] <pete-woods> mterry: thanks, will have a look
[13:58] <mhr3> Saviq, sorry, reflashed the phone 10minutes ago
[13:58] <Saviq> mhr3, nw
[13:58] <mzanetti> Saviq: what?
[13:58] <Saviq> mzanetti, if you open the preview for XDA (the first on in recommended)
[13:58] <Saviq> mzanetti, with fix-genericpreview branch, that is
[13:58] <mzanetti> Saviq: yep
[13:59] <Saviq> mzanetti, the list of screenshots gets confused
[13:59] <mzanetti> does it? looks good here
[13:59] <Saviq> mzanetti, I can't reach the far right edge
[13:59] <mzanetti> hmm
[13:59] <pete-woods> mterry: looks like that's it :D (http://bazaar.launchpad.net/~mir-team/unity-mir/trunk/view/head:/src/modules/Unity/Application/dbuswindowstack.h)
[13:59] <Saviq> mzanetti, I can overshoot and then it goes back some 5 GU or so
[13:59] <tedg> mhr3, Is ZG seeded for touch?
[13:59] <mhr3> Saviq, fwiw i did test that and it worked fine for me
[13:59] <mhr3> tedg, no
[13:59] <mzanetti> Saviq: not happening here
[13:59] <Saviq> mzanetti, I think it gets confused with the other images getting unloaded
[14:00] <mzanetti> yep, seems so
[14:00] <mzanetti> Saviq: it has some weirdness here too
[14:00] <mzanetti> Saviq: but not the one you describe
[14:00] <Saviq> mzanetti, if you overshoot more
[14:00] <mzanetti> Saviq: anyways, should I bump the cacheBuffer?
[14:00] <Saviq> mzanetti, it will destroy the first item
[14:00] <mhr3> tedg, chicken and egg problem, zg not there cause nothing uses it, nothing uses it cause it's not there... :/
[14:00] <mzanetti> Saviq: yep, seeing it know
[14:00] <Saviq> mzanetti, yeah, I'm thinking... probably
[14:01] <tedg> mhr3, upstart-app-launch uses it.
[14:01] <Saviq> mzanetti, as it's really a corner-case scenario (just over ListView.height items that are changing dimensions)
[14:01] <mzanetti> Saviq: it'll be destroyed anyways on any other interaction... so not a memory problem I'd say
[14:01] <Saviq> mzanetti, yeah
[14:01] <tedg> mhr3, We gen an error if it isn't :-)
[14:01] <mzanetti> Saviq: ack, will fix
[14:01] <Saviq> mzanetti, also, the initialWidth on them seems to low?
[14:01] <tedg> mhr3, And I thought the recent apps scope used it?
[14:01] <Saviq> mzanetti, our devices generally have 16x10 or so?
[14:01] <mzanetti> Saviq: ok, will check that too
[14:01] <tedg> mhr3, How else do you know recent apps?
[14:01] <Saviq> mzanetti, thanks
[14:02] <mhr3> tedg, i was told things trying to push data to it isn't a reason to seed it
[14:02] <mzanetti> Saviq: mind noting it down in the MR?
[14:02] <mhr3> tedg, shell does, we don't
[14:02] <Saviq> mzanetti, will do
[14:02] <mzanetti> Saviq: kinda in the middle of fixing other stuff in this branch
[14:02] <Saviq> mzanetti, yeah
[14:02] <tedg> *cough*bullshit*cough*
[14:03] <mzanetti> …
[14:05] <Cimi> mhr3, no MP for this? https://code.launchpad.net/~mhr3/unity8/expose-renderer-hint
[14:06] <mhr3> Cimi, whoops
[14:06] <Cimi> mhr3, remove the console.log
[14:06] <Cimi> mhr3, when you propose
[14:06] <nic-doffay> Saviq, strange thing happening there.
[14:06] <mhr3> whoops #2
[14:06] <Cimi> mhr3, but thanks for putting it ;)
[14:06] <nic-doffay> It doesn't look like the SequentialAnimation happens.
[14:06] <nic-doffay> But I threw in a ScriptAction to print at the end of it.
[14:06] <nic-doffay> And both do?
[14:07] <Saviq> nic-doffay, tried onRunningChanged in the Transition or the SequentialAnimation itself?
[14:08] <Saviq> nic-doffay, onOpacityChanged on primaryImage / searchIndicator, too?
[14:08] <mhr3> Cimi, https://code.launchpad.net/~mhr3/unity8/expose-renderer-hint/+merge/188848
[14:10] <larsu> Saviq: not getting the warning is actually a gcc regression: http://paste.debian.net/48653/
[14:11] <Saviq> larsu, oh, interesting
[14:12] <pstolowski> greyback_: re the problem with screenshot provider in mir - i'm getting this error for every app: https://pastebin.canonical.com/98374/
[14:12] <pstolowski> Saviq: ^
[14:13] <nic-doffay> Saviq, sorry I'm not following what you mean?
[14:13] <didrocks> Saviq: is https://bugs.launchpad.net/mir/+bug/1233870 a new bug? is that something under control from your point of view?
[14:13] <nic-doffay> The state changes fine.
[14:13] <Saviq> nic-doffay, did you try printing stuff onRunningChanged and onOpacityChanged
[14:13] <Saviq> nic-doffay, to see if the transitions are triggered / opacity animated at all
[14:13] <Saviq> nic-doffay, also, verify the states are changing as you expect them to
[14:13] <nic-doffay> Saviq, not yet, I just tried the states to see if they were hit.
[14:13] <pstolowski> greyback_: perhaps timing is different when results come from real scope
[14:14] <greyback_> pstolowski: ok, that's unfortunately due to you taking an app screenshot before the app has even drawn anything to screen. We're working around it by grabbing screenshots more (too) regularly
[14:14] <Saviq> didrocks, well, it's in progress?
[14:14] <nic-doffay> Saviq, it appears the transitions aren't affecting the values.
[14:14] <didrocks> Saviq: yeah, I think the management is asking for an ETA ;)
[14:15] <didrocks> Saviq: the spike shown in https://docs.google.com/a/canonical.com/document/d/1b-X9tN2Q9c_5r39XzA-Ppebbjuin5zVf9SkAGMcdp9Q/edit# is the same, right?
[14:15] <Saviq> didrocks, kinda not my place to determine ETA for Mir...
[14:15] <Saviq> kgunn, ↑↑?
[14:15] <pstolowski> greyback_: the scope just gives icon uri to the shell, so it's still the shell requesting it, so I suppose you need same workaround?
[14:15] <didrocks> Saviq: ok, full Mir-side, nothing on unity8?
[14:16] <Saviq> didrocks, if I read the bug correctly - yes
[14:16] <Saviq> didrocks, and it's the first time I've seen it, too
[14:16] <larsu> seb128: who do I talk to about gcc bugs? Upstream?
[14:16] <pstolowski> greyback_: btw, I just tested your mir branch on the phone, looks good now, except screenshot problem
[14:16] <didrocks> ok, was mostly checking that, thanks!
[14:16] <didrocks> let's wait for kgunn :)
[14:16] <larsu> seb128: gcc-the-compiler, not gnome-control-center ;)
[14:16] <nic-doffay> Saviq, got it figured minor typo.
[14:17] <Saviq> nic-doffay, it rarely is anything else - you need to dig more ;)
[14:17] <seb128> larsu, try doko on #ubuntu-devel
[14:17] <fudus> so unity8 on desktop will be pushed to 14.10? since it was dropped for 3.10
[14:17] <greyback_> pstolowski: yeah, the view will need to refresh that screenshot request at later stages. We've an updateScreenshots method right now that gets called for that very reason - the scope view will need to connect to that somenow
[14:17] <Saviq> nic-doffay, in a WTF moment, step back and look again
[14:17] <kgunn> didrocks: what's up...? breif update on making AP tests work...daniel on my team is going to do a quick hack on retries for getting input devices
[14:17] <kgunn> hoping it works...if so, will need to mp mir...
[14:17] <seb128> larsu, but desrt will probably advice you to directly go to upstream as well, our guys tend to be busy/not always responsive to pings
[14:18] <kgunn> didrocks: and just our luck...server api break as well :)
[14:18] <Saviq> greyback_, pstolowski yeah, but the scope won't be doing anything in that case, right?
[14:18] <nic-doffay> Saviq, it def looks better too. Ready for a re-review now.
[14:18] <pstolowski> Saviq: sure, it won't
[14:18] <Saviq> greyback_, pstolowski it's the shell that will trigger updates on the image
[14:18] <Saviq> nic-doffay, k
[14:18] <pstolowski> Saviq: yes
[14:19] <didrocks> kgunn: the unity8 spike on blank: https://bugs.launchpad.net/mir/+bug/1233870
[14:19] <larsu> seb128: ah okay. Thanks
[14:19] <greyback_> Saviq:  pstolowski: yes the scope has done it's job
[14:19] <didrocks> kgunn: I guess after the current discussion, this one is a blocker to switch Mir on by default
[14:20] <kgunn> didrocks: why ?
[14:20] <kgunn> didrocks: who discussed ? and how does it prevent development
[14:21] <kgunn> didrocks: on a lighter note...kdub is at 7:20am atm
[14:21] <seb128> larsu, why issue did you find?
[14:21] <didrocks> kgunn: rick, olli ;)
[14:21] <didrocks> kgunn: so, if we can get an ETA on that one, that would be helpful I guess
[14:22] <Cimi> mzanetti, give me a machine gun
[14:22] <kgunn> Cimi: sounds like i  need the machine gun first
[14:22] <larsu> seb128: -Wreturn-type doesn't work with g_return_if_fail macors anymore (or maybe with all macros?)
[14:22] <larsu> seb128: so when there's a return with a wrong type inside a macro, you don't get the warning
[14:22] <Cimi> kgunn, let's see who has better reasons :)
[14:22] <larsu> seb128: you do get it when the return is in normal code
[14:22] <seb128> larsu, ok, seems like a good bug to ping doko about
[14:22] <larsu> seb128: will do
[14:23] <didrocks> kgunn: can you update me once kdub is around?
[14:23] <kgunn> didrocks: acl
[14:23] <kgunn> ack even
[14:23] <didrocks> thanks :)
[14:24] <nic-doffay> Saviq, yeah just used to relying on a compiler to pick out my £!$%£"$%£ ups haha
[14:24] <olli> didrocks, re power spike: don't agree
[14:24] <didrocks> olli: that wasn't what was discussed? maybe I missed some words
[14:24] <didrocks> olli: I understood this one was blocking, was I wrong?
[14:25] <olli> didrocks, it blocked rick from using mir as default
[14:25] <Saviq> nic-doffay, I don't see a fade-out of the primaryIcon, though
[14:25] <Cimi> kgunn, having a look, you have reasons too… I'd sponson a couple of machine gun for our moments of anger
[14:25] <Saviq> nic-doffay, just a one-frame visible: false, and then a fade-in of activityIndicator
[14:25] <Cimi> *sponsor
[14:25] <Saviq> nic-doffay, the other way 'round works fine
[14:25] <didrocks> olli: maybe I extrapolated that it was a blocker for switching by default :)
[14:26] <didrocks> kgunn: nice to see the screen turning off now with Mir btw :)
[14:26] <Saviq> nic-doffay, actually no - the searchIndicator disappears in one frame, too
[14:26] <Saviq> nic-doffay, to test just use a longer duration on the animations
[14:26] <Saviq> nic-doffay, to clearly see that they fade out / in
[14:26] <nic-doffay> Saviq, taking a look.
[14:28] <mhr3> Saviq, should someone from #sdk review the thumbnailer branch, or are you going to top-approve it?
[14:28] <nic-doffay> Saviq, looks like the search indicator vanishes in one frame.
[14:28] <Saviq> mhr3, they should
[14:28] <nic-doffay> Saviq, primary icon looks fine to me though?
[14:28] <Saviq> nic-doffay, /me increases the durations to look clearly
[14:29] <olli> Saviq, kgunn, in the EU midnight HO from yday, we discovered that AP testcases/test suites might be using the wrong desktop file
[14:29] <tsdgeos> Mirv: ping again?
[14:29] <olli> and thomi wanted to fix these
[14:29] <olli> I assume he got pinned by the input bug
[14:29] <olli> is there anything we can give jfunk to get started on fixing these?
[14:29] <nic-doffay> Saviq, I reckon it's vanishing in one frame because running is set to false at the same time.
[14:29] <Saviq> nic-doffay, it definitelyis
[14:30] <Saviq> nic-doffay, just add PropertyAction at the end of the SequentialAnimation
[14:30] <Saviq> nic-doffay, or just base running off of opacity > 0
[14:30] <nic-doffay> Saviq, yeah that was it.
[14:30] <Saviq> nic-doffay, also, there's no need to provide both "to" and "from" on Transition
[14:31] <Saviq> nic-doffay, one is enough
[14:31] <nic-doffay> Saviq, yeah just wanted to be explicit though.
[14:31] <Saviq> nic-doffay, or well, in your case, just one Transition { reversible: true } would probably be enough
[14:32] <nic-doffay> Saviq, wasn't aware of that, will change it.
[14:32] <Saviq> nic-doffay, btw, good call with the "internal" states - I was thinking about that late yesterday
[14:33] <nic-doffay> Saviq, I think SnapDuration might be a bit short.
[14:33] <kgunn> olli: i'd have to defer to Saviq on that..but iirc it seemed the "moved desktop file" wasn't an issue?
[14:34] <nic-doffay> Although I'm not sure what would be best suited for what mhr3 brought up in the review.
[14:34] <Saviq> nic-doffay, 2*SnapDuration, for fade-out, fade-in, should be fine I think
[14:34] <nic-doffay> I personally think SlowDuration looks really cool.
[14:34] <kgunn> olli: basically thomi ended up working through setup issues (which ended up in the preamble of the current input bug)
[14:34] <Saviq> nic-doffay, you know where the designers are, don't you ;)
[14:35] <olli> kgunn, that's my understanding, and as such I don't think he was able to work on fixing the desktop file issue in the suites
[14:35] <Saviq> olli, yeah, looking through test suites
[14:35] <mhr3> Saviq, they're hiding under their desks everytime any of us raises eyes from the laptops :)
[14:35] <Saviq> olli, and making sure the commands they use to launch the app works
[14:35] <olli> Saviq, jfunk & team can do that if they know what to look for
[14:35] <nic-doffay> Saviq, the same ones who wanted to put the search indicator on the right? ;P
[14:36] <Saviq> nic-doffay, yes, them!
[14:36] <Saviq> olli, send them my way
[14:36] <Saviq> olli, but TBH there shouldn't be much in terms of "needs fixing"
[14:36] <olli> Saviq, so if you can give me some instructions of what's supposed to work from CLI I can have them go through the tests
[14:37] <Saviq> olli, thomi was just unlucky with the suite he tried yesterday  - it wasn't working on surfaceflinger either
[14:38] <Saviq> olli, effectively running a suite for an app - with unity8 running - should at least launch the app and bring it to front
[14:38] <olli> confirming: regardless of other outstanding AP/Mir issues
[14:38] <Saviq> olli, yes
[14:39] <Saviq> breakfast time!
[14:39] <olli> thx for the reminder Saviq
[14:39] <Saviq> ;)
[14:39]  * olli is having breakfast
[14:39] <olli> :P
[14:40] <nic-doffay> Saviq, ok well I'm done with those changes now.
[14:47] <fginther> Saviq, MacSlow, hope things work better today
[14:48] <Saviq> fginther, yeah, more stable, but sloooow - mir took over two of the VMs again ;P
[14:49] <fginther> Saviq, the issue of the long build queue should be better, retoaded was able to grab us another build server
[14:50] <fginther> Saviq, and I'll see what I can do about the VMs
[14:55] <Cimi> mzanetti, any idea why that needs fixing?
[14:55] <Cimi> mzanetti, apart from testing our patience?
[14:58] <Saviq> fginther, thanks
[14:58] <Mirv> tsdgeos: pong. I'm working in weird hours today, ie. back now. about the Qt patch?
[14:59] <mzanetti> Cimi: no, I haven't looked at the log yet
[14:59] <mzanetti> Cimi: now I have
[14:59] <tsdgeos> Mirv: yep
[14:59] <mzanetti> Cimi: did you read the logs?
[14:59] <tsdgeos> Mirv: can you have a look at it?
[14:59] <mzanetti> Cimi: seems pretty clear to me what happened
[15:00] <mzanetti> Saviq: mhr3: fixed all your comments in fix-genericpreview
[15:00]  * mzanetti moves back to switching-previews
[15:00] <Cimi> mzanetti, actually now there is a log
[15:00] <mzanetti> actually... /me takes a 10 minutes break
[15:00] <Cimi> mzanetti, there wasn't a log when I asked
[15:00] <Cimi> mzanetti, like, a big segfault page
[15:01] <mzanetti> Cimi: mhm... could be that publishing results to public jenkins takes a bit
[15:02] <mhr3> mzanetti, awesomeness, no more comments from me, it's great
[15:05] <Cimi> Saviq, any reason why if and switch were separate in GenericScopeView?
[15:05] <Cimi> Saviq, I just did this http://paste.ubuntu.com/6184223/
[15:07] <Mirv> tsdgeos: yep, looking it now and then trying to get sponsored. earlier today I didn't get to it yet
[15:07] <Cimi> Saviq, ignore that
[15:07] <Cimi> hah
[15:07] <tsdgeos> Mirv: awesome, tx
[15:12] <mhr3> how do i take screenshot of the phone's screen?
[15:14] <sil2100> bregma: ping
[15:14] <sil2100> bregma: we have real AP results for unity now, but it doesn't look too good
[15:15] <bregma> sil2100, any results are better than no results
[15:15] <sil2100> bregma: around ~31 failures per-platform
[15:15] <bregma> I'm seeing a lot of failure to get scope results, somcething that happens form time to time
[15:16] <bregma> and the non-Latin input methods are failing worse than usual
[15:16] <bregma> looks like someone may be playing with gnome-settings-daemon again, trying to break everything at the last minute
[15:16] <sil2100> bregma: right... IBus doesn't seem to 'switch on' - I think the tests need to be re-written
[15:17] <seb128> bregma, we didn't have any g-s-d changes for weeks
[15:17] <seb128> bregma, well, ibus changes
[15:19] <bregma> sil2100, the problem with our tests is the Gnome paradigm changed so it's not possible to successfully switch to using a non-Latin input method i the middle of a session: we used to be able to do that so we could test
[15:20] <bregma> now we'd have to run separate test runs for each local to test effectively, and we're looking for a simple workaround
[15:20] <karni> mzanetti: Do you think fix-genericpreview will make it to daily-proposed today?
[15:20] <karni> I just saw it got +1'ed
[15:20] <karni> I mean, approved
[15:22] <mzanetti> karni: yeah. I think Saviq want's to have a last look but I think we can make it today, yes
[15:22] <karni> mzanetti: yes. and, good! :)
[15:23] <sil2100> bregma: I didn't know that the changes were so radical...
[15:24]  * tsdgeos finds a bug in QSortFilterProxyModel
[15:24]  * tsdgeos goes to the corner and starts crying
[15:27] <tsdgeos> mzanetti: want to give a look at my test to see if there's not something horribly wrong before i start complaining?
[15:28] <tsdgeos> http://paste.ubuntu.com/6184295/
[15:28] <mzanetti> tsdgeos: heh... sure, can do
[15:28]  * mzanetti pities tsdgeos
[15:28] <tsdgeos> Saviq: greyback: ↑↑ can you also guys give it a quick look?
[15:28] <tsdgeos> it's pretty trivial tbh
[15:29] <greyback> tsdgeos: I know a trap when I see one :)
[15:29] <tsdgeos> :P
[15:30] <tsdgeos> pstolowski: ah, and you too, since this is afaics what is making your code not work
[15:30] <tsdgeos> pstolowski: http://paste.ubuntu.com/6184295/
[15:30] <tsdgeos> yay, the loader fix has been merged \o/
[15:31] <tsdgeos> now we just need the distro patch
[15:32] <Cimi> I forgot how to enable write access...
[15:32] <Cimi> on the device
[15:32] <Cimi> touching which file?
[15:33] <karni> Cimi: adb shell touch /userdata/.writable_image
[15:33] <Cimi> karni, thx!
[15:34] <karni> If you mean to make the system writable. then adb shell reboot
[15:34] <Cimi> karni, yep that one
[15:35] <mzanetti> tsdgeos: didn't try to run it yet, but your model seems to lack Q_OBJECT
[15:35] <tsdgeos> mzanetti: true, doesn't fix anything though
[15:35] <mzanetti> tsdgeos: do you have a qmake file for it?
[15:36] <tsdgeos> since the object doesn't really have it's own signals/properties/anything
[15:36] <tsdgeos> mzanetti: sure
[15:36] <tsdgeos> http://paste.ubuntu.com/6184338/
[15:39] <mzanetti> tsdgeos: wth
[15:39] <tsdgeos> what?
[15:39] <pstolowski> tsdgeos: so we get rowinserted instead of datachanged in the proxy?
[15:40] <tsdgeos> pstolowski: no, we get nothing, that's the problem :D
[15:40] <tsdgeos> we should get rowInserted
[15:40] <tsdgeos> we don't
[15:41] <tsdgeos> i.e. QCOMPARE(proxyRowsInsertedSpy.count(), 1); from line 118 is failing
[15:41] <tsdgeos> mzanetti: anything you find obviously wrong in the code?
[15:41] <mzanetti> tsdgeos: not sure... gimme a few more minutes
[15:43] <Saviq> tsdgeos, so what you're seeing is that the call to proxy.rowCount() updates something in the proxy internally and that fixes the issue?
[15:43] <tsdgeos> Saviq: yep
[15:44] <tsdgeos> and that's why when i add billions of debugging to pstolowski code
[15:44] <tsdgeos> it suddenly works :D
[15:44] <pstolowski> :/
[15:44] <Saviq> tsdgeos, right - yeah, nothing wrong I can see with your code
[15:46] <mhr3> sil2100, regarding thumbnailer, if new branches land to trunk, will things to go universe?
[15:46] <mhr3> sil2100, or rather, when will they?
[15:46] <tsdgeos> Saviq: and that rowInserted missing makes the qml list not "see" there's new stuff
[15:46] <Saviq> tsdgeos, yeah, of course
[15:46] <tsdgeos> and we end up with an empty thing
[15:46] <pstolowski> tsdgeos: at least you know the workaround... but this is scary
[15:47]  * tsdgeos wonders if this works in qt4
[15:47] <tsdgeos> nah
[15:47] <tsdgeos> let's try 5.2
[15:48] <tsdgeos> nah
[15:48] <Saviq> lol
[15:48] <tsdgeos> pstolowski: agreed, it is scary
[15:49] <Saviq> mhr3, on https://code.launchpad.net/~unity-team/unity8/fix-genericpreview/+merge/188355 - you only did functional review, right? code review is still on my plate?
[15:49] <mhr3> Saviq, yep, too much qml for my taste :)
[15:49] <Saviq> ;)
[15:50] <mzanetti> tsdgeos: so what exactly is the weird thing?
[15:50]  * mzanetti sees lots of weirdness
[15:50] <tsdgeos> mzanetti: what's weird for you?
[15:50] <Saviq> mzanetti, hum, the image in video preview still starts up some 10 GU high, not 16x10 aspect ratio?
[15:51] <mzanetti> tsdgeos: well, calling rowCount() shouldn't have any affect on the tests afterwards
[15:51] <tsdgeos> mzanetti: of course
[15:51] <tsdgeos> but besides that?
[15:51] <mzanetti> the other turned out to be my bad... missed the filter
[15:53] <mzanetti> Saviq: initialHeight: width * 10 / 16
[15:53] <mzanetti> Saviq: I think those previews are 4:3 aspect ratio
[15:53] <mzanetti> if you're having somewhat similar results on your video dash
[15:54] <Saviq> mzanetti, yeah, even so - they start up really narrow (low height)
[15:54] <mzanetti> Saviq: but I assume that can change depending on the backend
[15:54] <mzanetti> oh really? lemme check again
[15:55] <Saviq> mzanetti, karni and sorry, but it's not gonna happen today - I didn't even start the code review there - first thing tomorrow
[15:56] <mzanetti> fine with me
[15:56] <Saviq> dednick, you saw https://code.launchpad.net/~nick-dedekind/unity8/indicator-titles/+merge/186049/comments/432038 ?
[15:56] <dednick> Saviq: nope. looking
[15:57] <karni> ack, thanks for letting me know Saviq
[15:59] <Saviq> nic-doffay, why didn't you go for a single, reversible Transition? didn't work?
[16:00] <mzanetti> Saviq: I'm afraid the height: implicitHeight breaks the initialHeight
[16:01] <Saviq> mzanetti, huh, shouldn't matter at all - if it does - bug in LazyImage
[16:01] <Saviq> mzanetti, as implicitHeight is just animated between initialHeight and image.height
[16:02] <mzanetti> Saviq: yeah, there is something fishy with implicitHeight in there
[16:02] <mzanetti> Saviq: the "height: implecitHeight" shouldn't be needed, yet it changes things a lot
[16:04] <Saviq> mzanetti, damn, we need to get to the bottom of this
[16:04] <Saviq> nic-doffay, http://pastebin.ubuntu.com/6184445/ apply that to search indicator
[16:05] <Saviq> nic-doffay, if you agree, that is, and I'll happrove
[16:05] <Saviq> nic-doffay, you shouldn't have any values in a Transition btw, that's what States are for
[16:07] <mzanetti> Saviq: isn't there a binding loop in there?
[16:07] <Saviq> mzanetti, shouldn't?
[16:07] <mzanetti> Saviq: height: root.scaleTo == "width" ? implicitHeight * root.width / implicitWidth : implicitHeight
[16:08] <mzanetti> wouldn't implicitHeight change when changing height?
[16:08] <Saviq> mzanetti, that's image's implicitHeight
[16:08] <nic-doffay> Saviq, I wanted to add the PropertyAction at the end of the one.
[16:08] <mzanetti> yeah, and the image's height
[16:08] <nic-doffay> Which is why I didn't opt for the reversible.
[16:09] <Saviq> mzanetti, and no, implicitHeight never changes
[16:09] <mzanetti> Saviq: is that the initial sourceSize of the image?
[16:09] <Saviq> mzanetti, it's internal to Image - is set to what the image's actual size is after applying sourceSize
[16:09] <nic-doffay> Saviq, but it makes little difference to me. I'll change it to that.
[16:09] <mzanetti> after applying sourceSize... ok
[16:10] <Saviq> nic-doffay, it could be a PropertyAction, yeah, but it's just as well - we really want it running whenever it's visible, so...
[16:11] <tsdgeos> mzanetti: actually the move was wrong
[16:11] <tsdgeos> mzanetti: needs to be http://paste.ubuntu.com/6184443/
[16:11] <tsdgeos> but that doesn't help at all
[16:11] <dednick> Saviq: fixed the conflicts.
[16:11] <Saviq> dednick, thanks!
[16:11] <nic-doffay> Saviq, changed.
[16:11] <Saviq> nic-doffay, thanks!
[16:12] <dednick> Cimi: any movement on the ubuntu-settings-components branch in MP?
[16:12] <Cimi> dednick, nope
[16:12] <nic-doffay> Saviq, np
[16:13] <Cimi> dednick, the rest is fine, I didn't test the calendar
[16:14] <Saviq> nic-doffay, happroved
[16:15] <nic-doffay> Saviq, any idea why jenkins is unhappy?
[16:15] <Saviq> nic-doffay, flaky test, ignore
[16:17] <nic-doffay> Saviq, need your opinion on this too: https://code.launchpad.net/~nicolas-doffay/unity8/dismiss-keyboard/+merge/188599 it's my opinion that the InverseMouseArea is a bit flaky with the keyboard visible.
[16:17] <nic-doffay> If so I need to bring it up with the SDK guys.
[16:17] <Saviq> nic-doffay, the "z" there shouldn't be needed
[16:18] <Saviq> nic-doffay, and it took popover.contentHeight into consideration for a reason - otherwise you tapping on the popover to select a previous search or close it would unfocus the input field
[16:20] <nic-doffay> Saviq, ah I see. I'll change that. Still would like your thoughts on what I mentioned before though.
[16:20] <Saviq> nic-doffay, it's possible, IMA proved troublesome in the past
[16:21] <Saviq> dednick, is it designed that we now have "Incoming" instead of "Messages" and "Wednesday" instead of "Time & Date"?
[16:22] <dednick> larsu: ^ re Incoming ? i believe you changed that recently
[16:22] <nic-doffay> Saviq, most notably I noticed there were loads of issues with typing and then trying the inverse mouse area.
[16:22] <dednick> Saviq: re Time & Date, unfortunately yes, it is as designed.
[16:23] <nic-doffay> Either way I guess the QML is sound so we can land this? Pushed the modifications too Saviq.
[16:23] <Saviq> dednick, that I'm ~fine with, but "Incoming" sounds weird
[16:23] <Saviq> nic-doffay, I'll look at it closely tomorrow
[16:23] <tedg> Saviq, That is correct to design
[16:24] <Saviq> tedg, :S
[16:24] <mzanetti> Saviq: oh boy... it works *sometimes*
[16:24] <Saviq> mzanetti, uh oh
[16:24] <dednick> Saviq: yeah, the name isn't great, but i guess it's more "technically correct" seeing as it handles missed calls as well.
[16:24] <tedg> Saviq, Better discussed over several beers :-)
[16:24] <Saviq> tedg, we're planning those
[16:24] <tedg> +1
[16:26] <dednick> i would go for "Notifications" or something like that.
[16:27] <Saviq> dednick, approved
[16:27] <Saviq> 10 MPs awaiting...
[16:28] <Saviq> 2 hrs per MP...
[16:28] <Saviq> mterry, un-approving, then
[16:29] <mterry> Saviq, yeah, I'm a bit confused on why it's happening, will dig
[16:29] <Saviq> mterry, thanks
[17:02] <Saviq> kgunn, I think it worth mentioning for the wiki that PPAs will be easily available for those that want trunk Mir on their 13.10 installations
[17:02] <larsu> dednick: yes, design asked me to.
[17:27] <olli> Saviq, how do I run an AP test (say the one thomi used yday) from cli?
[17:28] <olli> cli/shell
[17:33] <Saviq> olli, autopilot run blah
[17:33] <Saviq> olli, where blah is the test suite name
[17:34] <Saviq> olli, on a readonly device - thomi's description on bug #1233944 has some details
[19:03] <thomi> morning
[19:05] <Saviq> hey thomi
[19:05] <thomi> hey man
[19:05] <thomi> gotta go help move some furniture, brb
[19:05] <Saviq> thomi, long story short: Mir tried to open the input devices before udev gave it the permissions
[19:09] <thomi> Saviq: yeah, I saw that
[19:09] <thomi> just building packages to test out the fix now
[20:18] <veebers> sergiusens: ping?
[20:29] <sergiusens> veebers, pong
[20:39] <veebers> sergiusens: hey, how you doing? I'm not sure I understand your response to my email (re: unity8 AP test dependencies etc.)
[20:40] <sergiusens> veebers, we had a very similar conversation that is about to happen with your emails ;-)
[20:41] <sergiusens> veebers, the best I can think of for those deps you say are missing is to add them to autopilot-touch
[20:42] <veebers> sergiusens: well, I guess you could make an argument for python-mock being there (req. by ubuntu ui toolkit) but I"m not sure unity8-fake-env belongs there) it really sounds like a work around
[20:44] <sergiusens> veebers, the problem with the latter is that it's using packaging for something it shouldn't
[20:44] <sergiusens> veebers, but anyways, if that does not depend on anything else I can take care of it
[20:45] <sergiusens> veebers, although it wasn't in my plans to work on getting non click stuff before click
[20:47] <sergiusens> veebers, just to reinforce, both of these are workarounds, the proper solution is to have autopilot work in daemon mode
[20:48] <thomi> what's "daemon mode"?
[20:48] <veebers> thomi: I also ask thomis question
[20:48] <veebers> err, sorry for the ping thomi :-\
[20:49] <sergiusens> thomi, the super nimble autopilot that we talked about during the sprint
[20:50] <thomi> sergiusens: oh, right. I think we're not going to have that any time soon, so we'd better stop building in workarounds/hacks and waiting for it to arrive
[20:50] <thomi> better to fix it properly for what we have today, IMO
[20:50] <sergiusens> thomi, well I don't know what the proper way is for unity
[20:50] <sergiusens> thomi, I'll leave that to you, the proper way for me is that said daemon mode
[20:51] <thomi> I don't really understand the problem you guys are talking about, as I lack context
[20:51] <sergiusens> thomi, don't go into read write mode and be able to test unity8
[20:51] <thomi> sergiusens: why is that a problem today?
[20:51] <sergiusens> thomi, which is a req we are aiming for for click apps and have it solved, but not for unity8
[20:52] <sergiusens> thomi, many packages
[20:52] <thomi> you're saying that the tests require additional stuff to be installed before they can run?
[20:52] <sergiusens> thomi, yes
[20:52] <thomi> ok
[20:53] <sergiusens> thomi, like the unity8-fake-env package
[20:53] <thomi> sergiusens: remember waaaaay back when you proposed the phablet-click-test-setup script, and I kept saying "yeah, but it'll need to handle dependencies", and you were all "nah, it'll be cool"... :P
[20:53] <thomi> seems that script really needs to handle dependencies :)
[20:53] <sergiusens> thomi, yeah, for click apps
[20:53] <sergiusens> thomi, never said anything about non click
[20:54] <sergiusens> thomi, want to play the remember game ;-)
[20:54] <thomi> sergiusens: how the application is packaged is irrelevant here - the test suite needs dependencies, and we're no longer installing the test suite using apt-get, so we need a replacement
[20:54] <thomi> whether the application is packaged as a click app or not is beside the point
[20:54] <thomi> or am I missing some nuance?
[20:54] <sergiusens> thomi, who came up with that req for ackaged apps?
[20:55] <thomi> sergiusens: you mean for the test suite? I don't understand your question, sorry
[20:55] <sergiusens> *ackaged == deb
[20:55] <thomi> the requirements on the test suite .deb files (like unity8-autopilot, for example) are set by the test authors
[20:56] <thomi> is that what you meant?
[20:56] <sergiusens> thomi, well let them figure out how to test without rw, from my pint of view, there is no reason they shouldn't keep doing what they are doing as they are image builders, not app devels
[20:57] <thomi> O.0 <- confusedface
[20:57] <thomi> the problem is that dev teams *are* nominating dependencies, but they're not getting installed
[20:58] <thomi> but naywya
[20:58] <thomi> *anyway
[20:58] <sergiusens> thomi, well that's easy, just install them
[20:58] <thomi> sergiusens: it'd be nice if that happened automatically when you grabbed the test suite, like it used to do when we could use apt-get. That's my complaint, and, I suspect veebers' as wel;l
[20:58] <thomi> it used to happen automatically, now it requires a manual step
[20:59] <sergiusens> thomi, phablet-test-run -n unity8-autopilot unity8
[20:59] <thomi> and what's worse, a manual step in several locations
[20:59] <thomi> sergiusens: but that doesn't work on a RO FS, for that, we need to use phablet-click-test-setup
[20:59] <thomi> which is the bit that's broken
[20:59] <veebers> sergiusens: for that to work with ubuntu-system/ro you need to make a change right?
[20:59] <sergiusens> thomi, as the name says phablet-CLICK-test-setup
[20:59] <veebers> ah, what thomi said
[20:59] <sergiusens> as in CLICK
[21:00] <veebers> sergiusens: right, but is unity8 even packaged as a click package?
[21:00] <thomi> sergiusens: yesterday you told me that the 'click' part of the name should be removed. Want me to find that in the scrollback?
[21:00] <sergiusens> I thought it was the QA team's problem to solve that
[21:00] <sergiusens> thomi, yeah, if we eevntually solve that
[21:00] <sergiusens> thomi, sroll back if you want
[21:00] <sergiusens> thomi, I also said I wanted click to work first
[21:01] <thomi> sergiusens: I'm not saying that it's necessarily your problem, just that we need a solution, and it's not going to happen if everyone keeps saying "oh yeah, the problem can be solved by writing this massive autopilot feature"
[21:01] <thomi> so instead, let's figure out how to solve the problem properly, and then get it done
[21:01] <sergiusens> thomi, well we had that meeting during the sprint to talk exactly about this; I'm just seeing a rerun
[21:02] <thomi> maybe my memory is broken (quite probable), but I don't remember a meeting about provisioning test suite dependencies?
[21:03] <sergiusens> thomi, nope, not about that
[21:03] <thomi> well, that's what I'm talking about ;)
[21:04] <sergiusens> thomi, it was more general and on a pad which I never wrote down the address for
[21:04] <thomi> sergiusens: would you like me to take the lead on this, and talk to the CI & QA teams and figure out what the correct solution is?
[21:04] <thomi> sergiusens: we keps all the links in a SS, one sec, let me find the URL
[21:05] <thomi> sergiusens: https://docs.google.com/a/canonical.com/spreadsheet/ccc?key=0AoH1kjVoZRmBdEJ6eWZXNnRoNnFnUDBBY0xvVWIwdWc&usp=drive_web#gid=5
[21:08] <sergiusens> thomi, seems https://docs.google.com/a/canonical.com/document/d/1sULsSfzHdTt_8ueJoD7U_vPsD9slC0qgjxBtLOOZaDc/edit wasn't fully passed in
[21:08] <sergiusens> thomi, I do recall one thing being said about test author being able to use whatever they wanted
[21:08] <sergiusens> as a pro for doing this
[21:09] <thomi> sergiusens: again though, that's a different topic. I remember *that* meeting - that was scoping out the remote test runner, to try and figure out whether we should build it or not. Yes, one of the advantages would be that it makes dependency installation easier, but it's a huge chunk of work, and I'm saying that we probably ought to solve this problem separately
[21:10] <thomi> since it's looking pretty unlikely that the remote runner will get done any time soon
[21:10] <sergiusens> thomi, tbh, it is the proper solution no matter when it happens
[21:11] <thomi> sergiusens: you may be correct, but a proper solution in 5 years time (or whevever) doesn't help us today, or for the next 5 years
[21:11] <thomi> I think we need  a solution that's cheaper to implement in the short term
[21:11] <sergiusens> thomi, the side option as we discussed with you and doanac two months ago was to add all the deps to autopilot-touch that we needed to test the image
[21:13] <thomi> that's certainly a possible solution. It still doesn't give us the ability to change dependencies easily across multiple projects, but I suppose it's better than nothing
[21:13] <thomi> sergiusens: could your test setup script not read the dependency list from the debian/control file, and install them along with the test suite?
[21:13] <sergiusens> thomi, well I don't actually _install_ anything
[21:14] <thomi> well, ok, s/install/write them to disk somewhere so they can be loaded by the test suite/
[21:14] <sergiusens> thomi, some of those dependencies could have weird requirements
[21:14] <sergiusens> thomi, some may require import paths for qt, other may have hard coded deps
[21:14] <sergiusens> s/deps/paths/
[21:15] <thomi> yeah, it's a tricky problem - are we any closer to having an overlayfs that works on the phones? Cos that would make this trivial
[21:16] <sergiusens> thomi, you can bug the kernel team about that, but there's a different kernel per device and some devices have too much of an old kernel
[21:16] <thomi> hmm, ok
[21:16] <thomi> sergiusens: you agree though that it would solve the problem, thought?
[21:16] <thomi> *though?
[21:17] <sergiusens> thomi, yes, it would solve the problem of not being able to install directly, but then I'm not sure why you don't just break into rw for these two apps under test
[21:18] <thomi> well, from a QA perspective, i think it's better to never have to break into RW. Secondly, it's unlikely that it'll be just two apps. It might be 2 apps today...
[21:18] <thomi> thirdly, it's better to have a single approach that will work across *all* apps, forever :)
[21:19] <sergiusens> thomi, I don't think you want this in production/released phones
[21:19] <thomi> an overlayfs? why not?
[21:20] <sergiusens> thomi, with overlay it would be easy to break confinement, override drm and stuff like that
[21:20] <sergiusens> thomi, nothing prevents it now, but I do see that it won't go long term
[21:21] <thomi> OK. I'll go talk to kernel-type people, and see if we can't get some traction on this. In the mean time, please stop telling people that autopilot daemon mode / remote runner is the solution to their problems :)
[21:22] <thomi> ...unless, of course, you want to write & maintain it ;)
[21:22] <sergiusens> thomi, ok, I'l just forward them to you then
[21:23] <robru> can somebody help me uninstall mir? it's seriously screwed on my system and following the uninstallation instructions just results in this: http://paste.ubuntu.com/6185686/
[21:32] <fginther> robru, I 'think' the ppa info is outdated. try: "sudo apt-get remove unity-system-compositor;sudo restart lightdm "
[21:33] <robru> fginther, thanks
[21:37] <robru> fginther, you are a lifesaver! Now I just need to figure out why I can't access launchpad and i might actually be able to get some work done today...
[21:38] <fginther> robru, glad it worked.
[21:40] <robru> fginther, you're not experiencing any issues accessing lp are you? multi-minute load times? frequent 'connection reset by peer' ?
[21:40] <fginther> robru, no, I don't see any issues today
[21:41] <robru> fginther, hummm, there's no vanguard in #is... can you point me at 'RT'? I don't know what that is
[21:42] <fginther> robru, hang on, it just isn't your day is it?
[21:42] <robru> fginther, nope :-/
[22:03] <Saviq> greyback, go away
[22:03] <greyback> Saviq: no you go away
[22:03] <Saviq> greyback, no YOU
[22:04] <greyback> Saviq: you times 100
[22:04] <Saviq> greyback, no, you whatever times you say times 100
[22:07] <Saviq> oh, it worked :)
[22:11] <greyback> Saviq: I simply couldn't deal with that numbers that high
[22:12] <greyback> yay for day off!
[22:12]  * greyback gone
[22:12] <Saviq> greyback, have a good one