[00:41] popey: I read all pings on IRC when I come back from work so if you need anything just write it and I'll read it when I can [02:42] does anyone know how to fix the orientation of an QML application? thanks [03:35] liuxg: http://developer.ubuntu.com/api/qml/sdk-14.10/Ubuntu.Components.MainView/#automaticOrientation-prop [03:36] mhall119, thanks for your reply. But I would like it to be horizontal display in the first place. [03:37] mhall119, it is auto, meaning that it could be portrait mode as well. Some of the games may like to be in the landscape mode, for example the car racing game. === chihchun_afk is now known as chihchun === chihchun is now known as chihchun_afk [05:28] good morning [06:50] mardy, good morning. Sorry to keep pinging you on that, but I'd like to have this fixed for Evernote to review the API keys. I'm still not able to add an account with the latest changes suggested, still getting the blank Accounts screen. Do you spot anything unexpected on this latest click? -> http://people.canonical.com/~dpm/click/com.ubuntu.reminders_0.5.230_armhf.click [06:51] dpm: hi! Did you try with a fresh image (wiping user data)? [06:52] dpm: I'm asking this, because even 0.5.229 was working for me (it has some other issues, but the account creation was working) [06:52] s/has/had/ [06:56] ogra_: hey :-) yes the haptic feedback should be fixed in the next release. tbh I hadn't even noticed it wasn't working :-/ === DanChapman_ is now known as DanChapman [06:58] mardy, hm, I'm using the device as my main phone, I wouldn't be too fond of wiping user data. Is there not another alternative? No way to debug why the blank screen is showing up? [06:59] dpm: well, yes, the usual way: [06:59] export OAU_LOGGING_LEVEL=2 [07:00] export OAU_DAEMON_TIMEOUT=9999 [07:00] killall online-accounts-service [07:00] online-accounts-service [07:00] then try again [07:06] mardy, actually, no need to wipe the device, I'll test on an emulator [07:42] Morning all [07:42] morning [08:06] Good morning all; happy Psychotherapy Day! :-D [08:07] dpm, code fix for localised departments and highlights in click index landed last night - just need to upload the translations now! [08:11] dpm: any luck? [09:04] JamesTait, awesome! [09:05] mardy, I left the emulator creation running while I was doing something else. It's finished now, so I'll give it a go [09:07] popey: morning :-), how can i trigger a rebuild for https://code.launchpad.net/~dpniel/ubuntu-weather-app/timezonePlugin/+merge/235389 [09:08] i can [09:08] running now [09:09] popey, great... thankyou [09:10] failed again [09:10] http://91.189.93.70:8080/job/generic-mediumtests-utopic-weather-app-python3/12/console [09:16] DanChapman: merge trunk? [09:16] popey: yeah forgot about that, just doing it now. [09:25] popey i don't get why it's failing on qtdeclarative5-dev being an unmet dependency. Could you get someone to take a look at it for me please? [09:29] dpkg-buildpackage: source distribution raring [09:29] thats why [09:29] should be building on utopic i think [09:29] * popey asks in -ci-eng [09:35] nik90: hey [09:40] DanChapman: try changing the control file and bump from raring to utopic? [09:51] mardy, good news: on the emulator, starting Reminders and clicking on "Add account" worked. However, after logging into Evernote, the app does not detect that there is an Evernote account. I.e. u-s-s-o-a returns to the app, but the app still prompts to add an account (previously it just loaded the account's notes directly). On u-s-s-o-a accounts I see the account, but no app is authorized to access the account: http://i.imgur.com/7o3Ax5B.png [09:51] so progress, but still no joy [10:08] hi [10:09] popey: still the same after bumping to utopic http://91.189.93.70:8080/job/generic-mediumtests-utopic-weather-app-python3/14/console [10:22] zsombi1: ping [10:46] DanChapman: wanna join us in -ci-eng ? [11:00] mihir: pong [11:02] oSoMoN, wow, not sure if you've seen this video, but the browser demo around minute 7 is incredibly fast! https://www.youtube.com/watch?v=TOuSfdCxekQ [11:03] well, at least to load [11:03] then the guy goes on to point some flaws [11:06] zsombi1: for tagging filters in event , we should use this in DetailFeildFilter , is that correct , http://developer.ubuntu.com/api/qml/sdk-1.0/QtOrganizer.Tag/ ? [11:07] zsombi1: we are filtering using this http://developer.ubuntu.com/api/qml/sdk-1.0/QtOrganizer.DetailFieldFilter/ [11:07] mihir: I guess so... renato___ might know more [11:08] dpm: OK, that is what I had with 0.5.229 [11:08] dpm: I'll try the latest one soon [11:08] mardy, ok, thanks. The link is here if you need it handy: http://people.canonical.com/~dpm/click/com.ubuntu.reminders_0.5.230_armhf.click [11:10] or here if you want to give it a go on the emulator: http://people.canonical.com/~dpm/click/com.ubuntu.reminders_0.5.230_i386.click [11:13] dpm: the manifest file still has "account-service": "share/accounts/services/com.ubuntu.reminders_evernote-account-plugin.service"... [11:13] argh [11:13] :-) [11:13] so it was not getting installed [11:16] dpm, thanks for the link! the browser loads really fast indeed, well done me :) my Italian is a bit rusty but I understand that the guy seems very disappointed by the rendering of theverge.com, I’m seeing something similar on krillin, to some extent, but not that bad [11:22] renato___: you about ? [11:31] mardy, I'm about to give up. I fixed the manifest, but still no way to grant access to Reminders to the account: http://people.canonical.com/~dpm/click/com.ubuntu.reminders_0.5.231_armhf.click [11:35] dpm: share/accounts/services/com.ubuntu.reminders_reminder.service [11:35] dpm: should be share/accounts/services/com.ubuntu.reminders_reminders.service [11:35] :-) [11:36] please excuse me while I go bang my head against the desk... [11:37] dpm: np :-) [11:37] happens to me all the time [11:37] (that's why the first thing I do when reviewing others' code is checking for spelling mistakes :-) ) [11:38] nik90: testing your location finding click [11:38] "Location Service Error!" in the clock [11:39] popey: did you follow the steps in the MP description? [11:39] popey: also which image and channel? [11:39] of course not! ☻ [11:39] * popey looks for the merge [11:39] popey: https://code.launchpad.net/~nik90/ubuntu-clock-app/implement-location-finding/+merge/231793 [11:39] ta [11:40] ubuntu-touch/devel-proposed 255 [11:40] lemme go through that merge [11:40] * nik90 looks at http://onlineslangdictionary.com/meaning-definition-of/ta [11:40] uk slang :P [11:41] hehe [11:41] it's often the first word a baby is taught [11:41] "Say taaaaaah" [11:41] I kept wondering what the hell "ta" meant everytime I saw you use it..I guess I know now [11:41] both my kids, their first word was "cat" [11:41] hehe [11:41] thats proper ! [11:44] nik90: did the prep steps, rebooted phone, opened clock, "Location service error!" [11:44] popey: can you close clock and run "sudo start ubuntu-location-service" then reopen clock [11:45] popey: btw did you get the location access prompt at all? [11:45] yes [11:45] i did [11:45] eh strange [11:45] start: Job is already running: ubuntu-location-service [11:45] still same [11:45] meh I need to reflash my phone back to utopic to test this out :/ [11:48] popey: I will report back in about 30 mins [11:48] mardy, success! I could get it to work on the emulator. Now I just need to figure out if it works on my phone and if it works on the packaged .debs to pass the Jenkins tests [11:51] dpm: it's probably because libaccounts is caching the wrong .service files [11:53] dpm: do "sqlite3 ~/.config/libaccounts-glib/accounts.db" [11:53] dpm: then give the ".dump" command, and paste the output (there shouldn't be a lot of sensitive info, just your account names) [11:55] ok, cool, just upgrading my phone [12:03] popey: I know how improbable this sounds but I promise this is not a bot and I really am up at six o'clock in the morning [12:03] :D [12:03] hello frecel [12:04] you needed something yesterday? [12:04] yeah, you have pm ☻ [12:06] zsombi1: okay , i'll speak to renato___ once he is around [12:06] done [12:07] thanks frecel ! [12:08] popey: you have no option to do it yourself or were you just being polite? [12:09] i do have the option to do it myself, yes [12:09] but i think that's rude ☻ [12:11] Well it would be if I wasn't informed about it before but since we had a conversation about it a while back I wouldn't care if you unpublished it yourself [12:27] mardy, here's the output of the commands, not sure if the logging is right, couldn't manage to kill the UOA service (no killall command on the rtm distro, pidof returns nothing for u-o-a-s) http://pastebin.ubuntu.com/8425156/ [12:27] although L14 looks suspicious [12:28] and L15 even more [12:29] dpm: L14 is the problem, it looks like it cannot load the Evernote plugin [12:29] dpm: what version of ubuntu-system-settings-online-accounts are you running? [12:30] mardy, 0.4+14.10.20140908~rtm-0ubuntu1 [12:31] dpm: you need 0.4+14.10.20140917-0ubuntu1 [12:32] mardy, aha - do you know when that is going to be uploaded to ubuntu-rtm? [12:33] popey: can we get store update for calendar tomorrow eod ? [12:33] popey: there are couple of MR pending i'll close them and then we can push it for again weekend testing :) [12:34] mihir: we sure can! [12:34] popey: great :) [12:34] I'll be afk on a train tomorrow, but if you ping balloons he can do the necessary [12:34] (I actually can't upload but he can) [12:36] dpm: no, let's ask dbarth [12:39] mihir, hi [12:39] hey renato___ [12:40] mihir, what do you need? [12:41] renato___: we were suppose to exclude disabled alarm from the filters [12:41] renato___: so for tag filters , i did try this but i couldn't find proper syntax to do that , http://developer.ubuntu.com/api/qml/sdk-1.0/QtOrganizer.Tag/ [12:42] renato___: for more info, https://bugs.launchpad.net/bugs/1361702 [12:42] Launchpad bug 1361702 in Indicator Date and Time "clock-app alarms need to be handled differently from other VTODO events" [Medium,Triaged] [12:42] renato___: as we are filtering events using http://developer.ubuntu.com/api/qml/sdk-1.0/QtOrganizer.DetailFieldFilter/ [12:42] mihir, alarms are different from events [12:43] mihir, each event can or can not have a alarm attached to it [12:43] nik90: ^^ [12:43] mihir, I believe that, if the event has a alarm attached it should be handle as a normal alarm [12:44] actually zsombi1 ^^ [12:44] renato___: the issue is , if the alaram is disabled , it shows up in calendar notification like this [12:44] renato___: https://launchpadlibrarian.net/183212190/disabled-alarm.png [2] https://launchpadlibrarian.net/183212222/calendarevent.png [12:45] renato___: as per yesterday's discussion with zsombi1 and charles we have to change filters in calendar [12:47] mihir, anyway you can filter alarms created by the sdk alarm API using the organizer filters [12:47] renato___: okay thanks. I'll try that and will let you know. [12:47] since but you need a way to diff these events from a normal event [12:47] how do you intend to do that? [12:48] renato___: as they have added tags to disabled alarams , so i guess we can exclude all disabled alarms with tag [12:49] mihir, in this case you will exclude only disabled events [12:49] mihir, do you want to exclude all events? [12:50] renato___: this is what we conclude yesterday, 2. ubuntu-calendar-app will ignore / not show anything with the "x-canonical-alarm" tag [12:50] thanks mardy === _salem is now known as salem_ [12:50] mihir, ok this is easy [12:51] mihir, just use a union or intersection filter with your current filter and a new detail Filter [12:51] renato___: okay , so we can modify this http://paste.ubuntu.com/8425302/ [12:52] and add one more Detail filter :) [12:55] mihir, well a bit more complex than that. Since there is no "matchFlag" for different [12:55] let me think about it [12:58] renato___: Ohh hmm okay. [13:01] mihir, I am not sure how to do this filter :D [13:01] mzanetti: hi, I put 2 UI concept images in that folder [13:01] mivoligo: cool, thanks [13:02] renato___: ahhh :( can't we just add tags to each filter and exclude them [13:02] mivoligo: looks nice. how would this work if there are already multiple level packs installed? [13:03] mihir, I did not find a way to create a filter that exclude a event with specific detail [13:03] mzanetti: let me know if you understand what's going on on them, I did't put any description :P [13:03] mzanetti: instead of + it will be > [13:03] mihir, o short time solution will be make this evens invisible on the UI [13:03] and + after the last levelpack [13:04] mihir, something like "visible: tag != "x-alarm"" [13:04] mivoligo: should this be some sort of carousel, or buttons? [13:04] mzanetti: can it be both? [13:04] sure [13:04] looks like a missing feature on QtOrganizer [13:04] :) [13:04] renato___: then that fix should be in UI :P [13:05] mivoligo: ok. one thing: it misses the difficulty selection [13:05] mzanetti: I'll put settings in separate window [13:05] mihir, until we have a proper solution on QtOrganizer, but this will take a while since we need to propose a API change [13:05] mivoligo: hmm... I don't consider difficulty being a setting tbh... more like a selection of what to play [13:06] zsombi1: charles ^^ [13:06] I will try discuss that with qt guys, to see if they have a solution for that [13:06] mihir: ?? [13:06] mihir: like if you completed the game on easy, you select "medium" and re-enter the level pack, everything will be locked again [13:06] mivoligo: ^ [13:07] mihir: sorry again... [13:07] mzanetti: hmm.. in my view you'll be able to have different settings for different levelpacks [13:07] zsombi1: renato___ said that we'll have to do that from UI way , until we have proper solution from QtOrganizer [13:08] mihir: filtering out the tag? yes, we said that also [13:08] mihir: renato___: we will drop QtOrganizer alarm backend when we get a datetime indicator API in place [13:08] zsombi1: yes...if you can read conversation between me and renato___ in above few lines [13:09] mihir: renato___: so we need a solution that works now [13:09] mivoligo: but its not really a setting... I think I'd really like to have the difficulty selection at the same page as the level pack selection... unless you totally disagree... [13:09] mihir: I read it [13:09] mivoligo: I'm afraid people will miss it otherwise too [13:09] zsombi1, I sugest something like: "visible: tag != "x-alarm"" [13:10] renato___: ok, that's good to me, actually it has to be OK for calendar app... [13:10] mihir, zsombi1 or you can filter only events with empty tags. But this is not safe [13:10] renato___: that would be little risky i believe [13:11] renato___: I do not need to filter anything. Every alarm has a tag: x-canonical-alarm set, and an additional one if it's disabled [13:11] zsombi1, the problems is that I did not find a way to create a filter where you exclude a value from the filter . you can only add values [13:11] renato___: mihir: so it is safe if you say an item is an alarm if the x-canonical-alarm tag is there [13:12] renato___: I saw that... and that's bad... [13:12] zsombi1, yes I know [13:13] mzanetti: hmm, what would you say if a player could select difficulty after hitting "play"? [13:14] mivoligo: would work, but must happen before the level selection screen is loaded [13:15] mivoligo: I personally would probably still put it on the same screen... [13:15] otherwise we have to add a new page step just for that [13:15] so renato___ i am not sure if from unionFilter we intersect this filter , i am not sure [13:18] mzanetti: you're right [13:20] mzanetti: so what we leave in settings would be: sound volume for effects, sound volume for soundtrack and reset score, right? [13:21] mivoligo: yeah... I think we should also have the other options... but just not show them by default [13:22] mivoligo: but for example if you press a couple of times on the icon in the about dialog you'll enter level pack creator mode [13:22] which would enable the other options too === salem_ is now known as _salem [13:22] and tell people in the level pack creation tutorial how to enable that for easy testing [13:23] or at least show them if you manually build the game for the desktop, which is most likely where you're creating the levels anyways [13:24] mzanetti: or we can put that enabling option in the tutorial [13:24] or that, yeah... fine with me too... depends a bit on what the turorial will look like [13:24] I thought about opening the browser with a link to my blog [13:25] and do the actual tutorial on a website [13:25] me too [13:27] mivoligo: here's the draft I'm working on: http://notyetthere.org/?p=421&preview=true [13:27] not sure if that link works for you [13:28] slightly outdated already... [13:29] mzanetti: Not Found [13:31] mivoligo: hmm yeah, its just a draft, not published yet [13:31] I think only when logged in as the editing user you can see it [13:31] mivoligo: well, not that important right now anyways [13:32] ok [13:32] zsombi1: so are we going for change in UI ? [13:32] mihir: looks like... .if you have the model in UI, then you have to change that... [13:41] zsombi1: that is in datetime-indicator right , i got lil confuse though sorry about that [13:42] mihir: me too... when U say UI, what are you referring to? UI is the User Interface of an app, a module, etc [13:42] mihir: so your app has a UI as well... [13:43] zsombi1: as I have understood renato___ suggested to change indicator UI (Not calendar-app) and make visible false which has tag [13:43] mihir: hehh???! [13:44] mihir, I was suggesting to change the app [13:44] mihir: ^ [13:44] renato___: ahh , got it :| i mis understood [13:44] mihir: sorry, i though the same way as renato___ :) [13:44] zsombi1: renato___ sorry i miss understood :) [13:45] mihir, on the event delegate make it invisible if it is a alarm [13:45] renato___: okay got it now :) [13:46] zsombi1: sorry for the confusion :| [13:47] mihir: np :) it happens.... [13:56] JamesTait, I've just seen your G+ post, nice work! Are all translations uploaded now? I can still see the departments in English only [13:57] dpm, I think we have Spanish translations uploaded, plus the one string I had to update for British English. [13:58] * JamesTait switches his phone to Spanish and reboots. [13:58] JamesTait, could we upload all the languages from LP? (I'm using Catalan, not Spanish) [14:00] dpm, we can, and will, of course. :) It's just a manual process atm, and we only have a limited number of people with the appropriate privileges. [14:00] oh I see [14:01] JamesTait, have you figured out a good way to do the uploads? Are you converting the .po files to SQL, or how did you end up doing it? === justCarakas1 is now known as justcarakas [14:02] dpm, we haven't automated it at all yet - it's busywork atm, but obviously that's not sustainable. We will be improving the process, we just haven't fleshed out exactly how yet. [14:03] JamesTait, ok, no worries, just asking out of curiosity [14:05] dpm, not a problem. :) [14:05] * zsombi1 eod [14:07] mzanetti: I put some ideas in the folder [14:08] mivoligo: cool. quite busy atm. will look at it tonight [14:08] mzanetti: no problem :) === _salem is now known as salem_ [14:16] balloons, it seems on this branch Jenkins keeps complaining about an invalid xml file, did you find out what it could be? https://code.launchpad.net/~dpm/reminders-app/account-click/+merge/230768 [14:20] dpm, fginther ack'd the problem yesterday, but I never saw a resolution. Regardless we know the tests that are showing the issue; it's with the account setup. That might also be causing issues [14:20] you can still see some of the run in the console [14:22] so what we can do is propose a seperate MP to fix the issue and see if jenkins is happy with that, then land yours [14:22] but ohh right, technically you are breaking something [14:24] popey, balloons, could one of you top-approve https://code.launchpad.net/~dpm/ubuntu-rssreader-app/fix-desktop-i18n/+merge/235970 and initiate a shorts upload to the store? It's the last app remaining to fix the .desktop files localization bug that vrruiz has been chasing for a while [14:24] ya [14:25] joey beat you to it [14:25] nice [14:25] balloons, dpm, the tests are hitting a general purpose timeout in the job. Are the reminders tests expected to take a long time? [14:26] reminders.tests.test_credentials.EvernoteCredentialsTestCase.test_add_evernote_account_must_set_provider is taking 20 minutes alont [14:26] fginther, ack, that makes sense. As I was saying to dpm, the tests on that MP just locks up forever [14:27] I wouldn't expect the test to take that long, no idea why it needs 20 mins [14:28] there's an exception and backtrace in that test, that appears to be where the 20 minutes disappears [14:32] dpm, the tests are not actually running for the testwithaccount tests, they lock up during account creation. It's an old bug we had that we think might have been due to a race condition. But it's no longer present in trunk [14:33] the process is forked and it always seemed like if the child process died for some reason the parent continues to wait [14:33] balloons, so you mean in that branch I've resurrected the issue? [14:42] dpm, yea it appears so. I was assuming it affected everything, but I remember you saying other things are landing fine [14:42] let me play with your actual branch vs trunk and see [14:44] ok, thanks balloons [15:15] ahayzen, do you think we should mark bug 1370874 as invalid for Music? As in, it's not something that can be fixed in the app, it needs to be fixed in the platform, right? [15:15] bug 1370874 in Ubuntu Music App "Music would pause when BT headset is switched off" [Critical,Confirmed] https://launchpad.net/bugs/1370874 [15:16] dpm, probably :) i'll do that [15:16] * ahayzen has been playing about with a bluetooth speaker lately finding/confirming bugs :) [15:17] dpm, done [15:17] dpm, i think i poked rsalveti about that bug so i think it is on his list IIRC [15:25] hi m-b-o! thanks for your comments on bug/1355835 - I think for now, I'd suggest to make the links to be to the mobile site, and in parallel I can initiate a conversation with TWC to fix it on their side [15:25] bug 1355835 [15:25] bug 1355835 in Ubuntu Weather App "Weather Channel doesn't display forecasts in Spanish" [High,Incomplete] https://launchpad.net/bugs/1355835 [15:26] dpm: yes, I think taht would be ok! [15:27] dpm: assigned it to me, will make a fix in the coming days [15:27] awesome, thanks m-b-o! [15:27] done [15:28] dpm thanks! :) [15:33] dpm, sorry i joined a vpn and disappeared so may have missed any further messages === salem_ is now known as _salem [15:39] dpm: heh.. weather forecast in spanish is simple, just make it "cálido y soleado" :p [15:39] * t1mp is glad to be in barcelona now :) almost don't need a weather forecast anymore [15:43] t1mp, I'll be in BCN tomorrow, is the wheather nice atm? [15:43] ahayzen_, no worries, didn't say anything else :) [15:43] dpm, cool :) [15:48] dpm: not as warm as next week, and some times a bit cloudy, but for my (Dutch) standards still very good :) [15:48] dpm: no need for long sleeves or trousers [15:48] t1mp, so you're around in underwear? [15:49] I generally wear the trousers, even in Barcelona [15:49] dpm: and the sea water is warm enough for swimming :) I went camping next to the beach a week ago [15:49] dpm: parse like this: "long (sleeves or trousers)". :) I wear shorts [15:50] ;) [16:03] mivoligo: hey, looking at the images. looks good! [16:07] mivoligo: should I just use the first level and paint the name/logo on top? [16:07] mivoligo: or better shipping the whole image as is as part of the level pack? [16:07] probably the latter === gatox is now known as gatox_lunch [16:23] balloons, popey, I need to go in a few mins, can I leave the shorts update to the store in your hands? [16:24] dpm: I can't, balloons can. [16:24] elopio, when you have a moment do you mind casting your eye over this again? https://code.launchpad.net/~andrew-hayzen/music-app/add-url-dispatcher-tests/+merge/233840 [16:24] mwaaha.. power [16:26] that said, dpm jenkins is down for maintenance atm [16:26] we'lll build and release once it's back up === dpm is now known as dpm-afk [16:29] balloons, cool, thanks === _salem is now known as salem_ [16:44] ahayzen_: on it. [16:45] elopio: can you ping me later when you are free [16:45] elopio, sweet thanks... i now just need to figure out how to double check it is working on my device as for me running album:// music:// starts the facebook app rather than music lol i must have broken something somewhere aha [16:45] nik90: pong. I have two eyes :) [16:45] your branch is on my other tab. [16:46] elopio: :) the test failed for 2 reason, the first being a missing dependency which is strange since AP tests pass on that exact page. Second is a genuine failure which I can fix rather quickly. [16:46] elopio: The one I need your help with is the missing dependency issue. [16:47] nik90: the tests are run during the build, so you need to add the deps on the first section of the debian/control. [16:47] the autopilot tests are run after the package has been installed, so it has a different set of packages. [16:47] ah ok [16:48] I will check the error to see which package you are missing. [16:48] I have been doing this on trial and error for a month. I was hoping to get better identifying the deps over time, but I'm still bad at it :) [16:49] I am missing Qt.labs.folderlistmodel apparently [16:49] even though I don't see it listed as being a dep of the clock app nor AP test. but I will investigate further [16:52] ahayzen_: as you added the decorator to the url dispatcher call, you don't need [16:52] 235 + logger.debug("Calling URL Dispatcher - " + path) [16:52] elopio, ah yes good spot ;) [16:53] ahayzen_: and last pita comment, pep8 recommends to capitalize all the letter of an abbreviation. [16:54] so URLDispatcher instead of UrlDispatcher. [16:54] elopio, ah yes ok i'll do that [16:54] ahayzen_: I'm really happy with your python code. I don't know much about the QML code, so it would be safer if you get somebody to review that part. [16:55] elopio, yeah i'm gonna try and get victor to check it as well as i need someone else to confirm the tests actually run and pass [16:55] elopio, thanks for the review :) [16:55] np. [16:56] renato___, what did you and mihir and zsombi agree on? [17:07] nik90: is that centerOf function coming from UbuntuTestCase ? === gatox_lunch is now known as gatox [17:22] Akiva-Thinkpad: ping [17:22] Akiva-Thinkpad: you about ? [17:24] elopio: yes [17:25] nik90: can you do one eye check on this code , i have done it from my side if you get couple of mins [17:25] https://code.launchpad.net/~akiva/ubuntu-calendar-app/All-Day-Events-Button/+merge/235396 [17:35] balloons, ping [17:35] mihir: I can't today [17:36] nik90: no issues :) [17:36] sry [17:36] nik90: no need of that :) i just did that , but wanted to double check nothing else :) [17:36] ah ok [17:37] :) [17:41] ahayzen_, pong [17:41] balloons, Recently i have seen two separate failures on jenkins, both passed with reruns. But they are both from a similar action, when selecting a listitem action. It looks like they either missed clicking the button or nothing happened? Do you think they are worth investigating? [17:42] balloons, these are the two failures http://91.189.93.70:8080/job/music-app-autolanding/612/console http://91.189.93.70:8080/job/music-app-autolanding/604/console [17:42] ahayzen_, we don't like inconsistencies [17:42] balloons, My current theory is that the listitem is doing that bounce animation and it gets the X pos at the wrong time, so maybe a wait for a property may help? Do you think that is plausible or does this look like autopilot itself or something else? [17:42] * balloons looks [17:42] balloons, if you watch the videos on them u can see it attempt to click but nothing happens [17:43] ahayzen_, ahh, that animation isn't accounted for [17:43] balloons, i think maybe not [17:43] and you are using a helper for this right? [17:43] balloons, yep there is a helper so it should be easy to add :) [17:44] balloons, yeah swipe_reveal_actions() ends with the pointing_device.drag() so i guess that needs to wait until the animation has finished? [17:46] balloons, ok i'll create an mp to add a wait :) [17:47] ahayzen_, ohh excellent. I agreee it's not accounted for and it should be [17:48] balloons, we want an all green ap board :) [17:48] *forever* lol [17:48] what's the qt api to parse .desktop? [17:48] if that's QSettings, is there a way to query for localized values? [17:48] like value("Name") to return the Name[current_locale] [17:55] balloons, putting an objectName on an UbuntuNumberAnimation should be accessible from autopilot right? [17:57] balloons, on no worries found there is actually an alias from the upstream component... [17:58] ahayzen_, ahh yea, that's what I was going to guess [18:06] balloons, https://code.launchpad.net/~andrew-hayzen/music-app/ap-wait-for-bounce-animation/+merge/236019 \o/ now to see what jenkins thinks [18:09] awesomesauce [18:09] :) [18:22] balloons, guess we'll want to push to the store again soon as the CI fix landed last night... do we wait for ^^ mp to land first for extra goodness? [18:26] ahayzen_, I'll wait for this mp [18:27] balloons, cool thanks [18:27] balloons, would be good to be green again on CI :) [18:30] Akiva-Thinkpad: you there? [18:30] mihir, yep [18:30] x : type === ViewType.ViewTypeWeek ? root.width/7*index : 0 [18:30] got it [18:30] Akiva-Thinkpad: :D that's quick fix :) [18:34] balloons, hmmm that mp got approved by jenkins do we just top approve it between us or wait for victor to come along? it is quite a small change [18:34] ahayzen_, yea, it's trivial. [18:35] balloons, cool done :) [18:48] mzanetti: I was thinking about putting the image of last unlocked level and on top of it some artwork characteristic to a levelpack. [18:50] mivoligo: hmm... I think we have a bit of a problem then.. Or I misunderstood your sketches. [18:51] mivoligo: I would have pushed a proposal for the main screen implementation. do you have a bit of time to try it? [18:51] mivoligo: as its not really working yet I pushed to to: lp:~mzanetti/machines-vs-machines/carousel-test [18:52] mzanetti: not at the moment, kids occupying my computer :D [18:53] mzanetti: I'll be back later I hope [18:53] ok. let me know. I need to give a bit instructions on how to try that branch [18:54] mivoligo: btw, I have 6 different songs now. one for each world [18:54] probably need to reorder enemies a bit. so that we have blue enemies only in the dark worlds to fit the somewhat darker music theme etc [18:57] mzanetti: hey, would you some time to provide a second opinion on an MP? You don't need to review the entire MP but rather just the overall theme. [18:57]