/srv/irclogs.ubuntu.com/2013/09/26/#ubuntu-unity.txt

=== _salem is now known as salem_
=== salem_ is now known as _salem
tsdgeosurg07:53
tsdgeosi lost the time indicator in unity707:53
tsdgeoswhat happened?07:53
arssonsame here07:56
=== iahmad is now known as iahmad|afk
mzanettiSaviq: I found a way to pass the highlight on to the items... in the end it was even easier than with the OpenEffect - as usual08:56
MacSlowSaviq, I've still no solution for the "UnityMenuModel is not a type"-failure... I tried all the cmake-foo I know without success.08:59
MacSlowSaviq, I don't know why it fails for the notifications qmltest but works for the Panel/Indicators one09:00
SaviqMacSlow, I'd assume the import path is incorrect still09:01
Saviqmzanetti, yeah, I thought as much09:02
mzanettiSaviq: only downside: as of now there is no way around using a Tile as base class for delegates - or you'll break the highlight stuff09:02
mzanettibut probably we had that dependency already in somehow before09:03
Saviqmzanetti, yeah, that's fine09:03
Saviqmzanetti, well, we should abstract out a "DashDelegate" that will handle that09:03
Saviqas Tile is a little too much for a default09:03
mzanettiprobably yes... but actually the Tile is exactly that. just badly named so to say09:03
mzanettiis it?09:03
SaviqMacSlow, as in do we have "UnityMenuModel" in the mock being loaded by the test?09:03
Saviqmzanetti, well, it does have a lot of UI already09:04
mzanettihmm, ok09:04
Saviqmzanetti, but that's fine for now09:04
Saviqmzanetti, it breaks for carousel09:05
mzanettido we have a scope with a carousel around?09:05
Saviqmzanetti, yeah, on desktop09:05
Saviqmzanetti, video09:05
mzanettiSaviq: local videos?09:06
mzanettibecause I only see tiles09:06
Saviqmzanetti, yeah, just drop something in ~/Videos09:07
mzanettistill nothing... do I need to fire up some media service?09:08
MacSlowSaviq, not sure what you mean... there's no UnityMenuModel at all in the mock notification09:09
SaviqMacSlow, there is a mock QMenuModel plugin09:10
SaviqMacSlow, in tests/mocks09:10
SaviqMacSlow, and it doesn't have UnityMenuModel in ther09:10
Saviqe09:10
SaviqMacSlow, it's probably enough if you'll have a UnityMenuModel.qml there with the properties you're setting09:11
MacSlowSaviq, trying...09:11
Saviqjust a dummy one to make QML swallow it - without really using it09:11
tsdgeosSaviq: https://codereview.qt-project.org/#change,66585 for the missing scopes09:14
Saviqtsdgeos, you ROCK, sir!09:14
tsdgeosnot fixed in 5.2 either09:14
tsdgeosSaviq: well, i'm not sure it's the *right* patch, it's a patch that makes it work and makes  to me, let's see what they think :D09:14
Saviqtsdgeos, that's enough for me ;)09:15
tsdgeosmakes to me -> make sense to me09:15
tsdgeosi lost the sense soemwhere09:15
mzanettiprobably in the loaders code09:15
tsdgeos:D09:15
=== iahmad|afk is now known as iahmad
Saviqmzanetti, ah, just noticed something in the navigation spec...09:25
Saviqmzanetti, "When a transition moves from last item  in a row to first item in the next row. The visible area of the scope view will scroll to animate the row transition. When the next item is not a visible items in the collapsed state of the category, it will expand."09:25
Saviqmzanetti, that's for previews09:25
Saviq(swiping between)09:26
nic-doffaymhr3, ping09:26
mzanettiSaviq: yeah? is that not what I'm doing?09:26
Saviqmzanetti, do you expand the category when you reach the end of the collapsed category?09:27
mzanettiSaviq: yes. but I collapse again when closing the preview if it was collapsed before09:27
mhr3nic-doffay, pong09:27
mzanettiSaviq: did feel really weird otherwise09:27
mzanettiimho09:27
nic-doffaymhr3, was away did you manage to sort that scope branch out?09:29
Saviqmzanetti, k, thought we're limited to the 6 items in preview already09:30
Saviqmzanetti, why do you collapse when closing?09:30
tsdgeosmzanetti: when you manually expand the category, do you update expandedIndex ?09:30
mhr3nic-doffay, i merged with some larger changes we needed which were pretty much touching the same code, but it'll take a day or two to get that branch in... you can just merge the branch i pointed you to, should be good enough for you09:31
mhr3nic-doffay, that branch being lp:~mhr3/unity8/search-in-progress09:32
MacSlowSaviq, just pushed the fix that makes the notifications qmltest pass again09:32
MacSlowSaviq, thx for the pointer09:32
mzanettitsdgeos: no, I set filter = false09:32
SaviqMacSlow, cool, thanks09:32
tsdgeosmzanetti: actually you should set expandedIndex better09:32
mzanettiSaviq: I restore the dash to what it was before opening the preview.09:32
tsdgeosotherwise bad things will happen09:32
tsdgeosnote that i'm working on replace expandedIndex with expandedScopeId09:33
mhr3nic-doffay, and the new one which will take time is lp:~unity-team/unity8/scope-isactive - but it's based off the search-in-progress one, so should merge cleanly09:33
tsdgeossince storing the index is bad09:33
mzanettitsdgeos: yeah, seen those bad things...09:33
tsdgeosso you may want to wait on that09:33
mzanettitsdgeos: but as I restore it to what it was before I don't hit those09:33
mzanettitsdgeos: there was a problem with expandedIndex... forgot what it was, let me check09:34
Saviqmzanetti, "NOTE: Selected category that was expanded remains so after exiting a preview. "09:34
mzanettiok...09:34
mzanettimissed that09:34
Saviqmzanetti, I think the desire is to just close the preview "in place"09:34
mzanettiok... will change09:35
Saviqmzanetti, i.e. switching preview really affects the dash like you would do it with your finger09:35
Saviqmzanetti, but please verify with Oren09:35
Saviqmzanetti, especially for the case "I expanded category 1, opened previews for category 2, got category 2 expanded by switching previews, closed previews"09:35
Saviqmzanetti, should category 1 get collapsed and remain collapsed?09:35
mzanettiSaviq: quite sure it should in that case09:36
nic-doffaymhr3, is the scope-isactive branch in a useable state atm?09:36
mzanettiSaviq: the bigger question is how to position the view after collapsing a gategory :D09:36
Saviqmzanetti, /me too, but let's just verify09:36
tsdgeosmhr3: pstolowski: is it safe to assume there's no categoryId named "" ?09:37
mhr3nic-doffay, it deps on changes to unity-core which didn't land yet, so not trivial to get working09:37
mhr3nic-doffay, but the search-in-progress has all the api you need09:37
Saviqmzanetti, you mean when you collapse 1 and expand 2, that you may get tricked by LVWPH for contentY?09:37
mzanettiSaviq: yep09:37
Saviqmzanetti, you'll manage ;)09:37
Saviqmzanetti, but you might need to do it in sequence09:38
mhr3nic-doffay, i suggest just merging search-in-progress into your branch and then when the search-isactive lands your branch should be fine (and the diff won't include the search-in-progress changes)09:38
mzanettiSaviq: do we actually use DashVideos still?09:38
mzanettiSaviq: I see it only used for mockvideoscopes09:39
mhr3nic-doffay, or your branch could even land first...09:39
Saviqmzanetti, prolly not09:39
Saviqmzanetti, not DashMusic either09:39
mzanettiSaviq: because that broke too as it handles the openeffect on its own09:39
pstolowskitsdgeos: libunity doesn't have constraint on it, so I would say it's not safe09:39
mzanettiSaviq: should I just drop it or fix it?09:39
Saviqmzanetti, drop09:39
mzanettiack09:39
Saviqmzanetti, we don't want it09:39
mzanetti+109:39
Saviqmzanetti, ScopeDelegateMapper should go away ASAP09:39
tsdgeospstolowski: uncool :D09:40
Saviqbut we still use it for Frequent Apps09:40
nic-doffaymhr3, cool :)09:40
* mzanetti never saw any "Frequent Apps" yet09:40
mhr3nic-doffay, or we can even mp search-in-progress by itself...09:40
mhr3otoh i did some minor fixes in search-isactive... don't want to backport those :P09:41
mhr3yey for doing things the complicated way :)09:42
tsdgeosbrr09:42
tsdgeoswow, actaully categoryId is empty for all the categories?¿09:43
tsdgeossomething's wrong somewhere09:43
nic-doffaymhr3, up to you dude, I'm easy.09:44
mhr3nic-doffay, i go for easy - merge search-in-progress into your branch09:45
nic-doffaymhr3, yeah that's what I've done.09:45
nic-doffaymhr3, so searchInProgress is the property, yeah?09:45
nic-doffayor at least will be the working one once the other branch lands?09:45
mhr3nic-doffay, yep, should do all you need09:45
nic-doffaymhr3, cool.09:45
nic-doffayta09:45
mhr3nic-doffay, should work right away09:45
Saviqjibel, thanks for going through our bugs!09:48
Saviqjibel, btw, do you know why the retracers are unable to get anything useful out of apport bugs from touch?09:49
jibelSaviq, yw, actually lot of the issues I find are already found, so it made more sense to me to triage the queue than submitting new ones.09:51
nic-doffaySaviq, not sure what's happening with the test fails on my branch with jenkins, all tests pass when I make them...09:51
jibelSaviq, I asked pitti last week about the retracers, but I don't think he had time to look in detail, I'll ask him again09:51
Saviqjibel, thanks09:52
Saviqnic-doffay, that's re: filter selectors?09:52
nic-doffaySaviq, yeah09:53
Saviqnic-doffay, the mediumtests-touch are unrelated09:54
Saviqnic-doffay, it still happens some times on phones for us for some reasons09:54
Saviqnic-doffay, looking at qmluitests09:54
Saviqnic-doffay, file:///tmp/buildd/unity8-7.81.3+13.10.20130924.2/Components/PageHeader.qml:494:25: Cannot assign to non-existent property "onExpansionCompleted"09:55
Saviq                             onExpansionCompleted: {09:55
Saviq                             ^09:55
nic-doffaySaviq, ah that's because of the what I mentioned in the comments further up. It relies on the multi selector branch.09:56
Saviqnic-doffay, yeah, so make sure that gets merged :)09:56
jibelSaviq, so, arm retraces on x86 is a bit hairy, for the moment the way to go is to run apport-retrace on the phone10:01
jibelunless we get an actual ARM box where we can do that, of course10:01
Saviqjibel, ah, apport-retrace!10:01
Saviqjibel, will that download dbg symbols and such/10:01
Saviq?10:01
tsdgeosmzanetti: since you're touching that code, do you think you can do https://code.launchpad.net/~aacid/unity8/storeExpandedCategoryNotIndex/+merge/187708 ?10:03
mzanettitsdgeos: sure10:03
tsdgeosgreat10:03
=== iahmad_ is now known as iahmad|afk
=== thostr_ is now known as thostr_lunch
Saviqmzanetti, re: lazy image... it's a bit tricky to keep aspect ratio, as the Image itself doesn't do this, and there's no way to find out that width/height are "unset" to calculate the other dimension... ideas?10:52
* greyback moving to office10:54
mzanettiSaviq: I think I did that already somewhere at some point10:54
Saviqmzanetti, yeah, but that's when you *know* which dimension is to be kept10:55
mzanettiSaviq: I think you can use the sourceSize to find the original aspect ratio10:55
mzanettibefore setting it10:55
Saviqmzanetti, *find* yes10:55
Saviqmzanetti, but I can't find out whether implicitHeight / implicitWidth is used10:55
Saviqmzanetti, let me rephrase...10:55
Saviqmzanetti, LazyImage { width: units.gu(10) ... }10:56
Saviqmzanetti, the image should scale to keep aspect ratio10:56
mzanettiyes10:56
Saviqmzanetti, but how do I know that this width is supposed to be used to calculate the height?10:56
Saviqmzanetti, Image itself won't scale the other dimension in that case10:57
Saviqmzanetti, sourceSize is capped by actual image size, btw10:57
mzanettiheight: sourceSize.height * width / sourceSize.width10:57
Saviqmzanetti, and why not width: sourceSize.width * height / sourceSize.height ?10:58
Saviqmzanetti, that's the thing - I can't know which one of those to calculate, without introducing a new property10:58
Saviqunless someone tells me otherwise, that is10:58
mzanettiSaviq: well... if you have width and height stored in the LazyImage10:59
mzanettiSaviq: then you can use those to calculate the Image's dimensions inside10:59
Saviqmzanetti, there is *always* width and height, that's the thing10:59
mzanettiright10:59
Saviqmzanetti, whether it's set externally or not - that's the question we can't answer11:00
mzanettiSaviq: probably introducing some new properties then :/11:00
Saviqit's just going to be equal to implicit*, or 0, but that's not something we should use to find out...11:00
Saviqmzanetti, yeah, s/sourceSize/desiredSize/ basically...11:01
mzanettiSaviq: I'd use imageSize, but yeah11:01
Saviqmzanetti, yeah, I'll think of some name11:01
mzanettiSaviq: wait... I remember11:02
mzanettiI think what I did is to keep the ratio and make it fit into width/height11:02
mzanettiso regardless who set those, the content will scale to the biggest possible, keeping the ratio11:02
mzanetticenteredIn, that is11:03
mzanettiSaviq: how does that sound?11:03
Saviqmzanetti, not sure, but then... it's all really about "scaleTo: width/height" - no need for actual values...11:06
Saviqso maybe we just need a string prop11:07
Saviq(ideally enum, but we can't do enums in QML yet)11:07
Saviqmzanetti, this way you'd just use it as usual, but set scaleTo: "width" in case you want it to do that11:08
Saviqor scaleTo: "height" otherwise11:08
Saviqand ignore it if you wanted to force both11:08
mzanettiprobably the most flexible one...11:08
Saviqmzanetti, centeredIn feels like it'd crop when we don't want to (we want to keep aspect ratio)11:08
Saviqit's stupid that Image doesn't have that11:09
mzanettiSaviq: why crop?11:09
Saviqmzanetti, I think I'm just not sure what you described above11:09
=== MacSlow is now known as MacSlow|lunch
mzanettiSaviq: ok. so the public API has width/height, as normal11:09
Saviqmzanetti, yeah, the only addition would be 'scaleTo: "width"/"height"'11:10
mzanettiSaviq: then the Image inside always keeps the aspect ratio11:10
Saviqmzanetti, I know what you mean11:10
Saviqmzanetti, but that's the thing - you don't know the aspect ratio11:10
mzanettiSaviq: sure you know11:10
Saviqmzanetti, what you're describing is PreserveAspectFit11:11
mzanettiSaviq: image's sourceSize holds that information (as long as you don't override it)11:11
mzanettiSaviq: yeah... that's it11:11
Saviqmzanetti, yeah, but how does the *user* of LazyImage know?11:11
Saviqmzanetti, ah, you mean to read it from tere11:11
Saviqthere11:11
mzanettiyes11:11
mzanettithe user doesn't care11:11
mzanettijust give it a place and put it in there with PreserveAspectFit11:11
Saviqwell, except we want to override sourceSize to preserve memory ;)11:12
mzanettiSaviq: sure, but with a bit of procedural code you can first read it, calculate your stuff and then overwrite it11:12
Saviqmzanetti, which means loading the image twice11:12
mzanettihmm... right... didn't think of that11:13
* Saviq is gonna go with the "scaleTo" approach :)11:13
Saviqmzanetti, pusehd11:26
SaviqMacSlow|lunch, works!11:27
tsdgeosis the CI bot on holiday?11:56
tsdgeoshttps://code.launchpad.net/~aacid/unity8/storeExpandedCategoryNotIndex/+merge/18770811:56
=== alan_g is now known as alan_g|lunch
=== MacSlow|lunch is now known as MacSlow
MacSlowSaviq, yeah saw that too :)12:01
Saviqtsdgeos, it's queued up probably12:02
=== hikiko is now known as hikiko|lunch
larsursalveti: hey, do you have any idea what could be causing bug #1226312?12:19
ubot5bug 1226312 in indicator-sound (Ubuntu) "[touch] Sound menu settings are overridden on second call " [High,Confirmed] https://launchpad.net/bugs/122631212:19
larsursalveti: unfortunately, I don't have a second sim card to reproduce this. I'm not even certain this is indicator-sounds' fault12:19
Cimimzanetti, hey, what happened to this? https://code.launchpad.net/~cimi/unity8/unity8.hud-2_hint-reveal-commit/+merge/18748812:21
mzanettiCimi: nothing it seems12:21
mzanetti:)12:21
Cimimzanetti, says needs fixing12:23
Cimiwhere is Unity.Application?12:31
mzanettiCimi: in qtubuntu and unity-mir12:32
mzanettiCimi: the tests seem to fail on the phones...12:33
Cimimzanetti, why?12:33
mzanettiCimi: I don't know...12:33
mzanettiCimi: have you run them on your phone?12:34
Cimimzanetti, nope12:36
=== dednick is now known as dednick|lunch
=== _salem is now known as salem_
=== alan_g|lunch is now known as alan_g
paulliumzanetti: https://code.launchpad.net/~paulliu/unity8/unity8-fixgenericpreview/+merge/18776212:59
paulliumzanetti: Also I fix the buttons to have the maxWidth. So it should looks better in wide screen.12:59
mzanettipaulliu: cheers13:02
=== hikiko|lunch is now known as hikiko
mzanettipaulliu: hmm.. this seems to be quite far away from what it should be13:23
mzanettipaulliu: I added some screenshots13:23
paulliumzanetti: ok13:29
paulliumzanetti: hmm.. let me fix it again..13:30
Saviqdandrader, standup?13:31
dednick|lunchSaviq: ping13:39
Saviqdednick|lunch, swallow, and pong13:39
=== dednick|lunch is now known as dednick
dednickSaviq: :) hopefully an easy one.13:40
Cimimterry, hey13:41
dednickSaviq: about the messaging indicator. Deisgn requires that snap decision messages (missed calls) are able to open the app by clicking the icon when you expand a menu. This works for text messages, but not for missed calls.13:41
mterryCimi, hello!13:41
Cimimterry, all good? :)13:41
mterryCimi, so here's status update on welcome-wizard...  I've fixed a lot of stuff, I'm working now on making it Mir-ready.  Still need to do the dynamic-language thing13:42
Cimimterry, I can work on it13:42
Cimimterry, however, some things seem to got broken13:42
mterryCimi, oh, I broke stuff?13:42
dednickSaviq: some functionality is missing from telephony-service which does this. I've raised a bug for it, but in the mean time, it's removing the item when you tap the icon, which seems a bit weird. Wondering if we should disable this for snap decisions for the time being until we have a fix, or just leave it13:42
Cimimterry, not sure you or the time :)13:42
mterryCimi, what's not working?13:43
Cimimterry, on my pc, the account page doesn't detect Andrea Cimitan as real name13:43
dednickSaviq: i'm talking about snap decisions in the messaging menu here.13:43
Saviqdednick, so not snap decisions... but messaging menu entries, really? :)13:43
dednickSaviq: yeah, well design originally called them snap decisions i think.13:44
Saviqdednick, I think we should just fix it :)13:44
Saviqdednick, and blame them for not fixing it ;)13:44
Saviqdednick, it was working before, no?13:44
mterryCimi, ah.  Did that ever work?  Even on phone when I go forward after setting that, then go back to start and hit that page again, it doesn't know my real name13:44
dednickSaviq: i dont know if it was ever working for missed calls13:45
Saviqdednick, did you talk to tiagosh / boiko about this?13:46
mterryCimi, well, I may have accidentally broken that, but I don't believe I touched the name code13:46
dednickSaviq: nope. i dug into the code and found it missing a peice.13:46
mterryCimi, also...  when testing on the phone, I found it helpful to chmod a-w /home/phablet/.config/ubuntu-system-settings13:47
mterryCimi, that way the system settings always come up13:47
Cimimterry, was working a month ago...13:47
Saviqdednick, so let's find out whether they know about that missing piece and see if they can add it - and let's leave our stuff alone - or if not - look for a temporary solution13:47
mterryCimi, OK, well, add that to our TODO then13:47
dednickSaviq: ok.13:47
paulliumzanetti: But for Video preview, I have to do that in another branch.13:48
mzanettipaulliu: ok13:49
kgunnmzanetti: tsdgeos ...were you're ears burning ? :) was going to start firing some bugs at you to help take some of the load off gerry13:49
kgunntsdgeos: can you look at https://bugs.launchpad.net/ubuntu/+source/mir/+bug/123112513:49
ubot5Ubuntu bug 1231125 in unity-mir "Music and Videos do not launch from the dash" [Critical,Triaged]13:50
mzanettikgunn: hit me13:50
tsdgeoskgunn: i guess :D Busy somewhere else, but just assign it to me and i'll put it on the "bug queue"13:50
dednickboiko: ping13:50
tsdgeoswhich is almost empty so it'll be soon13:50
boikodednick: pong13:50
kgunntsdgeos: yep...13:50
kgunnmzanetti: thanks...can you look at this one https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/121987113:51
ubot5Ubuntu bug 1219871 in unity8 (Ubuntu) "when idle with screen blank, unity8 generates hundreds of context switches per second" [High,Confirmed]13:51
kgunnmzanetti:  i think they found it with SF on...would be interesting to confirm in w/ very latest mir or not13:51
kgunnmzanetti: not sure if mir has made it out of staging yet13:51
dednickboiko: hi, you mind going over the above log between Saviq and I? related to the messagingmenu call activation in the telephony-service13:51
tsdgeoskgunn: ok, assigned it to my sel13:51
tsdgeosf13:51
mzanettikgunn: ok13:51
dednickboiko: we are missing dialer app launching from the menu activation (same as text app launch)13:52
boikodednick: in a meeting right now, I will check the logs afterwards13:53
dednickboiko: thanks13:53
boikodednick: but it was working before, might have been broken with the time being, but ok, can you report a bug on that?13:54
dednickboiko: https://bugs.launchpad.net/telephony-service/+bug/123140213:54
ubot5Ubuntu bug 1231402 in telephony-service "Can't launch dialer app from messaging menu." [Undecided,New]13:54
boikodednick: I remember the default action thing of libmessaging-menu was not being activated when clicking the icons13:54
boikodednick: thanks13:55
dednickboiko: yeah, that's broken at the moment, but working on a fix. Have it working for text messages, but missed calls do not work13:55
dednickmzanetti: i think we're good to go on the messaging menu if the review is ok. We will get the tap on icon fix another time.13:56
mzanettidednick: you opened a bug for that already, right?13:57
dednickmzanetti: yep13:57
mzanettiok. will do a last check and approve13:57
=== thostr_lunch is now known as thostr_
boikodednick: I guess it was never implemented for missed calls, because they didn't want to have a default action for missed calls, I can implement that, should be pretty simple14:09
mzanettiCimi: I have to leave for an hour or so. can help you afterwards if you're still stuck. try to run the tests on your device in the meantime14:09
dednickboiko: ok cool. not sure exactly what they want to happen, open dialer with number set maybe... might want to check with design.14:10
boikodednick: yep, will do14:10
dednickboiko: thanks for that14:10
tsdgeosi hate floats :D14:14
tsdgeos-50 -50 false14:14
tsdgeos-50 is not equal to -5014:14
tsdgeosargggg14:15
=== alan_g is now known as alan_g|tea
Cimihelp in running autopilot on the phone?14:27
Cimimaybe I'm missing required packages14:27
Cimithis is my error http://paste.ubuntu.com/6158929/14:30
=== alan_g|tea is now known as alan_g
Cimimzanetti, it's my turn to leave, I'll be back later and tonight14:37
Cimimzanetti, I have the error agove ^14:38
Cimiabove14:38
dednicklarsu: crash in inidicator-messages14:47
dednicklarsu: http://pastebin.ubuntu.com/6159005/14:47
dednicklarsu: and 'Hi!'14:47
dednicklarsu: also getting this every now and then. *** Error in `/home/phablet/dev/indicator-messages/install/libexec/indicator-messages/indicator-messages-service': corrupted double-linked list: 0x0004c530 ***14:48
dednicklarsu: but didnt get it the time it crashed14:48
larsudednick: hi :)14:48
larsudednick: is there a way to reproduce this?14:49
dednicklarsu: not really. just missed calling + dismissing the menu items.14:49
dednicklarsu: i'm getting this weird issue where items are not getting removed from the menu sometimes.14:50
larsudednick: cool, I'll do a code review of the section in that backtrace then.14:50
dednicklarsu: ta14:50
larsudednick: after you activate them?14:50
larsuor when the phone app should remove them?14:50
larsubecause you navigated to the call / message14:50
dednicklarsu: after i activate14:51
larsustrangeness14:51
larsudednick: I'll have a go with your branch later today or in the morning tomorrow14:52
larsuhopefully I'll hit the same issues14:52
dednicklarsu: thanks. it's happening every 2 or 3 times after i do an activate. Highlights another issue though, unity8 doesn' seem like it's handling recovery after you restart the indicator. The old menu items arent being deleted. Think it's something todo with the submenus14:54
larsunoted.14:55
tsdgeosmzanetti: can you do https://code.launchpad.net/~aacid/unity8/bug1230187_fix_showHeaderInEdgeCase/+merge/187808 ?14:56
tsdgeosactually i wanted to say Saviq in that sentence :D14:56
tsdgeosSaviq: ↑↑14:56
Saviqtsdgeos, on it14:56
Saviqtsdgeos, got a fail in testMaximizeVisibleAreaMoveUpAndShowHeader14:58
tsdgeoshmmmm14:59
tsdgeoswhat happened?14:59
tsdgeos0 != 0?14:59
Saviqtsdgeos, 0 != -7.blahe0515:00
Saviqtsdgeos, worked now15:00
tsdgeosyeah i got that 0 != almost0 once too15:00
tsdgeosnot sure why :-/15:00
tsdgeosnor how can i fix it to not happen15:00
tsdgeosit's not "wrong"15:01
tsdgeosit's just that someone decided to throw the odd bit up or down somewhere15:01
Saviqtsdgeos, yeah, dholbach's lockup is in WindowManager::run, too15:01
tsdgeos:/15:01
tsdgeoskgunn: Saviq: greyback: do you know where do i have to store local music/videos so that they get picked up? anywhere? or? regarding https://bugs.launchpad.net/unity-mir/+bug/1231125 first want to make sure i get it to work with surfaceflinger :D15:10
ubot5Ubuntu bug 1231125 in unity-mir "Music and Videos do not launch from the dash" [Critical,Triaged]15:10
Saviqtsdgeos, ~phablet/{Music,Videos}15:10
Saviqtsdgeos, search in dash for it to refresh15:11
tsdgeosoka15:11
tsdgeosprobably pushing a 800M video wasn't the smartest idea15:15
tsdgeosoh well :D15:15
tsdgeoshmmm15:25
tsdgeosi can't get it to play music15:25
tsdgeosi can get it to open the music player15:25
tsdgeosbut playing music? nope15:25
* tsdgeos reboots15:26
tsdgeosand for some reason15:26
tsdgeosi have one file15:26
tsdgeosand three icons of the song15:26
tsdgeosand i can't get the video scope to show my video either15:29
mhr3tsdgeos, not all codecs are supported15:30
mhr3mp4 wasn't working last time i tried15:30
tsdgeosmhr3: so isn't even listed on the scopes?15:31
mhr3no codec -> no metadata -> not knowing that is it a video15:32
tsdgeoswell15:32
tsdgeosso i can't play videos i recorded?15:32
tsdgeosor metadata you mean mimetype and length?15:32
mhr3tsdgeos, i mean that if it can't get the metadata it also wouldn't be able to play it15:34
mhr3so it's not stored as a video in the db15:34
tsdgeosmhr3: do you know who do i complain because i have one mp3 and 3 entries in the music thing?15:41
Saviqtsdgeos, jamesh15:53
Saviqtsdgeos, or, lp:unity-scope-mediascanner15:53
tsdgeos:D15:53
Saviqtsdgeos, you can try pstolowski, too ;)15:54
=== mzanetti is now known as mzanettifd
=== mzanettifd is now known as mzanetti
* tsdgeos scared, phablet-flash running for a long time16:01
mzanettitsdgeos: happened to me too quite often lately. just retry16:01
tsdgeosoh, just finished :D16:02
mzanettitsdgeos: do you have the link for that review still handy?16:04
* mzanetti had to restart16:04
tsdgeosmzanetti: which one?16:04
mzanettitsdgeos: are there more? all of them then I guess16:04
tsdgeosmzanetti: i think i got saviq to do them all16:05
mzanettiah ok16:05
tsdgeosno review from my side waiting16:05
* tsdgeos gets nothing on screen when using mir on the phone16:08
tsdgeoscome on16:08
tsdgeosnot again16:08
tsdgeosgreyback: any idea where to look at?16:09
tsdgeosunity8's running16:09
tsdgeossurfaceflinger is not16:09
tsdgeosoh16:09
tsdgeosand now i got it on screen16:09
greybacktsdgeos: patience :) Startup blocks due to HUD service not available16:10
tsdgeosok, can repro clicking on a music file not launching the music player16:10
tsdgeosnow16:10
tsdgeosthe image i had from yesterday or two days ago worked16:10
tsdgeossomething regressed :-/16:10
tsdgeostedg: anything you're aware of?16:11
tsdgeosanyway16:11
tsdgeoswill dig tomoorrow16:11
tsdgeosactually not16:11
tsdgeosi'm on leave tomorrow16:11
tsdgeoswill dig on monday :D16:11
* tsdgeos waves16:11
tedgtsdgeos, Have a good day!  :-)16:11
tsdgeosthank you sir16:12
tedgtsdgeos, I don't know of anything, but upstart-app-launch was updated.  It shouldn't break that though.16:12
Cimimzanetti, ping :)16:18
mzanettiCimi: pong16:18
Cimimzanetti, I have this http://paste.ubuntu.com/6158929/16:18
Cimimzanetti, while running on th phone16:18
mzanettiCimi: 'GRID_UNIT_PX' must be a string or a number, not 'NoneType'16:19
mzanettiCimi: export GRID_UNIT_PX=1816:19
mzanettibefore running the test16:19
Saviqjeez that guy16:22
Saviqback from holiday... 2 days off in a week...16:22
mzanetti?16:23
mzanettitsdgeos off again?16:23
mzanettiand you call me slacker :P16:23
Saviq;)16:25
Cimimzanetti, now complains about libs16:26
Cimimzanetti, like it's searching for amd6416:26
CimiRuntimeError: Expected library path does not exists: /home/phablet/unity8.hud-2_hint-reveal-commit/builddir/install/lib/x86_64-linux-gnu/unity8/qml/mocks.16:26
Cimioh it's in __int.py__16:27
mzanettiCimi: huh... weird. never saw that16:27
mzanettiSaviq: btw. I found what's wrong with ctrl+/ in my QtCreator16:29
mzanettiits a combination of deadkeys being broken in Qt5 and the german kbd layout16:29
Cimimzanetti, doesn't work16:37
Cimimzanetti, it required x11 it says16:37
mzanettiCimi: use phablet-test-run -n unity816:37
mzanettiCimi: but something is really weird with your stuff16:38
mzanettiCimi: did you compile on the desktop and then copy all the stuff to the phone?16:38
Cimimzanetti, no I did bzr branch on the phone16:38
Cimiand compiled etc etc16:38
Cimimzanetti, phablet-run complains about missing /home/phablet/autopilot16:39
mzanettiwhat?16:39
mzanettinever saw that either16:39
mzanettiCimi: can't be... if you execute "phablet-test-run -n unit8" on your desktop it really should work16:40
mzanetti(note that it uses the one installed in the system. so you might want to install the package first and then copy over all the files you modified to /usr/16:41
Cimimzanetti, http://paste.ubuntu.com/6159428/16:41
mzanettiom26er: do you know what this is? ^16:42
om26ermzanetti, let me try myself16:43
om26erCimi, restart the phone probably.16:44
om26erCimi, which image is that ?16:44
Cimiom26er, cdimage16:44
om26erCimi, it works on my phone which I just flashed. though I have ubuntu-system16:45
CimiI'm flashing ubuntu-system16:57
Cimilet's see16:57
SaviqCimi, I had that error, too, but it went fine to test16:58
CimiSaviq, AH, MAYBE I HAVE TO WAIT...16:58
Cimiops16:58
CimiSaviq, how much wait?16:58
SaviqCimi, no, no waiting, it started straight away16:58
SaviqCimi, you can try with -v to see more autopilot output16:58
=== alan_g is now known as alan_g|EOD
=== dandrader is now known as dandrader|lunch
CimiSaviq, seems like it's doing something17:00
SaviqCimi, well, you should see it on screen ;)17:00
Cimiyep17:01
Cimiwas just very slow17:01
mzanettihuh? why don't we have the make install target on the phone?17:07
mzanettiCimi: ok... ran into the same issue as you did17:16
mzanettiCimi: fixed the autopilot tests not require that damn make install any more17:16
mzanettiwhich also fixes the x86 issue you had17:17
Cimimzanetti, I edited the file anyway17:19
Cimibut yeah thx17:19
mzanettiCimi: still... that make install every time you change a file in order to run autopilot tests was the most annoying thing ever17:20
Cimimzanetti, tested hud tests on the device, they work17:22
mzanettiCimi: are you sure you are testing the correct unity8?17:23
Cimimzanetti, yes17:23
mzanettiyou might be a victim of: https://bugs.launchpad.net/unity8/+bug/120498217:23
ubot5Ubuntu bug 1204982 in Unity 8 "refactored autopilot tests are fail prone for wrong environment setup" [Undecided,Incomplete]17:23
CimiI think so...17:23
Cimimzanetti, pls help me tomorrow :)17:24
Cimidednick, I don't think I'm coming to the office tomorrow17:25
Cimidednick, I'll probably go direct home from the airport17:25
mzanettiCimi: I think your best bet would be to download a package from jenkins, install it on the phone and use phablet-test-run17:26
Cimimzanetti, my tests work on the pc and on the phone17:26
Cimiwhy jenkins complains?17:27
mzanettiI'm quite sure they don't work on your phone but you got fooled by what I reported in that bug17:27
Cimimzanetti, but does phablet-test-run deploys something?17:27
mzanettino17:27
mzanettithey run what is installed with apt17:28
Cimimzanetti, so how does the plugins and code get transfered?17:28
mzanettiCimi: it doesn't17:28
Cimimzanetti, so I need to create a package?17:28
mzanettithat's what I say... you are running the released stuff17:28
Ciminope then17:28
mzanettiCimi: well, you either create a package, or you download the one from jenkins17:28
Cimilet me download17:28
Cimimzanetti, link to jenkins packages? (sorry for asking)17:29
mzanettiCimi: in the comments jenkins posts to your merge request17:29
Cimiok17:29
* Cimi looks17:29
Cimimzanetti, yeah, they don't work17:32
Cimimzanetti, hud_button_appears doesn't17:33
Ciminope others don't as well17:33
Cimicrap17:33
mzanettiCimi: ok... now you can edit the files in /usr/lib/python2.7/dist-packages/unity8/17:34
mzanettiCimi: and once they work, copy the files (or diff) to your branch on the PC.17:34
mzanettiCimi: and you might vote for the bug17:34
Cimimzanetti, MismatchError: After 0.5 seconds test on HudButton.opacity failed: 0.0 != dbus.Double(0.008000016212463379, variant_level=1)17:39
Cimimzanetti, 0.5 wasn't enough :)17:39
mzanettiCimi: that's what I was afraid of17:39
mzanettiCimi: and the desktop tests are now ran on real hardware. that's why they work. imagine we would still use the VM's :D17:40
mzanetti5 secs probably wouldn't have been enough :D17:40
Cimiok17:40
Cimimzanetti, shall I disable that check?17:40
mzanettiCimi: probably yes... Actually I think such stuff should be tested with qmltestrunner17:41
mzanettiCimi: in theory the HUD tests should only test the interaction between the hud menu and the app17:41
mzanettiCimi: that button_appears stuff is only there because those were the very first tests that albert wrote, back then when we didn't have qmltests yet17:42
mzanettibut most of that stuff should be converted to qmltests17:42
Cimimzanetti, later though :)17:42
mzanettiyes17:42
mzanettiI'm afraid this is one of the things that never will change  :D17:43
=== dandrader|lunch is now known as dandrader
kgunn_mterry, hey, just curious...any luck integrating split greeter branch onto mir-on-mir ??18:35
mterrykgunn_, I put that aside after our discussion at the sprint to work on other features, like the welcome wizard18:35
kgunn_mterry, right...but i thot you were just trying to help cimi a bit. my bad, i thot you were going to eventually pick it back up18:36
kgunn_mterry, or rather...refresh my memory18:37
mterrykgunn_, the wizard is needing a fair bit of help18:37
mterrykgunn_, and I don't think it would be great (quality wise) to try to land split greeter so late.  That's the reason I wanted to delay in the first place18:38
Cimikgunn_, if jenkins stops complaining I have finished with the HUD stuff18:38
mterrykgunn_, so I figured no reason to pick it back up yet, I could work on wizard and startup image/animation18:39
mterryand bugs18:39
kgunn_mterry, yeah bugs of existing stuff priority18:40
kgunn_mterry, Cimi welcome wizard is technically ok to not have in place for v1....it made the deferred list18:40
mterrykgunn_, oh really?  :-/  It'd be really nice though.  But OK.18:41
mterrykgunn_, what is deferred list?  Everything that we don't have now?  :)18:41
Cimikgunn_, without wifi is a bit useless...18:41
mterryCimi, naw, it's a nicer introduction.  There's a bug how the first thing the user sees is the words "Right edge"18:41
mterryCimi, plus sets up language18:41
kgunn_mterry, you got it buddy18:50
mterrykgunn_, OK, well I'll look again at bugs assigned to unity8, but last time I looked, there wasn't anything super important greeter-wise.  If there still isn't, I'll plug away at wizard.  I would like to add it if possible18:51
mterryeven in a minimized just language/welcome form18:52
kgunn_mterry, i understand (so close you want to jam it in )18:52
mterrykgunn_, plus, without a split greeter, I feel like I need something OOBE related to land  :(18:52
kgunn_mterry,on the split greeter front...even tho not for v1, it18:53
kgunn_'d be great to have feedback for the mir guys18:53
kgunn_if you see instability18:53
mterryfair enough18:53
seb128mterry, I doubt any oobe is going to land for v118:54
mterryseb128, :(18:54
seb128sorry :/18:54
mterryseb128, you don't think we can squeeze in a wizard?18:54
seb128but we were asked to not do any design change or new feature18:54
seb128just fix bugs18:54
seb128well, you can try, good luck18:54
mterryseb128, I have a bug that says we'd like a welcome wizard...  :-|18:54
seb128mterry, I've some bug that says we would welcome control of the apps displayed on the lock screen18:55
kgunn_mterry, i bet if you showed up with a welcome wizard done....with at least a week to go, then they'd be happy to take18:55
seb128mterry, or control of the auth methods18:55
seb128mterry, that doesn't mean we are going to land those ;-)18:55
mterryseb128, now you're getting snippy  :)18:55
mterryj/k18:55
kgunn_mterry, seb128 no matter what...the works not throw away (think phone v2)18:55
mterrykgunn_, sure18:56
seb128mterry, :-P18:56
seb128well, what kgunn_ said18:56
seb128if you get something working in the next days, it might go in18:56
seb128but the priority is to make what we have solid18:56
seb128it's always tricky to land new code after betas18:56
mterryseb128, what happened to landing-stuff-until-the-11th-hour18:57
seb128good old times :/18:57
kgunn_mterry, trying to break the evil ways18:57
seb128mterry, I remember times where we were uploading 30 GNOME tarballs the day before hard freeze :p18:57
mterrykgunn_, seb128: alright, alright.  Quality is good, I just like wizards is all  /me kicks pebble18:58
* kgunn_ imagines mterry in wizard costume at home18:59
seb128mterry, we like wizards as well, do you have your wand with you? ;-)19:00
* mterry enters bug mode begrudgingly19:00
seb128mterry, if you need bugs I can assign you some, in memory of the good old desktop team time ;-)19:01
mterryseb128, I'll look around first  :)19:02
=== salem_ is now known as _salem
SaviqCimi, you can try with -v to see more autopilot output20:15
SaviqCimi, you can try with -v to see more autopilot output20:15
Saviquh oh20:33
Saviqmy focus behaves *weird* today20:33
* Saviq restarts the session20:33
=== _salem is now known as salem_
seb128bregma, hey, do you know if there are recent known issue about the compiz decorator (click going through)?21:02
seb128e.g can move by left click and dnd, and right click on the decoration displays the menu from nautilus21:02
seb128the order borders also don't allow to resize by dnd21:04
bschaeferseb128, yeah I believe ChrisTownsend fixed that21:12
bschaefercould be the wrong thing though :)21:12
seb128bschaefer, do you know when the fix is going to land?21:12
bschaeferseb128, hmm I think it hit trunk at lease?21:12
* bschaefer checks21:12
bschaeferChrisTownsend, also if you know ^21:12
bregmaI think it hit trunk21:13
bregmabut trunk has been waiting to land for a loooong time21:13
seb128I don't see a commit matching that21:13
seb128bregma, we had a landing 6 days ago it seems21:13
seb128bregma, https://launchpad.net/ubuntu/+source/compiz/1:0.9.10+13.10.20130920-0ubuntu121:13
bregmaChrisTownsend seems to be away from his desk right now21:13
ChrisTownsendI'm here21:14
ChrisTownsendTrying to grok the issue seb128 is seeing.21:14
bschaeferChrisTownsend, you remember that bug where the mouse was going through the decor?21:14
* bschaefer could be imagining things...21:14
bschaeferseb128, like this? https://bugs.launchpad.net/compiz/+bug/115826721:15
ubot5Ubuntu bug 1158267 in Compiz 0.9.10 "Regression: Broken (click-through) window decoration on some maximized windows" [Critical,Fix committed]21:15
bschaeferbut hmm that should have been released...21:16
ChrisTownsendbschaefer: That issue actually doesn't affect Unity users.21:16
* bschaefer is mis informed then21:16
seb128bschaefer, no, doesn't seem the same21:16
bschaeferChrisTownsend, ill leave it up to your brain :)21:16
seb128larsu had the same issue this week21:17
seb128so it's not only me21:17
ChrisTownsendseb128: That issue seems new.21:17
seb128yes21:17
ChrisTownsendseb128: So how do you reproduce again?21:17
seb128it started with the most recent landing21:17
seb128I would tend to blame http://bazaar.launchpad.net/~compiz-team/compiz/0.9.11/revision/378721:17
bschaeferChrisTownsend, could be the bug you're digging into right now?21:17
seb128ChrisTownsend, I don't know how to reproduce, it happens on some windows at some point21:18
seb128current on xchat-gnome21:18
ChrisTownsendbschaefer: Nah, I think it's a different issue.21:18
seb128my laptop was docked and now is undocked21:18
seb128so maybe it's due to a screen geometry change21:18
seb128the click go through the border21:18
seb128e.g I can't dnd the win by left click-dnd the decoration21:18
seb128or I can't resize on the borders21:18
seb128or I can't right click on the decoration to get the wm menu (I get the nautilus menu from under it when doing that)21:19
seb128it happens only to xchat-gnome atm21:19
seb128not to new things I open21:19
ChrisTownsendseb128: Did you happen to use Show Desktop recently?21:19
seb128couldbe21:20
seb128I keep hitting that one by mistake when alt-tabbing21:20
ChrisTownsendseb128: There is a click through issue after using Show Desktoip that's in Unity trunk waiting to land.21:20
seb128(I swear there is a bug there as well, where alt-tab order gets messed up)21:20
seb128oh ok21:20
seb128that could be it then21:20
ChrisTownsendLemme find the bug...21:21
seb128ChrisTownsend, https://bugs.launchpad.net/unity/+bug/1228915 ?21:21
ubot5Ubuntu bug 1228915 in unity (Ubuntu) "Regression in minimalization of open windows causes click-through in decoration" [Critical,In progress]21:21
seb128http://bazaar.launchpad.net/~unity-team/unity/trunk/revision/352821:21
seb128I guess21:21
ChrisTownsendYep:)21:22
seb128ChrisTownsend, thanks21:22
ChrisTownsendseb128: no problem21:22
=== salem_ is now known as _salem
SaviqMirv, so you know, just filed bug #1231738 about our FTBFS under Qt 5.1, will try and get fixed tomorrow23:03
ubot5bug 1231738 in Unity 8 "FTBFS with Qt 5.1 after merging "update section header on list change events"" [Critical,Triaged] https://launchpad.net/bugs/123173823:03
=== shiznix_ is now known as shiznix

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