=== duflu_ is now known as duflu === daek_ is now known as Daekdroom [03:53] https://code.launchpad.net/~unity-team/unity8/new-scopes-fix-grid/+merge/202593 [03:53] https://code.launchpad.net/~unity-team/unity8/new-scopes-fix-filtergrid/+merge/202594 [03:59] * karni eod [09:15] didrocks: hey, just writing the test plan for unity8 [09:15] didrocks: "Ensure that all AP tests are passing." [09:15] can't we have that automated? [09:16] mzanetti: there is andy's job I mentionned yesterday (ping doanac once he's around on the CI), it's a self-service testing [09:16] mzanetti: but for the rest, we need the CI team to provide an automated service for it [09:16] so, please write your queries down ;) [09:16] Cimi, ping? [09:16] where? [09:17] mzanetti: I guess email ev, asac and I [09:17] ok [09:17] (well, just CC me, I'm not in the CI team or in that chain of operations ;)) [09:38] . [09:40] yo [09:40] mhr3, pong [09:41] Cimi, just wondering, you do know about the design spec for the preview components, right? [09:41] there is a doc [09:42] wondering if anyone can point me in the way of getting 3 monitors going. I tried and failed again. [09:42] paulliu, you saw the doc as well, right? === rachelliu_ is now known as rachelliu [10:18] and otto is broken again [10:19] * tsdgeos goes to the crying corner === _salem is now known as salem_ === salem_ is now known as _salem [11:08] sil2100, hey, any idea what this is about? https://launchpadlibrarian.net/163037120/buildlog.txt.gz [11:12] mhr3: hi! Is it reproducible? [11:12] sil2100, yes, retried it now, just got a mail that it failed [11:14] mhr3: do you know what i need to run the unity-scope-tool ? demo-stuff ppa and some registry stuff i remember? [11:14] didrocks, you're bzrlib guru, seen something like that before ^^^? [11:15] mhr3: hum, this is bzr-builder itself. I got some in the paste and never got that worked done. That's one of the reason I enforce split mode and so, don't use recipes === _salem is now known as salem_ === alan_g is now known as alan_g|afk [11:57] tsdgeos, are those common failures that happen on many MPs? https://code.launchpad.net/~dandrader/unity8/draghandle_lp1269022/+merge/202484/comments/472332 [12:00] mhr3: yes. === alan_g|afk is now known as alan_g [12:05] dandrader: otto is silly [12:05] fails, then works, then fails again [12:05] noone knows why [12:05] dandrader: at the moment we are in the fail stage [12:08] Cimi: hi. You have been testing the side stage stuff lately, right? [12:09] Cimi: if you launch a sidestage app on the tablet, does that work properly? [12:11] tsdgeos, btw, would you have time to review it? https://code.launchpad.net/~dandrader/unity8/draghandle_lp1269022/+merge/202484 [12:17] sure [12:44] mzanetti, yes [12:44] mzanetti, it works [12:44] mzanetti, not 100% fine but runs [12:44] tsdgeos: Added comment - https://code.launchpad.net/~unity-team/unity8/new-scopes-fix-grid/+merge/202593 [12:45] Cimi: here, if I have only one side stage app running, the main stage is black, and swiping it to the right keeps the app where it is [12:45] tsdgeos: Any hints how would you unit test that? [12:46] karni: but compile unity8 trunk [12:46] and you'll see the bug is not there [12:46] mzanetti, yes, mine stage is black [12:46] karni: someone changed that made stuff be not centered [12:47] tsdgeos: The code was wrong, mathematically. I don't understand how that would be centered in unity8 without that chage :O :S [12:47] *change [12:47] karni: well, just look at the phone, it is centered [12:48] tsdgeos: Do you mean latest trusty-proposed? [12:48] yes [12:48] without demo-stuff [12:48] tsdgeos: It's not. That's where the screenshots came from. I flashed it before applying demo-stuff and applying them. [12:48] oh [12:48] now were getting somewhere [12:49] that's not what the comment says [12:51] tsdgeos: What did you mean by that last sentence? [12:51] your comment says [12:51] "The problem exists in both. [12:51] Screenshots above show latest trusty-proposed with demo-stuff ppa applied. [12:51] Screenshots below show same problem in unity-scope-tool:" [12:52] that is both have demo-stuff applied, no? [12:52] tsdgeos: Correct [12:52] well, if you consider unity-scope-tool having demo-stuff applied (you know what I mean), then yes [12:53] well, unity-scope-tool is only in new-scopes which is what demo-stuff gives you [12:53] so yes, i consired it the same [12:53] right [12:53] FTR just confirmed with Katie left alignment wasn't intentional. [12:53] tsdgeos: So you're saying I should flash clean image of trunk, and check it out? [12:54] yes and no [12:54] you can't run unity-scope-tool in "plain" unity8 [12:54] but what "yes", if you run unity8 and check the default dash scopes [12:54] that have 3 columns in the grid [12:54] they are perfectly aligned [12:55] karni: no need to flash the phone, just run it on your desktop [12:56] * karni investigates [12:56] lunch, back later === alan_g is now known as alan_g|lunch === dandrader is now known as dandrader|afk [13:37] Cimi: you pinged me yesterday [13:37] Wellark, yup [13:37] Wellark, I am fine at the moment :) [13:37] I assume you are trying to write an AP test for connecting to a wifi network [13:38] I would advice against it ATM, it's way too complicated to do properly [13:39] connecting to a wifi network requires actual HW to be available [13:39] or you have to mock all the relevant network-manager classes [13:39] I will get there eventually, but blocking mir merges just for that AP test is IMO unreasonable === dandrader|afk is now known as dandrader [13:40] Cimi: ^ [13:40] Wellark, there is a mock [13:40] Wellark, python-dbusmock [13:41] Cimi: which handles secret agents, etc correctly? [13:43] Wellark, don't know [13:43] Wellark, let's move to ubuntu-touch [13:43] Wellark, ask pitta there [13:43] Wellark, pitti === alan_g|lunch is now known as alan_g [13:56] karni: did you run trunk unity8 on your desktop? [13:57] tsdgeos: yes. and I think it's related to delegate size, but I haven't confirmed that yet. I'm adding tests for collapsedRowCount in FilterGrid. [13:57] cool [13:57] karni: do you think you could try to do this work directly over trunk instead of over the new-scopes branch? [13:59] tsdgeos: I asked Scott if he has his scope working with the old API. If he doesn't, I'd have to dedicate (significant?) time reproducing the problem on trunk. [13:59] But I understand where your question comes from. [13:59] karni: but why would this be scope dependant? [13:59] scopes just give data [13:59] the grid view layout should be the same for all, no? [14:00] tsdgeos: It should. Let me rephrase that. I think it's actually the delegate size that influences the spacing/marings. Because they're small in trunk, one can't reproduce the issue. [14:00] They're significantly larger in new unity8, thus easily reproducible. [14:01] Yes, grid view should be same in both. [14:01] karni: but you could still reproduce it in a test [14:01] tsdgeos: I'll try that [14:01] since there you can set any size for the delegates you want [14:01] perfect [14:15] tsdgeos: Is it legal for me to change API of qml/Component/FilterGrid? i.e. I believe 'collapsedRowCount' (which is settable) should not reflect real collapsedRowCount, when the model.count would simply not fill up the visible rows when collapsed. In other words - either I would introduce "maxCollapsedRowCount" and use collapsedRowCount to indicate *real* value of rows, OR use 'collapsedRowCount' as is (no API change) and internally keep a ... [14:15] ... smaller or equal value to collapsedRowCount elsewhere. [14:15] I hope it was clear what I said. [14:17] karni: i am not sure :D [14:17] I see. [14:17] karni: i guess it should be 1 instead of 2 when there's not enough items [14:17] That's right [14:17] but probably ask katie about it? [14:17] or 2 instead of 3 if there's not enough items [14:18] tsdgeos: Don't get me wrong. It's not a design decision - if we have "collapsedRowCount: 3" (say, 2 columns) and we have 2 items, it's clear we just display 2 items instead of 2 items and 4 blank spaces (2 empty rows) [14:18] What I mean is, the problem appears when there's only 2 elements in a filter grid that specifies collapsedRowCount: 2 [14:19] which is more than the model contains, and UI goes nuts [14:19] i got lost :D [14:19] though - I still need to reproduce that. [14:19] tsdgeos: quick mumble? [14:19] sure, give me a sec [14:19] thanks. 1 min, be right there [14:20] * karni is there, not sure which room to choose haha [14:31] tsdgeos: small success. FilterGrid goes nuts also in testFilterGrid, when I set maximumNumberOfColumns to 2 instead of 3, the layout goes from 3 columsn to 1 (!). So I guess I'm getting somewhere. [14:32] cool \o/ [14:32] Cimi: standup? [14:32] tsdgeos: That's regarding the rendering bug (latter MP), not the grid spacing, just yet. [14:32] one at a time! [14:32] inded [14:32] *indeed :D [14:42] Cimi: can you stay after standup for a moment? [14:45] can I get a quick review for a one-line change? [14:45] https://code.launchpad.net/~elopio/unity8/wait_for_category/+merge/202523 [14:46] elopio: want me to add you to the google calendar standup invite? [14:46] tsdgeos: kevin already added me. [14:46] ah [14:46] okidkoi [14:48] mzanetti: can't we have autolanding for trunk, and then manual promotion from trunk to the current version? [14:48] elopio: no [14:49] it sounds terrible that for a branch like mine, your machine will be stuck running for 3 hours. [14:49] elopio: I know [14:49] mzanetti: so you get to dry-run the manual-landing with https://code.launchpad.net/~elopio/unity8/wait_for_category/+merge/202523 :D [14:49] * tsdgeos hides [14:49] * mzanetti hides too [14:49] where should I complaint? didrocks? [14:49] elopio: I think asac [14:50] tsdgeos: that commit is for autopilot. we don't need it any more :P [14:52] I'll find some more townsmen in rage and go to him with torches. [14:57] didrocks: what are the MP guidelines in order to set the "Ready" field to yes? [14:58] ah found it. nvm [14:59] elopio, tsdgeos: does the branch comply with this checklist? https://wiki.ubuntu.com/Process/Merges/Checklists/Common [14:59] it's too long [14:59] i'm going to say no [14:59] :D [14:59] when it is, set the ready field in here to yes and come back to me: https://docs.google.com/a/canonical.com/spreadsheet/ccc?key=0AuDk72Lpx8U5dFlCc1VzeVZzWmdBZS11WERjdVc3dmc&usp=drive_web#gid=0 [14:59] yeah, no. It's not in sync with trunk because it cleanly merges into it. [14:59] mzanetti: I'm updating... [14:59] so we need to have an arhmf chroot now? [14:59] awesome [15:00] why have automatic testing [15:00] when you can force everyone to waste his time [15:00] this is nuts [15:03] mhr3: available for quick hangout? [15:06] Cimi: I've been reading the autopilot code for indicators. Let me know if you need a hand with that. [15:08] elopio, thanks in case [15:24] pete-woods: ping [15:27] tsdgeos: hi [15:27] pete-woods: just wanted to make sure i added a comment in https://bugs.launchpad.net/libhud-qt/+bug/1271036 in case you were working in a fix [15:28] Ubuntu bug 1271036 in libhud Qt "libhud-qt segfault when building on amd64 against Qt 5.2" [Critical,New] === om26er is now known as om26e === om26e is now known as om26er === dandrader is now known as dandrader|lunch [15:31] tsdgeos: I hadn't actually noticed that one, thanks for sorting it! [15:32] tsdgeos: I presumably would realistically have to wait for some new release of Qt there, though! [15:32] ? [15:33] pete-woods: you just get Mirv to distro-patch the patch [15:33] tsdgeos: ah, cool, will do that then! [15:33] thanks again! [15:33] yw [15:38] tsdgeos: just worked out that Mirv = Timo :$ [15:38] he he [15:39] nicks and names [15:39] that big first world problem :D [15:39] :) [15:39] add faces into the mix [15:39] and i'm lost [15:39] yes, I know what you mean [15:39] even when you've spoken on a hangout [15:41] tsdgeos: also, thought I'd let you know that progress has been made on the quit action [15:41] pete-woods: okidoki [15:43] with any luck, the unity8 side should need no changes === alan_g is now known as alan_g|tea [15:43] just need to do some patching up inside the uitk to listen to a "quit" signal [15:45] pete-woods: it will need for sure, the buttons in the hud bar are hardcoded :D [15:46] and i guess they want the button back [15:46] not just when you search for quit [15:48] tsdgeos: there should still be a "quit" labelled button in the toolbar [15:49] no i killed that long time ago [15:49] okay, fair enough [15:49] so i need to readd it [15:49] and make sure it calls whatever you need on it [15:49] the API should be exactly the same at any rate [15:49] cool [15:50] in the new HUD release, I've also included the Qt API, in-case you're interested [15:51] so if you wanted, you could just hook that straight into the QML, instead of using the one in unity8 [15:52] I've trimmed off anywhere it exposed dee, and that sorta thing === alan_g|tea is now known as alan_g [15:53] karni, i am now [16:01] pete-woods: cool, i'll have a look [16:02] tsdgeos: the plan is really to make the client library simpler / more reliable - I'll probably just made it a pure Qt lib once it's in use [16:09] mhr3: join me? :) https://plus.google.com/hangouts/_/72cpi855g63gciccj1rugpifuc [16:35] elopio: curious...do you have a bug ref for the new AP test for " install a click app with no u1 account associated" === dandrader|lunch is now known as dandrader [16:40] Saviq: are you going to give an update on Unity8 progress in the UE Live call in an hour? [16:41] he's on holiday [16:48] :( [16:48] I'll get him next week then === alan_g is now known as alan_g|EOD === dandrader is now known as dandrader|bbl [19:21] mterry: ping [19:21] kgunn, hello === dandrader|bbl is now known as dandrader [20:58] kgunn: no, that's just on my whiteboard. Do you need a bug for it? [20:59] elopio: no, its ok...was just cross referencing [20:59] on some stuff, saying it is dependent that we provide those tests first [21:00] kgunn: also, you know you can't install an app without u1 account, right? [21:00] kgunn: let me file the bug... [21:00] elopio: ok, " you know you can't install an app without u1 account".... so are you supposed to be ? [21:01] kgunn: no, you shouldn't. [21:01] I'm just saying, nevermind. [21:04] kgunn: https://bugs.launchpad.net/ubuntu/+source/unity-scope-click/+bug/1271722 [21:04] Ubuntu bug 1271722 in unity-scope-click (Ubuntu) "missing autopilot test: try to install an app without u1 account" [High,In progress] [21:04] elopio: hey thanks man === seb128_ is now known as seb128 === salem_ is now known as _salem