/srv/irclogs.ubuntu.com/2013/11/14/#ubuntu-unity.txt

veebersSaviq: if you're still around; I can't build unity8 package due to 2 failing tests. Are you seeing this too?00:45
robruSaviq, oh, sorry, didn't notice that there was already a branch for that. feel free to disregard mine, it's incomplete anyway01:10
robruveebers, ^^01:10
veebersrobru: cool, sounds good to me.01:34
veebersrobru: I was going to mention that my branch includes the tox stuff which eases running the tests with py27 and py301:34
Mirvcopied some newer Qt 5.2 beta1 builds over to qt5-beta2 from my other PPA, should not affect anything but tell me if it does06:53
=== iahmad_ is now known as iahmad
=== iahmad is now known as iahmad|afk
tsdgeosSaviq: so the pass objects to c++ one has been merged \o/08:47
tsdgeoshaven't had any comment in the compare singletons against themselves one yet08:48
=== kgunn is now known as Guest15371
=== iahmad|afk is now known as iahmad
Saviqtsdgeos, nice09:15
nic-doffaySaviq, tsdgeos mind if we pick up about that FilterGrid animation?09:26
nic-doffayFrom where we left off on Tuesday.09:26
Saviqtsdgeos, can you have a look at the height animation in LVWPH with nic-doffay?09:36
tsdgeossure09:36
tsdgeosnic-doffay: Saviq: what's the issue?09:36
Saviqtsdgeos, bug #122455209:37
ubot5bug 1224552 in Unity 8 "[Dash] Category expansion transition has varaible speeds" [High,In progress] https://launchpad.net/bugs/122455209:37
Saviqtsdgeos, and comment #3 in particular https://bugs.launchpad.net/unity8/+bug/1224552/comments/309:38
ubot5Ubuntu bug 1224552 in Unity 8 "[Dash] Category expansion transition has varaible speeds" [High,In progress]09:39
nic-doffaytsdgeos, the animation currently happens in FilterGrid.qml09:39
nic-doffayI was planning to keep it there.09:40
Saviqnic-doffay, not only there09:40
tsdgeosso you want it to expand at a continuous speed?09:40
Saviqnic-doffay, it's there in listviewwithpageheader, too09:40
Saviqtsdgeos, no, we need to expand it to $visible_height in a duration, then to $target_height in one frame09:41
tsdgeosSaviq: hmmm, yes and no, the only thing the LVWPH does is move the Y09:41
tsdgeosnothing to do with the height09:41
Saviqtsdgeos, and the opposite for collapsing - to $visible_height in one frame and then animate to $target_height09:41
Saviqtsdgeos, yeah, but still you put the comment in that they need to happen in sync ;)09:42
Saviqtsdgeos, would it be possible to apply a height behavior on LVWPH's delegates?09:42
Saviqtsdgeos, from LVWPH, I mean?09:42
Saviqtsdgeos, well, maybe I'm overcomplicating things... the only expandable thing we have is the grid...09:43
Saviqso maybe it's fine if we just leave it there09:44
Saviqtsdgeos, the height animation was never in sync with the y animation then anyway - only important thing is that y happens slower than height, doesn't it09:46
nic-doffaySaviq, is there a variable in FilterGrid which holds the shell height already btw?09:51
Saviqnic-doffay, dunno, doubt it09:51
tsdgeosnic-doffay: knowing the height of the shell is not going to help you anyway, lots of stuff to substract in there09:52
nic-doffaytsdgeos, what do you mean?09:55
tsdgeosnic-doffay: i mean we have the panel and stuff09:56
tsdgeosyou don't want to know the shell height09:56
tsdgeosyou want to know the genericscopeheight09:56
nic-doffaytsdgeos, is that just the height of the GenericScopeView09:57
tsdgeosis that a question?09:58
tsdgeosor?09:58
nic-doffaytsdgeos, yeah it is.10:02
tsdgeosthen i don't understand the question :D10:02
nic-doffaytsdgeos, how can I calculate the genericscopeheight, is it stored somewhere?10:04
tsdgeosnic-doffay: well the genericscopeview knows about it10:04
tsdgeosand it's what triggering the filtering10:04
tsdgeosso passing it around should not be that hard10:05
nic-doffaytsdgeos, is the genericscopeheight the genericscopeview height property?10:07
nic-doffayThat's the question basically.10:07
tsdgeosah no10:07
tsdgeosthat's just some crap i wrote10:08
tsdgeosbut it's an item10:08
tsdgeosso it has a .height property10:08
nic-doffaytsdgeos, you mean the GenericScopeView has a .height property, right? Just double checking because I intend to pass that through...10:10
nic-doffayIf I'm understanding you correctly? You mean it's best to use the GenericScopeView height to work out the animation?10:10
tsdgeosnic-doffay: well it's what contains the filter grid10:10
tsdgeosso yes10:10
nic-doffaytsdgeos, cool10:11
tsdgeosthat is what i'm saying10:11
tsdgeosbtw my leg is paining me a bit, sorry if i'm slower or grumpier than usual today10:17
mzanettiSaviq: about QtMultimedia in the test environment. Should I wait for jenkin's opinion or should I mock it straight away no prevent it from breaking at some point?10:47
mzanettis/no/to/10:47
Saviqmzanetti, just mock it, shouldn't be too difficult - not sure we need the mp3s at all, either10:52
mzanettiack10:52
Saviqmzanetti, if you can control the Audio component from the test, shouldn't be needed10:52
Saviqor well, you need to be able to control it, and once you mock the component, you won't need the mp3 ;)10:53
mzanettiyep10:53
Saviqtsdgeos, so, rejecting the 5.2 regression workarounds?10:58
Saviqi.e. https://code.launchpad.net/~aacid/unity8/singleton_52/+merge/19412310:58
Saviqhttps://code.launchpad.net/~aacid/unity-api/52regressions/+merge/19430310:58
tsdgeosSaviq: the unity-api needs the comparison one to land10:59
tsdgeosours is fine if we wait for the release10:59
Saviqtsdgeos, yeah, I know, I'd rather distro-patch than to disable the tests, TBH11:00
Saviqif we need it earlier11:00
tsdgeosok11:00
tsdgeosi'm really confused about this11:07
tsdgeosmy GenericScopeView is getting the "onDestruction" call11:08
tsdgeosbut the ListViewWithPageHeader inside doesn't get the destructor called11:08
tsdgeosany idea why that could happen?11:08
Saviqtsdgeos, only reason I can think of... ownership somehow b0rked? and QML doesn't destroy the LVWPH?11:10
Saviqtsdgeos, no idea why it would be b0rked, since it's a QML type like any other11:12
tsdgeosright11:14
tsdgeosthing is11:14
tsdgeoskind of works sometimes11:14
tsdgeosbut if the test is adding removing them a lot11:14
tsdgeosgets confused and breaks11:14
Cimidednick, ping11:16
dednickCimi: pong11:17
Cimidednick, hey mate11:17
Cimidednick, I was reading carousel code11:17
Cimidednick, and I see pressandhold signal11:17
Cimidednick, however it's triggered only if you stay within 1px11:17
Cimidednick, can you pls try to see if you can click and hold the selected element without moving?11:18
Cimidednick, we might just decide to enlarge the area11:18
tsdgeoswhat do i do manually to test lp:~mzanetti/unity8/fix-switching-previews-positioning ?11:21
dednickCimi: is it working for you at all?11:21
Cimidednick, I'm compiling11:21
mzanettitsdgeos: go to a scope where contentHeight > height11:21
Cimidednick, but I see there's code11:21
mzanettitsdgeos: move it so that a row of items is mostly covered by the header11:21
Cimidednick, it's though only fwd when the click is steady in position11:21
mzanettitsdgeos: open the preview => the items at the bottom should *not* be covered any more11:22
Cimidednick, which might be logically wrong11:22
dednickCimi: i dont think you understand. It's not working for me at all, no matter pressAndHold/click. The click signal is not getting from the Carousel to GenericScopeView because we've put the GenericCaroselLoader in between, which is not forwarding the click.11:22
Cimidednick, AAAAAAAAHN11:22
Cimidednick, ok11:22
Cimidednick, will fix that11:22
dednickthere's also other parameters which are expected to be bound when we're using the grid view, like expanded/filter.11:23
dednickCimi: i think it's probably better not to use the CarouselLoader at all and just decide when to display either carousel / grid in the GenericScopeView.11:24
Cimidednick, I see all now11:24
Cimidednick, indeed it will get complicated unless carousel and filter grid will share common interface11:24
tsdgeoswell11:25
tsdgeosi'm working on that11:25
dednickCimi: yeah.11:25
tsdgeoskind of11:25
tsdgeosbut stuck on other things11:25
tsdgeosso yeah fix it :D11:25
Cimitsdgeos, who are you talking to?11:25
tsdgeosyou guys11:25
Cimiok :)11:25
Cimitsdgeos, what's your ideal solution?11:25
tsdgeosthat = common interface11:25
Cimiok11:26
dednickCimi: you should be able to check the count in the GenericScopeView instead, by making the getRenderer function depend on the count.11:26
Cimidednick, unfortunately I think it depends on the model11:26
Cimidednick, let me dig into11:26
nic-doffayAre we able to land branches again guys?11:26
Saviqnic-doffay, not yet11:26
dandraderCimi, how far did you go with https://bugs.launchpad.net/unity8/+bug/115215011:28
ubot5Ubuntu bug 1152150 in Unity 8 "[DASH] diagonal swipe is recognized as a scroll" [High,Confirmed]11:29
nic-doffaySaviq, this is ready if you'd like to take it for a test drive: https://code.launchpad.net/~nicolas-doffay/unity8/category-transition-speed-fix/+merge/19520311:29
Saviqnic-doffay, k11:29
Cimidandrader, I did not go far: I had a discussion with Saviq and tsdgeos on how we should approach this11:29
Cimidandrader, we decided we should work on low level qt11:29
Cimidandrader, so we stopped going ahead11:29
Cimidandrader, I might unassign myself11:30
dandraderCimi, mind if I take that bug? I have nothing better to do at the moment11:30
Saviqnic-doffay, one nitpick: wrap "root.height = genericScopeHeight;" in { }11:30
Cimidandrader, it's qt bug though11:30
dandraderCimi, I know. I like digging into qt11:30
Cimidandrader, have fun then! :)11:30
Cimidandrader, I addigned you11:31
Cimi*Assigned11:31
=== _salem is now known as salem_
nic-doffaySaviq, sure thing11:34
Saviqnic-doffay, https://code.launchpad.net/~nicolas-doffay/unity8/category-transition-speed-fix/+merge/195203/comments/45063511:43
nic-doffaySaviq, reading11:47
tsdgeosmzanetti: the test is failing for me11:49
tsdgeosFAIL!  : qmltestrunner::GenericScopeView::test_hiddenPreviewOpen() 'verify()' returned FALSE. ()11:49
tsdgeos   Loc: [/home/tsdgeos_work/phablet/unity8/fix-switching-previews-positioning/tests/qmltests/Dash/tst_GenericScopeView.qml(149)]11:49
mzanettiinteresting11:49
tsdgeosmzanetti: can you test?11:50
* mzanetti wonders how he broke that one11:50
mzanettitsdgeos: sorry. forgot to update that one. it was still reading the context prop11:52
mzanettitsdgeos: fixed now11:52
dandraderSaviq, mzanetti what does it mean when a bug has a "affect Ubuntu UX" with a "fix commited" status? Does it mean that there's some design doc specifying it or something?11:58
dandradere.g. https://bugs.launchpad.net/unity8/+bug/115215011:58
ubot5Ubuntu bug 1152150 in Unity 8 "[DASH] diagonal swipe is recognized as a scroll" [High,In progress]11:59
dandraderI would ask the reporter, but he's gone11:59
mzanettidandrader: it menas that design has come to a conclusion on this11:59
mzanettidandrader: but it might be a design spec or a comment on the bug12:00
mzanettidandrader: in this case as Oren reported the bug I think the bug itself is the solution for the UX12:00
dandradermzanetti, hmm, ok. thanks12:02
=== rachelliu_ is now known as rachelliu
nic-doffaySaviq, it needs to take the y of the filter grid itself into account right?12:04
tvossSaviq, test ping12:20
Saviqtvoss, test pong ;)12:23
Saviqnic-doffay, yes, the y in relation to ScopeListView12:24
nic-doffaySaviq,  out of interest did you see this issue on the desktop? I'm not noticing it yet.12:31
Saviqnic-doffay, yes12:31
nic-doffaySaviq, so the best way to see it is to scroll up one row then collapse/expand right?12:31
Saviqnic-doffay, yes, but it's not even about seeing it - just think about it12:31
=== dandrader is now known as dandrader|afk
Saviqnic-doffay, nevertheless it is visible - but it's happening very quickly12:40
nic-doffaySaviq, yeah just wanted to ensure I could see the fix well though.12:44
=== alan_g is now known as alan_g|lunch
Saviqnic-doffay, here's one: http://ubuntuone.com/4AuII1GwHnbNI9C9yearMZ12:54
Saviqnic-doffay, you can see "Files & Folders" disappearing both when expanding and collapsing12:54
Saviqnic-doffay, that's the "you didn't take uncollapsedHeight into account" issue12:55
=== dandrader|afk is now known as dandrader
Saviqnic-doffay, and here's the other one http://ubuntuone.com/5wVUF7Jo3YD8X9Kcs6qHL612:58
Saviqnic-doffay, you can see it blinking when collapsing12:58
Saviqbut also when expanding it sometimes doesn't show all the items12:59
=== alan_g|lunch is now known as alan_g
mzanettimhr3: hi13:30
mhr3mzanetti, hey13:30
mzanettimhr3: looking at the preview spec there are some previews which are not fullscreen13:30
mzanettimhr3: is there already a plan on how to make that possible?13:31
mhr3mzanetti, yea... let's not go there yet :)13:31
mzanettimhm13:31
mhr3tbh i think the design should be changed13:31
mhr3it was under the assumption that those results can't have an image13:32
mhr3but they can13:32
mzanettimhr3: wait. the smaller previews should have an image too?13:32
mhr3design assumed they don't so they created small previews13:33
mhr3that's how i think it went13:33
=== dandrader is now known as dandrader|afk
mzanettimhr3: probably, yes. but so far the dash plugins don't have images except very few ones13:34
mhr3which is a bug13:34
mhr3majority of them do have images13:34
mhr3but there's some issue on the server13:34
mzanettiah ok. but maybe design doesn't even want those13:34
mzanettii for one think the amazon preview would be better off without the screenshot of the amazon website13:35
mhr3maybe13:35
mhr3but those small previews don't consider other-than-phone form factors13:35
mzanettiok. I'll clarify this with design13:35
mhr3therefore they suck :)13:35
=== pstolowski is now known as pstolowski|lunch
=== dandrader|afk is now known as dandrader
mhr3Saviq, i noticed that with my scopes split, unity goes for the mock scopes plugin when you ./run it14:09
mhr3Saviq, ok to just do http://paste.ubuntu.com/6415955/ or would you want something more elaborate?14:09
=== pstolowski|lunch is now known as pstolowski
Saviqmhr3, what's "unity-scopes-qml"? :D14:23
Saviqmhr3, and why would plugindir come from there - shouldn't unity8 (or unity-api, for that matter) define that path?14:25
=== dandrader is now known as dandrader|lunch
mhr3Saviq, it's me being consistent with naming :)14:45
mhr3Saviq, otherwise, pc installed by unity-plugin-scopes14:45
mhr3Saviq, sounds a bit odd that unity8 itself would define it14:46
mhr3although only because you're running it14:46
mhr3otherwise it kinda makes sense14:47
Saviqmhr3, yeah, that's why I said unity-api14:52
Saviqmhr3, it would define the interfaces and the place where to install them14:52
Saviqmhr3, both the implementations and unity8 would find out where to look for those build-time14:53
Saviqfrom the unity-api .pc14:53
mhr3Saviq, yea, that sounds good14:53
Saviqbiab14:53
mhr3so i'll just remove the unity-plugin-scopes' .pc file completely and move this var into unity-api's .pc, k?14:54
mhr3hmm, and maybe i'll keep it14:54
mhr3though not sure who'd need to know its libdir14:55
nic-doffaySaviq, I've sorted out the one issue, but where must the y be taken into account? I've tried testing as much as I can to check for faults.15:02
nic-doffayI'm not seeing anything after fixing the issue you mentioned in your comment.15:02
tsdgeosCimi: ping15:06
=== didrocks1 is now known as didrocks
tsdgeosdednick: ping15:11
dednicktsdgeos: yo15:12
tsdgeosdednick: i've updated https://code.launchpad.net/~aacid/unity8/noScopeView/+merge/194486 with a wait and the crash is gone, it is my current understanding that for reason even if the element has been removed it if it had a pending paint it gets painted again and then it crashes. I think this is something we should investigate but given the amount of changes in the renderloop in 5.1 and 5.2 i've opted for adding a TODO about it15:13
tsdgeoswhat's your opinion?15:13
tsdgeosSaviq: ↑↑↑15:13
Cimitsdgeos, pong15:14
CimiSaviq, was reading of that http://www.tomshardware.com/news/NAND-Flash-Flash-Friendly-File-System-F2FS-Jaegeuk-Kim,18229.html - do we have plans for ubuntu?15:14
tsdgeosCimi: so you're trying to find do a "base item" for the generic scope view items?15:14
Cimiyep albert15:14
dednicktsdgeos: what is the change in plugins/ListViewWithPageHeader/listviewwithpageheader.cpp for?15:16
dednickor just a change in logic pattern?15:16
tsdgeosCimi: ok, i may have been not clear this morning, but i was also doing it before i got sidetracked, this is what i had in Dash/DashBaseRenderer.qml http://paste.ubuntu.com/6416258/15:16
tsdgeosdednick: there's no change?15:17
tsdgeosor you mean the change in the last commit?15:17
dednicklast commit. you removed check for parentContext15:18
tsdgeosdednick: that was the reversal of the previous attempt to not make it crash15:18
dednicktsdgeos: oh right, so you were seeing it earlier? ok15:20
tsdgeosyeah15:20
tsdgeoson the same test15:20
dednickhm. wonder why this has suddenly caused issues15:20
tsdgeosdednick: it's not sudden15:23
tsdgeosdednick: the test was using a fake genericscopeview15:23
tsdgeosnot the real one15:23
tsdgeoswhich now i use15:23
dednicktsdgeos: ah. ok15:23
tsdgeosi can "fix" it15:24
tsdgeosadding gets here and there15:24
tsdgeosbut tbh i'd like to wait for 5.2 and then see if it is still happening15:24
tsdgeosand if it is15:24
tsdgeosinvestigate it more15:24
tsdgeosbecause it doesn't make sense that just adding the wait there it stops crashing15:24
dednicktsdgeos: ok, well if it's not a new thing and just now the test failing, then i think it's ok to just make the test pass for now and add a TODO15:25
tsdgeosthe todo is there already :-)15:26
dednickyup, i know, which is why i've approved :)15:28
mhr3tsdgeos, there was a talk some time ago that we shouldn't be putting QObjects into a model?15:48
tsdgeoshmmm15:48
tsdgeosdon't remeber about it15:49
tsdgeosand can't think of an immediate reason why not to tbh15:49
tsdgeosmhr3: c++ wise you mean, right?15:49
mhr3tsdgeos, yep15:49
mhr3tsdgeos, hm, i do remember a discussion about model and objects, although maybe the conclusion wasn't that there shouldn't be objects15:50
mhr3or maybe it was about not using QFoo* in there?15:50
mhr3instead to have QFoo instead15:51
tsdgeoswell, it depends on what you're trying to do :D15:51
tsdgeosdo you have an example we can talk over?15:51
mhr3i'm writing a new model and don't want to make mistakes from the beginning :)15:51
tsdgeosmhr3: i guess you just need to make sure you keep the ownership of the * clear if you use *15:57
tsdgeosthat's the only thing i can really think of atm15:57
mhr3yea, that makes sense15:58
Saviqnic-doffay, say you're expanding the second category16:20
Saviqnic-doffay, you don't want to animate from $currentHeight to $scopeViewHeight16:21
Saviqnic-doffay, but from $currentHeight to $scopeViewHeight - $currentY16:21
mhr3dednick, btw any development on the glib source in qt issue?16:22
=== greyback is now known as greyback|away
Saviqtsdgeos, does it crash in 5.2, too?16:23
tsdgeosSaviq: haven't tried, compiling and running the whole thing against 5.2 is not that easy (unless you run the ppa which i'm not sure i want to do and risk messing up my system)16:24
Saviqtsdgeos, I'm good with workaround + TODO, but if it's something that needs to be fixed in 5.2 anyway...16:25
Saviqtsdgeos, the ppa should be safe, + ppa-purge to go back16:25
tsdgeosi've never used ppa-purge16:26
tsdgeosbut ok, i'll check it tomorrow16:26
tsdgeoschanging the other Qt patch now16:26
tsdgeosi pinged lars and he did a quick comment to make me fix something so i need to fix it so he has no excuse not to continue looking at it :D16:27
Saviqtsdgeos, do do do :F16:27
Saviqtsdgeos, ppa-purge is very handy - it will downgrade all the packages, that you have installed from that ppa, to distro versions16:27
Saviqor at least to latest non-that-ppa versions16:28
Saviqand comment out the ppa16:28
SaviqCimi, well, it's not in the kernel yet, even, is it ;)16:31
SaviqCimi, our kernel folk would have to look into it and see the advantages16:31
nic-doffaySaviq, yeah so the individual category y. I was thinking of the scopeView y for some reason...16:32
nic-doffay Got it though, ta16:32
CimiSaviq, I read that motorola is using it on moto x and g16:32
Cimimoto g seems like a great phone where I would like to see ubuntu on it16:32
Cimiit's cheap16:32
Cimiand does good stuff16:33
=== dandrader|lunch is now known as dandrader
dednickmhr3: em, not really. have a better solution, but it's still just sending events16:38
=== salem_ is now known as _salem
=== _salem is now known as salem_
=== hggdh_ is now known as hggdh
=== greyback|away is now known as greyback
Cimidednick, so I've been digging on the carousel filter grid17:12
Cimidednick, in general I think those two are too different to be shared :\17:13
Cimithere's so much more custom code between scope view and filter grid than it's required for the carousel17:13
CimiSaviq, ^17:16
Cimias albert started http://paste.ubuntu.com/6416258/17:16
pete-woodsmterry: hi, did you ping me earlier about the MIR stuff? I can't find any trace of it, but my brain suddenly said, "hey you forgot something!"17:17
mterrypete-woods, yar, you said there were bug subscribers, but I don't see them17:19
Cimithere's also highlightIndex17:20
pete-woodsmterry: where do I look to verify?17:25
pete-woodsI thought I added them17:25
pete-woodsI put them on the launchpad project, not the source package, was that wrong?17:26
mterrypete-woods, https://launchpad.net/ubuntu/+source/libqtdbusmock17:26
mterrypete-woods, yeah.  I mean, both are good17:26
mterrypete-woods, but MIR specifically cares about Ubuntu maintainership17:26
pete-woodsmterry: okay, I will do that then17:26
veebersCan anyone here tell me the status of Optimus graphics on Ubuntu? Should I stay away and opt for Intel graphics? Is bumblebee a solution that just works?20:43
Saviqveebers, http://www.webupd8.org/2013/08/using-nvidia-graphics-drivers-with.html20:43
veebersawesome, thanks Saviq I'll take a look20:44
Saviqveebers, at least in my Dell I can switch between nVidia (by disabling optimus) and Intel (by enabling, but not using optimus)20:44
Saviqveebers, with  nvidia-prime Xorg crashes here, though :/20:44
veebersSaviq: sweet, sucks about the crash though :-\ Looks like I have a couple of other laptop options available to me now20:45
Saviqveebers, I've been happily using either nVidia or Intel on this laptop for two years now, just never really both at the same time20:45
SaviqI did experiment with bumblebee, but it's just never proven useful (I'm no gamer, though)20:45
Saviqveebers, truth is, if you don't need the umph from nVidia for gaming or such, probably better to steer away from it for all kinds of reasons20:46
veebersSaviq: right, that was my original thinking. But I've got a couple of laptops that I'm considering, one has nVidia Optimus so thought I would do some reasearch20:47
veebersSaviq: actually, if you have a moment, could I ask you a qml question?20:47
Saviqveebers, hit me, but latency might be high ;)20:53
veebersSaviq: ack, thanks. Is there a better way for me to pass an argument to this purely qml app so that it can be used as a property: https://code.launchpad.net/~veebers/stock-ticker-mobile-app/mock-server-for-testing/+merge/19497121:01
veebersI'm wanting to pass in the server url to use, but on startup it seems that the argument value isn't ready yet and thus initially uses the default21:01
Saviqveebers, yeah, so: QML objects are always instantiated with defaults21:19
Saviqveebers, and then the property values are evaluated and set21:19
Saviqveebers, so you always need to react to onPropertyChanged21:19
Saviqveebers, "get_data_server() ? get_data_server() : ..." you can use bracket-enclosed JS that you return a value from in a binding, so:21:20
Saviq{ var foo = get_data_server(); if (foo) return foo; else return "bar"; }21:21
Saviqsaves you one call to get_data_server()21:21
veebersSaviq: ah right, nice. But your saying that for instance this isn't doing what I expect it to (i.e. I expect that the dataserver arg (used in get_data_server() ) is available and would be used) : property string dataServer: get_data_server() ? get_data_server() : "http://finance.yahoo.com/"21:23
Saviqveebers, from what I can see things should be working, 'cause other props are bound to dataServer21:24
Saviqveebers, only potential missing thing is listening for args.values.dataserver changes21:25
Saviqveebers, but IIUC that comes as a command-line argument, so it should be safe21:26
veebersSaviq: odd, if I add debugging logging to get_data_server stating if args.values.dataserver is set or not, thre are some initial calls where it is not set21:26
Saviqveebers, yeah, I expect there may be21:27
veebersSaviq: yes it's an Argument that's set via command line, so it shouldn't change during operation21:27
Saviqveebers, well, but then it depends how the whole Arguments / Argument machinery behaves21:27
veebersSaviq: ah, good point21:27
Saviqveebers, there might be some cycles where it didn't yet pick up the args21:29
=== salem_ is now known as _salem
Saviqveebers, it's the first I've seen the Arguments component, do you know where it comes from?21:29
Saviqah, SDK ;)21:30
veebersSaviq: it's a uut component: http://developer.ubuntu.com/api/devel/ubuntu-13.10/qml/ui-toolkit/qml-ubuntu-components0-arguments.html21:30
Saviqveebers, so yeah, I'm not sure on its internals, but I expect there might be some racing involved, especially if you see the value being empty at first21:33
veebersSaviq: ack ok. I'll explore further. Is there any other methods available to me for a pure qml app to pass an argument either via command line args or env var that you know of?21:33
Saviqveebers, nope, there's nothing like that - those kind of things are meant to be done in c++ before starting the QML engine21:34
veebersSaviq: aye, that's the conclusion that I had come too as well. Thanks for clarifying :-)21:35
veebersSaviq: on that note, what sort of boiler plate would be needed to add that to the app (the c++ side)?21:35
Saviqveebers, very little21:42
veebersSaviq: hmm, I might explore that then21:43
Saviqveebers, create a QML app in QtCreator, it will give you a C++/QML template21:43
veebersSaviq: nice, thanks.21:43
Saviqveebers, file a bug against SDK, though, if you find that's indeed the case that the values are not ready straight away21:44
veebersSaviq: will do21:44
veebersSaviq: also (change of topic :-) ) fyi: https://code.launchpad.net/~veebers/unity8/ap_make_use_of_helpers_in_tests/+merge/19157521:44
Saviqveebers, yeah I tested it today and it was fine, wanted to have a look again21:47
veebersSaviq: cool thanks.21:47
Saviqveebers, one thing, though - there's some logging output from the helpers we should probably quiet down21:47
* veebers looks21:48
veebersSaviq: this is the "Restarting unity . . ." logs?21:48
Saviqveebers, yeah, and then the output from upstart21:54
veebersSaviq: cool, I'll take care of it21:54
Saviqveebers, cheers21:54
* veebers takes extra logging out into the woods to "take care of it"21:54
Saviqveebers, while you're at it, maybe it'd make sense to log the output from upstart, so that it shows up in -v21:55
veebersSaviq: good point, will do21:55
veebershmm, my qtcreator doesn't have the ubuntu branding etc. which leads me to believe that perhaps I need some more things installed. Anyone have an idea?23:32
Saviqveebers, ubuntu-sdk23:36
veebersSaviq: ack thanks. I'm installing as we speak :-)23:37
Saviq/away23:37

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