=== Malsasa_ is now known as Malsasa | ||
=== Malsasa_ is now known as Malsasa | ||
tsdgeos | mzanetti: https://code.launchpad.net/~aacid/unity8/scopeListPageHeaderScopeStyle | 08:31 |
---|---|---|
tsdgeos | camako: https://code.launchpad.net/~aacid/unity8/moreAsyncDash/+merge/241524 | 08:34 |
tsdgeos | err | 08:34 |
tsdgeos | camako: sorry | 08:34 |
tsdgeos | Cimi: https://code.launchpad.net/~aacid/unity8/moreAsyncDash/+merge/241524 | 08:35 |
tsdgeos | Saviq: CI is stuck again? | 08:39 |
Saviq | tsdgeos, I think yeah, they have issues with the phones all the time | 08:41 |
tsdgeos | seems i broke the Navigation tests when the fix for departments landed :/ | 09:27 |
* tsdgeos gets to fix them | 09:27 | |
* tsdgeos wants CI back | 09:27 | |
Saviq | tsdgeos, it should be back now | 09:40 |
Saviq | tsdgeos, any MP you'd like retried? | 09:40 |
tsdgeos | Saviq: with phones included? | 09:45 |
tsdgeos | Saviq: https://code.launchpad.net/~aacid/unity8/autopilot_drag_more | 09:45 |
tsdgeos | ubuntu-support-status --show-unsupported seems a bit confused by vivid | 09:53 |
tsdgeos | it's telling me that vlc is past support ?¿ | 09:53 |
tsdgeos | :D | 09:53 |
Saviq | tsdgeos, isn't it that because it's from universe? | 09:54 |
tsdgeos | maybe | 09:54 |
Saviq | tsdgeos, restarted | 09:54 |
tsdgeos | what's interesting is the "Can't be downloaded anymore" packages | 09:55 |
tsdgeos | helps clean up cruft from upgrade to upgrade | 09:55 |
Saviq | true | 09:55 |
tsdgeos | like libupstart-app-launch2-dev | 09:58 |
pstolowski | mzanetti, ping | 10:04 |
mzanetti | pstolowski: pong | 10:14 |
pstolowski | mzanetti, hey, do you know when is silo 9 going to land? anyone testing it? | 10:15 |
Cimi | tsdgeos, I was meant to test this morning but I am in a terrible pain (worse in months), I cannot even lay in bed, will try to do later | 10:15 |
tsdgeos | oh :/ | 10:16 |
tsdgeos | get better | 10:16 |
Cimi | I have enough of this :( | 10:16 |
mzanetti | popey: Saviq said he'll take over of friday. So I'm assuming he's testing it | 10:16 |
Cimi | it's a nightmare | 10:16 |
Saviq | mzanetti, I am | 10:18 |
mzanetti | pstolowski: that was for you, 3 lines above ^, not popey | 10:18 |
mzanetti | :) | 10:18 |
pstolowski | mzanetti, ok :) thanks | 10:19 |
pstolowski | Saviq, hmm, you were supposed to be off today?.. | 10:21 |
pstolowski | tsdgeos, hey, can you prepare rtm branch for your department-jumping fix? https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1343242 | 10:24 |
ubot5 | Launchpad bug 1343242 in unity8 (Ubuntu RTM) "Departments break if going to a subdepartment of Store" [High,Triaged] | 10:24 |
tsdgeos | pstolowski: i need to fix some qqmluitests that are failing because of it first | 10:24 |
tsdgeos | shouldn't take long | 10:25 |
Saviq | pstolowski, no, starting tomorrow | 10:28 |
Saviq | pstolowski, we don't do separate MPs for rtm, but a single "staging" branch, similar to https://code.launchpad.net/~unity-team/unity8/rtm-14.09-staging/+merge/244568 | 10:28 |
Saviq | pstolowski, and department jumping will have to wait for our rtm silo 7 to land first | 10:29 |
pstolowski | Saviq, sure, it will wait, i'm just collecting / preparing branches | 10:31 |
Saviq | pstolowski, got it | 10:31 |
Cimi | tsdgeos, landing 009 right? | 10:33 |
tsdgeos | Cimi: do not know | 10:33 |
Cimi | found a not-so-painful position | 10:33 |
Cimi | iirc was 9, let's see | 10:33 |
Cimi | yeah | 10:34 |
Cimi | tsdgeos, Saviq might be unrelated, but in silo 9 the manage dash text colors are white on grey | 10:38 |
Saviq | Cimi, right, I was supposed to write to tsdgeos about this | 10:39 |
tsdgeos | pstolowski: ping | 10:39 |
pstolowski | tsdgeos, pong | 10:40 |
Cimi | mmm | 10:40 |
tsdgeos | pstolowski: the count property of navigations, is it *always* avaialble, or only after navigation.loaded is true? | 10:40 |
Cimi | tsdgeos, I see frame skips switching to/from the reddit scope | 10:40 |
tsdgeos | you have good eyes sir :) | 10:41 |
Cimi | tsdgeos, well they are many :D | 10:41 |
tsdgeos | many eyes!? | 10:41 |
tsdgeos | run to a doctor! | 10:41 |
Cimi | lol | 10:41 |
Cimi | skipped frames | 10:42 |
tsdgeos | yes | 10:42 |
tsdgeos | that is known, this branch just makes things better | 10:42 |
tsdgeos | noone said it makes things perfectly smooth | 10:43 |
Cimi | tsdgeos, yes, I mentioned reddit because it seems to be most affected | 10:43 |
Saviq | tsdgeos, so, white-on-grey in Manage Dash header, we'd need the scopes scope to fix that, right? | 10:50 |
Saviq | tsdgeos, also, ETA on the navigation fix? could include in the silo, as have to rebuild anyway | 10:50 |
tsdgeos | Saviq: i guess yes | 10:50 |
tsdgeos | Saviq: the navigation fix is waiting on pstolowski's answer | 10:50 |
Saviq | tsdgeos, k | 10:50 |
=== _salem is now known as salem_ | ||
tsdgeos | i have a thing in the mock that makes the code not work, but not sure if the mock is doing what the real plugin does or not | 10:51 |
tsdgeos | because with the real plugin it seemed to work | 10:51 |
* Saviq will wait for that then | 10:51 | |
pstolowski | tsdgeos, count is the number of child departments of given department model. i think loaded can be false in some cases - "m_loaded = !treeNode->isLeaf() && treeNode->childCount() > 0;" - this is what mhr3 put there, not sure what was the purpose of that property | 10:51 |
tsdgeos | pstolowski: that doesn't answer my question on if count will always be valid or just after loaded is true :D | 10:53 |
Saviq | tsdgeos, I'm not sure about https://code.launchpad.net/~aacid/unity8/scopeListPageHeaderScopeStyle/+merge/244578, do we really want the scope to control the visuals? | 10:53 |
tsdgeos | Saviq: well, that's what they do everywhere, no? | 10:54 |
Saviq | tsdgeos, yeah, but they don't, here | 10:54 |
Saviq | tsdgeos, I mean you overrode the delegates, not actually using CardCreator and such? | 10:55 |
tsdgeos | yes | 10:55 |
tsdgeos | i mean, i just did that because you wanted it :D | 10:55 |
Saviq | tsdgeos, so yeah, the customizations will only control the header colours, which I'm not sure should be the case | 10:55 |
tsdgeos | i can revert it again | 10:55 |
tsdgeos | and you'll complain the header color is not the same header color than in the other scopes | 10:56 |
Saviq | tsdgeos, well, I wanted for the page header to have the correct color, it didn't seem like it did | 10:56 |
tsdgeos | what is "the correct" color? | 10:56 |
Saviq | tsdgeos, the one from designs ;) | 10:56 |
tsdgeos | i.e. the one used by the scopes | 10:56 |
tsdgeos | by not putting it in the scopes we have it in two places which means that they will get off sync easier | 10:56 |
tsdgeos | but i'll discard it and hardcode the colors | 10:57 |
Saviq | tsdgeos, I agree that the default in ScopeStyle seems wrong, simply, as they changed it in, I think, every scope | 10:58 |
Saviq | tsdgeos, so maybe the default is what we should change | 11:00 |
pstolowski | tsdgeos, it looks like count() will only be valid if loaded is true. (however, if it's a leaf, loaded will be false) | 11:00 |
Saviq | pstolowski, yeah, because a leaf has no children, so there's no loading | 11:01 |
tsdgeos | Saviq: you got me lost there | 11:02 |
tsdgeos | Saviq: default of what? | 11:02 |
Saviq | tsdgeos, hardcode for now | 11:02 |
Saviq | tsdgeos, and we'll find out whether we should change the default in ScopeStyle | 11:02 |
pstolowski | Saviq, yeah | 11:02 |
Saviq | tsdgeos, and maybe use it then | 11:03 |
tsdgeos | pstolowski: so loaded will be always false for leaves? | 11:05 |
pstolowski | tsdgeos, looks to me that way, yes. | 11:05 |
tsdgeos | that's terrible :/ | 11:06 |
dandrader | Saviq, could you show me the log with the test errors for https://code.launchpad.net/~dandrader/unity8/greeterRefactoring/+merge/243823 | 11:07 |
tsdgeos | pstolowski: we started to use the count to know if on a "jump" i'm on a leaf or not, but i can't use the count until it's loaded and now you say for leaves loaded is never true, so can't basically use rowCount :/ | 11:07 |
dandrader | Saviq, they pass on my machine | 11:07 |
tsdgeos | which means that fix doesn't work at all | 11:07 |
Saviq | dandrader, sure, sec | 11:07 |
pstolowski | tsdgeos, somehow it worked, i tested it... | 11:07 |
pstolowski | tsdgeos, but i hear you... | 11:08 |
tsdgeos | pstolowski: sure, because i'm ignoring loaded for now | 11:08 |
tsdgeos | but i should not | 11:08 |
* dandrader merges trunk and tries again | 11:08 | |
Saviq | dandrader, it could, in theory, be related to one of the other MPs in the silo, but nothing springs to mind (and having reverted your MP the tests passed) http://people.canonical.com/~platform/citrain_dashboard/#?distro=ubuntu&q=landing-009 | 11:09 |
dandrader | Saviq, so you're not getting these errors locally but on the silo? | 11:09 |
Saviq | dandrader, yes, locally | 11:10 |
Saviq | dandrader, just testing on trunk + your branch now | 11:10 |
tsdgeos | pstolowski: so either we decide that count will always be immediately available (as it seems to be in your plugin but not on my mock) | 11:12 |
pstolowski | tsdgeos, shall i report loaded=true for leaves then? we would need to think what happens when scope says 'no leaves' for deaprtment foo, and then changes its mind later with another search | 11:12 |
tsdgeos | of if we need to make count wait on loaded i need it to be true | 11:12 |
tsdgeos | pstolowski: loaded true for leaves would be nice | 11:13 |
pstolowski | tsdgeos, i cannot make count immediately available (well, it's always available, just 0 if not loaded ;)) | 11:13 |
tsdgeos | pstolowski: honestly i'd replace | 11:14 |
tsdgeos | m_loaded = !treeNode->isLeaf() && treeNode->childCount() > 0; | 11:14 |
tsdgeos | with | 11:14 |
tsdgeos | m_loaded = true; | 11:15 |
pstolowski | tsdgeos, hmm i would be more comfortable setting it true only if isLeaf() is true OR childCount > 0 | 11:18 |
tsdgeos | pstolowski: ok | 11:19 |
Saviq | dandrader, huh, now they passed, as you were, then | 11:19 |
pstolowski | tsdgeos, ok, i'll create branch for that | 11:20 |
tsdgeos | tx | 11:20 |
dandrader | Saviq, \o/ | 11:20 |
facundobatista | Hola! | 11:25 |
Saviq | o/ | 11:32 |
Saviq | tsdgeos, looks like CI is back indeed | 11:32 |
tsdgeos | yep | 11:32 |
Saviq | greyback_, just noticed where `hash -d` beats jump or wd, it actually gets expanded by the shell, so you can use it in all commands and it will Just Work™ | 11:43 |
greyback_ | Saviq: hmmm true | 11:48 |
greyback_ | but I've never missed that before, so I'll stick with jump :) | 11:49 |
Saviq | greyback_, sure, just saying | 11:56 |
greyback_ | Saviq: yep, good find tho | 11:57 |
Saviq | tsdgeos, please let me know when you have the test fix and the colour hardcoding, need to rebuild the silo then | 11:59 |
tsdgeos | yeah working on the test thing | 11:59 |
tsdgeos | it's not a test only change | 11:59 |
tsdgeos | needs reworkingg | 11:59 |
Saviq | tsdgeos, yeah I understand | 12:03 |
tsdgeos | and this think it's very fragile | 12:04 |
Saviq | tsdgeos, not trying to put pressure here, just asking for a ping when you're ready | 12:04 |
tsdgeos | broke it all on the rework | 12:04 |
=== pstolowski is now known as pstolowski|lunch | ||
dandrader | oh, it seems Jenkins has returned from the dead | 12:24 |
dandrader | tsdgeos, does that look familiar? https://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-vivid/203/?#showFailuresLink | 12:25 |
tsdgeos | dandrader: yeah fixing them | 12:25 |
tsdgeos | they snuck while CI was ahving a sleep | 12:27 |
=== dandrader is now known as dandrader|afk | ||
mzanetti | tsdgeos: hey, just accidentally flashed vivid to my dogfooding phone. and now that I see it with many installed apps, I noticed that the issue with disappearing icons on scrolling is back | 13:07 |
greyback | Saviq: could I get a +1 please? https://code.launchpad.net/~mir-team/qtubuntu/gles-sync/+merge/244621 | 13:14 |
greyback | anyone seeing this with building unity8 http://pastebin.ubuntu.com/9528509/ | 13:16 |
Saviq | greyback, no | 13:21 |
=== dandrader|afk is now known as dandrader | ||
=== pstolowski|lunch is now known as pstolowski | ||
tsdgeos | mzanetti: yes | 14:03 |
tsdgeos | mzanetti: there's so much i can do between "don't load everything to not kill the phone" and "things are not there if i scroll fast" | 14:04 |
mzanetti | I see | 14:05 |
tsdgeos | mzanetti: https://code.launchpad.net/~aacid/unity8/moreAsyncDash/+merge/241524 may or may not help | 14:05 |
tsdgeos | Saviq: for the test fix, we need a unity-scopes-shell fix | 14:08 |
tsdgeos | Saviq: should it go into a separate silo or the same silo you're preparing? | 14:08 |
Saviq | tsdgeos, if there's an MP up, let's land it together, pstolowski? | 14:14 |
Saviq | ah it's already in a separate silo | 14:15 |
Saviq | tsdgeos, let's land separately then, I'm just waiting for the manage dash colors then | 14:16 |
tsdgeos | ok | 14:16 |
tsdgeos | Saviq: https://code.launchpad.net/~aacid/unity8/scopeListPageHeaderScopeStyle/+merge/244737 | 14:25 |
Saviq | tsdgeos, thank | 14:26 |
Saviq | s | 14:26 |
tsdgeos | Saviq: https://code.launchpad.net/~aacid/unity8/autopilot_drag_more autopilot passed | 14:29 |
tsdgeos | maybe merge? | 14:29 |
tsdgeos | while we decide what to do with the qt code | 14:29 |
Saviq | tsdgeos, ok, you thinking it's better than before? apart from the backwards gesture? | 14:29 |
tsdgeos | Saviq: well it workarunds the backwards gesture thing | 14:31 |
Saviq | tsdgeos, think it would make sense to abstract to a function? | 14:31 |
Saviq | tsdgeos, the rate calculation at least | 14:31 |
tsdgeos | by making sure we just move in multiples of the move | 14:31 |
tsdgeos | Saviq: one line? | 14:31 |
Saviq | tsdgeos, hmm? 25-26 and 44-50 look the same? | 14:32 |
tsdgeos | they don't | 14:32 |
tsdgeos | one is forward and the other backward | 14:33 |
Saviq | hmm jump and width | 14:33 |
Saviq | ah ok | 14:33 |
Saviq | not like it couldn't be made work, but ok | 14:33 |
tsdgeos | sure, i can move rate, jump and divisions to a function that returns jump and divisions | 14:33 |
tsdgeos | the amount of code would be probably more than less | 14:33 |
tsdgeos | mzanetti: Saviq: one day, we need to code the ubuntushape to png cacher, i feel sad every time i think the billions of battery power we're wasting in shaping icons again and again and again and again | 14:36 |
Saviq | tsdgeos, bug #1311599 indeed | 14:39 |
ubot5 | bug 1311599 in ubuntu-ui-toolkit (Ubuntu) "UbuntuShape should be available as an image provider" [Undecided,Confirmed] https://launchpad.net/bugs/1311599 | 14:39 |
draven33 | need some help with compiz in 14.04 i activatewd dual loghin compoz and metacity an can't align the 3d windows on the cube they form outside the cube | 14:54 |
Saviq | greyback, I must say the global history in zsh trips me up every time | 15:07 |
greyback | Saviq: yep | 15:07 |
greyback | I tried turning it off but didn't succeed somehow | 15:07 |
greyback | but I got more used to ^R to use old commands | 15:07 |
Saviq | indeed, which is probably better in the long run anyway | 15:08 |
greyback | that was my thinking, but I still miss up-up-enter being reliable | 15:08 |
greyback | one of those things I wanted to fix, but never got the time | 15:08 |
Saviq | pstolowski, is https://code.launchpad.net/~stolowski/unity-scopes-shell/manage-dash-rtm/+merge/244117 supposed to be WiP still? | 15:25 |
Saviq | pstolowski, and is anything beyond what's in http://people.canonical.com/~platform/citrain_dashboard/#?distro=ubuntu-rtm&q=landing-007 needed in this silo (apart from the unity8 changes which I'll take care of? | 15:26 |
Saviq | ) | 15:26 |
om26er | rsalveti, this time after taking the photo, I got a crash again https://errors.ubuntu.com/oops/ca4ad358-846d-11e4-b78b-fa163e5bb1a2 | 15:27 |
om26er | and now I am sure I have the right tarball and the silo. | 15:27 |
Saviq | cwayne, hey, question for you, as I saw you had a PPA with galileo (the fitbit sync thing), does it work reliably for you? | 15:30 |
cwayne | Saviq: it did until i got the new fitbit, but i think i may just need to update it | 15:30 |
Saviq | cwayne, my wife got a Flex and it's only really working once after the dongle's plugged in / galileo restarted | 15:39 |
Saviq | cwayne, could be | 15:39 |
cwayne | Saviq: hm, i do recall the dongle needs to be plugged in to start it, but it should also try to respawn if it failed to start | 15:40 |
cwayne | Saviq: but once it's plugged in and restarted, it works? | 15:40 |
Saviq | cwayne, I mean that unless I unplug and replug, it doesn't work, it only works the first time after I plug the dongle and restart the service | 15:40 |
Saviq | cwayne, at least that seems to be the case, did not dig in too much | 15:40 |
cwayne | ah, that is odd | 15:41 |
Saviq | and there's so many warnings/errors in the log it's unclear which are benign... | 15:42 |
tsdgeos | dednick: do you think you could add a second line in the commit message of https://code.launchpad.net/~nick-dedekind/unity8/lp1385331.led/+merge/241417 explaing a bit more what's going on? i mean it does what seems quite a bit of refactor in plugins/Unity/Indicators | 15:47 |
pstolowski | Saviq, no, it shouldn't be wip. unity-scopes-shell needs additional fix from the vivid silo you tested today | 15:48 |
rsalveti | om26er: can you always get that crash? | 15:50 |
rsalveti | it seems quite weird, not necessarily related with that silo | 15:51 |
rsalveti | let me flash again and re-test that silo | 15:51 |
om26er | rsalveti, first camera-app crashed for me, then next time gallery-app crashed | 15:51 |
rsalveti | that's weird, shouldn't be related | 15:51 |
om26er | this silo does not feel safe to me | 15:51 |
rsalveti | and victor didn't get any crash | 15:52 |
rsalveti | well, will try to reproduce first | 15:52 |
rsalveti | this is the first time you got the right setup it seems | 15:52 |
om26er | the thumbnailer works in gallery, so the gallery crashed at the stage when it was creating thumbnails, same for camera-app it crashed after it took the photo | 15:52 |
ChrisTownsend | mzanetti: Hey, question about the Unity 8 windows. Is it possible to open a window already maximized, ie, no surface resize after the window is opened? | 15:58 |
mzanetti | ChrisTownsend: not yet | 16:02 |
tsdgeos | dednick: https://code.launchpad.net/~nick-dedekind/unity8/lp1385331.led/+merge/241417/comments/603644 | 16:06 |
ChrisTownsend | mzanetti: Ok. It would be very helpful for XMir apps to do this since resizing is not yet supported between XMir/Mir/Unity8 and opening maximized would be an easier short term solution. | 16:07 |
ChrisTownsend | mzanetti: Right now Xmir apps are basically useless in desktop mode. | 16:07 |
mzanetti | ChrisTownsend: noted. I'll see what I can do | 16:08 |
ChrisTownsend | mzanetti: Thank you! | 16:08 |
tsdgeos | dednick: how do i add/remove messages? | 16:15 |
dednick | tsdgeos: http://paste.ubuntu.com/9530101/ | 16:15 |
dednick | tsdgeos: need to install libmessage-menu-dev python-gi | 16:15 |
tsdgeos | dednick: and how do i remove them? | 16:16 |
dednick | tsdgeos: stop the script running | 16:16 |
tsdgeos | ah, ok | 16:16 |
=== alan_g is now known as alan_g|EOD | ||
Saviq | greyback, still here? could you prep an MP with the sigstop raise in mock AppMan? | 19:15 |
Saviq | (in unity8 that is) | 19:15 |
greyback | Saviq: am here. Ok | 19:15 |
Saviq | greyback, otherwise AP tests hang | 19:15 |
greyback | ah feck yeah | 19:16 |
greyback | Saviq: https://code.launchpad.net/~gerboland/unity8/sigstop-emit-for-AP/+merge/244781 shoulddo, not tested yet tho | 19:22 |
Saviq | greyback, why not in AppMan's c'tor? | 19:23 |
Saviq | just asking, not saying it's wrong | 19:24 |
greyback | Saviq: mainly to match what I did in qtmir | 19:24 |
Saviq | kk | 19:24 |
greyback | don't think it's possible to copy AppMan | 19:25 |
=== salem_ is now known as _salem | ||
dkessel | ChrisTownsend: good news. I cannot reproduce the freezing bug anymore with current unity8-in-lxc! i am actually currently using it | 21:16 |
ChrisTownsend | dkessel: Awesome, glad to hear it! | 21:25 |
dkessel | i cannot enable automatic crash reporting using the UI from lxc it seems | 21:32 |
ChrisTownsend | dkessel: I don't think there is automatic crash reporting yet in Unity 8. | 21:32 |
dkessel | ChrisTownsend: oh okay | 21:33 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!