/srv/irclogs.ubuntu.com/2016/03/15/#ubuntu-app-devel.txt

liuxgdoes anyone know where to find all of the theme names for ubuntu phone in the QML. I know there are names like Ubuntu.Components.Themes.Ambiance, what else?04:27
=== _salem is now known as salem_
nik90Elleo, hey, if I use DownloadManager{} instead of SingleDownload{}, is it still possible to show it in the indicator? If so, how?11:54
nik90I understand that "downloads" is a list of all the stuff that is going to be downloaded by the download manager. However I need a SingleDownload object to set its metaData11:55
nik90I looked at the web browser app code, at http://bazaar.launchpad.net/~phablet-team/webbrowser-app/trunk/view/head:/src/app/Downloader.qml12:01
nik90It seems they create dynamical SingleDownload{} objects, but I don't see any place where these objects get destroyed/cleaned up.12:01
nik90I would expect they cleaned up when the download finishes12:02
nik90oSoMoN, Can you explain this if you're free ^^12:02
Marin_Is anyone here to help with snappy-ubuntu-core12:03
ogra_Marin_, try #snappy12:03
Marin_first time here thanks12:03
oSoMoNnik90, Elleo wrote that code so he’s more capacitated than me to explain it, but let me take a look anyway12:04
nik90oSoMoN, I think I have a vague idea, but I cannot form a complete picture of how the various components are connected.12:08
nik90I don't mind waiting for Elleo if you have other things to attend to.12:08
Elleonik90: the downloads list in the DownloadManager component will contain downloads you create via SingleDownload as well12:09
Elleonik90: it's basically a list of all downloads created by your app, regardless of how they were created12:10
nik90Elleo, true, but I need to migrate from SingleDownload to DownloadManager to handle multiple downloads in Podbird. Otherwise it will stop downloading after the very first download (while the app is backgrounded).12:10
nik90So when I use DownloadManager, I am unable to understand, how to set the metaData of individual downloads in DownloadManager12:11
Elleonik90: you can create multiple SingleDowlnoad components dynamically12:11
Elleonik90: they'll automatically be queued by DownloadManager for you12:11
Elleonik90: in the order that they were created12:11
nik90Elleo, ah ok...so something like http://bazaar.launchpad.net/~phablet-team/webbrowser-app/trunk/view/head:/src/app/Downloader.qml12:12
nik90Elleo, and when I received the onFinished signal from the SingleDownload component, I can then destroy it?12:12
Elleonik90: yeah, but don't listen for onFinished in SingleDownload, listen for onDownloadFinished in DownloadManager, that way it'll also support the app being closed and the download still finishing12:13
Elleonik90: that provides you with both the path and the relevant SingleDownload component that finished12:14
nik90Elleo, ack. When I use the onDownloadFinished signal, I will need to compare the finished download ID with the list of temporary singleDownload objects to see which one it was and destroy that?12:15
nik90DownloadManager gives me the relevant SingleDownload component as well with the onfinished signal?12:15
nik90s/onfinished/ondownloadfinished12:16
Elleonik90: yeah, it gives you "download" which a SingleDownload and "path" which is where it ended up12:16
nik90cool, thnx12:16
Elleonik90: http://bazaar.launchpad.net/~phablet-team/webbrowser-app/trunk/view/head:/src/app/webbrowser/DownloadHandler.qml <-- this shows what the browser does there, there are other similar signals you can listen to for paused/resumed/etc.12:17
nik90Elleo, I saw that file earlier..What does DownloadsModel refer to?12:17
nik90I don't see that as a listed property in DownloadManager12:17
nik90nvr mind, that comes from the  webbrowserapp.private 0.1 import12:18
Elleonik90: that's just a webbrowser thing, it's basically the webbrowser's own sqlite db listing all old downloads12:18
Elleoyeah12:18
nik90I can just say download.destroy()12:18
nik90Elleo, thnx..this makes it clear.12:19
Elleonik90: yep, also iirc you can set autoRemove on the DownloadManager and it'll get rid of them for you when finished12:19
nik90Elleo, I only see cleanDownloads property in DownloadManager which just clears up the list. I can't be sure it also cleans up the SingleDownload object12:20
Elleonik90: yeah, I should probably do another little video explaining how SingleDownload and DownloadManager can be used together, good to have these questions asked and see where we need to improve docs12:21
Elleonik90: ah yeah, you may be right there12:21
nik90Elleo, oh man, those videos are freaking awesome.12:22
Elleonik90: regardless what you do it does actually automatically clean up the real internal download object, so all you're left with is a tiny SingleDownload component keeping the relevant properties in memory (id, state, error, etc.)12:23
Elleonik90: so there's only a few kbs memory hit if a dev doesn't manually destroy the object12:23
nik90Elleo, agree12:23
Elleonik90: thanks :) been away the past few days but going to do some more as soon as I get a free evening12:24
Elleonik90: I had a play with the dev version of podbird before I left (haven't tried the very latest version yet), seemed really good, only two things I wasn't sure on; one was the removal of the fuzzy background image at the top of the podcast pages; it looks good without it for some podcasts, but not as good for others (e.g. where their logos don't have such well defined edges), so we might want to chat with Kevin a bit about that12:26
Elleonik90: and I'm really not keen on losing the "Add new podcast" page, the interaction of pressing the "+" to get to a new page with identical header actions seems wierd to me12:27
Elleonik90: plus I'm planning on expanding that page a lot12:27
Elleonik90: so that when it's in the empty state it shows charts and recommendations from the podbird.org services12:27
Elleonik90: which seems more natural to reach via the "Add new podcast" menu option12:28
Elleoat least to me anyway :)12:28
Elleobut overall everything seemed really great12:29
ElleoI'll give the .5 release a go this evening to see what else has changed while I was away12:29
nik90Elleo, the fuzzy background removal was part of kevin's new design spec at https://drive.google.com/file/d/0B8NjAHEq9cvEY2VZb1lGczRKMVE/view. Although I have to agree that I liked the fuzzy background as well :)12:37
nik90I see your point regarding the add podcast page. I agree. The headers are too similar at the moment. I will bring that back to the old way. But I like to keep the EpisodesTab as the way it is if you like it.12:38
nik90Elleo, I think the most cool part is that we have playlists support in 0.7.5..so you should really try that out.12:38
Elleonik90: awesome, look forward to giving that a go :)12:48
Elleonik90: and yeah, episodes tap looks really good12:51
Elleonik90: nice having a way to access all the downloaded episodes from all podcasts mixed together12:51
Elleotab*12:52
nik90Elleo, I was thinking of integrating the in-progress downloads to the downloads view. This way users can cancel individual downloads, pause them if required etc. That was another thing that users reported bugs about.12:53
nik90that and the files indicator integration is my focus atm.12:53
Elleonik90: yeah, that sounds good12:53
=== charles_ is now known as charles
t1mpzsombi: so, I have a fix for the XFAIL, https://code.launchpad.net/~tpeeters/ubuntu-ui-toolkit/apl-dont-change-page-geometry/+merge/28905414:46
t1mpzsombi: current staging tests it with the old header, and your branch with the new14:46
t1mpzsombi: should we test both?14:46
zsombit1mp: I wouldn't bother tbh...14:47
zsombit1mp: we don't have resources to maintain both anyway14:47
t1mpzsombi: the whole error was there because we are trying to support both headers in a single APL anyway14:47
zsombit1mp: :(14:47
t1mpzsombi: anyway, the fix turns out to be not so difficult14:48
t1mpzsombi: it broke because the updates are not in the ideal order14:48
zsombiI see14:48
zsombit1mp: I cannto get yr MR14:48
t1mpwhat do you mean with "cannot get"?14:49
zsombit1mp: this MR is agains staging... so we shoudl get this prior to the APL MR14:58
t1mpzsombi: it works independently from the APL MR15:00
zsombit1mp: ok, let's hope CI will wake up and lat it go15:01
t1mpzsombi: I tested it with your MR too15:01
zsombit1mp: me too :)15:01
zsombit1mp: happroved15:01
t1mpthanks15:01
zbenjaminbzoltan_: so your script listed loads of packages... i decided to do a clean install15:01
zbenjaminbzoltan_: even kernels ...15:02
bzoltan_zbenjamin: my script?15:02
zbenjaminbzoltan_: the one that lists packages that are not from the archive?15:02
bzoltan_zbenjamin: check it if you do not believe :)15:02
zbenjaminbzoltan_: i do believe, but as i said i had a huge list of packages that came from nowhere. I guess broken or not completely removed ppa'S15:03
beunothere's an outage going on, FWIW15:03
zbenjaminbzoltan_: the system was in a very very weird state15:03
bzoltan_zbenjamin: I figured :)15:03
zbenjaminbzoltan_: clean install can not hurt from time to time ;)15:04
bzoltan_zbenjamin: yes15:04
bzoltan_zbenjamin: indeed many of the packages on a continously upgraded system fall out of the controll net15:06
zbenjaminwoah, new xenial did boot up so fast!15:07
balloonsrest in pieces my jenkins friend15:07
balloonsahayzen, you about?15:07
nik90Elleo, I have an issue. I was able to successfully migrate Podbird to DownloadManager. The integration went fine without breaking any existing thing. However, when I tried to batch download a group of episodes using a JS function, Podbird crashes. And now everytime I open Podbird I cannot download any episode since the internal SingleDownload{} objects still persist. Printing downloads.length shows 18 and I cannot do anything to clear it.15:54
nik90I already have cleanDownloads variable set to true.15:54
nik90I deleted the mp3 files manually, but not sure how to reset DownloadManager despite several times of reopening podbird.15:55
=== chihchun is now known as chihchun_afk
balloonsnik90, since you are about, can I get your opinion on always running qml tests on build?15:59
nik90balloons, hi16:00
nik90sure16:00
balloonscheckout https://code.launchpad.net/~nskaggs/ubuntu-clock-app/run-qml-tests-on-build/+merge/289065. This means jenkins will always run tests, but so will you on your desktop with this change16:00
balloonsis this a 'good' thing, or should we make things more explicit within jenkins and run make test after?16:00
* nik90 tests16:01
nik90balloons, It affects the way I run and build clock app on the phone. The tests are failing because I develop using 14.04 and run it on the phone. Since I am running on 14.04, all qml tests will fail since the sdk libs are unavailable. Also I get QXcbConnection: Could not connect to display16:04
Elleonik90: odd, you could try killing the ubuntu-download-manager process, it'd be good if you could push the code somewhere so I can give it a try (might not get to it until tomorrow now though), sounds like something might be going wrong in download manager somewhere16:04
Elleonik90: although I'm not sure why it isn't effecting the browser then16:04
nik90balloons, so this forces app devs to only use 16.04 or later16:04
nik90Elleo, sure, I will also provides steps to reliably reproduce it. FWIW restarting the phone fixed it.16:05
Elleonik90: okay, thanks16:05
balloonsnik90, well you could toggle it off, but I'm concerned that tests will fail, and running by default might upset people16:05
balloonsbut it would also ensure the tests are run when people are coding :-)16:06
balloonsnik90, the build should still finish though right, even though the tests faill16:06
balloonsnik90, obviously I don't want to force anyone into running new stuff16:06
nik90balloons, for me the build fails when the tests fail -> http://paste.ubuntu.com/15392814/16:07
nik90the test failures are being considered as errors rather than warnings by qtcreator16:07
balloonsok, well I have some concerns. I wish cmake made this a little easier16:08
balloonsso I think I will make it explicit inside jenkins and reject this16:09
nik90balloons, Yes. I am fully aware of cmake's abilities..it could have a ignore option while printing out the errors. But I can't say for sure.16:10
nik90not fully*16:10
ahayzenballoons, what's up ?16:12
balloonsI'll try doing it another way. It was more hoops and harder, but forcing it seems to make things painful, which is what I wanted to know16:12
ahayzent1mp, is there a silo for https://code.launchpad.net/~tpeeters/ubuntu-ui-toolkit/fasterWindowColor/+merge/288661 yet?16:12
balloonsahayzen, see above with nik90. I was going to mention how I'm implementing unit testing within jenkins16:13
ahayzenballoons, ok! :-)16:13
balloonsthe answer is, we'll make an explicit call. That does I suppose make things a bit harder and puts things back into your court perhaps16:13
balloonsit means back to playing with cmake16:13
balloonsahayzen, secondly, how's the changes coming for weather and music?16:13
ahayzenballoons, slowly, i'm coming up to coursework season :-(16:14
ahayzenhalf way through the last semester of uni \o/ :')16:14
balloonsright, no worries. Just thought I'd ask again.16:14
faenilahayzen: wow...keep tight! )16:15
ahayzenballoons, so i'd like to have them running each time a dev builds/runs, but as with nik90 i suspect most/all music tests would fail for me due to there being no QtMultimedia 5.6 on Wily16:15
ahayzenballoons, so probably best for just explicitly on jenkins, until we get a snappy desktop :')16:15
nik90+16:16
ahayzenballoons, can we have a build configuration for running the tests or something? so that it appears as an option in QtC ?16:17
t1mpahayzen: no silo yet for that MR16:17
t1mpahayzen: I wanted to have the fix for this one first https://bugs.launchpad.net/ubuntu/+source/ubuntu-ui-toolkit/+bug/155579716:17
ubot5Launchpad bug 1555797 in ubuntu-ui-toolkit (Ubuntu) "Theme loads MainViewStyle 1.2 when MainView has a dark backgroundColor set." [High,In progress]16:17
balloonsahayzen, an alternate cmake target might do that?16:17
ahayzenballoons, so then if i was running on xenial/latest release ... then i could select to use that config and run the tests while developing16:18
t1mpahayzen: so the fasterWindowColor MR is still landing in our staging now16:18
ahayzenballoons, and then jenkins could use that build config16:18
balloonsthat would be ideal. I'm not positive, but I'll try16:18
ahayzenballoons, i spotted coreapps like reminders have loads of configs (or whatever they are actually called)16:18
ahayzent1mp, ok thanks :-) i'll watch out for a silo so i can test :-)16:19
t1mpahayzen: okay. But a silo is the last step. First we verify that everything works, then we land it to staging, then we get a silo to land it in the image (with more testing)16:20
ahayzenballoons, like this https://drive.google.com/file/d/0B3XynHVKfrvMc0NlTENvdWRKZmc/view?usp=sharing you can see the different Autopilot ones and reminder ones16:21
t1mpMirv: where could we find the tgz that has the device deb files for this MR? https://code.launchpad.net/~tpeeters/ubuntu-ui-toolkit/fasterWindowColor/+merge/28866116:21
ahayzenballoons, if there was a 'jenkins' one16:21
t1mpMirv: ^ I got a bit lost with all the jenkins results and I cannot find the tgz any more16:21
ahayzent1mp, i you can point me to some debs i could test as well :-)16:21
balloonsahayzen, right, got it16:21
ahayzenballoons, do you think that is the best way ?16:21
nik90Elleo, you can find the branch and steps to reproduce the bug at https://code.launchpad.net/~nik90/podbird/migrate-to-download-manager/+merge/28906916:23
balloonsI think it would be the nice yes. However, jenkins first, then sdk, heh16:24
ahayzennik90, what do you think? could a run configuration be enough ?16:24
ahayzen:-)16:24
t1mpMirv: ah, I think I found one :)16:24
t1mpahayzen: https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-armhf-stable/502/artifact/artifacts/copy-to-phone-fasterWindowColor-1895.tar16:24
ahayzent1mp, \o/ i'll try and have a play in a bit16:24
nik90ahayzen, yes I think that would be best. This way we can use that particular build configuration last when fixing the tests after fixing a bug/implementing a feature and not all the time.16:25
ahayzenyeah agreed16:27
Mirvt1mp: please report back too if it works alright, I recently added the new qml-module-* packages16:27
Mirvwhich are now needed16:28
Elleonik90: great, thanks; will take a look at that once I've finished off this current bit of work (might not be until tomorrow)16:28
nik90Elleo, no worries, I will work on something else until then16:31
ahayzenballoons, any idea why this wasn't picked up by jenkins? https://code.launchpad.net/~vthompson/music-app/fix-1549557/+merge/28887117:04
balloonsahayzen, still about?19:22
balloonsif so, time for a little jenkins lesson19:22
ahayzenballoons, o/ i'm about for a few minutes, otherwise i'm around tomorrow19:23
ahayzenballoons, maybe best tomorrow :-)19:24
balloonsahayzen, just note the red jobs on https://core-apps-jenkins.ubuntu.com/computer/19:27
balloonsahayzen, if you see things like that, it's safe to cancel them. They generate again. That's why jenkins isn't running on music19:27
ahayzenso it's got struct trying to acquire a lock19:27
ahayzenah ok thanks :-)19:28
balloonsyou are welcome -- you can cancel them right?19:28
* ahayzen tries19:28
ahayzenballoons, yeah i've managed to cancel the music ones \o/19:29
ahayzencan see it rerunning now19:29
balloonsyou can cancel them all, but ;-) Awesome, now you now19:29
* ahayzen adds to his jenkins doc :-)19:29
balloonshttps://wiki.ubuntu.com/Touch/CoreApps/Jenkins#Troubleshooting now has a picture to go with #2 ;-)19:33
nik90Elleo, I figured it out :D .. well atleast have a faint idea why. There is this episode "Weekly Tech New - March 5, 2016" from the Podcast "Daily Tech News Show". That episode when you try to play/download it, causes Podbird to crash regardless of whether you use SingleDownload{} or DownloadManager{}.20:23
nik90Elleo, that said, I think in the event of an app crash, it is still important that UDM destroys its internal objects.20:23
nik90It could be that the URL is buggy. maybe not encoded correctly ..not sure20:24
nik90Elleo, more specifically (after further testing), it turns out that episode has a blank url. On creating a sample template app, I noticed that if you do downloadmanager.download(""), it causes the app to crash :P20:30
nik90that's the reason. I just need to check if the url is blank or not before sending it to download manager.20:31
=== salem_ is now known as _salem

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