| === chihchun is now known as chihchun_afk | ||
| === hyperair is now known as Guest84654 | ||
| === _salem is now known as salem_ | ||
| === chihchun_afk is now known as chihchun | ||
| === salem_ is now known as _salem | ||
| === chihchun is now known as chihchun_afk | ||
| === chihchun_afk is now known as chihchun | ||
| justCarakas | is there a way to send or read text messages in an app ? | 07:51 |
|---|---|---|
| mzanetti | nik90_: hey, tried to use the alarm clock today. doesn't seem to ring off for me. what's the state there? | 08:04 |
| mzanetti | is it supposed to work or still WIP? | 08:05 |
| JamesTait | Good morning all; happy Friday and happy World Population Day! :-D | 08:36 |
| === chriadam is now known as chriadam|away | ||
| nik90_ | mzanetti: Good morning | 08:38 |
| mzanetti | nik90_: Good morning | 08:38 |
| nik90_ | mzanetti: The alarm should work as expected. I woke up to them for the past 4-5 days | 08:38 |
| mzanetti | nik90_: hmm... can I verify on the command line somehow if the alarm is actually set? or any other means to debu git? | 08:39 |
| mzanetti | really doesn't ring ever here | 08:39 |
| nik90_ | mzanetti: the alarm is stored at .local/share/evolution/tasks/randomfoldername/tasks.ics | 08:40 |
| nik90_ | mzanetti: if you can get me that folder, I can check if EDS has saved it properly or no | 08:40 |
| nik90_ | not* | 08:40 |
| mzanetti | nik90_: ack. can't fetch it right now as I'm working on another device. but will get you that folder in a few | 08:41 |
| nik90_ | mzanetti: np, I will be here | 08:41 |
| mzanetti | cool | 08:41 |
| popey | https://github.com/loqui/im interesting | 08:55 |
| popey | its in the FFOS store. | 08:55 |
| popey | has WhatsApp support (unofficial) | 08:55 |
| davmor2 | popey: how do you change the title of a note book in reminders or for that matter delete one the only options I see when I click on a note book are search and refresh | 08:57 |
| popey | davmor2: do it on the web | 08:57 |
| dpm | davmor2, click on the edit button on the header, it should let you edit the title and the content of the note | 08:59 |
| davmor2 | popey: also if you edit a note in a notebook why does it move it auto change the notebook I'm in to first notebook. IE Notebook 1,2,3 I have a note in 3 I hit edit change the note text hit save and it default to save it in 1 | 08:59 |
| dpm | davmor2, and the notebook thing is probably a bug | 09:00 |
| popey | bugs.launchpad.net/reminders-app | 09:00 |
| davmor2 | dpm: I have =Notebooks + = if I click on the notebook in question I get < Personal nores + = hence wanting to change the title :) | 09:01 |
| dpm | davmor2, ah, sorry, I misread the notebook title change | 09:01 |
| dpm | I read "note" | 09:01 |
| davmor2 | So that is a feature request and a bug then thanks | 09:01 |
| davmor2 | dpm: you sure you want me to test this? | 09:01 |
| dpm | davmor2, indeed, it makes sense as a feature, I think | 09:02 |
| dpm | davmor2, I'm never sure to have you test things, as you always manage to break them! :) | 09:02 |
| davmor2 | dpm: also if you setup an account, and then open the app after the account is added it say you have no account add one click here, you click there it takes you to a page where you click accept and then goes back to the app, that doesn't seem right some how | 09:03 |
| dpm | davmor2, the workflow is right, but the message needs to change. The reason being, if you create the account outside the app, then you need to authorize the app to use the account the first time. If you create the account from within the app, it then does the authorisation transparently for you | 09:22 |
| dpm | but I noticed that one with the message too. We need to be able to detect whether there is genuinely no account, or whether there is an account that needs authorisation | 09:23 |
| davmor2 | dpm: oh indeed, it's not a wrong workflow but it saying there is No account when you know for a fact you just created one is the but thats meh | 09:24 |
| dpm | indeed | 09:24 |
| mzanetti | nik90_: here you go: http://paste.ubuntu.com/7779376 | 09:28 |
| mzanetti | nik90_: interesting thing that there are 4 alarms in there, should be 2 only | 09:28 |
| nik90_ | mzanetti: are you running the latest image? | 09:29 |
| mzanetti | nik90_: promoted one | 09:29 |
| mzanetti | not devel-proposed | 09:29 |
| nik90_ | mzanetti: ok | 09:30 |
| mzanetti | nik90_: but even if those 4 alarms would be what I want. I never heard any of those actually ring | 09:30 |
| nik90_ | mzanetti: does the saved alarm appear in the date time indicator? | 09:33 |
| mzanetti | nik90_: no... nothing in there | 09:33 |
| mzanetti | which probably explains why they're not ringing | 09:33 |
| nik90_ | yeah | 09:33 |
| nik90_ | can you create an alarm to ring in the next 5 minutes and see if it appears in the indicator? | 09:34 |
| mzanetti | ack | 09:34 |
| mzanetti | uuh. clock app crashed switching to the alarm tab | 09:35 |
| nik90_ | were you trying to create a new alarm? | 09:35 |
| nik90_ | or editing an existing one? | 09:35 |
| mzanetti | i was on the stop watch tab, wanted to switch to the alarms tab -> boom | 09:35 |
| mzanetti | now it works | 09:35 |
| mzanetti | nik90_: ack. set an alarm to go off in 5 mins, it appeard in the indicators | 09:36 |
| mzanetti | hmm... I have a suspicion... let me wait until this one went off and I'll play around a bit more | 09:38 |
| nik90_ | mzanetti: there is one case where the alarm might not ring, When you create a single type alarm and it rings. After which if you are try editing that alarm to set a new time, that will fail | 09:38 |
| * nik90_ checks for that | 09:39 | |
| mzanetti | that sounds like I could have fallen for a couple of times already | 09:39 |
| mzanetti | the other thing is, if you change the recurrance, I have the feeling that it changes it to the day last week or so | 09:40 |
| mzanetti | ringing now / | 09:40 |
| mzanetti | \o/ | 09:40 |
| mzanetti | ringtone is very stuttery though | 09:41 |
| mzanetti | like pauses for 3 seconds, plays half a sec, pauses again for 3 etc | 09:41 |
| mzanetti | that seems to be related to the screen being locked | 09:42 |
| mzanetti | turning the screen on makes it play smoothly, turning it off again makes it bad again | 09:42 |
| nik90_ | mzanetti: I don't notice the stuttery ringtone anymore in the devel-proposed. We had that issue a couple of images before since the system-settings was hogging a lot of CPU. | 09:49 |
| nik90_ | mzanetti: we have another bug in place where the alarm keeps ringing even after you dismiss it :) | 09:50 |
| mzanetti | nik90_: ok... | 09:50 |
| davmor2 | dpm, popey: https://bugs.launchpad.net/reminders-app/+bug/1340635 and https://bugs.launchpad.net/reminders-app/+bug/1340640 and https://bugs.launchpad.net/reminders-app/+bug/1340644 | 09:51 |
| ubot5 | Ubuntu bug 1340635 in Ubuntu Reminders app "Right workflow wrong wording on error on Initlial run of the app" [Undecided,New] | 09:51 |
| ubot5 | Ubuntu bug 1340640 in Ubuntu Reminders app "No way to Edit a notebook name" [Undecided,New] | 09:51 |
| ubot5 | Ubuntu bug 1340644 in Ubuntu Reminders app "Editing a note defaults to the First Notebook instead of the notebook it was edited in" [Undecided,New] | 09:51 |
| popey | thanks davmor2 | 09:51 |
| t1mp | nik90_: for https://code.launchpad.net/~nik90/ubuntu-ui-toolkit/improve-header-api-docs/+merge/226113 can you do a bzr commit -m "empty commit" --unchanged && bzr push, to see what happens? | 10:01 |
| nik90_ | t1mp: sure. | 10:01 |
| t1mp | nik90_: thanks | 10:01 |
| mzanetti | huh? what happened to the calculator? | 10:50 |
| === chihchun is now known as chihchun_afk | ||
| t1mp | kalikiana: up for a review? https://code.launchpad.net/~tpeeters/ubuntu-ui-toolkit/panelHeight/+merge/225493 | 11:10 |
| t1mp | kalikiana: ignore the bad jenkins results for now, its broken | 11:10 |
| kalikiana | tell me about it, I want to land stuff :-( | 11:11 |
| kalikiana | t1mp: can we have a test for the position? | 11:13 |
| === MacSlow is now known as MacSlow|lunch | ||
| t1mp | kalikiana: a test that verifies the new y-coordinate after changing height? | 11:16 |
| t1mp | kalikiana: yeah, makes sense, I'll work on that | 11:16 |
| kalikiana | cool | 11:19 |
| davmor2 | popey, dpm: I created another bug for adding new notes to notebooks, it is pretty much the same as the bug 1340644 bug but it is a slightly different workflow so created a new one do you want me to mention the other bug in both? | 11:26 |
| ubot5 | bug 1340644 in Ubuntu Reminders app "Editing a note defaults to the First Notebook instead of the notebook it was edited in" [Undecided,New] https://launchpad.net/bugs/1340644 | 11:26 |
| === MacSlow|lunch is now known as MacSlow | ||
| nik90_ | t1mp: the empty commit for https://code.launchpad.net/~nik90/ubuntu-ui-toolkit/improve-header-api-docs/+merge/226113 didn't trigger jenkins | 12:18 |
| nik90_ | zsombi: hey, Good Afternoon, you are back :) | 12:19 |
| nik90_ | t1mp: ok, I just saw the jenkins message :P | 12:19 |
| zsombi | nik90_: thanks, I'm back for few days, then will be away again | 12:20 |
| t1mp | nik90_: wow... weird failure | 12:21 |
| t1mp | hmm it ends with NoSuchProcess: no process found with pid 5311 | 12:21 |
| nik90_ | zsombi: ah ok | 12:21 |
| t1mp | nik90_: I'll happrove it anyway, see if it lands | 12:21 |
| nik90_ | t1mp: hmm | 12:21 |
| nik90_ | t1mp: ok..if it fails again, we can get the help of elopio | 12:22 |
| === _salem is now known as salem_ | ||
| t1mp | kalikiana: I re-happroved noArchErrors and englishButtons | 12:38 |
| kalikiana | t1mp: thanks | 12:39 |
| kalikiana | zsombi: added the test fixes https://code.launchpad.net/~ubuntu-sdk-team/ubuntu-ui-toolkit/blueCursor/+merge/225304 | 12:43 |
| zsombi | kalikiana: I was thinking, shouldn't we set the palette.foreground with alpha=0.2 instead of having the magic in the text inputs... | 12:45 |
| zsombi | kalikiana: you got comments | 12:49 |
| kalikiana | zsombi: "TextField changes"? | 12:51 |
| zsombi | kalikiana: you had color changes in textarea, shouldn't the same be done in textfield as well? | 12:52 |
| kalikiana | hmm I thought I did, checking | 12:53 |
| kalikiana | okay seems it's not in there | 12:54 |
| kalikiana | zsombi: if we want to have alpha in the palette there's a problem: we can't use the hex notation we agreed on | 13:02 |
| t1mp | kalikiana: why can't you use hex notation? | 13:02 |
| kalikiana | I'm not sure if in general it's expected to have alpha in the colors | 13:03 |
| kalikiana | t1mp: becasue it doesn't exist | 13:03 |
| zsombi | kalikiana: you can, you convert 0.2 into hex :) | 13:03 |
| kalikiana | zsombi: and writing it how? there's no standard for it afair | 13:03 |
| t1mp | kalikiana: http://qt-project.org/doc/qt-4.8/qml-color.html | 13:03 |
| zsombi | kalikiana: alpha is a value 0..255, and in Qt.rgba is (0..1) | 13:03 |
| zsombi | kalikiana: #AARRGGBB | 13:04 |
| t1mp | #AARRGGBB | 13:04 |
| zsombi | :) | 13:04 |
| kalikiana | oh | 13:04 |
| zsombi | I was faster :P | 13:04 |
| kalikiana | nice | 13:04 |
| t1mp | zsombi: I started typing that 5 minutes ago ;) but on this keyboard, there is a search-key instead of caps-lock.. and it works as super-key :s | 13:04 |
| zsombi | t1mp: yeah, excuses.... | 13:05 |
| kalikiana | t1mp: +1000 I love that the chrome book actually has a modern layout instead of following the stone age | 13:06 |
| kalikiana | few laptops do | 13:06 |
| t1mp | kalikiana: caps lock is handy some times | 13:06 |
| kalikiana | it's not, dude, nobody uses it | 13:06 |
| t1mp | I USE IT WHEN I AM UPSET! ;) | 13:06 |
| zsombi | kalikiana: only old farts.... as me :D | 13:06 |
| t1mp | kalikiana: but okay, it is not an essential key | 13:07 |
| t1mp | kalikiana: and it misses PgUp and PgDn keys.. which is not so handy | 13:07 |
| t1mp | kalikiana: other than that the keyboard layout is fine :) | 13:07 |
| kalikiana | t1mp: who needs those… :-P | 13:08 |
| t1mp | kalikiana: I need it to scroll back on irc in irssi | 13:08 |
| t1mp | kalikiana: ah, home and end buttons are also missing.. | 13:08 |
| kalikiana | t1mp: use a wheel :-D | 13:08 |
| t1mp | kalikiana: I need those to go to the beginning/end of a line that I'm editing | 13:08 |
| kalikiana | what about ctrl+left/right | 13:09 |
| t1mp | kalikiana: that only moves left/right one word | 13:09 |
| t1mp | on mac its Fn+left/right | 13:09 |
| zsombi | kalikiana: one more thing to that MR: shouldn't actually we add the blue color to the UbuntuColors as well? | 13:10 |
| kalikiana | zsombi: that was done in tim's mr | 13:10 |
| zsombi | kalikiana: then you would only need to use Qt.rgba() on the UbuntuColors.blue | 13:10 |
| kalikiana | actually lemme check if I can use the colors now | 13:10 |
| kalikiana | since it was merged | 13:10 |
| t1mp | zsombi: it has readonly property color blue: "#1ab6ef" | 13:11 |
| zsombi | t1mp: yes, I see it now, it slipped my eyes before | 13:11 |
| zsombi | kalikiana: so, if you use that, then you can use the rgba() function and set the alpha you used in the TextArea | 13:12 |
| t1mp | kalikiana: UbuntuColors.DarkGrey is also there | 13:12 |
| t1mp | kalikiana: and #888888 is LightGrey | 13:12 |
| kalikiana | seems like blue isn't | 13:12 |
| t1mp | err *blue/lightGrey/darkGrey (first letter small) | 13:13 |
| t1mp | kalikiana: it is, but it didn't merge to staging yet | 13:13 |
| t1mp | kalikiana: you'd need this as a pre https://code.launchpad.net/~tpeeters/ubuntu-ui-toolkit/NewColors/+merge/225230 | 13:14 |
| kalikiana | t1mp: aaaarg still approved :-( | 13:14 |
| kalikiana | so annoying | 13:14 |
| kalikiana | I don't really want pre at this point… I want to get rid of branches | 13:14 |
| t1mp | if you add a pre it should automatically land after the pre landed | 13:15 |
| davmor2 | kalikiana: there is a section in the api docs for UbuntuColor on developer.ubuntu.com | 13:15 |
| t1mp | and you can use the color names instead of re-defining them then | 13:15 |
| kalikiana | t1mp: "should" in real life means it will randomly fail | 13:15 |
| kalikiana | but what the heck I'll rebase on that | 13:15 |
| t1mp | davmor2: new colors will land soon | 13:16 |
| * kalikiana just a bit annoyed at this point with baby sitting more than anything else | 13:16 | |
| davmor2 | ohhh green by any chance? | 13:17 |
| davmor2 | seeing lots of green recently | 13:17 |
| t1mp | davmor2: yes, also green :) see https://code.launchpad.net/~tpeeters/ubuntu-ui-toolkit/NewColors/+merge/225230 | 13:17 |
| t1mp | davmor2: here is a short list of new colors https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1335826 | 13:18 |
| ubot5 | Ubuntu bug 1335826 in Ubuntu UI Toolkit "Update UbuntuColors with new colors" [High,In progress] | 13:18 |
| zsombi | it takes too much time now in CI to get things landed.... | 13:19 |
| kalikiana | zsombi: btw with regard to "old fart" http://xkcd.com/1393/ | 13:26 |
| t1mp | http://xkcd.com/303/ %s/compiling/landing/g | 13:30 |
| t1mp | kalikiana: ^ | 13:30 |
| kalikiana | zsombi: https://code.launchpad.net/~ubuntu-sdk-team/ubuntu-ui-toolkit/blueCursor/+merge/226457 | 13:33 |
| kalikiana | t1mp: :-D | 13:33 |
| zsombi | kalikiana: will check it asap | 13:34 |
| elopio | seb128, or Laney or jgdx: can you please review https://code.launchpad.net/~elopio/ubuntu-system-settings/refactor_about_tests/+merge/225425 ? | 13:49 |
| jgdx | elopio, sure | 13:50 |
| elopio | thanks! | 13:50 |
| seb128 | elopio, oh, I just looked at that, looks fine from me (though I didn't look too much into details because I don't know autopilot that well, but I didn't see anything obvious wrong and jenkins seems happy) | 13:51 |
| seb128 | so +1 from me | 13:51 |
| seb128 | let's see jgdx says | 13:51 |
| elopio | thanks seb128. | 13:51 |
| jgdx | elopio, might have to merge in https://code.launchpad.net/~jonas-drange/ubuntu-system-settings/1211804-allow-technology-preference/+merge/226125 | 13:51 |
| seb128 | elopio, jgdx: not sure I see the point of those license boilerplate changes | 13:52 |
| elopio | jgdx: looks like you are right. I'll make it a prerequisite. | 13:52 |
| elopio | seb128: the license used to say it was LGPL in some files, GPL in some others. | 13:53 |
| seb128 | elopio, that's only the __init__.py one | 13:54 |
| seb128 | which is fine fixing | 13:54 |
| seb128 | the other ones looks like pointless | 13:54 |
| elopio | seb128: the other ones are to use the same template. If you are missing one of the GPL header paragraphs it can cause you legal problems. | 13:59 |
| elopio | it's unlikely, and a minor problem for sure | 13:59 |
| elopio | but on so many projects we have found weird things on the headers that we are trying to use the same template everywhere. | 14:00 |
| elopio | seb128: I can revert that on this branch, and propose it on a different one to discuss only that if you prefer. | 14:00 |
| seb128 | elopio, don't bother, but I'm going to put your changes on hold because they might conflict with some other work due to that | 14:04 |
| seb128 | so your call | 14:04 |
| seb128 | if you want to see that in the next landing better to rever the changes out of the init | 14:04 |
| elopio | you are convincing :) | 14:06 |
| jgdx | elopio, I'm seeing this on 28 tests http://pastebin.ubuntu.com/7780470/ | 14:42 |
| jgdx | I'm most likely lacking something | 14:43 |
| jgdx | elopio, I have to go away for some hours. Be back later. | 14:44 |
| elopio | jgdx: I'll take a look. I think that you must run the tests with autopilot3 after my previous branch. | 14:51 |
| === chihchun_afk is now known as chihchun | ||
| === chihchun is now known as chihchun_afk | ||
| nik90_ | popey: u having fun with the back and forth email with mhall119 :P ? | 16:28 |
| popey | haha | 16:29 |
| ahayzen | t1mp, are you around to look at some strange behaviour we are having with the searching? | 17:26 |
| === renato is now known as Guest85133 | ||
| t1mp | ahayzen: yes | 17:42 |
| ahayzen | :) | 17:42 |
| ahayzen | t1mp, this is our WIP mp... https://code.launchpad.net/~andrew-hayzen/music-app/add-sdk-search-support/+merge/226045 | 17:42 |
| ahayzen | t1mp, the behaviour we are seeing is... | 17:43 |
| === ken_ is now known as kenvandine | ||
| ahayzen | t1mp, if you start searching on one tab...then cancel the search... move to another tab and start searching...it *sometimes* (usually after repeating enough times)... doesn't actually search and ur result from the previous tab is sometimes there? | 17:43 |
| ahayzen | t1mp, furthermore sometimes u can see it flicker between the tabs as you press the search button | 17:44 |
| ahayzen | t1mp, is this todo with the way i've done our MusicPage and the implementation of the searching... or is this an SDK issue? | 17:45 |
| * t1mp reading the MR | 17:46 | |
| t1mp | ahayzen: what do you see flicker between tabs? | 17:46 |
| ahayzen | t1mp, i see the title of the previous tab appear then switch back to the current... then the text field appears | 17:46 |
| ahayzen | t1mp, that happens more rarely than the issue (maybe because it happens so fast) | 17:47 |
| === Guest85133 is now known as renato___ | ||
| ahayzen | t1mp, but we suspect somehow you end up on tab A but actually searching on tab B as the text field sometimes has what you had when you were searching in tab B and would explain why it appears like nothing is happening | 17:47 |
| t1mp | seeing the title of the previous tab is weird | 17:48 |
| ahayzen | t1mp, yeah thats what i thought | 17:49 |
| t1mp | hmmm | 17:49 |
| t1mp | [ 1%] Merging translations into com.ubuntu.music_music.desktop | 17:49 |
| t1mp | /bin/sh: 1: INTLTOOL_MERGE-NOTFOUND: not found | 17:49 |
| t1mp | I have a new laptop, so probably I still need to install some dependencies | 17:49 |
| ahayzen | errr | 17:49 |
| ahayzen | ah i had that | 17:49 |
| ahayzen | t1mp, try installing intltool | 17:50 |
| t1mp | that helps, but I still get errors when starting the app: | 17:53 |
| t1mp | tim@C720:~/dev/add-sdk-search-support$ qmlscene music-app.qml | 17:53 |
| t1mp | file:///home/tim/dev/add-sdk-search-support/music-app.qml:832 Type MusicStart unavailable | 17:53 |
| t1mp | file:///home/tim/dev/add-sdk-search-support/MusicStart.qml:235 GenresModel is not a type | 17:53 |
| ahayzen | woah | 17:53 |
| ahayzen | t1mp, what are you runing this on? | 17:54 |
| t1mp | on my laptop | 17:54 |
| ahayzen | looks like your missing mediascanner2 qmlplugin | 17:54 |
| ahayzen | t1mp, ensure you have all of our depends in here http://bazaar.launchpad.net/~music-app-dev/music-app/trunk/view/head:/debian/control | 17:54 |
| ahayzen | t1mp, also note i've been doing this on device (utopic with latest devel-proposed image) as my laptop is trusty | 17:55 |
| t1mp | seems like I have all the dependencies | 17:57 |
| t1mp | I am also running trusty | 17:57 |
| t1mp | it won't work on trusty? | 17:57 |
| ahayzen | t1mp, ah yes mediascanner2 could be interesting on trusty... and sdk doesn't have search on trusty for me? | 17:58 |
| ahayzen | ...but that is a strange error your ending up with... | 17:58 |
| ahayzen | t1mp, oh do u have the coreapps ppa? | 17:58 |
| ahayzen | t1mp, yeah thats what it is...coreapps ppa has the new ms2 which includes genremodel :) | 17:58 |
| ahayzen | t1mp, this one https://launchpad.net/~ubuntu-touch-coreapps-drivers/+archive/ubuntu/daily | 17:59 |
| t1mp | I have a mediascanner2.0, but no qtdeclarative5 mediascanner package.. | 18:00 |
| ahayzen | t1mp, you need qtdeclarative5-ubuntu-mediascanner0.1 to be installed | 18:00 |
| t1mp | I have that one | 18:01 |
| t1mp | maybe I need the version from the ppa to get GenresModel | 18:01 |
| ahayzen | t1mp, yeah you need to have a version after ~20 may or something | 18:01 |
| ahayzen | coreapps ppa has 0.101+14.10.20140627 | 18:02 |
| ahayzen | t1mp, however if ur on trusty i still think it won't work | 18:02 |
| t1mp | trusty has 0.100+14.04.20140403-0ubuntu1 | 18:02 |
| ahayzen | t1mp, i get 'PageHeadState is not a type' on my machine | 18:03 |
| t1mp | I'm using a spare laptop now with trusty.. my main laptop has utopic, but I had to send it for repair, I'll wait for that one to come back | 18:03 |
| t1mp | ahayzen: which version of UITK? | 18:03 |
| ahayzen | t1mp, and we get this from ms2 'pure virtual method called | 18:03 |
| ahayzen | terminate called without an active exception' | 18:03 |
| ahayzen | ^^ which has been fixed in qt5.3 | 18:03 |
| t1mp | ahayzen: did you import Ubuntu.Components 1.1 for PageHeadState? | 18:04 |
| ahayzen | yep i changed all of them to 1.1 as u can see in the mp | 18:04 |
| ahayzen | t1mp, qtdeclarative5-ubuntu-ui-toolkit-plugin: | 18:05 |
| ahayzen | Installed: 0.1.46+14.10.20140520-0ubuntu1~0trusty2 | 18:05 |
| ahayzen | t1mp, that is coming from our ppa... hmm so do we just need the package updating again? ... and will it work with qt5.2 we were under the impression things will start breaking as trusty doesn't have qt5.3 | 18:06 |
| t1mp | true, that's tricky | 18:06 |
| ahayzen | t1mp, but even if that works u won't be able to test because mediascanner2 will explode with its bug when you try search etc | 18:06 |
| t1mp | newer UITK works on trusty and with qt 5.2 | 18:07 |
| t1mp | I have the same problem with old UITK package | 18:08 |
| t1mp | let me compile an UITK from bzr.. | 18:08 |
| ahayzen | t1mp, this is the mediascanner2 bug for reference https://bugs.launchpad.net/mediascanner2/+bug/1334843 | 18:08 |
| ubot5 | Ubuntu bug 1334843 in mediascanner2 "Mediascanner2 crash on model change with header in listview" [Undecided,New] | 18:08 |
| ahayzen | t1mp, it is failing because u have an old mediascanner...but if u update it is pretty broken on trusty :/ so ur best off using a device to test | 18:09 |
| t1mp | ahayzen: I replaced the model: SortFilterModel { } by model: 10 to test | 18:11 |
| ahayzen | hah | 18:11 |
| t1mp | also doens't work.. but I can see that the PageHeadState works with a more recent UITK | 18:11 |
| ahayzen | what happens then i suspect something bad | 18:12 |
| ahayzen | oh like the rowCount errors will be everywhere and there was anotehr one... | 18:12 |
| t1mp | so if you bzr branch lp:ubuntu-ui-toolkit && cd ubuntu-ui-toolkit && qmake && make && ./export_modules_dir.sh, then you can run your app with the newer UITK | 18:12 |
| t1mp | well.. you could if there was a working mediascanner for trusty | 18:12 |
| ahayzen | hah | 18:12 |
| ahayzen | t1mp, i've been testing on my device | 18:12 |
| ahayzen | t1mp, you should just be able to run in the branches dir to push the app to device $ click-buddy --dir . --provision | 18:14 |
| ahayzen | (assuming u have click-buddy installed ;) ) | 18:15 |
| t1mp | probably not, I just installed ubuntu on this laptop a few hours ago | 18:15 |
| ahayzen | hehe | 18:16 |
| t1mp | I have to leave now | 18:18 |
| ahayzen | ok no worries | 18:18 |
| t1mp | so the flickering of the previous title is weird.. do you think you can reproduce that with a small test program without dependencies? | 18:18 |
| ahayzen | i'll see if i can figure out if it is us or not and yeah if i think it isn't us...i'll try and build a test app | 18:18 |
| ahayzen | t1mp, thanks for ur help so far | 18:18 |
| t1mp | and the wrong search results is also strange. The contents basically only sets the textfield in the header, doesn't deal with the results | 18:19 |
| t1mp | I'm not ruling out UITK, it is all new stuff so not many people used it so far | 18:19 |
| ahayzen | t1mp, it is like it is searching on the other tab | 18:19 |
| ahayzen | t1mp, (the one that flickered) | 18:19 |
| t1mp | that other tab title should only become visible when that tab is active | 18:20 |
| t1mp | maybe it quickly switches tabs before going to search mode? | 18:20 |
| ahayzen | t1mp, yeah something funky going on here lol | 18:20 |
| ahayzen | t1mp, i'll continue playing about see if i can narrow it down a bit | 18:21 |
| t1mp | ahayzen: in the MR I only see one PageHeadState, where do you have the other pages where you can search? | 18:21 |
| ahayzen | t1mp, that is in our generic MusicPage {} | 18:22 |
| ahayzen | t1mp, all of our pages are based of MusicPage which is a Page {} itself | 18:22 |
| ahayzen | t1mp, but searching will appear on the Artists/Albums/Tracks tabs (pages) | 18:23 |
| t1mp | ahayzen: I see now | 18:24 |
| t1mp | ahayzen: can you check that tabs.selectedTabIndex doesn't change when you get the flickering? | 18:24 |
| ahayzen | t1mp, yep i'll add tracing to that and the page states and see if i can figure out anything | 18:25 |
| ahayzen | t1mp, anyway i gotta go in a minute as well so i'll investigate further tonight | 18:26 |
| t1mp | hmm | 18:27 |
| t1mp | I see you have several Tabs/Page/PageStack instances defined directly inside the MainView | 18:27 |
| t1mp | perhaps the MainView gets problems determining which is the active page | 18:27 |
| t1mp | if you move those inside the PageStack you avoid that | 18:28 |
| ahayzen | t1mp, hmmm but they are all visible: false ... and we should have one pageStack... | 18:28 |
| ahayzen | t1mp, oh so the hidden ones should be inside the pageStack? | 18:28 |
| ahayzen | t1mp, i gotta go i'll try moving things around later thanks for the pointers o/ | 18:29 |
| t1mp | yes, could be related to that | 18:29 |
| t1mp | they are not visible, but the MainView has only one page listed as activePage internally, that's not the same as visible | 18:30 |
| t1mp | I gotta go too | 18:30 |
| t1mp | have a nice weekend :) | 18:30 |
| ahayzen | t1mp, ok thanks.... you to o/ | 18:30 |
| mhall119 | DanChapman: btw, http://bazaar.launchpad.net/~mhall119/ureadit/trunk/view/head:/models/CommentListModel.qml shows what I was talking about earlier about flattening a comment thread | 18:51 |
| DanChapman | mhall119, heh i was half way through writing a message to you about that :-D how weird. Thanks i'll take a look, i can't download the reddit app for some reason to see it in action | 18:52 |
| mhall119 | DanChapman: :) | 18:53 |
| mhall119 | DanChapman: how are you trying to download it? | 18:53 |
| mhall119 | from the Ubuntu Store scope on a device? | 18:53 |
| DanChapman | through the store, yes. Getting a rather generic "error downloading or installing" message. It happens straight after tapping install so i presume it's the download failing | 18:55 |
| mhall119 | beuno: ^^ that's like the third person I've heard say they can't download apps from the store, any idea what's going on? | 18:57 |
| matiasb | mhall119: beuno is not around today; afaict, everything seems ok (there were some firewall issues yesterday that could have caused some problems, but that should be all fixed now) | 19:10 |
| mhall119 | thanks matiasb | 19:11 |
| matiasb | mhall119: np, if you get any more details, or cases, let me know | 19:11 |
| ahayzen | balloons, ping | 20:27 |
| ahayzen | t1mp, if your there i just added tracing http://pastebin.ubuntu.com/7781949/ ... if you look at the last few lines you can see artists page has its state changed to 'search' but the textfield appears to be from the songs page! | 20:37 |
| balloons | ahayzen, poing | 20:38 |
| ahayzen | balloons, i've been thinking about doing refactor/cleanup of our ap tests at somepoint in the near future.... | 20:38 |
| ahayzen | balloons, and was wondering if our tests should be python3 rather than python2? is that the long term goal? | 20:39 |
| ahayzen | balloons, as if me or victor did a cleanup it could be a good opportunity to move | 20:39 |
| balloons | ahayzen, yes, the tests should be py3 | 20:49 |
| ahayzen | balloons, ok cool | 20:49 |
| ahayzen | balloons, i may do that over the new week or so if i get stuck in qml stuff :) ... as it is getting annoying having all our methods in a random order with strange namings (songsPage vs songsTab?) ... and i suspect there is some stale code in there as well | 20:51 |
| balloons | ahayzen, I'd recommend looking at http://developer.ubuntu.com/apps/platform/guides/acceptance-testing-using-the-page-object-model/ | 20:51 |
| balloons | and yes, you likely could sync and cleanup object names and the qml and tests at the same time :-) | 20:52 |
| ahayzen | balloons, was it the clock-app ap test that where split into different classes/modules per tab? | 20:52 |
| ahayzen | balloons, thanks for the link :) | 20:52 |
| balloons | ahayzen, clock was the first, but others show it as well | 21:00 |
| balloons | generally we write anything new in it | 21:00 |
| ahayzen | balloons, cool | 21:00 |
| balloons | reminders is done the same way | 21:00 |
| ahayzen | balloons, oh yeah and while looking at the system-apps i saw they have QML tests? are they beneficial in any way or should all of ours be python/autopilot | 21:15 |
| balloons | ahayzen, you should consider qml tests for your custom components | 21:16 |
| ahayzen | balloons, what is the purpose of them? just to test a component rather than a use case/user story? | 21:16 |
| * nik90_ is proud to hear clock do it first ;) | 21:21 | |
| balloons | ahayzen, it's different tech. I would recommend using them for any low level tests you might want to do as well | 21:21 |
| balloons | nik90_, :-) | 21:21 |
| ahayzen | balloons, ok i may have a look at those after any refactoring is done | 21:22 |
| gerlowskija | balloons: ping | 21:23 |
| balloons | gerlowskija, pong | 21:24 |
| balloons | ahayzen, definitely.. feel free to ping with specifics | 21:24 |
| ahayzen | balloons, cool thanks | 21:25 |
| gerlowskija | balloons: I saw you commented on this MP: https://code.launchpad.net/~gerlowskija/ubuntu-calendar-app/revised-fix-for-bug1334883/+merge/226049. Is there anything that I can do to help take a look at it? | 21:25 |
| balloons | gerlowskija, ohh my did that not land? | 21:26 |
| * balloons looks | 21:26 | |
| gerlowskija | no, it's still getting UNSTABLE on one of the test runs | 21:26 |
| balloons | I see http://91.189.93.70:8080/job/generic-mediumtests-utopic/974/artifact/calendar_app.tests.test_new_event.NewEventTestCase.test_delete_event_must_remove_it_from_day_view%20%28with%20mouse%29.ogv | 21:27 |
| gerlowskija | stupid realization, but I didn't realize the autopilot tests were screen recorded. | 21:31 |
| balloons | gerlowskija, no worries! yea, it's in the test output | 21:31 |
| gerlowskija | thats awesome. Though the failure isn't. | 21:31 |
| balloons | anyways, yea, let me grab and try locally. If it works locally, then it gets a bit harder but not impossible | 21:32 |
| gerlowskija | ok cool. It looks like the test ends (terminates/crashes?) early. The test should go on to delete the event. | 21:34 |
| balloons | gerlowskija, it seems it just tries to find the event and then ends | 21:35 |
| balloons | perhaps using an eventually on the assert would be useful | 21:35 |
| * balloons runs | 21:36 | |
| gerlowskija | Hmm.. based on the test name, and this line: | 21:36 |
| gerlowskija | day_view = day_view.delete_event(event.name, len(original_events) > 0) | 21:36 |
| gerlowskija | I would expect it to go through deleting the event, but I didn't write the test, so that's just my impression | 21:37 |
| balloons | ok, it does run on the desktop :-) | 21:37 |
| balloons | gerlowskija, the source looks weird.. | 21:37 |
| balloons | see line 68 in test_new_event? def get_new_events()? | 21:38 |
| gerlowskija | yep | 21:38 |
| balloons | mm.. not part of your mp.. weird it's in trunk I guess? | 21:39 |
| gerlowskija | Yeah, I didn't change anything in the autopilot tests other than removing a skip on the last test | 21:39 |
| balloons | I'm making some changes.. | 21:39 |
| balloons | gerlowskija, http://paste.ubuntu.com/7782180/ | 21:40 |
| balloons | I like_method for private stuff.. but otherwise, just moving it out | 21:41 |
| balloons | now as for the logic.. | 21:41 |
| balloons | I think we can fix that too, since you are fixing https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1334833 | 21:42 |
| ubot5 | Ubuntu bug 1334833 in Ubuntu Calendar App "After adding an event, the TimelineBase on DayView duplicates the old EventBubbles" [High,Confirmed] | 21:42 |
| gerlowskija | I *think/hope* I'm fixing 1334833 haha | 21:43 |
| balloons | gerlowskija, well let's change the logic and see eh> | 21:43 |
| balloons | I hope so too.. so many events in the stack! | 21:43 |
| gerlowskija | ok. I'll re-push with the _get_new_events() change you had, and see if that helps. Thanks as always balloons: | 21:46 |
| balloons | gerlowskija, one sec | 21:47 |
| gerlowskija | sure | 21:47 |
| balloons | I missed adding self as the arg for def _get_new_events(self): | 21:47 |
| balloons | and there might be more things :-) | 21:47 |
| balloons | I'm guessing elopio actually set it up this way.. but I do wonder why | 21:48 |
| balloons | gerlowskija, I think I'll just push something.. the diff is getting bif | 21:51 |
| gerlowskija | Haha, ok then. I'll be interested to see what the final changes are! | 21:52 |
| gerlowskija | Anything I should do then? I feel like I asked what I could do to help but you ended up getting stuck with all the work yourself ;-p | 21:53 |
| balloons | no worries | 21:55 |
| balloons | I think it's good now | 21:55 |
| balloons | gerlowskija, lp:~nskaggs/ubuntu-calendar-app/filter_dupes | 21:59 |
| gerlowskija | Cool, looks like it works for me on my desktop (though it was originally too) | 22:03 |
| gerlowskija | balloons: So the plan is to delete my existing branch, link your newer one to the 2 bug reports, and propose for merging? | 22:04 |
| balloons | gerlowskija, yep, let me know when you push and I'll sic jenkins on it | 22:04 |
| === salem_ is now known as _salem | ||
| balloons | gerlowskija, no, I would just merge my branch into yours and then push a new rev | 22:04 |
| balloons | same mp and everything | 22:04 |
| gerlowskija | Ok. My bzr skills are a little lacking...off to the man page! | 22:05 |
| balloons | gerlowskija, :-) | 22:07 |
| gerlowskija | haha | 22:07 |
| balloons | bzr merge lp:~nskaggs/ubuntu-calendar-app/filter_dupes | 22:08 |
| balloons | bzr commit | 22:08 |
| balloons | bzr push | 22:08 |
| balloons | :-) | 22:08 |
| gerlowskija | balloons: you're taking all the fun out of it ;-p But thanks! | 22:08 |
| balloons | gerlowskija, haha.. sorry! | 22:11 |
| gerlowskija | I re-pushed, balloons. Let's see jenkins do its worst! | 22:15 |
| balloons | gerlowskija, kk, it's running | 22:16 |
| Nadir92 | Hi. | 22:21 |
| balloons | gerlowskija, flake8 issues | 22:26 |
| balloons | can you fix? | 22:26 |
| gerlowskija | Yep, just a sec. | 22:27 |
| gerlowskija | balloons: re-pushed with a fix (I think). | 22:34 |
| * balloons keeps fingers crossed | 22:52 | |
| ahayzen | balloons, still there? | 23:02 |
| balloons | gerlowskija, we broke it | 23:10 |
| balloons | http://91.189.93.70:8080/job/generic-mediumtests-utopic/984/testReport/junit/calendar_app.tests.test_new_event/NewEventTestCase/test_add_new_event_with_default_values_with_mouse_/ | 23:10 |
| ahayzen | balloons, When trying to land https://code.launchpad.net/~verzegnassi-stefano/music-app/transparent-bg-for-ubuntushapes-labels/+merge/225589 we noticed that in the last few failures different tests were failing with dbus errors, we thought maybe it was because the packages were still syncing. However on this https://code.launchpad.net/~andrew-hayzen/music-app/abstract-row-second-iteration/+merge/226549 i've just | 23:18 |
| ahayzen | got the same issue, I only changed 1 line of code between the revisions (added a margin to something) and this test worked one time and not the other. Any ideas what is going on? | 23:18 |
| balloons | ahayzen, yes it'd be worth looking deeper at the tests to see why the random fails | 23:20 |
| balloons | off the top, nothing simple | 23:20 |
| ahayzen | balloons, hmm maybe in our refactor hehe | 23:20 |
| ahayzen | balloons, ok so just rerun for now? | 23:20 |
| balloons | yes :-) | 23:20 |
| ahayzen | balloons, can u retrigger for me so i don't have to empty commit? | 23:22 |
| balloons | yepper | 23:27 |
| ahayzen | thanks | 23:32 |
| ahayzen | balloons, success \o/ lol | 23:37 |
| balloons | :-) | 23:37 |
| balloons | lol | 23:37 |
| elopio | gerlowskija: balloons: you are fixing the bug that causes duplicate events \o/ | 23:41 |
| elopio | what's weird on this run is that the machine has many events: | 23:42 |
| elopio | http://91.189.93.70:8080/job/generic-mediumtests-utopic/984/artifact/calendar_app.tests.test_new_event.NewEventTestCase.test_add_new_event_with_default_values%20%28with%20mouse%29.ogv | 23:42 |
| balloons | elopio, well we tried.. seems something is still a bit off | 23:42 |
| elopio | isn't jenkins starting a new machine for every run? | 23:42 |
| balloons | elopio, that's not weird, in so much as events don't delete well | 23:42 |
| balloons | calendar isolation isn't working atm, I believe | 23:42 |
| elopio | start_time, end_time = time_label.text.split(' - ') | 23:43 |
| elopio | ValueError: need more than 1 value to unpack | 23:43 |
| elopio | anyway, this error means that there is an event whose time label doesn't match {start time} - {end time} | 23:43 |
| elopio | it might be a previosly existing all day event? | 23:44 |
| elopio | I didn't write the helper to work on that case. | 23:44 |
| elopio | hum, my calendar from trunk doesn't work nicely now. | 23:47 |
| elopio | balloons: gerlowskija: actually, if you look at the video, it shows a new event with no time, and the bubble is on today and tomorrow. | 23:50 |
| elopio | there could be a problem creating an event for the last half hour of the day. | 23:51 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!