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

=== duflu_ is now known as duflu
=== zequence_ is now known as zequence
Saviqtsdgeos, o/08:18
tsdgeosSaviq: \o08:18
Mirvtsdgeos: I found a note about dropping 'Don-t-use-ebx-in-the-x86-register-allocator.patch'.. I'd be replacing it with 'V4-JIT-save-restore-ebx-GOT-ptr-on-linux-x86.patch' + 'V4-fix-range-sorting.patch' + 'V4-regalloc-fix-interval-splitting-when-register-pre.patch', does that sound sane?08:40
tsdgeosMirv: looks correct yes08:41
Mirvthank you08:44
Saviqmzanetti, top-ack https://code.launchpad.net/~unity-team/unity8/new-scopes-clean-to-trunk/+merge/209459 ?09:09
Saviqmzanetti, I'd like you to take another stab at testing landing-01309:09
Saviqdidrocks, could we have packaging review of ↑↑?09:10
didrocksSaviq: yeah, but not now, like in an hour or so :)09:10
Saviqdidrocks, sure, thanks09:10
* didrocks sees everything fallind down09:10
didrocksfalling*09:10
Saviqdidrocks, yeah, I saw Larry's and Paul's emails...09:11
didrocksSaviq: and it's only part of the story :p09:11
Saviqdidrocks, I can imagine09:11
MirvSaviq: on the plus side I've locally run all AP:s for #256 and it seems like everything is still passing with exception of gallery-app failing 1 test for me, so it's not like disaster would have happened after #25009:19
SaviqMirv, cool09:19
MirvI just hope my local run is not giving more positive results than what the automated tester would give09:19
Cimidednick, ping10:24
dednickCimi: yo10:24
mhr3Saviq, so how do we look on landing scopes?10:24
Cimidednick, ciao10:24
Cimidednick, I'm reusing the unity menu model stack for welcome wizard10:25
Cimiand I arrived at this menuitemfactory10:25
Saviqmhr3, want to, today10:25
Saviqmhr3, we're fixing a flaky ap test, found one small UX issue10:26
Cimidednick, is there a way to get only the list of the access points?10:26
mhr3Saviq, very well, i'll land some extra raciness fixes meanwhile10:26
dednickCimi: hm. not sure other than maybe using a filter model.10:27
Cimidednick, what's a filter model?10:29
Cimidednick, something that doesn't pass items in the model?10:29
dednickCimi: i mean a qt filter model. like when you filter out items in a model based on a property10:30
Cimiok10:31
dednickCimi: It's possible there should be a different indicator profile for the welcome wizard.10:31
Cimidednick, can you help me on that?10:31
Cimidednick, or guide me10:31
Cimidednick, where can I create this profile?10:31
dednickCimi: the profile is in the indicator itself in which case tedg would be your man to ask (not sure if it's the thing to do really).10:32
dednickCimi: otherwise you should probably be using the "phone_wifi_settings" profile10:33
CimiI'm using that for now10:34
didrocksSaviq: answered on the MR10:49
Saviqdidrocks, /me too10:49
Saviqdidrocks, only LP didn't refresh its INBOX yet...10:50
didrocksah :p10:51
Saviqdidrocks, now10:51
Saviqdidrocks, anyway, new scopes have nothing to do with libunity any more, and there's not a desktop version of those dependencies10:51
didrocksSaviq: shouldn't they be recommends only though?10:51
didrocksSaviq: and we seed the new scopes10:52
didrocksso that, on desktop, people are not forced to installed onlinemusic for instance10:52
didrocksand we give more granularity10:52
Saviqdidrocks, they are10:52
didrockshum?10:53
didrocks39-Recommends: ${unity-default-masterscopes},10:53
Saviqhmm10:53
didrocks40+         unity-scope-scopes,10:53
didrocksso Recommends10:53
didrocksno*10:53
Saviqdidrocks, you're right10:53
didrocks\o/10:53
didrocksSaviq: just reminds me to check they are all seeded when this branch lands (once you downgraded them as recommends)10:53
Saviqdidrocks, so what do you think, click and scopes depends, but mediascanner2 and onlinemusic recommends?10:54
didrocksSaviq: why click? I would say scope-scopes depends only10:54
Saviqdidrocks, yeah, I pung ogra this morning, didn't respond yet, though10:54
didrocksSaviq: I can do the seeding if needed10:54
Saviqmhr3, opinion ↑↑↑?10:54
Saviqdidrocks, ok10:55
didrocksscope-scopes is the scope to enable/disable, right?10:55
Saviqdidrocks, ultimately, yes10:55
didrocksso if you want to depends on anything, that one, the rest is surely just recommends10:55
didrocksand we seed them10:55
Saviqok10:55
dednickCimi: dont think adding a new profile is the way to go on this one. maybe just use the settings profile. Think it's a bit of overkill just for something we're really only displaying first time you use the phone.10:55
Saviqmhr3, only -scope-scopes as Depends, rest as Recommends?10:56
dednickCimi: so I'd just go for filter model if you can.10:56
Cimidednick, yes but it contains other menuitems, no?10:56
tsdgeosdamn, i broke something in my machine10:56
mhr3Saviq, no, click is deps too10:56
tsdgeos"file:///home/tsdgeos_work/phablet/unity8/new-scopes-clean-to-trunk/builddir/plugins/typelist.qml:2:1: plugin cannot be loaded for module "AccountsService": The file '/home/tsdgeos_work/phablet/unity8/new-scopes-clean-to-trunk/builddir/plugins/AccountsService/libAccountsService-qml.so' is not a valid Qt plugin."10:56
mhr3Saviq, we're all the time saying that scopes and apps are two required components10:56
Saviqmhr3, right10:56
Saviqdidrocks, ↑10:56
tsdgeosdoes anyone get this?10:57
dednickCimi:    ./builddir/src/Panel/Indicators/client/indicators-client -profile phone_wifi_settings10:57
Saviqtsdgeos, no10:57
Saviqtsdgeos, ./build -c ?10:57
tsdgeosSaviq: i nuked builddir, same thing10:57
dednickCimi: open up the "indicator-menu" section and deselect visual representation10:57
didrocksmhr3: Saviq: is click available on the desktop? I think it's not10:58
didrocksmhr3: Saviq: and we have that preview session where we don't want to promote click on the desktop for now10:58
Saviqdidrocks, sure it is, with the unity8 preview session10:58
Saviqdidrocks, uhm hum10:58
mhr3why not, the preview session does mir10:59
dednickCimi: you should be able to set up a filter for type "unity.widgets.systemsettings.tablet.accesspoint"10:59
Saviqtsdgeos, that on new scopes?10:59
didrocksmhr3: there is still the xorg one10:59
didrocksmhr3: will it crash without the click scope?11:00
tsdgeosSaviq: yes11:00
mhr3didrocks, of course it won't, but will you be able to launch anything? no11:00
dednickCimi: using a Utils.SortFilterProxyModel11:00
didrocksmhr3: so gallery depends on click scope?11:01
didrocksand so on?11:01
didrockslike unity-scope-mediascanner2 depends on unity-scope-click?11:01
mhr3didrocks, click surfaces apps11:01
didrocksright11:01
didrocksbut as of today unity-lens-applications is a recommends from unity711:02
didrocksas every other scopes11:02
didrocksany reason to not be the same for click scopes?11:02
mhr3didrocks, wasn't it recommends until now?11:05
Saviqdidrocks, you can't really launch apps other than through the click scope on unity8 (except for those on the launcher)11:05
didrocksmhr3: yeah, it was11:05
didrocksmhr3: that's why I wonder why we are going to change this to depends11:05
mhr3didrocks, then why did you ask to change it in the first place? :)11:05
didrocksmhr3: did I? I didn't, I'm arguing to keep it as recommends11:06
mhr3well, pks wise it's really an optional dep, yes unity is useless without it, but you can write your own apps scope to make it useful11:07
mhr3s/pks/pkg/11:07
didrocksyeah, so keeping it as recommends + seeding it makes sense to me11:07
mhr3so i'm fine with keeping everything recommends + seeding11:07
didrocksok, thanks mhr3 :)11:07
didrocksSaviq: ^11:07
dednickCimi: something like this: http://pastebin.ubuntu.com/7145753/11:07
Cimidednick, trying11:08
Saviqdidrocks, mhr3, ack11:08
dednickCimi: except with the sound replaced with network11:08
Saviqdidrocks, mhr3, so depends on scopes, recommends+seed everything else?11:08
mhr3Saviq, recommends on everything,  no?11:09
didrocksmhr3: scopes-scope isn't mandatory?11:09
Saviqdidrocks, no11:09
mhr3same way apps is11:09
didrocksok, all recommends as it was11:09
Saviqdidrocks, not code-wise11:09
mhr3unity is useless, but it works11:10
didrocksand we'll seed everything11:10
didrocksmhr3: come on, there is the launcher! don't bring back everything to scopes :)11:10
didrocksso then, we can land the new scope infra?11:10
didrocksas thostr_ told on the ML that we were only blocking on landing?11:10
* greyback hates Mondays, accidentally reverted code written on Friday11:11
mhr3didrocks, launcher doesn't list everything :P11:11
tsdgeosSaviq: ok, it seems i had something weird exported in my shell, i switched to a new terminal and now it works :S11:11
mhr3didrocks, Saviq is finishing a fix for some flaky tests, should be able to land it in a bit11:12
Saviqtsdgeos, :)11:12
Saviqdidrocks, pushed the change11:12
Saviqgreyback, no .~1~ file?11:13
greybackSaviq: *mwah* totally forgot those11:13
* Saviq wipes11:14
* Saviq goes for a shower11:14
* Saviq goes to an auto-wash11:14
Saviqmzanetti, top-ack https://code.launchpad.net/~unity-team/unity8/new-scopes-clean-to-trunk/+merge/209459 ?11:27
Saviq mzanetti, I'd like you to take another stab at testing landing-01311:27
Saviqmzanetti, actually, wait a bit, will rebuild with the latest changes11:27
mzanettiSaviq: ok11:27
Saviqdidrocks, can you upload the updated seed to landing-013?11:28
mzanettiSaviq: this is now everything merged together?11:28
Saviqdidrocks, we need unity-scope-{scopes,click,mediascanner2,onlinemusic}, the rest can go away11:28
Saviqmzanetti, yes11:28
didrocksSaviq: sureeee11:28
didrocksSaviq: uno momento11:28
Saviqdidrocks, thanks11:28
didrocksSaviq: mhr3: dropping libunity-scopes-json-def-phone, right?11:29
didrocksin addition to the rest of scope11:30
mhr3didrocks, ack11:30
Saviqdidrocks, not sure why it was there in the first place11:30
didrocksunity-scope-video-remote is handled by mediascanner2?11:30
mhr3yes11:30
didrockswaow, all was spread out all over the seeds11:30
didrockslet me clean this11:30
Saviqdidrocks, well, in that case I should probably drop that, and the perl thingy to parse it?11:30
Saviqdidrocks, yeah11:30
Saviqdidrocks, from unity8's debian/control?11:30
didrocksSaviq: yeah11:30
* Saviq does11:31
mhr3didrocks, also -scope-onlinemusic should go byebye11:31
didrocksSaviq: snif, my perllllll :=11:31
didrocks:)11:31
Saviq;)11:31
Saviqit's a unicorn!11:31
didrocksheh, I'll blame mhr3 for this Friday evening hack :p11:31
didrocksmhr3: and -home11:31
mhr3didrocks, and you loved every bit of that friday evening :P11:31
didrocksmhr3: soooo much11:32
Saviqdidrocks, pushed11:33
Saviqdidrocks, do we have a process for landing a thing risky enough like the new scopes?11:33
didrocksseed updated11:33
Saviqlike additional QA?11:33
didrocksSaviq: yeah, it's just asking for some manual dogfooding from our team11:33
Saviqdidrocks, ok11:33
didrocksin addition to all AP tests that you run + all tests that you are doing as usual of course11:34
didrocksok, generating the metapackage11:34
Saviqdidrocks, you're dputting the seed to landing-013?11:34
Saviqdidrocks, ah ok11:34
didrocksSaviq: yeah, once the package is generated11:34
Saviqdidrocks, yeah, of course (re: testing)11:34
didrocks(it's a 2 step process)11:34
Saviqdidrocks, right, remember now11:34
* didrocks just relooked at his perl script and cries on all the $11:35
didrocksSaviq: mhr3: looking good to you? http://paste.ubuntu.com/7145879/ -click and -onlinemusic were already in the seed, I just moved them)11:39
Saviqdidrocks, +111:40
* didrocks dput to landing-01311:40
didrockswill need a reconfigure then11:40
Saviqdidrocks, will I be able to, or do you need to?11:41
didrocksSaviq: you can't, as it's about adding on more component11:41
Saviqdidrocks, yup, what I thought11:41
didrockshum, it's building11:42
didrocksSaviq: should I just stop the monitoring and relaunch it?11:42
Saviqdidrocks, sure, go for it11:42
didrocksSaviq: uploaded, configure and restarted with watch only11:45
Saviqdidrocks, yup, thanks11:45
didrocksyw!11:45
didrocksoh, miss the reject11:45
didrockslet me see why11:45
didrocks(from my dpugt)11:45
mhr3didrocks, no onlinemusic11:47
mhr3didrocks, it's handled by mediascanner2, same as onlinevideos11:47
didrocks12:28:33    Saviq | didrocks, we need unity-scope-{scopes,click,mediascanner2,onlinemusic}, the rest can go away11:48
didrockshum?11:48
didrocksok, I'll do a second upload then :)11:48
Saviqoups11:48
didrocksSaviq: you're mean! :)11:48
Saviqdidrocks, no, we still have it in unity8, too11:48
Saviqdidrocks, sorry11:48
didrocksno worry ;)11:48
didrocksok, refreshing with only11:49
Saviqpushed11:49
didrocks * unity-scope-scopes11:49
didrocks * unity-scope-click11:49
didrocks * unity-scope-mediascanner211:49
davmor2mhr3: do a search do the online music and videos show up then?11:52
didrocks  * Removed unity-scope-onlinemusic from touch11:54
mhr3davmor2, yes, just provided by different pkgs11:56
davmor2didrocks: does this mean I have to keep my eyes out for a landing of the new scopes?11:56
didrocksdavmor2: yeah, this is going to happen very soon :)11:56
davmor2Saviq: I see QT devs have replied to your bug :(11:56
Saviqdavmor2, yeah, tvoss did, too11:56
Saviqdavmor2, but yeah, I was expecting that11:56
Saviqdavmor2, they have a point, and we need to work out a compromise of sorts11:57
davmor2Saviq: Meh!11:57
Saviqsomeone needs to budge ;)11:58
davmor2Saviq: I blame it all on popey for finding the bug11:58
Saviqdavmor2, :)11:58
* popey loads is gun and looks for messengers to shoot11:59
popey*his11:59
* davmor2 takes popey 's gun off him before he shots himself ;)11:59
didrocksSaviq: seems it's still picking up 116, waiting a publisher cycle to get 11712:04
Saviqdidrocks, k12:04
didrocksSaviq: all build, tell us if that's enough once the testing is done (or if you need to rebuild and so on)12:14
didrocksSaviq: but all on you now :)12:14
Saviqdidrocks, yup12:14
Saviqmzanetti, ↑ can you do a round of testing of landing-013?12:14
didrocksSaviq: I didn't rebuild unity8 though, not sure if your latest packaging changes are there12:15
mzanettiSaviq: sure. but I just set up my phone for autopilot testing the right edge stuff before you pinged me before. can I finish that first or is this so high priority that I need to get started immediately?12:15
Saviqdidrocks, they don't really matter for dogfooding, but yeah, /me does12:16
Saviqmzanetti, sure, it's fine12:16
=== MacSlow is now known as MacSlow|lunch
thostr_Saviq: have you noticed a crash after applying silo 13?12:21
Saviqthostr_, no, any more data?12:22
thostr_Saviq: let me check if apport has written anything12:22
thostr_Saviq: crash file is there, not sure how helpful though12:29
Saviqthostr_, please do /usr/share/apport/whoopsie-upload-all12:29
Saviqthostr_, and after that's completed, send the .crash somewhere12:30
thostr_will do12:30
popey14012:44
popeybah12:44
=== _salem is now known as salem_
sil2100Saviq, mhr3: guys, what about the transition to the new scopes? How's it proceeding?12:47
Saviqsil2100, almost there, tweaking the last flaky test and we'll be in for dogfooding soon12:47
sil2100Saviq: awesome ;)12:48
mhr3sil2100, silo for 31 for a prereq for that pls12:49
mhr3sil2100, 002 still not freed :P12:49
sil2100mhr3: pfff ;p12:51
sil2100Nitpicks ;p12:52
didrocksSaviq is faster than I at the unassign war :)12:52
Saviq;)12:53
bregmahey all:  when I launch a lot (but not all) applications in the Unity8 desktop preview session they abort during startup with a failure to create an instance, and Unity8 logs the message 'ApplicationManager REJECTED connection from app with pid 8208 as no desktop_file_hint specified' .. anyone have any idea what might be going on with that?12:53
bregmathese are core apps, there are desktop files12:53
Saviqbregma, sounds like unity8 and upstart have issues communicating that the apps are being launched12:53
Saviqgreyback, can you support bregma please ↑?12:54
greybackok12:54
bregmathings like Sudoku and browser-app all work OK12:54
greybackbregma: hey, how are you launching these apps? Does it go through upstart-app-launch?12:54
Saviq/food12:54
bregmagreyback, I would imagine, this is stock Unity8 (although, as usual, it may need further tweaking for the desktop)12:55
bregmadamn you, Dropping Letters, I only want to see if you launch12:56
greybackbregma: can I install the PPA to try it?12:56
bregmagreyback, https://launchpad.net/~unity8-desktop-session-team/+archive/custom12:57
greybackbregma: usually that message due to an app starting up which shell wasn't expecting. upstart-app-launch should always notify shell that the app is launching. So either u-a-l or unity-mir is broken12:57
greybackbregma: ok, installing now12:58
bregmayou need the qtubuntu from the PPA for a workaround in a Qt5.2, the rest of the things in there are for (a) mouose support and (2) session logount support12:58
bregmagreyback, if you could take a look at qtubuntu bug #1295309 that would also be swell12:59
ubot5bug 1295309 in qtubuntu "qtubuntu-desktop fails using Qt 5.2 on Mir" [Critical,In progress] https://launchpad.net/bugs/129530913:00
greybackbregma: sure, will review13:02
greybackheh "feature"13:02
sil2100mhr3: silo for your prereq13:05
mhr3sil2100, ty13:05
=== alan_g is now known as alan_g|lunch
=== MacSlow|lunch is now known as MacSlow
Cimidednick, can I use that plugin from the ubuntu system settings?13:22
dednickCimi: hm. not the Utils one.13:23
Cimidednick, so probably I have to copy/paste13:23
dednickCimi: yup. unless qt 5.2 has offered a qml filter model now.13:24
dandraderSaviq, finally noticed that we no longer (since January!) have the "Approved by Foo" line added to commit messages. Do you know why? And, secondly, should we try to get it back somehow?13:34
dandraderI find them useful as the reviewer/approver shares part of the responsibility over a commit (one more guy you can poke about it)13:35
greybackbregma: how does one log out of a unity8 session?13:37
Saviqdidrocks, ↑13:37
didrocksplease file a feature req. Not sure when I can safely add that though13:38
bregmagreyback, if you have everything from the PPA installed, pull down one of the indicators and switch to indicator-session (it doesn't have an icon in the top bar for some reason)13:38
bregmathen click on logout13:38
greybackbregma: on unity8-x11, mouse gestures aren't working (I can't unlock the greeter with my mouse). We've a "-mousetouch" switch for unity8 that should fix that.13:40
bregmagreyback, yeah, that package is pretty much unsupported at the moment and should probably be removed13:42
greybackbregma: ah ok13:42
bregmait needs to have a window manager added for most featuures to work anyway13:42
greybackyeah I expected it to be a bit useless :)13:43
bregmaain't nobody got time fo' dat13:43
greybackword13:43
greybackbregma: ok, trying the unity8-mir, I just get blank screen. Where do I look for logs?13:45
bregmagreyback: ~/.cache/upstart -- that problem sounds like the QT5.2 problem13:45
greybackbregma: ibEGL warning: unsupported platform (null)13:46
greybackterminate called after throwing an instance of 'boost::exception_detail::clone_impl<boost::exception_detail::error_info_injector<std::runtime_error> >'13:46
greyback  what():  MesaNativePlatform::create_internal_client is not implemented yet!13:46
greybackinteresting, I wonder what I broke13:46
bregmawhat version of Mir do you have installed?13:48
bregmathat should have been fixed by 0.1.713:49
bregmayou might need to upgrade13:49
greybackI have updated all, but now I can't even start mir_demo_server_shell13:52
mzanettielopio: hi13:53
=== alan_g|lunch is now known as alan_g
mterryMacSlow, if you do work on the boot animation, please feel free to do it in ~unity-team/unity-system-compositor/new-gl-screen14:00
bregmagreyback, are you using proprietary drivers by any chance?14:00
MacSlowmterry, ok14:00
mterryMacSlow, last week I already made it use a black background and no text14:01
mterryMacSlow, so it should mostly be an asset swap and maybe some small tweaks on your end14:01
greybackbregma: crisis averted14:01
MacSlowmterry, still busy in notification-land... might be much later today I'll be able to switch to the loading-screen14:01
mterryMacSlow, no rush14:02
greybackbregma: have you tried this on nouveau hardware? I have some text characters missing, maybe driver bug?14:04
greybackit's not exactly buttery smoother either14:05
bregmagreyback, no, all my test hardware is Intel14:07
greybackand graphics corruption and crash....14:07
greybackbregma: well anyway I've approved the qtubuntu qt5.2 fix14:10
greybackand once I get used to the invisible cursor, I'll launch some apps14:11
bregmagreyback, it's the 21st century, get a HDPI touchscreen14:12
tsdgeosone day someone should investigate this errors we're getting14:15
tsdgeosQtDBus: cannot relay signals from parent QObject(0xabb0e330 "") unless they are emitted in the object's thread QThread(0x15a9c68 ""). Current thread is QThread(0x158c8c0 "").14:15
=== greyback is now known as greyback|lunch
=== dandrader is now known as dandrader|afk
MacSlowmterry, doh... imap... I'll wait half a day before considering to reply to anything :)14:39
mterrykgunn, if you have some time today, I'd appreciate you removing telephony-service branches from the split silo.  And adding in the lp:~mterry/lightdm/fake-no-change-mp branch  -- this should be enough for people to play with functionality14:40
mterryMacSlow, I know, right!  Everyone in that thread is doing it  ;)14:40
mterrykgunn, (we can remove the telephony stuff because it landed in trusty)14:40
=== pete-woods is now known as pete-woods-lunch
Saviqmterry, I can do, so we don't want telephony-service in the silo any more?14:44
mterrySaviq, right.  And we want to add lightdm via lp:~mterry/lightdm/fake-no-change-mp14:45
Saviqmterry, yup, doing14:45
Saviqmterry, although the service won't disappear from the silo, does it need to? looks like it does?14:45
Saviqi.e. there's not a newer version in distro14:46
mterrySaviq, we had a version land today in trusty I thought14:46
=== dandrader|afk is now known as dandrader
mterrySaviq, ah, it entered with an older date than the silo has14:47
mterrySaviq, yeah, we can delete the one in the silo14:47
Saviqmterry, yeah, ok, we'll have to delete14:47
Saviqsil2100, can you please delete telephony-service from landing-004 ppa?14:47
sil2100Saviq: just the package from the PPA, right? Sure14:48
mterrysil2100, right14:48
Saviqsil2100, yup14:49
Saviqmterry, so only lightdm needs building?14:49
sil2100Saviq, mterry: it's removing14:49
Saviqthanks!14:49
Saviqmterry, ok, lightdm is gonna build soon14:50
mterrySaviq, for the nonce yeah14:50
kgunnthanks..i was majorly distracted :)14:50
tsdgeosSaviq: ok, i know what's going on with that AP test that fails14:52
Saviqtsdgeos, good14:52
Saviqmterry, please add commit message14:52
tsdgeosnow i only need to fix and add a test :D14:52
Saviqtsdgeos, looking forward to it! :)14:53
mterrySaviq, whoops, done14:53
Saviqmterry, cheers14:53
Saviqdidrocks, uh oh14:54
Saviqunity-scopes-master-default14:54
Saviqdidrocks, we need that gone, too14:55
Saviqactually it's not there maybe? /me no get it14:56
Saviqbut upgrading ubuntu-touch didn't say the old scopes can be removed...14:56
Saviqmhr3, is it expected that we still see songs instead of albums in the local music carousel?15:00
mhr3Saviq, yes15:01
mhr3Saviq, the design has a separate category for albums, and we have a branch that does that, but music-app needs to support launching those first15:01
Saviqmhr3, well, songs should be random or something, no? so that we don't see the same cover all the time?15:02
mhr3Saviq, not really specified anywhere afaict15:02
* Saviq doesn't think there should be "songs" at all in the dash, not for surfacing15:03
mhr3convince design :)15:03
elopiomzanetti: hello.15:03
mzanettielopio: hi. I'm struggling with getting one of the autopilot tests working again. Would you have time to help me?15:03
elopiomzanetti: I do. Which one and from what branch?15:04
mzanettielopio: I'll ping you again in a few minutes, will prepare instructions for you to reproduce15:04
elopiothat's ok15:05
mhall119mhr3: http://91.189.92.89/api/scopes/ubuntu-14.04/15:06
=== greyback|lunch is now known as greyback
mhr3mhall119, wooow, looks.. consistent :)15:11
SaviqMacSlow, hey, bfiller reported https://bugs.launchpad.net/unity8/+bug/1296777, can you please have a look when you have a chance?15:11
ubot5Ubuntu bug 1296777 in Unity 8 "avatar displayed incorrectly in notification" [High,New]15:12
mhr3mhall119, so mono font here and there would be nice though ;)15:12
mhall119mhr3: where and where?15:12
MacSlowSaviq, ah ok15:13
mhr3mhall119, the code snippets for example15:14
mhall119ah, yes, I can do that15:17
tsdgeosSaviq: https://code.launchpad.net/~aacid/unity8/LVWPHcullLostItems/+merge/21245715:17
mzanettielopio: I think I've solved it. The issue seems to be that QML *sometimes* optimizes the AppPreview to be a DashPreview, sometimes doesn't.15:18
mhr3mhall119, would be also good if the categorization you did was connected with the docs, i guess right now you did it manually?15:18
mzanettielopio: I've removed the select_single by classname and only fetch it via objectName now15:18
elopiomzanetti: hum, that might be a problem.15:18
mzanettiwhy?15:19
Saviqtsdgeos, on it15:19
elopiomzanetti: because we have to define some helpers in AppPreview that don't apply to DashPreview, like install()15:19
Saviqtsdgeos, TODO somewhere for a test?15:20
Saviqtsdgeos, if you think not → not15:20
elopiomzanetti: but that will be a problem later, when I have to update the click scope tests.15:20
mzanettielopio: but I have the impression that this select by classname doesn't really work well with QML anyways15:20
tsdgeosSaviq: honestly i would not know how to test it, but i can add a TODO saying "think how to do this"15:20
mzanettielopio: as just changing one property can cause it to change classname in the qml context15:21
elopiomzanetti: that could be even worse, we rely on it too much.15:21
elopiomzanetti: oh, yes, I hate that. I had some problems before with CheckBoxes and Switches because of that.15:21
Saviqtsdgeos, your call15:21
tsdgeosSaviq: i'll add it, someone invests some time and decides if to do the test or remove the todo15:22
Saviqtsdgeos, yup15:22
elopiomzanetti: they are working on an alternate way to match the helpers with a QML class.15:22
mzanettielopio: yeah, had to change some in my branch now, just because I somewhat touched a file, but yeah, the real problem is that this optimization might even change at runtime15:22
=== alan_g is now known as alan_g|tea
elopioI think that will make it a little better.15:22
mzanettielopio: what's wrong with objectNAme?15:22
mzanettielopio: that one will stay the same unless you really change the code in way that you have to update tests anyways15:23
elopiomzanetti: objectName is perfect. The thing is that autopilot currently needs the name of the python class to match the name of the QML class.15:23
mzanettielopio: no it doesn't15:23
elopioand that's sometimes painful, not just your example.15:23
tsdgeosSaviq: pushed15:23
mzanettiselect_single(objectName="dashPreview", isActive=True) works fine15:23
Saviqtsdgeos, thanks15:24
elopiomzanetti: yes, that works, but it returns an instance of the python class that matches the QML class.15:24
mzanettiwhich sounds correct to me...15:26
elopiomzanetti: you will see some tests on test_emulators.py with self.assertIsInstance15:26
elopiothose check that we return the right instance. If they pass for you, things are ok.15:27
mzanettielopio: yeah, changed that to just check for GenericScopeView instead of DashApps15:27
mzanettigiven that the line above checks if scope.id == apps.scope, I don't think its necessary to check for the isInstanceOf too15:28
elopiomzanetti: then that's not ok.15:28
elopiomzanetti: go to unity8/shell/emulators/dash.py15:28
mzanettiok15:28
elopioyou will that the DashApps class has an extra method, get_applications.15:30
elopiothat one will not be available if the instance returned is GenericScopeView.15:30
mzanettielopio: yeah, moved that into GenericScopeView15:30
elopiowe need a way to solve that.15:30
elopiomzanetti: ok, that's one way :)15:30
elopionot the nicest one, because not all GenericScopeViews should have a get_applications method15:31
mzanettiyeah...15:31
elopiobut due to the current autopilot implementation, there'15:31
elopios not much we can do about it.15:31
elopiomaybe raise an exception if we call it with a scope that doesn't show apps.15:31
mhall119mhr3: yeah, manually for now, because I didn't see any way to derive that information from the docs15:32
=== alan_g|tea is now known as alan_g
elopiomzanetti: I have a meeting, I'll be back in 1 hour, but it seems you have it properly covered.15:32
elopiojust one detail, your changes might conflict with what tsdgeos is doing for new scopes.15:33
mzanettielopio: same here...15:33
tsdgeosnoOoOoOoOoOoO15:33
mzanettielopio: well, in that case we'll merge it15:33
mhr3mhall119, can't doxygen do some tags?15:33
mhr3mhall119, otherwise it's just asking to get stuff out-of-sync15:34
mhall119mhr3: I'm not really a doxygen expert15:34
=== pete-woods-lunch is now known as pete-woods
mterrySaviq, this is the 1st step of changing how jenkins unlocks: https://code.launchpad.net/~mterry/unity8/unlock-script/+merge/212170  You seemed interested last time we talked15:50
Saviqmterry, yeah, I'll look through, but we definitely need ACK from QA and CI folks15:50
kgunnSaviq: just one more query on 5.2 render loop update issue, so do we need to put another thread in unity-mir for the screen off state to make qt happy ?15:51
Saviqmterry, a) ship it in unity8-autopilot, not unity815:51
mterrySaviq, sure...  read the description though, this is a very incremental small change and does what you want -- encapsulates more info inside the component.  This change itself shouldn't need QA signoff, but the next step where I change the jenkins scripts will15:51
Saviqkgunn, it's not that15:52
Saviqkgunn, there's two threads in QML threaded rendering: GUI and render15:52
Saviqkgunn, GUI processes almost everything, render just does the scenegraph thing15:52
Saviqkgunn, on 5.0, they acted more or less independently15:53
Saviqkgunn, now they do a sync once every frame15:53
mterrySaviq, done15:53
Saviqarenson, o/15:53
mzanetti\o15:53
arensonhello15:53
Saviqarenson, if you can use one nick across IRC, that's helpful :)15:53
Saviqkgunn, and if that sync doesn't return, the GUI thread is ultimately blocked15:54
Saviqkgunn, so we basically can't block the render thread, unless we tell Qt to stop rendering via an exposeEvent15:54
kgunnSaviq: oh wow, so yeah that seems weird15:54
arensonfor some reason it says josharenson is taken on this server, perhaps I'll just make it arenson across servers15:54
Saviqkgunn, read https://bugreports.qt-project.org/browse/QTBUG-3767715:54
kgunnSaviq: thanks...i'll read up15:55
Saviqarenson, must be "josharenson" was registered by someone maybe, not here at this time, though15:55
kgunnstill, based on your description...seems like a funny decision15:55
arensonmaybe it was me at some point....15:55
Saviqarenson, weird, NickServ doesn't know it, you should be able to15:55
Saviqarenson, type /nick josharenson now?15:55
=== arenson is now known as josharenson
josharensoncool15:57
josharensonchanged back on canonical too... sorry for the confusion15:57
Saviqjosharenson, no worries15:57
tsdgeosSaviq: how is going testing the LVWPH fix for the AP test?15:59
tsdgeosand does anything else in new-scopes needs me fighitng against it?16:00
Saviqtsdgeos, unity8 just built16:00
SaviqI think16:00
Saviqtsdgeos, so will run it now16:00
Saviqdidrocks, I think I reconfigured landing 013 to not include ubuntu-touch-meta, can you add it back?16:01
Saviqsil2100, maybe you're around ↑16:03
sil2100Saviq: ok16:03
sil2100Saviq: doing16:03
Saviqsil2100, thanks16:03
Saviqmterry, looks like landing-004 is good to go?16:05
didrocksSaviq: tssss :p16:05
Saviqdidrocks, yeah :|16:05
mterrySaviq, define good to go?16:05
Saviqmterry, everything built16:06
Saviqmterry, and ready for testing16:06
mterrySaviq, yeah, everything built.  I'm going to send out an email shortly after I do a double-check test myself16:06
Saviqmterry, yup, cool16:06
mterrySaviq, thanks btw!16:06
Saviqmterry, cheers16:06
sil2100Saviq: reconfigured16:08
Saviqsil2100, thanks!16:08
MacSlowmzanetti, tsdgeos: how does one usually get a models role-types known on the QML-side?16:10
mzanettiMacSlow: not sure I understand the question16:11
tsdgeosMacSlow: read the documentation of the model16:11
MacSlowmzanetti, tsdgeos: is some Qt/moc macro needed in the C++-sources?16:11
tsdgeosor in its absense read the c++ of the model :D16:11
tsdgeosMacSlow: besides the roleNames function?16:11
tsdgeosno16:11
MacSlowtsdgeos, I need to make a SortFilterProxyModel work16:12
SaviqMacSlow, it takes int as models I'm afraid16:12
Saviqs/models/roles/16:12
tsdgeosMacSlow: ok, what's the question?16:13
mzanetti:)16:13
SaviqMacSlow, so you just need to know the id of the role16:13
MacSlowSaviq, well sure... but I've seem it used in other places in unity8 and there symbolic names were used16:13
MacSlowSaviq, I wanted to avoid that :)16:13
SaviqMacSlow, ah then yeah, you need Q_ENUM16:14
MacSlowSaviq, argl... so I do need a new branch for unity-notifications :)16:14
SaviqMacSlow, ;)16:16
SaviqMacSlow, good thing is, these land together now16:16
MacSlowSaviq, didn't expect your change-request there on my MP to be that involved16:16
SaviqMacSlow, i.e. we can put unity8, unity-notifications in the same silo, and they'll land in the same time16:16
didrocksSaviq: once sentence update on the Qt issue?16:17
didrocks(preparing the landing team meeting)16:17
didrocksso that you don't have to come16:17
Saviqdidrocks, Qt said "you're doing it wrong", we're disagreeing to some extent, but looking for a local solution anyway, until we can convince upstream otherwise16:18
didrocksSaviq: thanks :)16:18
didrocksSaviq: you are the one leading it, right?16:19
Saviqdidrocks, reverting Qt change is not an option, it's a rewrite of the threded rendering engine16:19
Saviqdidrocks, with tvoss taking over from an arch PoV16:19
Saviqdidrocks, but for now I'm the PoC still16:19
didrocksSaviq: perfect… and good luck! :)16:19
tsdgeosSaviq: mterry: so the split branch conflicts a bit with new-scopes-clean-to-trunk, given that AFAIK new-scopes-clean-to-trunk is the "blocker" for commiting anything else would make sense to merge it in already in split?16:21
mzanettiSaviq: is this about the one with the events being queued while not rendering?16:21
mterrytsdgeos, OK.  Can do16:21
Saviqtsdgeos, we should land it today16:21
Saviqtsdgeos, so by tomorrow it will hopefully be in trunk16:21
tsdgeosok16:22
mterryStill good to get on top of the merge16:22
Saviqugh, just got 5 dbus failures on it :|16:23
tsdgeoswoot?16:26
mhr3Saviq, btw once you have a moment - happy with16:29
mhr3https://code.launchpad.net/~mhr3/unity-scopes-shell/expose-attributes/+merge/212213 ?16:29
mhr3Saviq, and we need to talk about filters, how do you want those exposed16:29
Saviqmhr3, looking good, think we're not limiting ourselves with text and optional icon? didn't we have some place where there was no text, actually?16:30
mhr3Saviq, there was always at least a number or something16:31
mhr3Saviq, and noone's saying the value can't be a dict of something16:31
Saviqmhr3, ok16:31
Saviqmhr3, as filters go, I think they way they were done until now was pretty good16:32
Saviqmhr3, and without looking at some visuals... pretty difficult to say otherwise16:32
mhr3ok, will check that out16:32
=== dandrader is now known as dandrader|lunch
* tsdgeos EODs16:55
* Saviq too17:11
Saviqo/17:11
MacSlowSaviq, no longer racy -> https://code.launchpad.net/~macslow/unity8/modal-snap-decisions/+merge/21098817:15
MacSlowmzanetti, if you like Karma... this gives you an easy way to harvest it :)  https://code.launchpad.net/~macslow/unity-notifications/modal-snap-decisions/+merge/21248317:26
mzanettiMacSlow: ok. will review17:27
MacSlowmzanetti, thx17:27
mzanettiMacSlow: I think we require a checklist17:30
MacSlowmzanetti, it didn't save the text for that?17:30
MacSlowmzanetti, one sec17:30
MacSlowmzanetti, but now it is17:32
=== alan_g is now known as alan_g|afk
=== arenson is now known as josharenson
mterrygreyback, no, I haven't seen that behavior from USC...17:39
greybackmterry: would you mind double-checking on your hardware, to see if it's my setup or not?17:40
mterrygreyback, I can.  Are you blocking on this right now or can I get back to you in a bit?  I'm in the middle of something17:41
mzanettiMacSlow: I hate to say it, but IMO that enum should be defined in unity-api and tested for its existance17:42
greybackmterry: not blocking me, so no rush17:43
MacSlowmzanetti, it wasn't missed before... so why now?17:43
mzanettiMacSlow: before it was only used internally in the backend.17:44
mzanettinow it is used in Unity8 too17:44
mzanettiwhich means we have a inter-repo dependency on that enum17:44
mzanetti=> unity-api defines it17:44
MacSlowmzanetti, what process is to be followed to get that done?17:44
mzanettiMacSlow: add the enum to unity-api, remove it from the backend and file the mp as usual17:45
mzanettiMacSlow: later the silo takes care for all 3 landing at the same time17:45
mzanettihence the "Are there any related mps..." question in the checklist17:45
MacSlowmzanetti, Saviq: I guess that then needs to go into lp:unity-api/include/unity/shell/notifications/Enums.h ?!17:48
mzanettiMacSlow: this is really specific to the model. put it inside the model17:49
mzanettithe stuff in Enums.h is more generic, appliying to many models/structures17:49
mhall119mhr3: what 'tags' were you referring to earlier?17:52
mhall119I'm digging through the doxygen output trying to find something to use to categorize thigns17:52
mhr3mhall119, nothing specific, was mostly wondering if doxygen has something like that17:56
mhall119not that I can see17:57
mhall119and since all of the classes except testing are in the same namespace, I don't see any alternative to manual mapping17:58
mhall119FWIW, the script defaults to putting things into the "Searching" section unless told otherwise17:58
=== dandrader_ is now known as dandrader
mhr3mhall119, http://www.stack.nl/~dimitri/doxygen/manual/grouping.html18:05
=== alan_g|afk is now known as alan_g
mhall119mhr3: those might work, are any of them used in the current docs?18:40
mhr3mhall119, no18:42
mterrytedg, I saw the greeter-volume changes landed for indicator-sound, awesome!19:25
mterrytedg, but I thought https://code.launchpad.net/~mterry/gsettings-ubuntu-touch-schemas/volume/+merge/209158 was in the same silo?19:25
mterrytedg, can you add that to the next indicator silo?  Otherwise the volume/mute syncing won't do anything19:26
tedgmterry, Ah, bummer, I put the volume changes in the player settings silo, I forgot to include that one.19:26
tedgmterry, Sure, not sure when that'll be, if you get one throw it in there as well.19:26
mterrytedg, k19:27
tedgmterry, Told you we should put the setting in the indicator sound package ;-)19:27
mterrytedg, hrrm19:27
tedgI think that's all the indicator-sound stuff though, the project is flushed. Which is exciting.19:28
mterrytedg, :)19:28
mterrygreyback, do you know much about what makes the osk keyboard become "maximized" (unity-mir triggers some of its keyboard code off when the keybaord surface becomes maximized)19:51
Saviqmterry, there's a direct comm channel between qt and maliit, if that's what you're asking19:58
Saviqmterry, that says 'hey, a field got focused'19:58
Saviqmterry, and then there's an API between ubuntu-keyboard and unity8 (over a socket) to tell it what's the keyboard state19:58
mterrySaviq, I think I'm interested in ubuntu-keyboard more than maliit.   Found an issue where it is passing clicks through the keyboard into shell with split branch19:59
Saviqas well as directly back to Qt for the Qt.inputMethod19:59
Saviqmterry, than you want this ↑19:59
Saviqmterry, OSKController.qml19:59
mterrySaviq, seems to be because OSKController never sees that the keyboard surface gets maximized19:59
mterrySaviq, right.  It has code to watch the surface state.  This state gets triggered without my branch, but doesn't with19:59
mterryTrying to find code that actually "maximizes" the ubuntu-keyboard surface20:00
=== dandrader is now known as dandrader|bbl
mterrygreyback, testing your PPA, will be back (hopefully)20:11
mterrygreyback, seems to work fine to me (stop lightdm and restart lightdm both killed USC)20:14
mterrygreyback, I wasn't logged in20:14
mterrygreyback, should I have tried a unity8 session and then stopped lightdm?20:14
greybackmterry: ok, strange it behaves differently for me. Will have a look tomorrow20:24
* greyback eod20:24
=== ajmitch_ is now known as ajmitch
=== salem_ is now known as _salem
=== dandrader|bbl is now known as dandrader
Saviqthe scopes have landed!23:02
dandraderSaviq, go get some sleep23:23
dandradereod23:24
Saviqmterry, are we ready to rebuild silo 4 on top of new scopes?23:55
Saviqmterry, they got merged23:56
mterrySaviq, uh, OK.  Let me merge from trunk.  I'm also debugging a weird issue with the ubuntu keyboard23:56
mterrySaviq, that's why I didn't send out the email23:56
mterrySaviq, but I'll merge now23:56
Saviqmterry, ok cool, let me know23:57

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