[06:57] Saviq: i see my branch has been merged... is it already on the image? [06:57] zbenjamin, http://people.canonical.com/~ogra/touch-image-stats/211.changes [06:58] zbenjamin, not yet apparently [06:58] zbenjamin, the next one [06:58] Saviq: :/ [06:58] zbenjamin, ah no [06:58] zbenjamin, yeah, it's there in 211 [06:58] zbenjamin, sorry, eyes didn't wake up yet :) [06:58] Saviq: but not on the RTM image i guess [06:58] zbenjamin, no, it's in a silo for that [06:58] zbenjamin, awaiting QA sign-off [06:59] zbenjamin, http://people.canonical.com/~platform/citrain_dashboard/#?distro=ubuntu-rtm&q=landing-001 [06:59] Saviq: ok because thats what i use , on the devel-proposed image the click packagage signing checks are enabled and make it impossible to install click packages from the SDK [06:59] zbenjamin, I believe that was pulled back again [07:00] Saviq: ok then i'll check that later [07:00] zbenjamin, if you get me a .click package I can test out for you, just flashing 211 [07:06] morning [07:08] Saviq: sent you a mail [07:08] zbenjamin, tx, will check in a mo [07:11] Saviq: ping [07:12] i didn't realize your seeLessStuckBottom did so many things different to mine [07:12] in particular you changed expandedCategoryId to expandedCategoryItem which is wrong [07:13] tsdgeos, was not me, was Daniel actually [07:13] tsdgeos, http://bazaar.launchpad.net/~unity-team/unity8/seeLessStuckBottom/revision/1128 [07:14] good [07:14] code is mor declarative [07:14] and doesn't work :) [07:16] shall i simply revert the whole change? [07:17] tsdgeos, what doesn't work there? I didn't notice any wrong behavior when testing? [07:17] Saviq: well, storing the pointer is wrong [07:18] you can't compare pointer delegates [07:18] because the next delegate represent the same category will obviously have a different pointer [07:18] tsdgeos, right, so what's broken is expansion across searches? [07:18] that's why we store the category id and not the pointer [07:18] Saviq: and expansion across scrolling [07:19] expand, scroll, scroll back, broken [07:19] tsdgeos, right [07:19] that might get fixed once/if we merge the branch i did to keep all the categories created [07:20] yeah, but not across searches [07:20] * Saviq was never fond of the persistence of the expansion TBH [07:20] across searches that is [07:21] zbenjamin, status in bug #1361349 will tell you what's in RTM btw [07:21] bug 1361349 in url-dispatcher (Ubuntu RTM) "Should open scope:// URIs through unity8-dash" [Undecided,In progress] https://launchpad.net/bugs/1361349 [07:21] zbenjamin, hmm and I didn't get no email? [07:21] ah [07:22] junk [07:22] Saviq: so you consider my click package junk? ;) [07:22] zbenjamin, thunderbird does [07:23] Saviq: so where do you want that reversion? in a new separate branch or is part of two_see_more_bugfix ? [07:24] zbenjamin, yeah, pkcon install-local installed it fine [07:24] tsdgeos, the same, just name it "three_see_more_bugfix" ;) [07:25] ok [07:33] Saviq: awesome thx for testign [08:05] tsdgeos, isn't like the indentation incorrect here? https://code.launchpad.net/~aacid/unity8/dashOverviewFromTempScopePreview/+merge/232378 [08:05] Cimi: no [08:05] unless you mean that the whole list is [08:05] then yes [08:06] tsdgeos, yes [08:07] tsdgeos, btw, sorry for missing the expansion behaviour, I didn't look too closely as Daniel top-acked it already [08:07] tsdgeos, sounds like we need a test, but let me try and verify that that's still the behaviour we want (persistency across searches that is) [08:08] tsdgeos, btw, your popover test fix didn't help apparently :/ [08:08] yeah i saw [08:08] can try making that last test a trycompare [08:08] will do in a bit [08:15] tsdgeos, so you basically use valgrind to slow stuff up during a test run locally? [08:15] that's what i did yesterdya yeah [08:15] nice ;) [08:15] sometimes it is unworkable [08:16] since it takes too much [08:16] and then tryCompare fails [08:16] because takes more than 5 sec [08:17] you know what? [08:17] the test fails all the time here now :D [08:26] Saviq: and the test is righfully changing [08:27] tsdgeos, oh interesting [08:27] something's bringing back the focus to the text field after clicking in a popover result here in tryPageHeader [08:28] and from what i can see it isn't us [08:31] now what made this break, i do not know [08:33] the pluin is from 26th [08:33] pluin -> SDK [08:33] so may well be that [08:40] Saviq: yeah SDK [08:41] tsdgeos, yay : [08:41] they made their popup destruction restore focus [08:41] which i can see how it makes sense [08:41] but broke us [08:43] i think i can workaround, let me see [08:43] tsdgeos, since when do bindings with functions introspect the function body ?¿ [08:43] they don't afaik [08:44] it may just happen that you get the binding updated for some other reason [08:44] tsdgeos, http://pastebin.ubuntu.com/8167099/ [08:44] tsdgeos, that's due to find*Child [08:44] :S [08:44] or actually findInvisibleChild [08:44] weird [08:44] because findChild looks at children [08:45] Saviq: no, can't workaround it [08:46] tsdgeos, I think I saw the bad behaviour too [08:46] but can't pinpoint when [08:46] Saviq: shall i do a quick MR against SDK adding a "propery bool restoreFocus: true" to Popup? [08:47] tsdgeos, hmm, why would it restore focus? [08:48] Saviq, just pushed the fix for the "white-bar on top of sim-unlock snap-decision" to my branch... I guess it's too late for the silo, right? [08:48] MacSlow, it landed already, yeah ;) [08:48] Saviq: because is the new default, i jus want to expose the property so we can set it to false [08:48] MacSlow, but that's fine, we'll get another one [08:49] Saviq, I could swear I had that fix pushed before... I don't remember seeing it when I did local tests... no idea how that slipped [08:49] tsdgeos, looks like https://bugs.launchpad.net/ubuntu-ux/+bug/1353966 ? [08:49] Ubuntu bug 1353966 in Ubuntu UX "[SDK] Slider does not work in a List View" [Medium,Triaged] [08:50] hmm [08:50] Saviq: no? [08:50] tsdgeos, no ;) [08:50] Saviq, I'll to a "paranoid" check of all of my other branches... I don't want to be surprised like that again :) [08:50] aargh devel branches ffs [08:50] Saviq: https://bugs.launchpad.net/bugs/1234982 is the bug it tries to fix [08:50] Ubuntu bug 1234982 in ubuntu-keyboard (Ubuntu) "[osk] Keyboard does not auto-hide when no longer needed" [Undecided,Confirmed] [08:51] don't see how it helps [08:51] but it's the bug number attached to the change [08:52] tsdgeos, http://bazaar.launchpad.net/~ubuntu-sdk-team/ubuntu-ui-toolkit/staging/revision/1211 then [08:52] Saviq: yes, that one [08:52] see the change in popupbase [08:53] tsdgeos, mhm, talk to zsombi please [08:54] tsdgeos, or well, if we create the popup before we focus (but we do create it on focus don't we?) [08:54] oh yay, got a popup in the middle of the screen now [08:54] because it restored focus... uugh [08:55] Saviq: yes, we create it on focus, only way to know we have to create it [08:55] tsdgeos, mhm, I'm not totally sure I agree with that behaviour [08:56] but maybe I don't know everything === alan_g is now known as alan_g|afk [09:42] Saviq: our qmluitests take 2h in CI ?¿ [09:43] tsdgeos, yup, an hour to install deps / build + 40 sthg to run the tests :/ [09:43] :/ [09:43] tsdgeos, I have plans to improve that, but time... [09:43] sure :D === alan_g|afk is now known as alan_g [09:55] Wellark, huh, is it expected that one of the SIMs goes "no signal" when the other is on call? [09:57] * Saviq gets No signal all the time on one of the SIMs, need to swap them... [10:07] hmm so the no signal is a lie [10:09] Saviq, looks like we might have indeed issues with the carousel and card size [10:09] Saviq, i was trying with scope tool [10:09] Saviq, do you have a category handy I can override? [10:09] I don't have enough music to show me carousel [10:10] Cimi, just get more music? [10:10] Cimi, or override any random category? [10:11] Saviq, yeah but some others are broken too [10:11] Saviq, the carousel seems to clip [10:11] so I was wondering what we want to support [10:11] Cimi, yes, because it only supports overlay [10:11] Cimi, that's fixed in the scope already [10:11] Cimi, don't look at carousel, that is fine [10:12] Saviq, I tried this in grooveshark http://paste.ubuntu.com/8167638/ [10:12] Cimi, look at when you tap on a result and open [10:12] Cimi, carousel is fine, leave it [10:12] ok [10:12] this is broken btw [10:12] shows text clip [10:12] yes, because it's not overlaid [10:12] when it should be [10:12] carousel assumes there's nothing below the card (per design) [10:13] Cimi, and that's fix committed to the scope already [10:13] Cimi, what is not fine is the single large card when you open an artist search [10:15] Saviq, how do I open an artist search? [10:15] Cimi, just tap on an artist in music (the tool does not support canned queries, so in the real thing) [10:17] I don't have artists?? [10:17] I have few coldplay albums [10:17] I tap it opens albums [10:17] oh maybe I need two artists [10:17] easy joke incoming "Coldplay don't produce art thus no artist" [10:20] ;) [10:24] Saviq: yes [10:24] j/k it's like when Skrillex added this sample at the end of one of his songs https://www.youtube.com/watch?v=Gs3ocG5yW88 [10:24] but the signal strength should return [10:24] when you end up the call [10:24] Saviq: you can still check with awe and abeator [10:25] Wellark, yeah weird thing is stuff works fine, but the indicator just goes to no signal for no reason [10:25] Saviq: it goes to no signal if ofono tells it to go no signal [10:25] Saviq: so A) there really is no signal [10:25] Wellark, sure, I understand, I didn't say that's your fault [10:25] B) ofono is not updating the signals properly :) [10:26] Wellark, just wanted to know your experience [10:26] Saviq: your experience matches mine [10:26] except that the signal should recover once you hang up the phone [10:26] sure it did [10:26] but then it was switching back'n'forth after that [10:26] even though everything seemingly worked [10:27] * Saviq wonders if the names should be mapped to SIM serial numbers... [10:27] Saviq: the back and forth is another bug [10:27] which will get fixed on the next ofono planging [10:27] Wellark, oh ok, yeah, my 2nd SIM does that, regardless which SIM I put in [10:28] I mean I swapped them and it's still the second one that jumps about [10:29] Saviq: we're hitting a few asserts on tests when running on a debug qt (i.e. one that has asserts enabled) [10:29] :( [10:29] QFATAL : qmltestrunner::Dash::test_temp_scope_dash_overview_all_search_temp_scope_favorite_from_all() ASSERT: "e->isAccepted()" in file items/qquickitem.cpp, line 4490 [10:30] tsdgeos, oops [10:30] huh, interesting [10:30] which is ultra weird [10:30] since it's a keyevent [10:30] and i don't see us having key events on that test [10:30] it seems the second SIM slot is slow for data... [10:30] Saviq: yes [10:30] Saviq: i think someone mentioned 3g is only available on first [10:31] rings a bell [10:31] well then the indicator is wrong [10:31] says H still [10:35] ask around, my bell may be broken :D [10:44] * Saviq sends stupid photos around, wonder when the sensor will break [10:50] 006087 [10:51] * Saviq managed to remember a 16-random-char password for U1 SSO... #brainisstupid [10:51] Saviq, I put another artist, no luck [10:51] Cimi, search for it? [10:52] Cimi, what do you see in the Music scope? [10:52] albums [10:53] Cimi, that's Music or My Music? [10:54] music [10:54] Cimi, where are those albums from? [10:54] local [10:54] here [10:54] Cimi, and it says "My music" above them? and there's Grooveshark results below? [10:55] yes [10:55] Cimi, I only get artists there, not albums... [10:56] Cimi, do you have tags in your music at all? [10:56] Saviq, yes I have artist [10:57] albums, genres, tracks [10:57] inside my music navigation [10:57] Cimi, I've artists, albums, genres, tracks... [10:58] Cimi, so your music selection must really not sit together with the media scanner ;P [11:02] Saviq, so can you tell me what I can put in scope-tool? [11:02] pls? [11:02] Cimi, I don't think you'll get the exact result with the scope tool as you can't override the result data [11:02] Cimi, in card tool [11:02] I'll unassign myself for now then [11:03] I copied 3GB of music [11:03] still no artist [11:04] Cimi, http://pastebin.ubuntu.com/8168021/ [11:04] trying thaty [11:04] Cimi, and see that implicitHeight is wrong [11:04] tryCard? [11:06] ah basically we want to not allocate height for summary here>? [11:10] Saviq: http://paste.ubuntu.com/8168055/ [11:13] Cimi, implicitHeight should always be the real, minimum, height of the card (no forced height) [11:13] Cimi, because it's used in vertical journals where there's no "allocation" happening [11:14] Saviq, it is weird because we have text: cardData && cardData["summary"] || ""; \n\ height: text ? implicitHeight : 0; \n\ [11:14] Saviq, maybe we need text != "" ? [11:14] on height? [11:16] Cimi, well, text === "" in that case [11:16] Cimi, that seems to be the problem === _salem is now known as salem_ [11:18] starting unity8/trunk gives me a black window and some warnings about a lightdm mock module not being found [11:21] does anyone have an idea what could be going on? [11:22] Holas [11:26] larsu, how are you starting it? [11:27] larsu, ./build.sh and ./run.sh are best set up to take care of it now [11:27] Saviq: running the binary directly (but setting QML2_IMPORT_PATH to what run.sh sets it to) [11:27] run.sh doesn't work for me, because I don't have unity8 installed on my system [11:28] larsu, you need LD path too [11:28] larsu, just copy data/unity8.conf to ~/.config/upstart [11:28] hm? Doesn't qml load .so files directly? [11:28] oh, interesting [11:28] that makes it work, thanks [11:29] larsu, if you care about the dash, copy unity8-dash.conf, too [11:29] oooh, colored indicators [11:29] larsu, ;) [11:30] but note that by default the dash that launches with ./run.sh is the system-wide one (so it will fail in your case), we need to fix the run script to support the split-dash situation [11:30] larsu, means you got the mock indicator backend [11:30] dash works for me [11:30] seems to be a mock as well [11:30] larsu, just an image ;) [11:31] see the SURFACE part? ;) [11:31] larsu, yeah, we can't do window management under X11 ya know [11:31] ya, makes sense === MacSlow is now known as MacSlow|lunch [11:32] I'm just trying to test whether doing s/StatusIcon/Icon works [11:33] larsu, not sure why you'd get mock indicators, though :/ [11:34] Saviq: a stray QML2_IMPORT_PATH. Now I don't get any indicators and the system's dash [11:35] unless you copied unity-dash.conf to upstart and you don't have unity8 installed, there can be no dash as upstart doesn't know the job... [11:35] unity8-dash [11:35] and even if you did copy it [11:35] the job doesn't know how to run the non-installed dash [11:35] I copied it [11:35] so should fail [11:35] I have a dash though [11:35] larsu, inside unity8? [11:36] ya... [11:36] larsu, that's just an image [11:36] mzanetti, gonna kill ya ↑ [11:36] we need to replace that image :P [11:36] ? [11:37] nah... [11:37] I like it :) [11:37] mzanetti, I'm gonna defer people to you every time someone mentions a dash inside unity8 under x11 :P [11:37] we might stamp a red "Facsimile" on it though [11:37] argh, now it stopped working again [11:37] mzanetti, we have a yellow SURFACE stamped on it now [11:37] aanyway [11:38] * Saviq goes to try out BT in the car + /food [11:38] "unity8 start/running" - but no window [11:38] Saviq: can you pair something now? [11:38] Saviq: bon appetit [11:38] larsu, .cache/upstart/unity8.log [11:38] Saviq: I still can't pair anything BT [11:38] mzanetti, it's supposed to work, will find out soon (don't have anything BT at home other than laptops and phone) [11:38] s [11:39] that would be awesome [11:39] * Saviq wonders how will google sheets behave with no internets [11:39] Saviq: they don't [11:39] oh neat: *** Error in `/home/lars/src/unity8/trunk/builddir/src/unity8': corrupted double-linked list: 0x0000000000ef3a10 *** [11:39] just don't [11:39] hmm [11:39] mzanetti, I have the sheet opened, didn't complain in plane mode yet ;) [11:39] also, a binding loop [11:40] larsu: what are you trying to do? [11:40] mzanetti: run unity from trunk (just shelved my changes) [11:40] larsu: shouldn't crash then :/ [11:40] larsu: you using ./run.sh? [11:40] yep [11:41] hmm... well, sometimes I have such crashes when it doesn't rebuild everything. eg. after shelving [11:41] larsu: try a ./build -c [11:41] if its still crashing then, we need to debug [11:41] k [11:44] mzanetti: same crash [11:44] meh [11:44] I wonder why this happens now - it worked 10 minutes ago [11:44] yeah... weird [11:44] there's a bunch of image provider warnings as well [11:45] they shouldn't be fatal though [11:45] for the launcher icons [11:45] no, they aren't. The fatal one is the currupted double linked list error [11:45] works in gdb of course :( [11:46] ... [11:46] * mzanetti tries current trunk [11:46] working fine here [11:47] hmm...that's not good [11:47] might be related to copying unity8-dash.conf to ~/.config/upstart? [11:47] * larsu tries without [11:48] indeed [11:48] I still get the dash with my currently running apps though [11:49] larsu: I can't repro even with copying the file over [11:49] what's the currently running apps? [11:49] it's a heisenbug. I'm getting it again now [11:49] mzanetti: gnome-terminal, firefox, thunderbird [11:50] larsu: how are they related to unity8? [11:50] I don't know? [11:59] mzanetti: got it in gdb now: http://paste.debian.net/plain/117983 [12:02] hmmm plugins/Unity/Indicators/rootactionstate.cpp:56 [12:02] * mzanetti looks [12:03] looks like a noreml connect call to me - maybe @menu is dangling? [12:03] yeah... [12:03] m_menu is set from the outside using setMenu() [12:03] maybe the outside deletes it at some point [12:03] right [12:03] dednick: hey, can you maybe have look? ^ === alan_g is now known as alan_g|lunch [12:04] dednick: first guess would be m_menu is parent-less and QML engine garbage collects it [12:05] well, assuming the pointer ends up in QML at some point [12:05] yeah. its a Q_PROPERTY(). so most likely that's it [12:06] dednick: thanks for the trace, seems helpful [12:06] err larsu ^ [12:06] mzanetti: hm. i remember something similar with the actions i think [12:08] mzanetti, dednick: manually setting a parent in that function doesn't resolve it though [12:09] I think the ownership is determined when it first reaches the qml context [12:09] and it can't be changed? [12:09] and as setMenu is called *from* QML that ship has sailed at that point [12:09] yes it can be changed [12:09] but qml doesn't notice? [12:10] yes, it does, but doing that here would probably be too late already (as the connect below crashes already) [12:10] and even if it would work, it would be a bad hack [12:10] but if you want to try, its something like setOwnership in QQmlContext iirc [12:11] yeah it's a hack - I'm just trying to suport your hypothesis :) [12:11] larsu: http://qt-project.org/doc/qt-5/qqmlengine.html#setObjectOwnership [12:11] hm, I don't have acces to the qmlengine in here though [12:12] huh. looks weird to me. if the menu was destroyed, why is it being set? [12:13] dednick: mzanetti's idea is that it is set and GCed right after [12:13] right after what? [12:13] setting it [12:13] but it's crashing in the set. [12:14] yeah... which doesn't line up I just realize [12:14] ah right. Is the GC running in the same thread? [12:14] hmm... actually [12:14] #11 0x00007fffd0b3087e in RootActionState::setMenu (this=0xcab270, menu=0xcfd530) [12:14] no it's not [12:14] so it might stil be that [12:14] but judging by how often I see it, I doubt it [12:14] it would need to be gced at exactly the right time every time [12:14] larsu: it doesn't always crash for you, right? [12:15] mzanetti: no, but more than half the time [12:15] can you try qDebug() << menu->parent() [12:15] in gdb a bit less often [12:15] would be interesting what it says in the non-crashing case [12:15] mzanetti, hah! it did work (sheets with no wifi) [12:15] sure [12:15] mzanetti, it only saves the changes after you press "ready" [12:16] Saviq: last time I tried it it broke the whole doc [12:16] mzanetti, so I was able to do the changes and it synced them up [12:16] mzanetti, wfm [12:16] mzanetti, but, BT, did not, my car kit requires a reverse pairing (it's the car kit that initiates and says what pairing code to use) [12:16] mzanetti, so it found the phone and as soon as it tried to connect it failed [12:17] hah, note != comment [12:18] larsu: has this only been recent? [12:18] mzanetti: gcc complains that unitymenumodel only has a parent(QModelIndex &) function [12:18] dednick: haven't run unity in a while [12:18] this must have to do with lp:~nick-dedekind/unity8/lp1328646 [12:18] dednick: just tried it and hit this error [12:18] now that's strange [12:19] messed around with the object ownership of models [12:19] is qasbatrctitemmodel not a qobject? [12:20] it is [12:20] and it should have a QObject* parent() method [12:21] i wonder if something is trying to access it's parentItem, which it doesnt habve [12:21] but is accessed through the qml parent property [12:22] larsu: is it latest image? [12:22] dednick: it's unity8 trunk on my laptop [12:22] larsu: ah. running desktop shell in window? [12:23] ./run i mean [12:23] ./run.sh [12:24] mzanetti: http://paste.debian.net/plain/117987 [12:24] dednick: yes [12:25]