=== kalikiana__ is now known as kalikiana_
=== alblasch is now known as ablasche
=== wgrant_ is now known as wgrant
dholbachgood morning08:09
=== dholbach_ is now known as dholbach
=== lan3y is now known as Laney
=== Laney is now known as Guest1882
=== Guest1882 is now known as Laney
JamesTaitGood morning all! :-D09:59
=== greyback_ is now known as greyback
nerochiarozsombi: 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:43
zsombinerochiaro: morning sire!!! indeed a good start, but got few bugs to solve :) the StateSaver works in the same way as usual.10:44
zsombinerochiaro: Loader must have an ID as it will be the parent of the loaded element10:45
nerochiarozsombi: interesting. it looks like in some case it doesn't work even if the loader has an id. i'll investigate further10:46
zsombinerochiaro: so you have a component loaded by the Loader and the component uses the StateSaver?10:46
nerochiarozsombi: yes10:47
zsombinerochiaro: we have test cases for that, check them10:47
nerochiarozsombi: ok10:48
nerochiarozsombi: i seem to be doing all correctly, same as the test case, but I don't see the state being restored11:22
nerochiarozsombi: in the app, of course it works in a separate case11:22
zsombinerochiaro: so you don't see the state restored in your case?11:27
nerochiarozsombi: no11:27
zsombido you see any warnings?11:27
nerochiarozsombi: i put a onSavedPropertyChanged: console.log(savedProperty) and the value is not hte value i saved. no warnings from statesaver either11:28
nerochiarozsombi: i know the saved value is different because if i cat the file .config the right value is there11:28
zsombinerochiaro: so no warnings on the console from StateSaver?11:28
nerochiarozsombi: no11:28
zsombinerochiaro: weird... show me the branch11:29
nerochiarozsombi: one moment11:29
nerochiarozsombi: lp:~amanzi-team/gallery-app/gallery-app-listview-state/11:33
nerochiarozsombi: check Components/MediaViewer.qml11:33
nerochiarozsombi: (i will have to run out for 15 minutes soon)11:34
nerochiarozsombi: so take your time11:34
zsombinerochiaro: ok, will check it11:34
nerochiarozsombi: thanks11:35
=== _salem is now known as salem_
zsombinerochiaro: cannot see such a QML file...11:41
zsombinerochiaro: ah, under Components/MediaViewer there is one11:42
mzanettipopey: hi12:08
mzanettipopey: could you please test again if this works with a smaller image? https://code.launchpad.net/~mzanetti/reminders-app/add-image-support/+merge/19903112:08
popeymzanetti: ok12:09
nerochiarozsombi: news ?12:21
zsombinerochiaro: on standup, but couldn't find your StateSaver there...12:21
mzanettipopey: and I've added the cmake dep to the script in here: https://code.launchpad.net/~mzanetti/reminders-app/switch-to-cmake/+merge/19937712:22
nerochiarozsombi: rc/qml/Components/MediaGrid.qml line 4812:25
nerochiarozsombi: sry I had pointed you to the wrong file before12:27
zsombitimp: you completely forgot this! https://code.launchpad.net/~zsombi/ubuntu-ui-toolkit/pickerpanel/+merge/199250 ;)12:44
timpzsombi: no it is 4th on my to review list ;)12:44
zsombitimp: last year it was the first on your list :D12:44
zsombiyou downgraded it???!12:44
nerochiarozsombi: done with standup ?12:55
zsombinerochiaro: still12:58
nik90zsombi: btw nice touch on the SDK dialer component. I am currently transitioning the clock app to use them :)13:15
zsombinik90: it has been there for a while ;)13:16
nik90zsombi: I know, but only recently I really took notice of that component.13:16
=== matiasb1 is now known as matiasb
zsombinik90: glad that you'll take it in use :)13:20
nik90zsombi: 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:21
zsombinik90: +113:24
nerochiarozsombi: i've worked around the problem for now, but it would be still better if we could figure out what's going on13:39
zsombinerochiaro: ended the standup, lemme check the file13:39
zsombinerochiaro: who loads this MediaGrid.qml?13:42
nerochiarozsombi: rc/qml/PhotosOverview and that one is loaded by rc/qml/MainScreen13:42
zsombinerochiaro: hmm... weird...13:49
nerochiarozsombi: fails for you too ?13:49
zsombinerochiaro: I have checked the code, all seems to be OK, I'll try to build it13:50
nerochiarozsombi: ok. i'm testing on desktop at the moment13:50
zsombiuhh... dependencies...13:50
nerochiarozsombi: just apt-get install gallery-app13:50
nerochiarozsombi: i mean build-dep gallery-app13:51
nerochiarozsombi: sucess @ build ?14:10
zsombinerochiaro: still builds14:10
zsombinerochiaro: done, success14:10
zsombinerochiaro is there any script to help it running on desktop?14:13
nerochiarozsombi: just run src/gallery-app14:14
zsombinerochiaro: actually there is something that overwrites the savedDistance to 0 upon next start...14:20
nerochiarozsombi: what is that ?14:20
nerochiarozsombi: it has first to be restored, then overwritten, if the StateSaver is working the way it should. and i never see it being resotored14:20
zsombinerochiaro: what is what? the one which overwrites? don't know... something that happens after the state is restored...14:21
zsombinerochiaro: will have to dive more into it14:21
zsombinerochiaro: but will check it tomorrow...14:22
nerochiarozsombi: 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 overwritten14:22
nerochiarozsombi: i won't be around tomorrow, but can you please send me an email ?14:22
zsombinerochiaro: 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
zsombinerochiaro: sure14:22
nerochiarozsombi: perfect, i'll check once in a while and come up if there's more to discuss14:23
zsombinerochiaro: cool!!14:23
nerochiarozsombi: thanks14:23
zsombinerochiaro: ywc14:23
nerochiarozsombi: i am trying with other lists that are loaded in the same way, same problem14:30
nerochiarozsombi: just to confirm14:30
zsombinerochiaro: then it's worth reporting a bug!!!14:37
nerochiarozsombi: even if i can't repro in a simple exaple ?14:37
zsombinerochiaro: well, not then... give me few days to hunt for it, ok?14:38
nerochiarozsombi: 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
nerochiarozsombi: since you suggested to use preferredHighlightEnd in the first place ...14:40
zsombinerochiaro: few days was before I found the solution to the bug in DatePicker, so I'll recall my few days :D14:42
nerochiarozsombi: ok then :)14:49
mhall119Wellark: ping me when you're around14:55
nerochiarooSoMoN: 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/19836415:00
oSoMoNnerochiaro, are you able to successfully run the tests for this MR on your device?15:16
nerochiarooSoMoN: they were all passing on my device last i tried. i think the crashes were on mako ?15:18
oSoMoNnerochiaro, 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 run15:35
nerochiarooSoMoN: ok. i'm upgrading the phone to the latest ubuntu-system from channel trusty15:38
oSoMoNnerochiaro, you should probably use trusty-proposed to test against the latest stuff15:40
om26eroSoMoN, who is working on gallery-app now ?15:41
nerochiaroom26er: i'm doing some work on it, but it's not "my app"15:41
oSoMoNom26er, noone is actively working on it, I’m doing AP test fixes if needed15:42
om26ernerochiaro, oSoMoN  got it. Do we have someone filling gusch place ?15:43
om26ersoonish ?15:43
nerochiaroom26er: not yet15:43
oSoMoNom26er, not yet, but we’re in the process of hiring15:43
om26ernerochiaro, oSoMoN ok. Thanks15:43
=== gatox is now known as gatox_lunch
Wellarkmhall119: ping16:28
mhall119Wellark: pong16:29
Wellarkplease use the "other" IRC server if you need to reach me urgently. I sometimes miss stuff on Freenode :)16:29
Wellarkmhall119: what's up?16:30
mhall119Wellark: 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:30
Wellarkmhall119: short answer: no16:31
mhall119can we get one?  Having unuseable icons in the HUD is annoying16:31
Wellarkmhall119: agreed.16:31
mhall119Wellark: so who do I need to bribe to get this on the schedule for this cycle?16:32
Wellarkmhall119: thostr_16:32
mhall119ok, I'll send him an email16:33
Wellarkmhall119: np.16:33
Wellarkmhall119: other option would just be removing the icons until we have API for them. would be easier.16:34
mhall119would be, but making them useful would be better16:34
balloonsnik90, ping16:34
Wellarkmhall119: well, sure. it's just that we must make compromises on features vs. man-power16:35
Wellarkand unfortunately you are the first person to ask about the toolbar icons in 9 months or so.16:36
mhall119man, we have *got* to get that engineer cloning program working16:36
timpWellark: what's the toolbar-item-hint?16:36
Wellarkjust saying that it has not been a priority feature :(16:36
balloonsnik90, does this have the online search in it? https://code.launchpad.net/~nik90/ubuntu-clock-app/more-stopwatch-tests/+merge/19392216:36
timpWellark: it may be simpler to add an iconName property to the Action. No need to pass the icon itself16:36
timpbut the only icons from the theme can be used16:36
timp(just an idea, mybe a bad one)16:37
Wellarktimp: the hud toolbar items are a bit special16:37
timpmhall119: nobody has time to make the engineer cloning program work ;)16:37
mhall119timp: 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 list16:37
Wellarkthey are basically predefined actions that an application may or may not define for it self16:37
=== gatox_lunch is now known as gatox
nik90balloons: pong16:58
nik90balloons: no it doesn't have online search in it16:58
balloonsnik90, see above about the merge.. kk, just wanted to double check that indeed it did not :-) I didn't see it but ;-)16:58
nik90balloons: i removed it since when I looked at it more closely, I realised it was quite similar to the local search16:59
oSoMoNnerochiaro, 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.zip16:59
balloonsnik90, right I remember our conversation.. perfect17:00
nik90popey: ping17:00
nerochiarooSoMoN: i'm already installing it17:01
nerochiarooSoMoN: thanks for the heads up though17:01
popeynik90: pong17:01
popeynik90: just on the phone for an hour now, but type at me anyway17:01
nik90popey: sure17:01
nik90popey: I was just wondering if the core apps PPA updates trickle down to the phone automatically?17:02
nik90popey: or if I should request it specifically?17:02
popeynik90: they do not17:02
popeywe request click packages to be updated by sergio and I confirm in the store17:03
balloons^^ indeed, I push things to be landed, like your changes.. It's already been requested17:04
nik90balloons: ah okay17:04
nik90I will remember that17:05
popeybeuno: Changes can not be accepted and published at this time. Please try again later.17:12
beunopopey, you're so impatient17:13
beunolet's see17:13
popeyI know right?17:13
beunopopey, I'll have to bring down some logs, which will take a bit17:16
nerochiarooSoMoN: all tests seems to pass correctly on my maguro device17:20
oSoMoNnerochiaro, running them on my device now17:21
beunopopey, found the broken service, chasing it with IS17:25
popeythanks beuno17:30
oSoMoNnerochiaro, I’m still getting the same errors I was getting in Dec17:32
oSoMoNnerochiaro, gotta go now, can you describe how exactly you run the tests on your device in a comment to the MR?17:32
=== davmor2_ is now known as davmor2
=== balloons_ is now known as balloons
=== bfiller is now known as bfiller_afk
=== daker_ is now known as daker
=== kalikiana__ is now known as kalikiana_
=== balloons is now known as Guest99854
beunopopey, this happened to you when doing what?18:07
popeyapproving this app https://myapps.developer.ubuntu.com/dev/click-apps/173/changerequest/18:07
beunopopey, k, so there's a bug somewhere18:11
beunoI'll need to chase that tomorrow with a dev18:11
popeykenvandine: ^18:12
ahayzenballoons_, ping18:13
balloons_ahayzen, pong18:14
ahayzenballoons_, 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/19990618:14
balloons_ahayzen, most certainly18:14
balloons_ahayzen, at first glance it seems some things failed while running18:16
balloons_but the test output is cryptic :-) I don't have super power on the toolkit18:16
ahayzenballoons_, i looked at it and didnt have a clue wht was going on :/18:16
balloons_you could ask Tim to re-run it, but my guess is something you've changed broke it18: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
ahayzenballoons_, thx18:17
balloons_all the tests failed.. doesn't seem like it ran properly18:18
balloons_ahayzen, ask tim to re-run, and if the results look funny again, we'll ping ci18:18
=== balloons_ is now known as balloons
ahayzenballoons, ok thx... think timp has gone by now?18:19
balloonsmm.. perhaps18:19
balloonsahayzen, I'll try and re-run it18:20
ahayzenballoons, thx :)18:20
beunopopey, kenvandine, fixed the problem and approved the update19:09
kenvandinebeuno, thx19:09
popeythank you19:10
balloonsahayzen, it passed, but we need someone from that team to approve. I can't do it19:32
balloonsmight have to wait19:32
ahayzenballoons, cool... ah ok i was wondering wht tht message meant19:50
=== salem_ is now known as _salem
=== bfiller_afk is now known as bfiller
nik90timp: thnx for getting started on https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1259917. The timing is impeccable.20:34
ubot2Launchpad bug 1259917 in Ubuntu UI Toolkit "Using a Qml Loader in combination with Pagestack and Tabs results in incorrect positioning" [High,In progress]20:34
nik90balloons: got a moment?20:49
balloonsnik90, shoot20:49
nik90balloons: https://bugs.launchpad.net/ubuntu-clock-app/+bug/126684120:49
ubot2Launchpad bug 1266841 in Ubuntu Clock App "1 test error, when running in the emulator" [Medium,Incomplete]20:49
nik90balloons: how long does it take for the click package to be updated?20:50
balloonsnik90, ahh, sergio hasn't landed it yet20:50
nik90balloons: can you confirm if my asserting in that bug report is correct?20:50
balloonsyou are correct20:51
nik90balloons: what is sergio's nick? I like to ask him some questions related to core apps click packages version numbering and stuff.20:53
=== Ursinha is now known as Guest50378
=== Ursinha-afk is now known as Ursinha
nik90popey: 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
popeynik90: tvoss i think.23:11
nik90popey: ok. I will check with him tomorrow.23:12
nik90popey: and accordingly update the progress in the email thread23:12
=== Ursinha is now known as Ursinha-afk
=== Ursinha-afk is now known as Ursinha
=== _salem is now known as salem_
=== salem_ is now known as _salem

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