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

Saviqmornin'08:33
Saviqwhat did I miss? ;)08:33
Zhenech[GitHub] Subscribed to credativ/puppet-anysyncd notifications08:36
Zhenechups08:36
mhr3Mirv, btw do you have mediascanner2 in the qt5.2 transition? we added a qml plugin to it very recently, so it probably needs rebuild08:45
tsdgeosSaviq: not much i'd say08:45
Saviqmhr3, does it use private headers? if not - it should not need rebuilding08:46
Saviqtsdgeos, cool beanz08:46
tsdgeosSaviq: i don't think what Thomi says makes any sense, or at least any sense for my problem08:47
tsdgeosSaviq: so i guess all i have left is debug this stuff myself?08:47
mhr3Saviq, no, but doesn't it need to relink against the core5a binary?08:47
Saviqmhr3, ah, right, damn ABI break08:47
Saviqtsdgeos, so you think your issue is not what I described?08:48
tsdgeosSaviq: you say objectName evaluation is "not taken into account" or something08:49
tsdgeosi say "the object is simply not there"08:49
tsdgeosor not updated at all08:49
tsdgeosi can scroll a list and autopilot vis will still show the old children08:49
tsdgeoseven after restarting autopilot vis08:49
Saviqtsdgeos, I might not have read into your email properly08:50
tsdgeoswell i did not add that extra info (the objects not updating)08:50
tsdgeosbut still it can't be waht you say08:50
tsdgeossince i have08:50
tsdgeosobjectName: foo + index08:51
tsdgeosand all children names are08:51
tsdgeosfooNumber08:51
tsdgeosand the index is not really changed there08:51
tsdgeosso all items have the evaluation08:51
tsdgeosjust one at least is missing08:51
tsdgeosof course it still could be something similar to what you say but i don't think it may be the easiest explanation08:52
Saviqtsdgeos, what I was saying (the problem we encountered) was that ap managed to select a object0 before it became object1 after the binding got updated08:53
tsdgeosSaviq: right, but how can that cause object0 not existing at all?08:54
Saviqtsdgeos, sure, it can't08:55
tsdgeosand that's what i'm seeing08:55
Saviqtsdgeos, so yeah, as I said I might not have read your email properly - was Sunday evening after all ;D08:56
tsdgeossure :-)08:56
Saviqtsdgeos, so yeah, please reply to my email saying I'm full of shit08:57
Saviqoh yay, no WiFi after flashing...08:57
Mirvmhr3: incidentally, I found out about mediascanner2 on Friday and I've just built it08:58
mhr3Mirv, very well08:59
tsdgeosSaviq: so back to my question, do you have any clue how/where to start debugging that?08:59
Saviqtsdgeos, /me looks09:01
tsdgeosSaviq: ah, and we discovered some compiz regression that made some tests fail09:09
tsdgeosSaviq: https://code.launchpad.net/~aacid/unity8/vjog_compiz_workaround09:09
tsdgeosbasically it's not obeying window sizing sometimes09:09
Saviqtsdgeos, hmm after applying the patch on top of new-scopes-cleanup it doesn't seem to unlock the screen even... does it work for you?09:11
tsdgeosit was defenitely working on friday09:12
tsdgeosyeah works today too09:13
tsdgeoshttp://paste.ubuntu.com/7066602/09:13
tsdgeosSaviq: i'm doing this with 5.0 though09:14
Saviqtsdgeos, I'm on 5.0 currently, too09:15
tsdgeosok09:15
tsdgeosand you get stuck on greeter when rnning autopilot?09:15
Saviqtsdgeos, yes09:17
tsdgeostry again? :D09:17
Saviqtsdgeos, looks like it's looking for the home scope and fails09:17
tsdgeosSaviq: are you up to date?09:17
tsdgeosor maybe i didn't push09:17
tsdgeosthat can very well be09:17
Cimitsdgeos, did you have any luck with autopilot?09:17
Saviqtsdgeos, ah, actually it didn't switch to apps scope on startup09:18
Saviqtsdgeos, s/didn't/doesn't/09:18
tsdgeosCimi: well, i thinking i have found an autopilot bug, proving it is hard though09:18
tsdgeosCimi: did you manage to get past the "everything fails" point?09:18
tsdgeosSaviq: nothing left for pushing here09:19
Cimitsdgeos, I stop when you told me you were dealing wih it :)09:19
tsdgeosCimi: ok09:19
CimiI can try again now09:19
Saviqtsdgeos, yeah, the "switch to apps on startup" seems somewhat flaky :/09:19
Cimitsdgeos, just read your mail09:20
tsdgeosSaviq: weird, never had that problem on friday09:21
Cimitsdgeos, when did this dashcategory error start?09:21
tsdgeosCimi: i have no clue09:21
Cimitsdgeos, it's a new scopes error or also in trunk?09:21
tsdgeosCimi: the test works in trunk but the whole QML tree is different, so that proves nothing09:26
Saviqbiab09:48
om26ertsdgeos, are you taking new-scopes introduced regressions as bugs or should I wait for it to first land in trunk ?09:56
tsdgeosom26er: if it is indeed a regression/bug, sure09:57
om26ersure, the scrolling in the Apps scope have apparently become much jerky on my mako -- will report in lp.09:58
=== pete-woods is now known as pete-woods-afk
Cimitsdgeos, ok they fail for trunk here as well11:17
Cimitsdgeos, on a different machine11:17
Cimitsdgeos, I have this QQuickLoader.isCurrent failed error11:17
tsdgeosno idea :11:18
tsdgeosyou'll have to try to dig more yourself11:18
tsdgeosotoh it should not be that hard for us to run the autopilot tests11:18
Cimiyeah :(11:18
Cimitsdgeos, are you trying to11:36
Cimiunderstand as well?11:36
tsdgeosCimi: no, it works for me, i'm fixing the tests in new-scopes that don't work because of the code changes11:36
Cimi@unity who else has issues with autopilot run unity8 ?11:37
* mzanetti has *always* issues with AP :D11:37
Cimimzanetti, can you run11:37
mzanettiCimi: I fiddled a while with it last week and got it to run eventually11:37
Cimiautopilot run unity8 > log11:37
* mzanetti tries11:38
Cimithen a nice grep FAIL log | pastebinit :)11:38
mzanettiCimi: anything you're interested in particular?11:44
mzanettiCimi: I don't want to run all of them now because they either fail or It'll mess up my desktop if I kill my notification system11:44
Saviqtsdgeos, Cimi, the QQuickLoader.isCurrent thing, AFAICT, is dash not switching to Apps on startup11:50
CimiI'm replying to a mail, will be back soon :)11:51
Cimiand reply to you :)11:51
CimiSaviq, so we have a fix for that12:05
Cimi?12:05
=== MacSlow is now known as MacSlow|lunch
SaviqCimi, no, that's something I found just today with tsdgeos's cleanup branch12:21
CimiSaviq, weird12:21
CimiSaviq, I have that bug with trunk12:21
Cimi;)12:21
SaviqCimi, well, if you're going "autopilot run unity8", you're not actually running trunk most probably, but the system-wide installed version12:22
SaviqCimi, you need `make -C install`, `PYTHONPATH=tests/autopilot autopilot run unity8` to run the local build12:22
Saviqalways check out what autopilot says ni "loading tests from..."12:23
Saviqin12:23
CimiSaviq, oh!!12:24
CimiSaviq, ok, nevermind :D12:24
=== _salem is now known as salem_
=== alan_g is now known as alan_g|lunch
tsdgeosSaviq: actually now i'm having a problem with the correct scope not being loaded, it's sad, isCurrent is true, but it's not really the current one, having a look12:54
mzanettididrocks: when you have some time, could you please review the packaging changes in here and related branches? https://code.launchpad.net/~mzanetti/unity-api/new-screenshot-and-focusing-api/+merge/19981013:10
didrocksmzanetti: sure, will do in ~20, that's ok?13:11
mzanettididrocks: not urgent at all13:11
didrocksok ;)13:11
=== MacSlow|lunch is now known as MacSlow
didrocksmzanetti: hum, where are your packaging change? :)13:43
didrocksor the diff isn't correct?13:43
mzanettididrocks: well, it's only bumping unity-application-iml version13:46
didrocksmzanetti: ah ok, that part that we marshmall13:47
mzanettididrocks: and the others bump their "Provides" and "Requires" accordingly13:47
mzanettiso yeah, that's the question. is that enough?13:47
didrocksmzanetti: yeah, that's perfect :)13:47
mzanettinice, thanks13:47
didrocksmzanetti: yw, commented (to +1)13:48
mzanettipurrfect13:48
mzanettigreyback: jfi ^13:48
greybackmzanetti: ack from me so13:48
=== alan_g|lunch is now known as alan_g
kgunnmhall119: mornin', hey i put in for 2 sessions at uds...and they're still in "proposed" can you help get them on the calednar?14:00
kgunnhttp://summit.ubuntu.com/uds-1403/kgunn72/meetings14:00
mhall119kgunn: what track?14:13
mhall119kgunn: you need to get a track lead to approve it and schedule it14:13
mhall119track leads are listed here: http://summit.ubuntu.com/uds-1403/tracks14:13
kgunnmhall119: thanks14:15
MacSlowmterry, did you have better luck with the newer version of the egl-spinner or didn't you try yet?14:20
mterryMacSlow, I didn't try yet...  Let me do that14:20
elopiotsdgeos: is there something I can do to help with the autopilot updates?14:20
tsdgeoselopio: i'm on it, just need some more time to debug some stuff14:21
elopiotsdgeos: ok, let me know if you need a hand.14:21
MacSlowmterry, vanvugt tried it earlier today on his mako too and it worked there too.14:21
tsdgeostx14:21
mterryMacSlow, hrm14:22
MacSlowmterry, I hope you'll have better luck this time around14:22
MacSlowmterry, when it crashes again a backtrace would be helpful if you can get one14:22
mterryMacSlow, I did find where it crashed...  But will see if it happens again14:23
mterryMacSlow, works like a charm  :)14:23
MacSlowmterry, rev7 with the text-rendering?!14:24
mterryMacSlow, yup.  might as well merge that into the shared USC branch then14:24
MacSlowmterry, awesome!14:24
mterryMacSlow, has Christina seen this yet / had a chance to comment?14:24
mterryMacSlow, I know she was OK with the general idea, but curious about details14:25
MacSlowmterry, I cc'ed here in the last email (~30 minutes ago) which has a link to the latest video... so I hope she'll reply with some feedback14:25
mterrycool14:25
MacSlowmterry, from my side it basically just cleaning up the code and making it nicer to maintain... so I would (after that) hand off to you and trun back to notification-duties.14:26
mterryMacSlow, sure14:27
MacSlowmterry, although I _love_ GL! :)14:27
mterry:)14:27
mterryMacSlow, well if Christina wants changes you can dive back into it  :)14:27
MacSlowmterry, I take any excuse to do that :)14:28
MacSlowkgunn, you didn't read that *waves.hands* ;)14:28
greybackkgunn: in "alsamixer", if you hit F5, there's a "mic boost" option, maybe you can turn it down?14:33
Cimimhr3, dednick btw, tomorrow we can meet 1pm in front of brixton tube station14:34
dednickCimi: ok. cool14:35
mhr3Cimi, ok14:39
Cimikgunn, if you want to stay in mumble, maybe we can try different settings, or later when you have time14:42
Cimitsdgeos, Saviq14:44
Cimicimi@vostro:~/Development/unity8/new-scopes-cleanup/builddir$ grep -B 3 FAIL LOG | pastebinit14:44
Cimihttp://paste.ubuntu.com/7067998/14:44
tsdgeosthats' bad14:45
tsdgeosstuff like test_passphrase_screen_wrong_password should be passing14:45
Saviqyeah, something weird with shift, too - "FFFFFf" instead of "FFFFFF"14:45
tsdgeosi'm down to http://paste.ubuntu.com/7068005/ failing14:46
Cimitsdgeos, some context?14:46
Cimigrep -B3 ?14:46
tsdgeosCimi: what do you mean some context?14:46
Cimitsdgeos, apart the error14:47
Cimisorry14:47
Cimithe name of the failing test14:47
Cimitsdgeos, I run tests  with > log14:47
tsdgeosCimi: they fail because they are checking toatlly wrong stuff14:47
Cimithen I grep -B3 FAIL log14:47
* greyback needs to run to bank, bbiab14:49
mzanettidandrader: so, this Model of apps, that's still the ApplicationManager, right?14:49
kgunnCimi: i got about 10 min...wanna try real quick14:51
kgunnmumble14:51
tsdgeosand actually the 4 are basically the same14:52
tsdgeosthe preview opening doesn't work14:52
tsdgeosbecause we changed the way preview opening works14:52
dandradermzanetti, no, it's a simple ListModel of qml items14:52
dandradermzanetti, it's a list model of MirSurfaceItems (qquick items holding mir surface gl textures)14:52
dandradermzanetti, then ListView has a Window.qml delegate that takes a suface and reparent it to itself14:52
mzanettidandrader: ah, but shouldnt the MirSurfaceItems be the Delegates?14:53
mzanettiand the model still hold ApplicationInfo objects?14:53
mhr3Cimi, l4d or l4d2? :)14:53
mzanettidandrader: at least that's what I though of when doing the right edge14:54
Cimimhr3, l4d214:54
mzanettidandrader: so all we'd need to do in that case would be to replace "Image" with "MirSurfaceItem" in the SpreadDelegate.qml14:54
dandradermzanetti, no, MirSurfaceItems are created from the qpa/cpp/appMan side. it essentially wraps a gl texture that's bound to a mir (EGL) surface14:54
dandradermzanetti, so that the qml side can manipulate that texture as a QQuickItem fairly easily,14:55
mzanettiok... so the ApplicationInfo objects will become MirSurfaceItems?14:55
dandradermzanetti, no, they're are still there14:55
mzanettibut then we have 2 models14:55
dandradermzanetti, yes14:56
mzanettiok... I think I understand14:56
mzanettibut is that really way to go?14:56
dandradermzanetti, the concept of an application is one thing, more abstract, a client surface is another one14:56
mzanettiyeah.14:57
mzanettithat's why I said, the MirSurfaceItems should be the delegates14:57
dandradermzanetti, and stage wants to arrange an shuffle around surfaces/client windows14:57
mzanettibut the stage still needs to represent all running apps14:57
mzanettiand not some other model of surfaces14:58
mzanettidandrader: so we can't create such MirSurfaceItems in QML as we like?14:58
mzanettifor example, one in the app-spread, one in the runningappsgrid etc14:58
mzanettibut still showing the same surface14:59
dandradermzanetti, at the moment no. mir tells the QPA that there's new surface and then QPA creates a MirSurfaceItem that makes use of it, so that the mir surface can be used by the QML scene15:03
dandradermzanetti, then QPA informs the QML side that there's a new MirSurfaceItem available15:03
dandradermzanetti, QML side (unity8) then reparents it to the view delegate to have it on the scene15:04
mzanettidandrader: what I mean is this: I'm ok if we need to create such MirSurfaceItems somewhere in C++, but I don't think we should only have that model available in qml... Can't we wrap that somehow? So that I can just say this in QML:15:04
mzanettiAppSurface { appId: "foobar" }15:04
mzanettiand that then just finds its MirSurfaceItem somehow and paints its contents in the AppSurface {} item15:05
mzanettiso we can still do things like:15:05
mzanettiListView { model: ApplicationManager; delegate: AppSurface { appId: model.appId } }15:05
mzanettiotherwise this sounds like a hugely complex thing to me to match surfaces with apps in QML all over the place15:06
dandradermzanetti, I think it would be good if you checked out the current stage code to get an idea on how that all fits together15:07
mzanettidandrader: what is the current stage code?15:08
dandradermzanetti, lp:~unity-team/unity8/mirCompositor15:08
mzanettiok. I'll check it out15:08
dandradermzanetti, also an application can have many surfaces (on the desktop at least)15:10
mzanettidandrader: true... but that doesn't really invalidate my point15:10
dandraderjust something to keep in mind15:11
mzanettiyeah, sure15:11
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
greybacksomeone do me a favour please: run "xev | grep -A 3 KeyRelease" and hit SysReq15:51
Zhenechgreyback, http://paste.debian.net/86897/15:53
greybackZhenech: perfect, thank you15:53
Zhenechit's a thinkpad, so I actually pressed Fn+PrtSc if that matters15:54
* greyback has almost managed to remap his macbook's eject key to sysreq15:54
greybackZhenech: I've a macbook, no such key so playing with remapping15:55
greybackI've mapped eject to printscreen, which is close :)15:55
Zhenechpost it on the web, newer thinkpads miss that one too15:55
greybackwill do15:55
tsdgeoseean: is it ok for you if i kill the get_details autopilot tests? they don't do much, do they?15:57
tsdgeoswhat?15:57
tsdgeoseean: sorry15:57
tsdgeoselopio: read ↑↑↑↑15:57
elopiotsdgeos: I need that to check the application details shown on the click scope16:03
elopioif you remove the test, I might break the helper without noticing16:03
tsdgeos?¿16:03
tsdgeoselopio: ↑↑16:06
tsdgeosnot sure i understand what you men16:06
tsdgeosmean16:06
MacSlowmterry, pushed rev8 of the egl-spinner, which should be easier to read and alter. Until we get feedback from Design, I'll leave it as-is. If you run into any issues just ping me.16:07
mterryMacSlow, thank you!16:08
MacSlowmterry, yw16:08
elopiotsdgeos: I mean that if something changes on unity and get_details doesn't return what I expect it to return16:16
elopiothen the click scope tests will start failing without a clear reason16:16
elopiowhy do you want to remove them?16:16
tsdgeoselopio: because i don't see the point16:17
tsdgeosi mean it's not testing anything else than "we're showing stuff on screen"16:18
elopiotsdgeos: it's testing that get_helpers returns the title, the subtitle and the description.16:19
elopiothe real user tests is in the click scope, where we check that the title, subtitle and description match the ones of the application we are opening.16:19
elopiobut lets say we remove this test in unity.16:19
elopioand for some reason, unity no longer shows the description anymore.16:20
elopiowithout the test, we are with no clue about how it will affect other tests that might be using the helper.16:20
tsdgeosah16:22
tsdgeosso you're using that helper from somewhere else?¿?16:23
tsdgeoselopio: ↑ ?16:23
elopiotsdgeos: yes. On the unity-scope-click suite.16:24
elopiotsdgeos: all the things you will see with tests in test_emulators.py are because I'm using them somewhere else.16:24
tsdgeoselopio: well16:24
tsdgeosthat sucks16:24
tsdgeosi mean i suck probably :D16:24
elopiothey are not user tests, just to make sure that the helpers remain working.16:24
tsdgeosis that documented?16:24
tsdgeosbecause i had no idea16:24
tsdgeoselopio: maybe it's better you fix those tests then16:25
tsdgeosthose = unity816:25
tsdgeosbecause i'm not going to go over unity-scope-click suite and fix them to work with new-scopes16:25
elopiotsdgeos: yes, mzanetti told me to add a big comment on test_emulators.py explaining it.16:25
elopioit's after the copyright.16:25
mzanettipoor elopio... first needed to convince mzanetti and now tsdgeos :D16:26
elopiotsdgeos: you won't need to go over unity-scope-click suite as long as you keep the signatures of the helpers.16:26
tsdgeoselopio: i can't keep the signature of the helpers16:26
tsdgeosbecause everything has changed in previews16:26
tsdgeosit's a different world16:26
elopioand if you have to break a signature, then you talk to the devs of the scope tests, and coordinate with them the updates.16:26
tsdgeosright16:26
elopioin this case, I'm the dev of the scope tests :)16:26
tsdgeosi don't want to do that16:26
Saviqtsdgeos, don't want to do what?16:27
tsdgeosbe a good citizen16:27
Saviq;)16:27
tsdgeosfeeling moody today16:27
tsdgeossorry16:27
Saviqtsdgeos, those helpers are ours, we need to commit to them, and yeah try and maintain the signatures, but obviously in this case we'll need to break them16:28
Saviqclick scope tests will need to be redone just as well16:28
elopiotsdgeos: I'm here to help with whatever you need.16:28
elopioI'm used to moody devs, don't worry ;)16:28
tsdgeoselopio: well basically16:29
tsdgeosthe get_details thing doesn't apply anymore16:29
tsdgeossince you can have a random number of stuff in a preview16:29
tsdgeosthere is not just one title16:30
elopiotsdgeos: what I suppose we should do is to make it specific for the application preview.16:30
tsdgeosyou can have 10 of them16:30
elopiotsdgeos: the application preview will still return title, subtitle and description, right?16:30
Saviqelopio, I don't think so, that knowledge needs to move towards the click apps scope16:30
tsdgeosthe application preview doesn't exist in our autopilot world16:30
Saviqelopio, it's the scope that decides what will be displayed, we have no control over it, effectively16:30
tsdgeosis just a preview the same it was16:30
tsdgeosthat sentence made no sense16:31
tsdgeosis just a preview as the others16:31
tsdgeosso that's why i want to remove the whole get_details thing16:31
elopiotsdgeos, Saviq: yes, so we need to move this get_details to the autopilot helpers that live in unity-scope-click16:31
tsdgeosi don't think it makes sense16:31
Saviqtsdgeos, and we need to facilitate that ↑↑16:31
elopiotsdgeos: then, I agree with you that it needs to be removed.16:31
elopiojust, we need to be in sync so the tests in unity-scope-click are broken just for a little time.16:32
Saviqelopio, or not at all, we can land them together with new-scopes16:32
elopiotsdgeos: remove the helper, remove the test, and when we have all the other helpers working, I will  prepare a branch for the scope.16:32
tsdgeoselopio: cool thanks16:32
elopiotsdgeos: thanks to you.16:33
tsdgeosSaviq: elopio: i just pushed something that makes the last tests i had written down as not passing pass again, hopefully it should not regress the rest of the tests17:04
tsdgeoswill check first thing tomorrow in the morning17:04
=== dandrader is now known as dandrader|lunch
=== alan_g is now known as alan_g|EOD
=== dandrader|lunch is now known as dandrader
Saviqmzanetti, hey, tried hi-dpi Unity7? worth checking out probably :)19:34
Saviqmzanetti, it's in the archive already, iiuc19:34
bschaeferSaviq, yup it is, dont open the dash or hud though :)19:43
popeyis there an easy way on current build of unity 8 on nexus 7 to get the sidestage, or do I need the bleeding edge stuff shown at MWC?19:55
=== dandrader is now known as dandrader|bbl
Saviqpopey, no, no easy way, and it won't be enabled soon either, it was using a hack to rotate the shell, which we need to replace with a proper implementation20:42
popeySaviq: thought so, thanks20:55
=== jhodapp_ is now known as jhodapp|brb
=== jhodapp|brb is now known as jhodapp
kgunnmhall119: ok...so i got my sessions approved but not scheduled22:10
kgunnhttp://summit.ubuntu.com/uds-1403/all/22:10
kgunnsee i'm listed here...but no date/time22:10
kgunnany ideas (day before :)22:10
mhall119kgunn: still up to the track lead to schedule it22:13
kgunnug...they've all gone home22:13
hitsujiTMOkgunn: any one i've clicked on has a time22:26
hitsujiTMOahh, yours is the only one without a time22:28
mhr3mhall119, do you use your @canonical mail?22:35
=== dandrader|bbl is now known as dandrader
=== salem_ is now known as _salem
=== jhodapp is now known as jhodapp|afk
mhall119mhr3: sometimes, why?23:28

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