[08:09] <dholbach> good morning
[09:59] <JamesTait> Good morning all! :-D
[10:43] <nerochiaro> zsombi: good morning. hope you had a good start of the year. I have a quick question for you, how does the StateSaver work when elements are loaded by a Loader ?
[10:44] <zsombi> nerochiaro: morning sire!!! indeed a good start, but got few bugs to solve :) the StateSaver works in the same way as usual.
[10:45] <zsombi> nerochiaro: Loader must have an ID as it will be the parent of the loaded element
[10:46] <nerochiaro> zsombi: interesting. it looks like in some case it doesn't work even if the loader has an id. i'll investigate further
[10:46] <zsombi> nerochiaro: so you have a component loaded by the Loader and the component uses the StateSaver?
[10:47] <nerochiaro> zsombi: yes
[10:47] <zsombi> nerochiaro: we have test cases for that, check them
[10:48] <nerochiaro> zsombi: ok
[11:22] <nerochiaro> zsombi: i seem to be doing all correctly, same as the test case, but I don't see the state being restored
[11:22] <nerochiaro> zsombi: in the app, of course it works in a separate case
[11:27] <zsombi> nerochiaro: so you don't see the state restored in your case?
[11:27] <nerochiaro> zsombi: no
[11:27] <zsombi> do you see any warnings?
[11:28] <nerochiaro> zsombi: i put a onSavedPropertyChanged: console.log(savedProperty) and the value is not hte value i saved. no warnings from statesaver either
[11:28] <nerochiaro> zsombi: i know the saved value is different because if i cat the file .config the right value is there
[11:28] <zsombi> nerochiaro: so no warnings on the console from StateSaver?
[11:28] <nerochiaro> zsombi: no
[11:29] <zsombi> nerochiaro: weird... show me the branch
[11:29] <nerochiaro> zsombi: one moment
[11:33] <nerochiaro> zsombi: lp:~amanzi-team/gallery-app/gallery-app-listview-state/
[11:33] <nerochiaro> zsombi: check Components/MediaViewer.qml
[11:34] <nerochiaro> zsombi: (i will have to run out for 15 minutes soon)
[11:34] <nerochiaro> zsombi: so take your time
[11:34] <zsombi> nerochiaro: ok, will check it
[11:35] <nerochiaro> zsombi: thanks
[11:41] <zsombi> nerochiaro: cannot see such a QML file...
[11:42] <zsombi> nerochiaro: ah, under Components/MediaViewer there is one
[12:08] <mzanetti> popey: hi
[12:08] <mzanetti> popey: could you please test again if this works with a smaller image? https://code.launchpad.net/~mzanetti/reminders-app/add-image-support/+merge/199031
[12:09] <popey> mzanetti: ok
[12:21] <nerochiaro> zsombi: news ?
[12:21] <zsombi> nerochiaro: on standup, but couldn't find your StateSaver there...
[12:22] <mzanetti> popey: and I've added the cmake dep to the script in here: https://code.launchpad.net/~mzanetti/reminders-app/switch-to-cmake/+merge/199377
[12:25] <nerochiaro> zsombi: rc/qml/Components/MediaGrid.qml line 48
[12:27] <nerochiaro> zsombi: sry I had pointed you to the wrong file before
[12:44] <zsombi> timp: you completely forgot this! https://code.launchpad.net/~zsombi/ubuntu-ui-toolkit/pickerpanel/+merge/199250 ;)
[12:44] <timp> zsombi: no it is 4th on my to review list ;)
[12:44] <zsombi> timp: last year it was the first on your list :D
[12:44] <zsombi> you downgraded it???!
[12:55] <nerochiaro> zsombi: done with standup ?
[12:58] <zsombi> nerochiaro: still
[12:58] <zsombi> in
[13:15] <nik90> zsombi: btw nice touch on the SDK dialer component. I am currently transitioning the clock app to use them :)
[13:16] <zsombi> nik90: it has been there for a while ;)
[13:16] <nik90> zsombi: I know, but only recently I really took notice of that component.
[13:20] <zsombi> nik90: glad that you'll take it in use :)
[13:21] <nik90> zsombi: I did find two small bugs, will report them a bit later when I am done with the MP. But otherwise it has helped remove quite a bit of code from the clock app.
[13:24] <zsombi> nik90: +1
[13:39] <nerochiaro> zsombi: i've worked around the problem for now, but it would be still better if we could figure out what's going on
[13:39] <zsombi> nerochiaro: ended the standup, lemme check the file
[13:42] <zsombi> nerochiaro: who loads this MediaGrid.qml?
[13:42] <nerochiaro> zsombi: rc/qml/PhotosOverview and that one is loaded by rc/qml/MainScreen
[13:49] <zsombi> nerochiaro: hmm... weird...
[13:49] <nerochiaro> zsombi: fails for you too ?
[13:50] <zsombi> nerochiaro: I have checked the code, all seems to be OK, I'll try to build it
[13:50] <nerochiaro> zsombi: ok. i'm testing on desktop at the moment
[13:50] <zsombi> uhh... dependencies...
[13:50] <nerochiaro> zsombi: just apt-get install gallery-app
[13:51] <nerochiaro> zsombi: i mean build-dep gallery-app
[14:10] <nerochiaro> zsombi: sucess @ build ?
[14:10] <zsombi> nerochiaro: still builds
[14:10] <zsombi> nerochiaro: done, success
[14:13] <zsombi> nerochiaro is there any script to help it running on desktop?
[14:14] <nerochiaro> zsombi: just run src/gallery-app
[14:20] <zsombi> nerochiaro: actually there is something that overwrites the savedDistance to 0 upon next start...
[14:20] <nerochiaro> zsombi: what is that ?
[14:20] <nerochiaro> zsombi: it has first to be restored, then overwritten, if the StateSaver is working the way it should. and i never see it being resotored
[14:21] <zsombi> nerochiaro: what is what? the one which overwrites? don't know... something that happens after the state is restored...
[14:21] <zsombi> nerochiaro: will have to dive more into it
[14:22] <zsombi> nerochiaro: but will check it tomorrow...
[14:22] <nerochiaro> zsombi: i don't think it ever gets restored. if i put an onChanged handler on it I never see the value being restored and hten overwritten
[14:22] <nerochiaro> zsombi: i won't be around tomorrow, but can you please send me an email ?
[14:22] <zsombi> nerochiaro: right, I see that. There;s somewhere between the time the state is about to be restored, but it is overruled by something...
[14:22] <zsombi> nerochiaro: sure
[14:23] <nerochiaro> zsombi: perfect, i'll check once in a while and come up if there's more to discuss
[14:23] <zsombi> nerochiaro: cool!!
[14:23] <nerochiaro> zsombi: thanks
[14:23] <zsombi> nerochiaro: ywc
[14:30] <nerochiaro> zsombi: i am trying with other lists that are loaded in the same way, same problem
[14:30] <nerochiaro> zsombi: just to confirm
[14:37] <zsombi> nerochiaro: then it's worth reporting a bug!!!
[14:37] <nerochiaro> zsombi: even if i can't repro in a simple exaple ?
[14:38] <zsombi> nerochiaro: well, not then... give me few days to hunt for it, ok?
[14:40] <nerochiaro> zsombi: ok, i'll let you upgrade from tomorrow to "a few days" :) but maybe then you can also look into this at the same time: you will notice that if you scroll around the list, especially past the edges, you will get binding loops for preferredHighlightEnd having a loop with height. We tried to figure this out with tim but couldn't get to the bottom of it.
[14:40] <nerochiaro> zsombi: since you suggested to use preferredHighlightEnd in the first place ...
[14:42] <zsombi> nerochiaro: few days was before I found the solution to the bug in DatePicker, so I'll recall my few days :D
[14:49] <nerochiaro> zsombi: ok then :)
[14:55] <mhall119> Wellark: ping me when you're around
[15:00] <nerochiaro> oSoMoN: what do you think we should do regarding this MR that has been pending for quite a while now ? https://code.launchpad.net/~amanzi-team/gallery-app/gallery-app-save-state/+merge/198364
[15:16] <oSoMoN> nerochiaro, are you able to successfully run the tests for this MR on your device?
[15:18] <nerochiaro> oSoMoN: they were all passing on my device last i tried. i think the crashes were on mako ?
[15:35] <oSoMoN> nerochiaro, I’m unable to retrieve the debs from the links in the MR comments posted by the CI job (getting 404s), I’ll trigger a new CI run
[15:38] <nerochiaro> oSoMoN: ok. i'm upgrading the phone to the latest ubuntu-system from channel trusty
[15:40] <oSoMoN> nerochiaro, you should probably use trusty-proposed to test against the latest stuff
[15:41] <om26er> oSoMoN, who is working on gallery-app now ?
[15:41] <nerochiaro> om26er: i'm doing some work on it, but it's not "my app"
[15:42] <oSoMoN> om26er, noone is actively working on it, I’m doing AP test fixes if needed
[15:43] <om26er> nerochiaro, oSoMoN  got it. Do we have someone filling gusch place ?
[15:43] <om26er> soonish ?
[15:43] <nerochiaro> om26er: not yet
[15:43] <oSoMoN> om26er, not yet, but we’re in the process of hiring
[15:43] <om26er> nerochiaro, oSoMoN ok. Thanks
[16:28] <Wellark> mhall119: ping
[16:29] <mhall119> Wellark: pong
[16:29] <Wellark> please use the "other" IRC server if you need to reach me urgently. I sometimes miss stuff on Freenode :)
[16:30] <Wellark> mhall119: what's up?
[16:30] <mhall119> Wellark: I spoke to ted a couple weeks ago about enabling the HUD toolbar icons from QML apps, and he said that the QML Actions API needs to be able to send the toolbar-item-hint property over DBUS for the HUD to bind it to an icon, is there a roadmap for implementing that?
[16:31] <Wellark> mhall119: short answer: no
[16:31] <mhall119> can we get one?  Having unuseable icons in the HUD is annoying
[16:31] <Wellark> mhall119: agreed.
[16:32] <mhall119> Wellark: so who do I need to bribe to get this on the schedule for this cycle?
[16:32] <Wellark> mhall119: thostr_
[16:33] <mhall119> ok, I'll send him an email
[16:33] <mhall119> thanks
[16:33] <Wellark> mhall119: np.
[16:34] <Wellark> mhall119: other option would just be removing the icons until we have API for them. would be easier.
[16:34] <mhall119> would be, but making them useful would be better
[16:34] <balloons> nik90, ping
[16:35] <Wellark> mhall119: well, sure. it's just that we must make compromises on features vs. man-power
[16:36] <Wellark> and unfortunately you are the first person to ask about the toolbar icons in 9 months or so.
[16:36] <mhall119> man, we have *got* to get that engineer cloning program working
[16:36] <timp> Wellark: what's the toolbar-item-hint?
[16:36] <Wellark> just saying that it has not been a priority feature :(
[16:36] <balloons> nik90, does this have the online search in it? https://code.launchpad.net/~nik90/ubuntu-clock-app/more-stopwatch-tests/+merge/193922
[16:36] <timp> Wellark: it may be simpler to add an iconName property to the Action. No need to pass the icon itself
[16:36] <timp> but the only icons from the theme can be used
[16:37] <timp> (just an idea, mybe a bad one)
[16:37] <Wellark> timp: the hud toolbar items are a bit special
[16:37] <timp> mhall119: nobody has time to make the engineer cloning program work ;)
[16:37] <mhall119> timp: toolbar-item-hint AIUI, is a DBUS property that tells the HUD to bind an action to an icon in the HUD, rather than (or in addition to) the list
[16:37] <Wellark> they are basically predefined actions that an application may or may not define for it self
[16:58] <nik90> balloons: pong
[16:58] <nik90> balloons: no it doesn't have online search in it
[16:58] <balloons> nik90, see above about the merge.. kk, just wanted to double check that indeed it did not :-) I didn't see it but ;-)
[16:59] <nik90> balloons: i removed it since when I looked at it more closely, I realised it was quite similar to the local search
[16:59] <oSoMoN> nerochiaro, CI job finished, you can grab the debs for testing at http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/1949/artifact/work/output/*zip*/output.zip
[17:00] <balloons> nik90, right I remember our conversation.. perfect
[17:00] <balloons> ty
[17:00] <nik90> :)
[17:00] <nik90> popey: ping
[17:01] <nerochiaro> oSoMoN: i'm already installing it
[17:01] <nerochiaro> oSoMoN: thanks for the heads up though
[17:01] <popey> nik90: pong
[17:01] <popey> nik90: just on the phone for an hour now, but type at me anyway
[17:01] <nik90> popey: sure
[17:02] <nik90> popey: I was just wondering if the core apps PPA updates trickle down to the phone automatically?
[17:02] <nik90> popey: or if I should request it specifically?
[17:02] <popey> nik90: they do not
[17:03] <popey> we request click packages to be updated by sergio and I confirm in the store
[17:04] <balloons> ^^ indeed, I push things to be landed, like your changes.. It's already been requested
[17:04] <nik90> balloons: ah okay
[17:05] <nik90> I will remember that
[17:12] <popey> beuno: Changes can not be accepted and published at this time. Please try again later.
[17:12] <popey> halp!
[17:13] <beuno> popey, you're so impatient
[17:13] <beuno> let's see
[17:13] <popey> I know right?
[17:16] <beuno> popey, I'll have to bring down some logs, which will take a bit
[17:17] <popey> k
[17:20] <nerochiaro> oSoMoN: all tests seems to pass correctly on my maguro device
[17:21] <oSoMoN> nerochiaro, running them on my device now
[17:25] <beuno> popey, found the broken service, chasing it with IS
[17:30] <popey> thanks beuno
[17:32] <oSoMoN> nerochiaro, I’m still getting the same errors I was getting in Dec
[17:32] <oSoMoN> nerochiaro, gotta go now, can you describe how exactly you run the tests on your device in a comment to the MR?
[18:07] <beuno> popey, this happened to you when doing what?
[18:07] <popey> approving this app https://myapps.developer.ubuntu.com/dev/click-apps/173/changerequest/
[18:11] <beuno> popey, k, so there's a bug somewhere
[18:11] <beuno> I'll need to chase that tomorrow with a dev
[18:12] <popey> ok.
[18:12] <popey> kenvandine: ^
[18:12] <kenvandine> thx
[18:13] <ahayzen> balloons_, ping
[18:14] <balloons_> ahayzen, pong
[18:14] <ahayzen> balloons_, happy new year! ... i'm having an issue with the jenkins bot wondered if u could interpret the output for me https://code.launchpad.net/~andrew-hayzen/ubuntu-ui-toolkit/fix-swipe-delete-001/+merge/199906
[18:14] <balloons_> ahayzen, most certainly
[18:16] <balloons_> ahayzen, at first glance it seems some things failed while running
[18:16] <balloons_> but the test output is cryptic :-) I don't have super power on the toolkit
[18:16] <ahayzen> balloons_, i looked at it and didnt have a clue wht was going on :/
[18:17] <balloons_> you could ask Tim to re-run it, but my guess is something you've changed broke it
[18:17] <balloons_> this is where I'm looking: https://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/4417/
[18:17]  * ahayzen wonders wht could be broken with such a small change...
[18:17] <ahayzen> balloons_, thx
[18:18] <balloons_> all the tests failed.. doesn't seem like it ran properly
[18:18] <balloons_> ahayzen, ask tim to re-run, and if the results look funny again, we'll ping ci
[18:19] <ahayzen> balloons, ok thx... think timp has gone by now?
[18:19] <balloons> mm.. perhaps
[18:20] <balloons> ahayzen, I'll try and re-run it
[18:20] <ahayzen> balloons, thx :)
[19:09] <beuno> popey, kenvandine, fixed the problem and approved the update
[19:09] <kenvandine> beuno, thx
[19:10] <popey> thank you
[19:32] <balloons> ahayzen, it passed, but we need someone from that team to approve. I can't do it
[19:32] <balloons> might have to wait
[19:50] <ahayzen> balloons, cool... ah ok i was wondering wht tht message meant
[20:34] <nik90> timp: thnx for getting started on https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1259917. The timing is impeccable.
[20:34] <ubot2> Launchpad bug 1259917 in Ubuntu UI Toolkit "Using a Qml Loader in combination with Pagestack and Tabs results in incorrect positioning" [High,In progress]
[20:49] <nik90> balloons: got a moment?
[20:49] <balloons> nik90, shoot
[20:49] <nik90> balloons: https://bugs.launchpad.net/ubuntu-clock-app/+bug/1266841
[20:49] <ubot2> Launchpad bug 1266841 in Ubuntu Clock App "1 test error, when running in the emulator" [Medium,Incomplete]
[20:50] <nik90> balloons: how long does it take for the click package to be updated?
[20:50] <balloons> nik90, ahh, sergio hasn't landed it yet
[20:50] <nik90> balloons: can you confirm if my asserting in that bug report is correct?
[20:50] <nik90> assertion*
[20:51] <balloons> you are correct
[20:53] <nik90> balloons: what is sergio's nick? I like to ask him some questions related to core apps click packages version numbering and stuff.
[20:54] <balloons> serguisens
[20:54] <nik90> thnx
[23:11] <nik90> popey: do you happen to know who the best person would be to talk to about the wake/unlock devices when an alarm notification is triggered?
[23:11] <popey> nik90: tvoss i think.
[23:12] <nik90> popey: ok. I will check with him tomorrow.
[23:12] <nik90> popey: and accordingly update the progress in the email thread