=== rmescandon is now known as rmescandon|afk === rmescandon|afk is now known as rmescandon [09:28] nik90, before i spend too much time on it, does this error mean anything to you? http://paste.ubuntu.com/8960443/ [09:28] nik90, in clock-app AP tests [09:34] Good morning all; happy World Architecture Day! :-D [09:38] hi liuxg, could you comment on bug 1369799 [09:38] bug 1369799 in Ubuntu App Developer site "Adding new scope example code" [Critical,In progress] https://launchpad.net/bugs/1369799 [09:39] dpm, departments is being finished right now, publishable in a moment [09:39] \o/ [09:39] dpm, OK. I'll take a look at it. [09:44] davidcalle, hey, do you have 5 minutes for a question about scopes? :-) [09:45] rpadovani, sure :) [09:45] davidcalle, thanks! First of all, congrats for your tutorial. I don't know C++, and I a couple of nights I was able to create a basic scopes! [09:46] rpadovani, thanks, it was the same for me ;) [09:48] davidcalle, so, I'm developing the duckduckgo scope, the problem is there are two type of queries, with same json response. Sometimes result are best in one query, sometimes in the other, sometimes are the same. So I have to do query1, compile my data, do query2, take response from query2 and merge in query1 where there isn't a response. How can I do it? [09:48] dpm: [09:48] https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1391689 [09:48] Ubuntu bug 1391689 in Ubuntu Reminders app "UriHandler doesn't emit onOpened if the app needs to be started" [High,Triaged] [09:48] :/ [09:49] dpm: and https://code.launchpad.net/~reminders-app-dev/reminders-app/uriHandler/+merge/239961/comments/593940 [09:49] mzanetti, yeah, I saw it last night [09:49] ah ok [09:50] IMO we should land the branch so penk can start testing with it when the app is running. the launching will start working when the platform bug is fixed [09:50] davidcalle, e.g: infobox is one of the most interesting field, here there isn't https://api.duckduckgo.com/?q=ferrara&format=json&pretty=1 but here yes https://api.duckduckgo.com/ferrara&format=json&pretty=1 [09:51] mzanetti, sounds good to me [09:51] davidcalle, on the other hand, here there isn't the answer https://api.duckduckgo.com/3+2&format=json&pretty=1 but here yes https://api.duckduckgo.com/?q=3+2&format=json&pretty=1 [09:51] (please note the ?=q) [09:59] rpadovani, what you can do is call get another time (with another "root"), parse both, put the infobox, when you find it, in a QVariantMap and then use insert() to add it to another map (the one you use to build your result), for example. [09:59] s/with another/with a different [09:59] rpadovani, do you need data from both, or just the one with infobox? [10:00] davidcalle, the best I think is merging the two results, to have all informations available [10:01] take query1 and fill all possible, take query2 and fill what's blank [10:02] rpadovani, ok, then, you can simply use "map_you_want_to_parse.insert("other_info", qvariantmap_containing_the_other_json)" To get a new JSON key in your existing map. [10:03] davidcalle, awesome, thanks! [10:03] davidcalle, I'll try asap, if I'll have other questions I'll ping you ;-) [10:03] rpadovani, np, if you have any issues, ping me :) [10:03] thanks! [10:12] popey, ping [10:12] ahayzen: pong [10:12] popey, could you build a new click please ? :) [10:12] popey, readme has been updated [10:12] from what bzr rev? [10:13] popey, 736? (latest remix) [10:13] k [10:16] popey, assuming you think it is good, me and victor agreed last night that this click could be a candidate for the store [10:16] ok, I'll get the wheels in motion [10:17] popey, thanks :) [10:17] thank you! [11:00] where can i find the source code of preinstalled ubuntu touch scopes? [11:00] dslul: any in particular? [11:00] yes, the software center scope [11:00] lp:unity-scope-click i think [11:01] yeah https://launchpad.net/unity-scope-click [11:01] thank you :) [11:02] np [11:07] dpm: pushed [11:20] brendand: hey [11:21] brendand: it means that the bottom edge wasn't visible and happens when the clock app didn't have focus on start. [11:21] brendand: I remember briefly talking about this with you earlier. And you said there was an upstream bug reported about this. [11:22] nik90, that is meant to be fixed now [11:23] cool, thanks mzanetti [11:25] brendand: are these failures consistent? [11:25] nik90, yeah [11:30] hmm [11:32] davidcalle, I have a very strange issue... on a PC the code works, on another PC doesn't work with this error: http://paste.ubuntu.com/8962497/ [11:32] Both utopic, both 64bit, both with last updates [11:37] rpadovani, ouch, that's an annoying issue with the API: the path to the scope ini file is too long and some part of the API stack doesn't like it. I guess your user name on the pc it doesn't work makes the path too long. Sending that to an API dev to see if there is a workaround until the big is fixed... [11:37] thanks! [11:55] balloons: would you be able to join the clock app uos session today? It would be nice to discuss things with you about the testing suite ;) ? [12:14] is there any way to see the newest apps in the store? [12:14] sadly not :(( [12:15] the only way I discover new stuff right now is if I check something else, or search, and spot something new by chance… [12:15] https://search.apps.ubuntu.com/api/v1/search?q=architecture:armhf&size=1000&page=1 ☻ === rmescandon is now known as rmescandon-brb [12:17] rpadovani, looks like there is no easy solution for now, except changing the domain or name of the scope to something shorter :/ This is really impractical, I'll get back to you in the afternoon if I have more info. [12:17] dpm ^ [12:17] davidcalle, thanks! [12:17] rpadovani, np [12:18] popey: is that API documented? [12:18] is it always ordered by most recent addition? [12:19] Hello there, I need some help with setting up gettext() in a scope.. [12:28] hi [12:28] kalikiana: ask JamesTait [12:29] does anyone here know anything about packaging? [12:30] kalikiana, most relevant match for search keywords wins; ties are decided by average rating; ties after that point are ordered most-recently-updated first. [12:31] kalikiana, so if you entered no search terms, popular, recently-updated packages should come first. [12:32] rpadovani, can you send an e-mail to the phone mailing list with your problem? We can get a scopes API developer to look at it there [12:35] zbenjamin: works!! even when adding a new kit to an existing project [12:35] kalikiana: awesome - o :) === rmescandon-brb is now known as rmescandon [12:42] JamesTait: cool, exactly what I want then - one more q: is there a way to get a URL that can be used to install it? [12:44] hi,I am using the ubuntu-sdk and when I go to the publish tab I can't create click packages.The "click" column is missing. [12:44] zmaj: disabled? or completely absent? [12:44] absent [12:45] kalikiana, yes, but you'll need to be authenticated to be able to download. In the search results (clickindex:package object) is a 'self' link - that takes you to the full package details. Follow that link, and you want the download_url property. [12:45] zbenjamin: hmmm not sure I've seen that - for me occasionally the device isn't being detected so I re-create the kit and that does it - might be worth a try still [12:45] er [12:45] zmaj: ^^ [12:45] sorry [12:45] yeah [12:46] kalikiana, alternatively, you can provide the fields parameter: https://search.apps.ubuntu.com/api/v1/search?q=architecture%3Aarmhf&fields=title%2Cdownload_url&page=1&size=1000 [12:47] kalikiana, and then you get the download_url directly in the search results. (For clarity, I just added `&fields=title,download_url` to the URL popey gave you.) [12:51] JamesTait: I was more thinking of the equivalent of the "install" button in the store scope; I don't really need to save the file, I'd just like to see new apps and install them right away [12:52] kalikiana, AFAIK the only way to install them atm is to download the .click file and then install it with either `click install` or `pkcon install-local`. [12:54] kalikiana, to help with the download part of that and deal with logging in and such, I hacked together https://code.launchpad.net/~jamestait/+junk/click-support-tools [12:55] kalikiana, note the word *hacked* in there. ;) It isn't pretty, but it does work. === jgdxx is now known as jgdx [13:01] in qt creator...in my devices tab,I only have a broken 'cat:' emulator...whenever I try to remove it,it reappears when I restart qt creator [13:25] dpm, mhall119: do you know if we meant to have the ToC at the top of http://developer.ubuntu.com/apps/platform/guides/online-accounts-developer-guide/? [13:30] dholbach, I don't know, I think mardy passed the html doc to mhall119 [13:31] dpm, dholbach: correct, I prepared the HTML file as you see it, but then I don't know if the ToC should be there or not [13:32] dpm, will do [13:32] dholbach: I think that at list the biggest sections should have a ToC: "OA for application developers" vs "OA for service developers" [13:35] mardy, I'm happy either way - I just felt like I hadn't seen this on other articles on developer.u.c before [13:35] and just wanted to ask back === dpm_ is now known as dpm [13:52] ubuntu online summit to start in 7 minutes: come join http://summit.ubuntu.com/uos-1411/meeting/22327/intro-by-michael-hall-keynote-by-rick-spencer/ [13:53] sigh, I'm at uni all today, cannot join anything :/ [13:53] rpadovani, dawww [13:54] can you skip class and tell your prof that you got super important stuff todo? [13:54] akiva-thinkpad, lol, I already skipped two weeks to go to devices sprint, cannot skip other lessons [13:55] heh [13:58] rpadovani: if you put only one earplug, with the other ear you can still listen to the lectures :) [13:59] t1mp, could be interesting :D [14:00] rpadovani, I support you doing well in school. No worries about missing the sessions [14:01] :-) === rmescandon is now known as rmescandon|lunch [14:15] JamesTait: I think I'll make do with typing the names in the store for now… maybe there's even a slim chance this actually makes it into the store proper some day [14:15] thanks anyway [14:16] different thing: is there a way to let a web app keep running while music is playing? libre.fm keeps stopping in the middle of a song [14:16] kalikiana, no [14:16] meh. today is not my lucky day I guess :-] [14:16] kalikiana, but there is work to integrate the webapp backend with media-hub [14:16] shoul eb done by RTM hopefully [14:17] so your playback will go on, the app will be stopped though [14:17] kalikiana: when I have a bit of free time I'll write a proper libre.fm client rather than just the webapp [14:18] kalikiana: then I can get it to give media hub a big playlist in advance [14:18] just a reminder; the online summit has started https://www.youtube.com/watch?v=PvKXxp2y3xw#t=16 [14:19] join #ubuntu-uds-plenary [14:19] Elleo: that would be sweet! [14:19] * kalikiana hugs Elleo [14:19] :) [14:20] kalikiana: also vaguely working on a libre.fm scope: https://lh6.googleusercontent.com/-vfuw3Nn_7Co/VFbXrpApqLI/AAAAAAAAAp0/OqrBV_sZ0cY/w426-h757/librefm-scope.png [14:20] kalikiana: but that's more to be able to dowload music straight to the music app, rather than for listening to the radio streams [14:26] Elleo: hmm for me it's mostly: listen, listen, listen, listen, download, listen… [14:26] looks nice, though [14:27] kalikiana: yeah, that's my general approach too; but that doesn't fit so well with the scope concept, so I'm probably not going to be doing radio stuff in the scope [14:28] kalikiana: and the app will have a download button that sends stuff to the music app (or anything else that receives music) too [14:29] Elleo: maybe there can be a Listen button per album or artist [14:30] and behind the scenes whether it's streamed or cached, I don't mind, I just don't need to download everything [14:31] (well, at least as long as I have no spare SD for this phone, I can't download much :-|) [14:31] kalikiana: yeah, that'd be good === oSoMoN is now known as oSoMoN|afk === rmescandon|lunch is now known as rmescandon [15:03] if i have a list type in a setting, how can i get the string selected? config["listname"].get_string() does not work [15:03] nik90: you around ? [15:04] mihir: yeah, attending the uos sessions, what's up? [15:04] nik90: great , got some question if you have time !! [15:04] mihir: shoot [15:05] nik90: with respect to new design , we might have to use bottom edge with multiple views , then how do we maintain pagestake ? [15:06] mihir: what do you mean? You can continue to use pagestack alongside the bottom edge component. [15:06] mihir: clock app uses a regular pagestack combined with the bottomedge [15:08] nik90: hmm okay. [15:19] akiva-thinkpad, I placed your session on the schedule tomorrow [15:19] dholbach, cool beans [15:20] brilliant [15:20] nik90: i see this error and also noticed that you've encountered the same issue [15:20] qml: WARNING! Do not put Page/Tabs/PageStack inside another Page because that causes confusion which is the active page that sets the title and actions. [15:20] nik90: can we ignore this , application is behaving as exptected though. [15:20] expected* [15:21] mihir: yeah that can be ignored. I see that in the address book as well === oSoMoN|afk is now known as oSoMoN [15:50] UOS session about "App/Scope development training events" - starting in 10m at http://summit.ubuntu.com/uos-1411/meeting/22365/community-1411-appdev-events/ === akiva-thinkpad is now known as akiva-yellow-roo === chihchun is now known as chihchun_afk [16:11] nik90: okay great :) [16:45] balloons: ping [16:46] popey: bottom edge is on his way :D [16:46] \m/ === chihchun_afk is now known as chihchun === akiva-yellow-roo is now known as akiva-is-yellow [17:01] mihir, pong [17:01] balloons: i am trying to implement bottom edge in calendar. [17:01] balloons: how can we achieve this in AP, i have no clue. [17:05] mihir, totally doable. you just generate the swipe. [17:05] mihir, the clock ap test has this. you can copy [17:06] balloons: okay, great , let me look at that. === rmescandon_ is now known as rmescandon_|afk [17:10] nik90, clock asks for location now? [17:10] anyone interested in testing/ reviewing U1Db.Query improvements to export role names? https://code.launchpad.net/~kalikiana/u1db-qt/indexRoles/+merge/211771 [17:10] * kalikiana added some tests and improved the example [17:11] brendand: yes [17:11] brendand: we show it in the main page [17:11] nik90, could that be causing the test failures? [17:11] it was in the pipeline for a long time. Was waiting for the location service to be reliable before merging it. [17:11] brendand: hmm I doubt it [17:12] brendand: the bottom edge relies on the alarm model to be loaded before showing it. [17:12] nik90, so you added code to dismiss the dialog? [17:12] brendand: no I didn't [17:13] brendand: I haven't thought about it since I usually run the tests on my laptop and phone where I have already granted access to the location [17:13] nik90, ahem [17:13] nik90, that won't fly in ci [17:13] brendand: isn't there a way to see a video of what's happening when running the test in ci? [17:14] nik90, not on the dashboard since its run on hardware there [17:14] nik90, probably there is video from the ci jenkins job [17:15] I guess then location could be the point of failure then [17:15] brendand: if this is it, then it should cause **all** clock tests to fail [17:15] not just the alarm tests [17:16] nik90, they are all failing [17:16] then location is the failing point [17:16] brendand: Do you have a link to the code that the camera app uses to get past this issue? [17:17] I can have a branch up and ready asap and you could check if that passes with ci [17:17] nik90, that would be the code where it doesn't show the trust prompt [17:17] nik90, ask Kaleo about it [17:18] brendand: ok.. so is the dialog blocked only while running the tests or all the time? [17:19] gah… again this crap trying to overwrite '/usr/lib/x86_64-linux-gnu/qt5/qml/Ubuntu/PushNotifications/plugins.qmltypes', which is also in package qtdeclarative5-ubuntu-push-notifications-plugin:amd64 0.1-0ubuntu1 [17:19] * kalikiana really looking forward to click updates on desktop [17:19] nik90, it only gets in the way of running the tests [17:19] ok [17:41] kalikiana: you got to stop signing your email since I cant read them on dekko :P [17:45] nik90: let's team up and tickle Daniel until he has no choice but to fix that :-D [17:46] :P [17:46] DanChapman: ^^^^^ [17:46] :D [17:46] * kalikiana tickles DanChapman [17:50] nik90 kalikiana: Hah!, yeah it's a right pain in the **** atm. There should be support for multipart/signed in the next few weeks. For composition though that that will be some time further down the line [17:52] balloons: i did try something similar but couldn't succeed , i'll try more. [17:52] and will ping you back [17:52] DanChapman: reading would be a great start! [17:53] mihir, kk [17:54] balloons: if you get sometime you can play around that , https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/bottomEdge/+merge/241590 === chihchun is now known as chihchun_afk === dholbach__ is now known as dholbach [17:58] * kalikiana off for dinner, very hunrgy now due to that person called cheeseburger having come up all the time in the keynote === chihchun_afk is now known as chihchun [18:12] Question; was the "Why Go" session cancelled? I don't see it on the schedule. === chihchun is now known as chihchun_afk [18:17] akiva-is-yellow: I can see it in the list of unscheduled meetings [18:18] akiva-is-yellow: I don't know what happened to it [18:18] t1mp, wierd [18:18] akiva-is-yellow: who scheduled it? [18:18] DPM I think... nik90 said he moved some things around [18:18] DPM was supposed to get the speaker. it was scheduled for today [18:22] akiva-is-yellow: dpm asked me to [18:22] i removed it [18:22] popey, okay thanks [18:22] * akiva-is-yellow goes delete the reddit post :P [18:23] heh [18:23] sorry [18:31] renatu: ping [18:31] mihir, hi [18:31] hi renatu [18:32] renatu: i am trying to use bottomEdge , but facing some issue in refreshing object, i just wanted to make sure i am not doing anything wrong. [18:32] renatu: this is what is happening ---> http://youtu.be/b5DglmkGcbo [18:32] Quote: "At the moment we do "Punch people in the face" out of the box". [18:33] the object refreshed , on second swipe . [18:33] mihir, what is the problem? [18:34] renatu: the problem is when i swipe and open bottom edge , it doesn't take latest property value. [18:34] renatu: http://paste.ubuntu.com/8968664/ [18:34] so in the above example case, dayStart keep changing on each swipe. [18:35] renatu: so it takes the older dayStart value instead of new one. [18:35] mihir, this is expected the component is create only once [18:35] renatu: how can we avoid this , as in calendar , we change values on each swipes [18:38] renatu: but when i open second time , it takes correct value , why is that? [18:38] mihir, the component is created at the application startup and stay alive until you dismiss the page [18:38] after dismiss the page it will reload the component [18:39] renatu: hmm but if you notice on video , i am not dismissing the component, i just back and open again using bottom edge. [18:39] mihir, do you want to update the "date" with a most recent value? [18:39] renatu: yes [18:39] dismissing == hide the component [18:40] renatu: here is MP , https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/bottomEdge/+merge/241590 [18:40] mihir, you can track "isReady" property (this property is set to TRUE when the page is fully visible) [18:41] or, "bottomEdgePageLoaded" which is set to "true" when the page is loaded [18:41] the second one can cause problems since the page can be loaded and stay invisible for a while [18:41] renatu: i did try that , but i might be doing it wrong. [18:42] I recommend you to create a page with empty time and set the time when the "isReady" property change to true [18:43] renatu: this is how i am doing it right now , http://paste.ubuntu.com/8969256/ [18:43] this does not work [18:43] mihir, I recommend you to create a page with empty time and set the time when the "isReady" property change to true [18:44] renatu: okay , you mean to say when currentDay time is being set , have i understood correct? [18:44] mihir, or set the bottomEdgePageSource with a QML file [18:46] renatu: okay [18:46] i'll try and will update you. [18:47] mihir, something like that: http://paste.ubuntu.com/8969351/ [18:47] mihir, this one: http://paste.ubuntu.com/8969361/ [18:50] renatu: may be this should work , let me try. [18:53] renatu: nope that didn't work, it always pass null instead of assinged value [18:54] renatu: i did this http://paste.ubuntu.com/8969463/ [18:54] renatu: if you need full source code for reference , http://paste.ubuntu.com/8969465/ [18:55] could you print the value that you have in the currentDay on the implementation of onIsReadyChanged, and check if it is a valid value [18:56] check if it is called [18:59] renatu: it is being called [18:59] hi akiva-is-yellow, I've tried hard to get a speaker, but the two I approached ended up declining, sorry :/ [18:59] renatu: it prints actual value , qml: Current Day is Thu Nov 20 00:00:00 2014 GMT+0530 [18:59] qml: Current Day is Fri Nov 21 00:00:00 2014 GMT+0530 [18:59] dpm, np; I didn't have any luck either. [18:59] mihir, ok then report a bug I will take a look later [18:59] sorry [19:00] renatu: this is how printed, http://paste.ubuntu.com/8969593/ [19:00] renatu: sure i'll do that. [19:00] renatu: shall I file it in address-book?\ [19:00] yes [19:00] renatu: okay thanks for your help !! [19:07] mzanetti, any idea what is causing the test to fail? https://code.launchpad.net/~reminders-app-dev/reminders-app/uriHandler/+merge/239961 it seems the arguments are not being read correctly by looking at [19:07] http://91.189.93.70:8080/job/generic-mediumtests-vivid/172/testReport/junit/reminders.tests.test_reminders/RemindersTestCaseWithAccount/test_add_notebook_must_append_it_to_list/ [19:08] dpm: hmm... I did drop the -testability argument [19:08] dpm: because that should have landed in Qt by now [19:08] but I can add it back if still needed [19:09] I've no idea, that's an area I'm not familiar with [19:09] dpm, davidcalle I just send a mail to ubuntu phone list about the error I have with the scope, hope it's clear enough, please add other details if you think are needed [19:09] great, thanks rpadovani! === dpm is now known as dpm-afk [19:11] rpadovani: edit mode should be ready for testing now [19:12] mzanetti, cool, I'll do after dinner [19:12] tomorrow only appointment in the morning is the reminders hangout, so I can go to bed later \o/ [19:16] rpadovani, thanks! That's very thoughtful :) [19:17] mihir, I did a quick test and the property is getting set correct, the problem is that the NewEvent page does not care about the date value after the component loaded [19:17] mihir, you need to fix that on NewEvent component [19:17] mihir, to update the fields if the value of date change [19:45] * popey pokes balloons - let me know when you're about. [20:01] popey, don't pop balloons [20:01] lolz [20:06] I'm about [20:06] what's up popey? [20:06] lets talk music [20:06] \o/ [20:07] do we have a bug for these failures? [20:07] popey, errrm balloons we had a few bugs related to music mocking IIRC? [20:09] balloons, or was that the bug about the new way to mock that i'm thinking of? [20:13] ahayzen, popey this is the metabug: https://bugs.launchpad.net/ubuntu-touch-session/+bug/1376423 [20:13] Ubuntu bug 1376423 in Ubuntu Touch session manager "Provide script to set up a temporary user session" [Undecided,New] [20:13] balloons, what/where were the changes you did to calendar and are they likely to work with music/mediascanner? [20:14] balloons, thanks for the link, too many bugs lol [20:15] ahayzen, we have been gutting all the mocking stuff out in anticipation of that bug being fixed. So we've tried not to mock anything, or as little as possible [20:16] for music that might mean the device shreds your music folder on the device when run [20:16] hmm [20:16] we sort of expect the session to be correct on device. [20:16] we can make some allowances on the desktop if they are easily done [20:20] This looks to me like something that isn't a high priority for anyone? [20:20] Except those of us affected by it. [20:23] makes me think we should submit the app for inclusion into the image given it's no worse than it was before. [20:24] +1 [20:24] ++ [20:25] balloons, is there no way to workaround the issue? (or is it not worth it?) and what is the timeframe for this new mocking? [20:26] the timeframe is not this month sadly. pitti is working on it, but hasn't started yet [20:26] so do not wait for it [20:26] ok, I'll submit it for review then. [20:26] ahayzen, the workaround is to assume a clean env on the phone [20:26] balloons, but how do we 'fix' the issue? lol [20:26] which is ok for now as in the dashboard it is clean [20:27] balloons, ugh that is nasty :/ ... i don't wanna lose my 8GB of music on device ;) [20:27] on the desktop, we can mock [20:27] ..or 50GB on desktop [20:27] ahayzen, or figure out why it's not working manually [20:27] vthompson, you been following this ^^ ? [20:27] ahayzen, I thought you had done that already so I was comfortable with music because of it [20:27] I didn't realize it broke again [20:27] balloons, i believe it may be related to that extra env var we put in to 'fix' it at one point [20:28] https://blueprints.launchpad.net/ubuntu/+spec/qa-v-testability [20:28] balloons, it has been broken for ages...but originally we thought it was autopilot itself as it was around the time of every app failing CI lol [20:29] popey, so recommendations? i guess we should investigate (remind ourselves why) it is failing and see if we can do anything for the short term? [20:30] balloons, note that a green image is one criteria we have for the golden milestone ... [20:30] so the tests need to pass [20:30] My recommendation is we get someone from the QA team to fix it. ☻ [20:30] ahayzen, so the new app Ap [20:31] the the remix AP tests; they are broken, or they work but ? [20:31] and music is the only app that currently has huge amounts of failures ... the landing team ignored them since we knew there was a new app in the pipe [20:31] balloons, they work on desktop but not device [20:31] ahayzen, and why not on device? [20:31] balloons, exactly the same as trunk [20:31] all other tests pass ... [20:31] balloons, because the mocking fails [20:31] balloons, or thats what i thought [20:32] ahayzen, ok, so we should actually look and just get them passing [20:32] i guess the question is why is it the only app that doesnt pass [20:33] therev seems t be something special about it [20:33] ahayzen, balloons sorry, I'm late to this party [20:33] ogra_, when it was working we had to do some fun and games to get mediascanner2 mocking to work [20:34] the underlying issue is there isn't a helper for mediascanner [20:34] popey, you may want to hold the click... [20:34] why? [20:34] popey, we've been talking to jdstrand about a bug and know what it is i suspect a fix will be soon [20:34] so it's up to music to deal with the fallout. The QA team is addressing this lack of testability this cycle with the services providers, but until then we have what we have [20:34] but why does that not fail in i.e. gallery ? [20:34] popey, basically delete any track that is in one of your playlists from your device and then the playlist appears empty [20:34] i assume it uses as much of mediascanner as the music app [20:35] if you don't depend on platform services heavily you aren't as encumbered [20:35] ahayzen: I will file a bug but not add it to the sheet, so I can add it to the sheet when we're ready. just ignore the bug from me which will be coming shortly [20:35] ogra_, our tests are in python not c++ so we can't access some of the mediasacnner helper methods? [20:35] balloons, ahayzen is it worth going back to the old way of not mocking just as a short term solution? [20:35] popey, cool i'll let you know when we are clear to go again :) [20:35] k [20:35] vthompson, that was my recommendation. We gutted the mocking from everyone else [20:36] ahayzen, gallery does just uas AP and pythong too i think ... perhaps it is a good start to look at these tests [20:36] *use [20:36] * ogra_ cant type anymore after a loooong day [20:36] I mean I'm sure they will work without the mocking.. you will just lose your music [20:37] ahayzen, so what branch should I pull? [20:37] If that's what we need to do to get a check mark for inclusion then I think it's the best option [20:38] I'll propose something [20:38] balloons, lp:music-app/remix is the latest remix [20:38] just making sure [20:48] mzanetti, I spent 39 minutes to test functionality, and I didn't read the code yet! [20:48] https://code.launchpad.net/~mzanetti/reminders-app/new-edit-mode/+merge/241226 [20:48] thanks balloons [20:49] it's awesome anyway, I think a lot of apps could use this code :-) [20:50] rpadovani: a lot of those issues are in the TextEdit though [20:51] meh... you've been testing too good :D [20:51] some of those I was aware, but decided its too much work... like the clicking twice of the list button [20:51] mzanetti, I just reported all issues, the ones that are about textedit we have to report upstream and link in our bugtracker :-) None of them are too annoying, could be merged right now IMO [20:52] rpadovani: I'll go through your list and fix what's fixable with a reasonable amount of work [20:53] mzanetti, thanks! [20:53] rpadovani: in the meantime I fixed this one for real: https://code.launchpad.net/~reminders-app-dev/reminders-app/uriHandler/+merge/239961 [20:53] lets hope jenkins shuts up on it [20:54] noooo. I comitted a po change [20:55] DanChapman: seen https://code.launchpad.net/~mzanetti/reminders-app/new-edit-mode/+merge/241226 ? [20:55] something that might be useful in dekko? [21:01] popey: here's a screenshot with some example stuff (in case you want to blog about it :P) [21:01] http://i.imgur.com/dejnDU7.png [21:02] * popey hugs mzanetti [21:02] popey, geez, stop suggesting dekko features before reading signed multipart messages works ! [21:02] :) [21:02] hhe [21:03] lol [21:03] patches welcome :þ [21:03] ogra_ or just don't sign your messages :-D [21:03] popey: mzanetti that looks awesome!! :-D [21:03] i could just always use dekko ... then i wouldnt sign :) [21:05] DanChapman: there's a class called "FormattingHelper" in reminders. you can attach it to a TextEdit and it'll give you nice qml properties to set this stuff [21:05] DanChapman: it still needs a bit more love... rpadovani can tell you about it :D [21:06] DanChapman: but I think its a good start [21:06] feel free to copy it and ask me for help on how to use it [21:07] DanChapman: speaking of features to be implemented before signed multipart messages, what happened to the default BCC setting? Did I just miss it or were there problems implementing it? [21:12] mzanetti: ok great thanks. I'll take a look at it later! Re: bcc that is coming in the next update. I needed to get the multiple sender identities in first. so you will be able to either multi-select any of your current identities to add to cc/bcc or provide a string list of "Other addresses". [21:13] ah, I see [21:13] cool stuff :) thanks [21:16] mzanetti, where is ! [21:16] making text red but no doesnt work :P [21:17] lol [21:17] * mzanetti checks if QML supports that [21:18] next step: smiles support, focusing on (b2b) [21:19] ogra_: nope :( [21:19] not even the WebView [21:19] lol [21:19] they are good citizens [21:19] how can they drop the 90's just like that [21:20] smileys support is something I actually miss in the messages app [21:28] ahayzen, I'm testing / playing with it now https://code.launchpad.net/~nskaggs/music-app/drop-ap-mocking/+merge/241627 [21:28] balloons, just saw that...how can i sanely test it? [21:28] balloons, move Music->Music.old or something? [21:29] as in without losing my music [21:29] ahayzen, yep.. but fair warning, I've not yet run on the device, only dekstop [21:29] balloons, i'll wait :) [21:29] desktop was still safe from music.. ahayzen to be fair there is no removal of music actually [21:29] the tests will just fail is all, and you'll get the 3 new music in there [21:29] but yea, waiting is good [21:29] ah ok balloons it just assumes there is nothing in Music [21:29] ahayzen, right [21:29] cool i understand now [21:31] balloons, i assume all the tests pass on device still :P as we've only been able to run on desktop [21:31] ahayzen, I'm thinking for the non-mock stuff on devices we won't patch mediascanner either [21:31] balloons, erm we need to somehow ensure that mediascanner2 has actually scanned the files if we don't patch [21:32] balloons, that was the reason we used to patch in the early days [21:32] ahayzen, in theory that should happen automagically right? [21:32] no harm in leaving it.. it's patching now, but . . . [21:32] balloons, problem is you don't know when..it could happen too late...unlikely but could [21:32] vthompson, any thoughts? ^^ [21:32] ahayzen, we could wait for it [21:33] balloons, i think you could probably get away with removing it as by the time the app starts and AP finds the window it will have indexed it [21:33] ahayzen, probably something worth having as a test somewhere.. maybe better in mediascanner [21:33] balloons, only bad thing i can think of is if the models suddenly refresh lol [21:33] balloons, there were talks of making there helper methods visible in python? [21:33] what do you mean? [21:33] let me find an example hang on [21:35] balloons, eg here http://bazaar.launchpad.net/~unity-team/unity-scope-mediascanner/trunk/view/head:/tests/test-music-scope.cpp#L54 [21:36] balloons, they call methods to populate ms2 i think elopio ? mentioned it would be nice if we had them in python then we could just call them [21:36] balloons, but anyway the more important thing at the moment is to get autopilot running not the patching...yet [21:37] ahayzen, ahh right, exactly. [21:37] they need to provide us with helpers [21:40] balloons, so what are the changes doing, still mocking the ms2 db? [21:41] vthompson, assuming there is nothing in Music [21:41] vthompson, at the moment yes, but trying to remove that too [21:41] * balloons testing the initial commit [21:42] blows up nicely as expected :-) [21:42] :/ [21:42] hah [21:43] "as expected" [21:43] balloons, did you move your music outside of $HOME? [21:43] If it's in home I think it will still get rescanned [21:43] vthompson, yeah this is what i was saying, but we don't know *when* it is rescanned [21:43] vthompson, I have no music on this device.. I don't think, let me see [21:43] ahayzen, right away I'd figure [21:43] vthompson, i *think* we could get away with not patching the db though? [21:44] the device will assume it's clean, and that mediascanner will pick up the files [21:44] ahayzen, exactly, I don't think we need to patch [21:44] vthompson, well if you copy something over MTP with music app running notice the delay until the models are refreshed but i think it is minimal [21:44] we can add a check to wait for it to pick them up [21:44] ahayzen, balloons but if we don't patch, we might get phantom errors if ms2 doesn't see the files in time [21:44] ...but would we still need to patch desktop? or am i now totally confused [21:44] vthompson, on the desktop the full mocking is still occuring, which we'll leave for now for convience sake [21:45] vthompson, right. so maybe we want more control, that's still a question.. I'm playing now [21:45] balloons, yea, makes sense to leave it in there. We probably just don't want to mock the db on the device [21:45] +1 [21:45] I'm shooting for no mocking on device [21:46] if you want to somewhat safely move your music so it's not seen, you can probably do a "mv Music .Music" so it's invisible to ms2 [21:47] balloons, maybe we'll need to mock the app's db though? otherwise it might see recent items [21:47] vthompson, interesting.. that's a question at you I suppose [21:47] vthompson, on device it is assuming to be a clean setup as it would be in CI [21:48] right ^^ [21:48] vthompson, so when we run it would be up to us to clear out dbs [21:48] *our [21:48] ...as in manually [21:48] lol, I have music, it sees it :-) [21:48] ahayzen, I suppose that's reasonable [21:48] and my recent [21:49] balloons, clear your .local/share/com.ubuntu.music/ [21:49] and maybe your .config/com.ubuntu.music [21:49] vthompson, what tab do the tests assume you start on? or do they just always switch straight away? [21:50] hmm.. so things are running, but mediascanner might not play nice [21:50] ahayzen, should start in Albums [21:50] each time [21:50] vthompson, yeah the app itself..but i think the tests always move to another tab anyway so it doesn't matter? [21:50] we'll start simple and have a clean device [21:51] yeah otherwise the restoring of queue could break some tests as well lol [21:53] cool [21:54] * balloons runs full test suite [21:55] balloons, are you using adt-run instead of phablet-test-run to run the tests? [21:56] ahayzen, yes [21:56] balloons, cool just checking so i know when i test :) [21:57] ahayzen, you'll need http://mirrors.kernel.org/ubuntu/pool/main/a/autopkgtest/autopkgtest_3.7.1_all.deb and make sure you pass the password for the device [21:57] balloons, ok thanks :) [21:59] balloons, ahayzen do you think the tests might have issues if during the course of the test suite it adds an item to the recent? [21:59] vthompson, i don't think so [21:59] vthompson, i don't think the tests ever look at recent? [22:00] ahayzen, yea, but it'd dictate which tab is started on [22:00] I get this when I try to run: [22:00] FileExistsError: [Errno 17] File exists: '/home/phablet/.cache/mediascanner-2.0' [22:00] vthompson, yeah but the first thing all the tests do is move to another tab? [22:00] vthompson, the bigger issue is it restoring the queue [22:00] hm, queue too [22:00] vthompson, as the queue count is then incorrect which it does check for [22:01] balloons, are we able to clear .local/share/com.ubuntu.music before/after each test? [22:01] ahayzen, vthompson we can think about wiping the db only on click.. But I'd rather not remove any files I think for safety [22:01] balloons, i think we may have to [22:02] balloons, otherwise some tests may fail [22:02] balloons, but they may be ok so try it without first [22:03] ahayzen, doing a full run, no mocking with non-clean db and non-clean mediascanner [22:03] yep :) balloons how is it going? [22:04] * balloons looks at log [22:06] ahayzen, some failures [22:11] ahayzen, http://paste.ubuntu.com/8972709/ [22:11] balloons, thanks [22:11] ahayzen, that's really raw. I'll make tweaks now to remove the db and mediascanner db each time [22:12] this one seems strange... StateNotFoundError: Object not found with name 'LibraryListModel' and properties {'objectName': 'trackQueue'}. [22:12] and StateNotFoundError: Object not found with name 'MusicaddtoPlaylist' and properties {'objectName': 'addToPlaylistPage'}. also seems odd [22:15] rpadovani: https://code.launchpad.net/~mzanetti/reminders-app/new-edit-mode/+merge/241226/comments/594431 [22:16] ahayzen, same error on music_app.tests.test_music.TestMainWindow.test_add_song_to_queue_from_albums_page even removing the db first [22:16] balloons, it seems weird surely the Queue should be there :P [22:16] ahayzen, vthompson: fyi, since I will be driving my car in a bit, I snagged playlists.js and copied it into place, and playlists work great :) [22:16] jdstrand, sweet thanks for testing :) [22:16] I'll grab the new click when it becomes available too [22:17] ahayzen, well do you want to have a play with it now or ? [22:17] would you like to merge the basic changes in https://code.launchpad.net/~nskaggs/music-app/drop-ap-mocking/+merge/241627 or ? [22:17] mzanetti, great work, thanks! I'm bit worried about the focus, but I think autofocus could help a lot [22:17] rpadovani: yeah, it helps indeed [22:17] balloons, i'm not sure why it is doing that...i can take a look... ideally we want to get autopilot passing :) so i reckon work on your mp until it passes? [22:18] rpadovani: I also move the cursor to the end so in 99% of the cases you would just start typing now [22:18] rpadovani: if you want something else, you want to tap above it and there the issue doesn't happen because there is text [22:18] mzanetti, yes, you're right. I do some tests, then I can approve it I think. Do you want to wait dpm to top approve? [22:19] I don't mind :) If I put it up for review in my opinion it's good enough :P So it's up to you guys [22:19] balloons, are all your latest changes pushed? [22:20] rpadovani: I did see some weirdness with checkboxes disappearing, so there might be a bug somewhere still... let me know if you can reproduce, otherwise I guess I'll find it sooner or later [22:21] ahayzen, yep [22:21] balloons, right now to destroy my device lol [22:21] ahayzen, we can work it till things pass no worries [22:21] :) [22:21] ahayzen, lol, backup your Music folder and everything should be fine [22:22] moved my Music folder to Music.old, created a new Music folder and then restarted [22:22] ahayzen, I think vthompson said to make it .Music instead so mediascanner doesn't pick it up [22:22] probably smart [22:23] ah yes [22:23] mzanetti, I can reproduce it 100% of times: the height of the image of box is set to 0: http://paste.ubuntu.com/8972832/ [22:24] mzanetti, mhh, this is simpler: http://paste.ubuntu.com/8972846/ [22:26] ahayzen, dinner time for me, back later [22:26] balloons, cool thanks for the work so far [22:26] ahayzen, let me make that branch a team branch [22:26] balloons, good plan :) [22:26] * ahayzen better have dinner soon as well [22:28] done [22:32] rpadovani: interesting... [22:32] rpadovani: ah... I know what it is [22:32] you don't have grid units exported [22:32] will fix [22:33] mzanetti, great! [22:36] rpadovani: pushed [22:37] balloons, "Failed to update click AppArmor rules: no root privileges in testbed" [22:38] mzanetti, works well now :-) I also read the code and, as I understand, seems good, so approve and top approved :D [22:38] awesome :) [22:40] Well, as soon as my network works :/ [22:44] :D [22:44] rpadovani has netnonworks [22:45] rpadovani: the next reminders release is really going to be a great one :) [22:47] mzanetti, sure! It will be legen - wait for it - dary :D [23:13] ahayzen, did you pass -p ? [23:13] balloons, yeah realised that just then mucked up the actual cmd [23:13] now whats happenning "adt-virt-ssh: WARNING: ssh connection failed." [23:13] adt-run ubuntu-calculator-app/ com.ubuntu.calculator_1.3.283_all.click --- ssh -s adb -p 1234 [23:15] balloons, i'm doing ... ADT_AUTOPILOT_MODULE="-v music_app.tests.test_music.TestMainWindow.test_play_pause_library" adt-run --click com.ubuntu.music --- ssh -s adb -p PASS [23:15] ahayzen, looks good [23:15] ahayzen, well actually... [23:15] it did work but now it doesn't [23:15] ADT_AUTOPILOT_MODULE="-v music_app.tests.test_music.TestMainWindow.test_play_pause_library" adt-run test_folder --click com.ubuntu.music --- ssh -s adb -p PASS [23:16] balloons, test folder being? on the desktop? [23:16] yep, pass the branch folder [23:16] since you want adt to use the tests from the local branch not the ones from trunk [23:17] well i may have pushed them manually by my adb seems broken i'm restarting [23:18] balloons, what does this mean? "adt-virt-ssh: WARNING: ssh connection failed. Retrying in 3 seconds..." [23:18] means what it says.. is everything connected? [23:18] balloons, i can run adb shell or phablet-shell and do what i want [23:18] is the password correct? [23:18] yep [23:19] without a password it actually connects...just then can't install packages [23:21] I get Ran 18 tests and 5 failed.... one was probably because I forgot to delete my playlists [23:22] vthompson, \o/ [23:22] vthompson, after the playlists test i guess it would need to clear the db or is there only 1 test that touches playlists? [23:23] vthompson, do you know which 5 failed? [23:23] ahayzen, getting a list now [23:26] http://paste.ubuntu.com/8973779/ [23:26] ahayzen, ^ the list [23:27] vthompson, interesting [23:27] ahayzen, I'm running again just to verify it's the same [23:27] vthompson, can you try with clearing the db before running [23:28] ahayzen, yea, but that'd really only affect the first test. the only other thing we store is the queue [23:28] vthompson, as in on each run so it is afresh for each test [23:28] ahayzen, hmmm, the queue is probably what's doing it! [23:28] vthompson, just add to the setup and delete of .local/share/com.ubuntu.music [23:29] right, but I think the issue is we check the queue and assume it should be 0 [23:29] it can be whatever it was previously [23:29] * balloons embarks on an upgrade [23:30] vthompson, yep and u may need to clear the settings as well otherwise it'll try to restore the queueIndex ? [23:30] ahayzen, yea, but that should be valid, right? [23:31] vthompson, not if your not restoring the queue? as we'll be clearing the queue [23:31] vthompson, or are you saying we should mod the tests? [23:32] I say we should just mod the tests, I guess. Maybe we should mod the setup you're saying? [23:32] ^ I suppose moding the setup might be better [23:33] vthompson, yeah i don't mind either [23:33] vthompson, *some* of the tests read the current queue count and then figure out what it should be but others assume 0 [23:33] vthompson, i guess we should really assume a clean env between each test? [23:33] ahayzen, yea, that'd be the best methodology [23:33] vthompson, so clearing the db/settings is the better option assuming ^^ ? [23:34] This time I got 3 failures, but 2 of them were not in the same list of failures last time? [23:34] ahayzen, agreed [23:34] magic [23:34] vthompson, we may have some waits to add with the defer'd/incubating we have now? [23:35] ahayzen, that's also possible [23:35] vthompson, what were the failures? [23:35] ahayzen, I'm using phablet-test-run so I can pick off and debug certain tests [23:36] vthompson, i was just wonder what the traceback was that usually gives you a clue [23:36] This time? It was test_mp3, test_artists_tab_album (like last time) and test_swipe_to_delete_song [23:37] vthompson, could you put the full output into pastebin? [23:38] yea, it almost looks like the app crashes [23:38] oh god [23:39] ah damn i was using the wrong ap tests no wonder phablet-test-run wasn't working :P [23:40] ahayzen, you still didn't/hadn't done a provision? [23:40] vthompson, no i pushed the tests from on of my columnflow branches :P [23:43] ahayzen, here's a single failure http://paste.ubuntu.com/8974061/ [23:44] hm, I get some apparmor related failures, I think, now [23:44] yeah i'm getting loads of them [23:44] this used to be easy to run [23:46] i managed to run a test \o/ [23:46] ahayzen, maybe I don't get the failure when I run the full suite [23:47] vthompson, have you added lines to clear the dbs before/after each test? [23:47] nope, have not done that yet [23:52] vthompson, i guess it should restore the settings as well incase the test assume that repeat/shuffle are set to their defaults? [23:52] vthompson, although i *think* most of the tests cover that [23:55] vthompson, on my first run I got... Ran 18 tests in 370.854s FAILED (failures=5) [23:57] hm, pull now and lets try. I added a change to delete the Databases dir