=== chriadam|away is now known as chriadam === _salem is now known as salem_ === salem_ is now known as _salem [06:57] good morning [08:20] tmoenicke: hey... how's the progress with the keyboard testing? [08:23] tmoenicke: we would like to start integrating the shell tests with some OSK usage [08:23] tmoenicke: do you have some emulators ready we could reause? [08:23] reuse [08:24] mzanetti: lets talk to bill about this in the afternoon [08:31] Good morning all, happy Autistic Pride Day! :-D [08:36] hum [08:36] is there a "ubuntuorange" color in the toolkit? ;-) [08:46] hi seb128, the sdk guys can probably give a better answer, but I haven't found it, I just specify "#DD4814" :) [08:46] dpm, ok, that's cheating :p [08:47] dpm, hey btw, and thanks ;-) [08:47] :) [08:47] I was going to do that but I figured out that it would be nice to have a named color for that one [08:47] I guess quite some apps want to use to give an ubuntu look [08:50] seb128: no, there is no UbuntuOrange yet https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1098209 [08:50] Ubuntu bug 1098209 in Ubuntu UI Toolkit "define the ubuntu orange color in the SDK" [Wishlist,Confirmed] [08:57] timp, thanks ;-) [09:08] om26er: can you review this? https://code.launchpad.net/~schwann/gallery-app/gallery-video-autopilot/+merge/170015 [09:08] gusch, looking [09:08] om26er: thx [09:14] dpm, seems like the developer.ubuntu.com/packaging link is broken [09:14] dpm, do you know what happened there? [09:23] dpm, FWIW http://developer.ubuntu.com/resources/tools/packaging/ is broken too [09:35] dholbach, looking at it now [09:37] dpm, thanks [09:39] guys, do we have a way to detect the running QML application name? (i.e. desktop file name, main QML file)? Arguments SDK component aint seem to provide that info... [10:02] gusch: I'm a bit stuck with autopilot (I don't have much experience with it). [10:02] gusch: when I return the "cancel icon", I know that I need tools.back from the toolbar. But is there a way with autopilot to do that besides adding objectNames to the back/cancel button everywhere? [10:03] timp: you can use any property [10:03] gusch: an easy way to do it is by looking for the button with text "Cancel" or "Back", but that will break on systems that use different languages [10:03] timp: correct [10:03] gusch: #toolbar_items = toolbar.select_single("ToolbarItems") #return toolbar_items.back [10:03] without the # [10:03] that doesn't work, but you get the intention. I want to get the back-property of toolbar_items [10:04] use select_many instead of select_single [10:04] timp: ^ that might help [10:04] why many? there is only one [10:04] for sure? [10:05] the problem is AttributeError: Class 'ToolbarItems' has no attribute 'back'. [10:05] in qml I have this in ToolbarItems: property Item back [10:06] I just don't know how to access it from python [10:06] "back" is a QML property? [10:06] yes [10:07] which type? [10:07] Item [10:08] well, I noticed that now I have several subclasses of ActionItem (including Button, ToolbarButton), and if select all ActionItems it doesn't seem to include Buttons [10:08] then return toolbar_items.select_many("QQuickItem")[0] [10:08] then "return toolbar_items.select_many("QQuickItem")[0]" [10:08] or whatever index the back item has (that's a least not worse than the current test ;) [10:08] ohh I need to use the cpp classname, not the qml name? [10:08] yep [10:09] you can run "gallery-app -testability" - and "autopilot vis" [10:09] the select the gallery, and you can browse the QML tree nicely [10:10] ok, thanks [10:15] maybe I can do it all via the objectName [10:23] gusch: it is a bit confusing to me. I have ActionItem, and subclasses of that called ToolbarButton and Button (in qml) [10:23] gusch: autpilot vis shows me a classname "Button" for the classic buttons, and "ActionItem" for the ToolbarButtons [10:24] ActionItem for ToolButtons ?!? [10:24] that sounds weired [10:24] gusch: and it seems that the classname for autopilot must match exactly, so I cannot say "ActionItem" and select all instances of (subclasses of) ActionItem [10:24] but if you can set a unique objectName, that would be the best for testing for course [10:25] timp: yes, the names have to match exactly [10:25] gusch: yes, I was trying that. but it doesn't work for me because of the classname [10:25] some times it is ActionItem (for back buttons), some times Button (for cancel buttons) [10:26] I guess it'll work if I introduce an additional get_toolbar_back_icon function in addition to the get_toolbar_cancel_icon [10:27] just weird that for ToolbarButton I need to search for ActionItem instead. [10:27] the full implementation of ToolbarButton is this: https://pastebin.canonical.com/93000/ [10:27] perhaps it doesn't change the className because I didn't add properties to it [10:28] that would mean the tests that I am planning will break if I ever add a property to ToolbarButton [10:31] timp: damn - but might be [10:32] gusch, Hi! gallery-app crashes on 'Auto Enhance' btw [10:32] gusch: thanks for the help. I'll have a break now and then I give it another try. [10:32] om26er: yep https://bugs.launchpad.net/gallery-app/+bug/1180345 [10:32] Ubuntu bug 1180345 in gallery-app "Auto enhance crashes gallery app and deletes photo" [Critical,Confirmed] [10:33] om26er: you could do a quick fix in C++ if you want ... [10:34] om26er: in src/photo/photo.cpp comment line 805 (QApplication::processEvents();) [10:36] gusch, cool, that works [10:39] om26er: well - it blocks the UI - so I need another (proper) fix === oSoMoN is now known as oSoMoN|afk [11:24] gusch, should I keep your workaround with my test branch ? [11:25] om26er: good question [11:25] om26er: I'd say yes - and update the bugreport [11:27] ok [11:29] om26er: just to recap: all the autopilot tests that are in the branch at the momnent for the browser plugin are working, right ? i just tried and they all seem to pass successfully [11:29] om26er: didn't you say there were some failing ? [11:29] nerochiaro, yes, I have disabled a few so you would not see them failing [11:30] look in test_history.py [11:30] om26er: ok, thanks [11:36] om26er: oh, i thought you had disabled them with the decorator, not with comments [11:37] om26er: so it's basically only test_history and test_chromeless, right ? [11:37] nerochiaro, yep [11:38] oSoMoN|afk: when you're back i have one question for you on a traslations-related issue [11:38] oSoMoN|afk: please pingme === MacSlow is now known as MacSlow|lunch === greyback is now known as greyback|lunch [12:06] gusch, I added tests for auto-enhance and 'delete last photo' .. https://code.launchpad.net/~om26er/gallery-app/add_test_for_auto_enhance/+merge/170054 [12:06] for the latter case the bug would need to be fixed before the branch goes in. [12:06] om26er: cool - looking === oSoMoN|afk is now known as oSoMoN [12:15] nerochiaro: ping [12:18] om26er: looks good - I guess now it's my turn ;) [12:18] gusch, yep :D [12:20] oSoMoN: hi, i've been trying to figure out why my translation files were being installed under /usr/local/share instead of /usr/share and I'm not really finding any difference with the other packages that use translations. Wonder if you could have a quick look at lp:~amanzi-team/ubuntu-ui-extras/ubuntu-ui-extras-initial and confirm that debuild does the same for you and give any clue [12:22] nerochiaro: sure, I’ll look into it shortly and get back to you === greyback|lunch is now known as greyback [12:28] oSoMoN: ok, i'll go for some food in the meantime === _salem is now known as salem_ === chriadam is now known as chriadam|away [12:43] nerochiaro: can you review the camera? https://code.launchpad.net/~schwann/camera-app/camera-video-orientation/+merge/170053 [12:46] dpm, could you find out anything? [12:49] nerochiaro: independently from the issue you asked me to look into, building a package with debuild fails with the following error: [12:49] dh_install: qtdeclarative5-ubuntu-ui-extras-plugin-autopilot missing files (usr/lib/python*/dist-packages/ubuntu-ui-extras/*), aborting [12:49] zsombi: could you review https://code.launchpad.net/~tpeeters/ubuntu-ui-toolkit/toolbarItems/+merge/170070 ? [12:50] zsombi: don't happrove yet, when you think the MR is good, I'll first send an e-mail to ubuntu-touch mailing list [12:50] dholbach, back online at http://developer.ubuntu.com/resources/tools/packaging/ - it was my fault, I had reshuffled a lot of the content on the resources page and had made the packaging page private by mistake [12:52] boiko: ping [12:53] gema: pong [12:53] boiko: I'd like to have a quick chat with you as to where you are functionality - wise with the phone-app [12:54] boiko: we are going to start testing it in the lab and I'd like to know what to expect [12:54] and how to prioritise the testsing [12:54] gema: yep, nice! [12:55] dpm, thanks [12:55] boiko: so I had an interesting morning today not being able to do some things with the app, it'd fail to hang up, or to send the conference id to a conference call [12:55] boiko: it'd also not be able to call to the provider on a service number [12:56] boiko: are these things on your TODO or do you think they are there [12:56] gema: so conf call is not supported yet [12:56] ok [12:56] gema: failing to hang up is something new to me [12:56] salem_: ^ [12:57] boiko: I was calling the canonical conference system [12:57] I had tried to enter the conf id [12:57] then tried to hang up [12:57] didn't happen [12:57] gema: oh, you mean DTMF then ok [12:57] gema, ah, I have seen this before. I think it only happens if you try to hangup from the dtmf screen. [12:58] salem_: waht is the dtmf screen? [12:58] gema, the screen to type numbers during an ongoing call. [12:58] salem_: ah, ok [12:58] salem_: which screen should I use to hang up? [12:59] gema, well, you should be able to hangup from that screen, but as it seems we have a bug there. I would suggest you to tap back, and then hangup from the main screen [13:00] salem_: ack, do you want me to raise a bug? [13:00] gema: please do [13:00] ack [13:00] gema, that would be great. thanks [13:00] salem_: I'll try dtmf in today's image to see if it works [13:01] boiko, ok. I think this bugs is not 100% reproducible, but I have seen it before [13:01] salem_: ok, let's see what I can find here [13:02] boiko, salem_ : https://bugs.launchpad.net/phone-app/+bug/1192158 [13:02] Ubuntu bug 1192158 in phone-app "Unable to hang up from dtmf screen" [Undecided,New] [13:02] I used your nomenclature, feel free to change it :) [13:03] boiko: on the bright side I was able to gather some energy consumption data with a call that lasted 28 mins [13:03] (until credit ran out) [13:03] gema: oh, and how did it go? [13:03] boiko: I still need to do the math, I will let you know soon [13:04] how long I expect the battery to last with an engaged call [13:04] nerochiaro: that said, regarding your issue, I’ve added some debugging messages to src/Ubuntu/Components/Extras/Browser/po/CMakeLists.txt, and the value of CMAKE_INSTALL_LOCALEDIR when building a package is share/locale, which looks correct [13:06] gema: not sure how long, cause battery usage does not depend exclusively on the phone-app [13:07] boiko: I know, that's why we are measuring the whole system :) [13:07] boiko: but I need the phone-app to behave if we are going to measure this kind of thing [13:07] gema: yep [13:07] i.e. being able to make calls, keep them up, hang up [13:08] boiko: do you have an ETA for the conferencing? === MacSlow|lunch is now known as MacSlow [13:12] oSoMoN: yes, it looks correct, but then if you look under obj-i686-linux-gnu/ you will see that /usr/local/ is prepended to that path [13:13] gema: that's DTMF, conferencing is something different (creating a conf call on the phone) [13:13] gema: I'll test that, and I bet salem_ will also give it a try too [13:13] nerochiaro: I don’t see anything there, probably because the package fails to build [13:14] nerochiaro: have you tried building from a clean copy of your branch? [13:14] gema: actually there is a simple test for that: call some other phone, and press the keypad buttons to see if any sound is heard on the other side === dholbach_ is now known as dholbach [13:15] oSoMoN: i'm always doing that from a clean branch now. i'm using debuild -uc -us -b and the obj-i686-linux-gnu/ dir is there even if the build fails (which i confirm it does with the issue you mentioned, i'm fixing it) [13:16] boiko: it is not at the moment [13:17] gema: so there is indeed a but there, the phone has just finished flashing, let me try it here [13:17] oSoMoN: ok, fixed the issue you were noticing before, pushed [13:17] oSoMoN: but there are other problems, i'm working on them [13:17] boiko: ack [13:19] gema: just tested using today's image, and DTMF seems to be working here, I'll do some more tests [13:19] nerochiaro: next error when building package: [13:19] dh_install: tmp/ubuntu-ui-extras-initial/obj-x86_64-linux-gnu/tests/unittests/Ubuntu/Components/Extras/libubuntu-ui-extras-plugin.so exists in debian/tmp but is not installed to anywhere [13:19] nerochiaro: does the package actually build successfully on your machine? [13:19] oSoMoN: no, it does not, i was trying to approach one issue at a time. and the translation one is the one i could not explain === francisco is now known as Guest7324 [13:20] oSoMoN: this one you just pointed out to me I know where it comes from [13:20] nerochiaro: well how can you tell there’s an issue with the package if the package doesn’t build at all? [13:20] oSoMoN: because i see the translation files installed in the wrong dir [13:20] gema: are you using today's image? [13:21] nerochiaro: fix the package build first, and then see if there are issues, not the other way around [13:22] oSoMoN: fair enough [13:33] nerochiaro: can you review the camera? https://code.launchpad.net/~schwann/camera-app/camera-video-orientation/+merge/170053 [13:33] om26er: here the delete fix https://code.launchpad.net/~schwann/gallery-app/gallery-delete-last-photo/+merge/170077 [13:33] boiko: can you review? https://code.launchpad.net/~schwann/gallery-app/gallery-previewmanager-mediacollection/+merge/169998 [13:34] gusch: probably after the standup only, is that fine? [13:34] boiko: ok - no problem [13:35] boiko: that branch sits and waits for about a week already ;) [13:35] gusch: hehe, ok :) [13:39] gusch: are all gallery autopilot tests supposed to pass on my local machine? [13:39] gusch: i'm stuck with some build issues to solve ASAP, but i can try to take care of that later [13:39] nerochiaro: ok [13:40] gusch: I fixed all the button tests in my branch, but I have two (seemingly unrelated) fails: [13:40] timp: yes [13:40] FAIL: gallery_app.tests.test_photo_viewer.TestPhotoEditor.test_photo_editor_crop(with mouse) [13:40] FAIL: gallery_app.tests.test_photo_viewer.TestPhotoViewer.test_double_click_zoom(with mouse) [13:40] om26er: ^ [13:40] in the crop test, I don't see the frame for cropping when the test is run. [13:40] timp: but they all run fine locally and even on Jenkins [13:41] timp: whaaaat?!? [13:41] in the double_click_zoom test, the mouse cursor moves to the center of the photo, but I don't see zooming [13:41] gusch, om26er oh. doubleclick to zoom, and cropping also don't work for me when I run gallery myself [13:41] ah.. [13:42] probably I broke that. [13:42] because I commented out some HUD stuff. [13:42] ^so that I can run it without having a HUD. But the code is broken because of my chagnes [13:42] *changes [13:42] timp: you shouldn't do that ;) [13:43] boiko: no, I am using the first saucy image, I will be upgrading soon [13:43] gema: ok [13:43] gusch: I don't have Ubuntu.HUD [13:43] boiko: for some reason I cannot turn it on right now, need to look into that [13:43] gusch: last time I installed shell stuff it broke my whole system. (that was months ago) [13:43] boiko: we are looking at 7.3 hours worth of battery [13:44] with an active call [13:44] pmcgowan: ^ [13:44] timp: well, then uncooment it, and push hope the best, and jenkins will test it [13:44] timp: well, then uncomment it, and push hope the best, and jenkins will test it [13:44] gema, good [13:45] gusch: yes, I will do that. [13:47] hum [13:47] did anyone look adding qtsystems (https://qt.gitorious.org/qt/qtsystems) to the touch image? [13:47] that might be a question for #ubuntu-touch [13:53] gusch, timp, ah sorry, unity notifications are not working, looking now [13:54] om26er: should be sorted timp broke the tests himself ;) === ckpringle_ is now known as ckpringle [14:08] Kaleo, hey [14:08] Kaleo, I've a quick question for you about keyColor if you are around === oSoMoN_ is now known as oSoMoN [14:17] seb128: sure [14:18] Kaleo, is there any way to use that in a ListItems.Standard? [14:18] Kaleo, I was talking with tiheum, the icon he has atm are using a #cccccc grey rather than #888888 [14:19] he recommended using keyColor to change the grey [14:19] but I'm not sure that works with listitems? [14:22] seb128: where did you get keyColor from? [14:22] seb128: you know it's part of the Icon class right? http://developer.ubuntu.com/api/ubuntu-12.10/qml/mobile/qml-ubuntu-components0-icon.html [14:23] Kaleo, right ... I guess my question is "can we set the property for an icon: embedded in a ListItem.Standard"? [14:23] seb128: http://developer.ubuntu.com/api/ubuntu-12.10/qml/mobile/qml-ubuntu-components-listitems0-standard.html#icon-prop [14:23] or do I need new icons with the proper grey? [14:23] seb128: you are probably confused by that bad API & documentation for ListItem.Standard [14:23] gema, kind of interesting comparison here http://www.anandtech.com/show/6440/google-nexus-4-review/2 [14:24] seb128: ultimately the API should be: [14:24] seb128: ListItem.iconSource and ListItem.iconName [14:25] seb128: the source would be any url to any image file [14:25] seb128: iconName would be the name of the icon in the theme [14:25] seb128: right now the icon property can take 2 different values [14:25] seb128: either a URL to an image file [14:25] pmcgowan: yep, we are pretty close to those figures :) [14:25] seb128: or any Item derived class [14:26] gema, yep, and I like the way they describe the testing [14:26] Kaleo, if I set an iconSource, does it use an Icon {} element to build the widget, and can I set keyColor: for that Icon{} in some way? [14:26] pmcgowan: I will definitely be reading that in detail [14:26] seb128: so no it does not use an icon [14:27] Kaleo: fyi, I'd propose to add iconName to ActionItem (all ListItems, Buttons, ... inherit from that now) and property var icon can be deprecated [14:27] seb128: since the API does not accept an icon name [14:27] seb128: it would not work [14:27] Kaleo, ok, I guess that settle it, I will ask for new icons [14:27] Kaleo, thanks [14:27] seb128: so all you can do right now [14:27] seb128: is to create an Icon class yourself [14:27] seb128: ListItem.icon: Icon {} [14:27] pmcgowan: although I am expecting us on the nexus 4 to be pretty close to their figures (since it is mainly hw usage all those tests) [14:27] seb128: (and there is no "iconSource" property) [14:28] gema, right [14:28] pmcgowan: but it is comforting to know we are :) [14:28] Kaleo: there is, from its parent ActionItem [14:28] exactly [14:28] Kaleo: but it is still ignored in the list items [14:28] timp: don't make it more confusing for seb128 [14:28] :) [14:29] timp: so yes, please make sure API includes iconName & iconSource [14:29] timp: and wherever we are using Item icon, we probably need to keep it for now since people rely on it [14:29] timp: but we can make it so that it does not accept a URL anymore [14:29] Kaleo, I've the feeling I will run into https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1190751 if I try your workaround ;-) [14:29] Ubuntu bug 1190751 in Ubuntu UI Toolkit "ListItem.Standard icon's item geometry is incorrect " [Undecided,New] [14:29] timp: (since iconSource will support that) [14:30] seb128: yes, that's a dupe btw [14:30] seb128: we can fix that [14:30] Kaleo, sorry for the dup, I looked for reports bug didn't find it [14:31] seb128: no worries [14:32] Kaleo: Item icon was added because iconSource was not flexible enough, but I don't remember the specific use-case [14:32] brb food [14:33] Kaleo, timp: thanks for the help! [14:34] Kaleo: hmm. instead of iconName: "name" it is now possible to say icon: Icon { name: "name" } [14:36] timp: yes that's what I told seb128 [14:39] timp, Kaleo: I tried that with an icon and ran into the scaling issue ... I will keep the color issue instead until that bug is fixed :p [14:39] "pick your bug" ;-) [14:40] seb128: k [14:51] om26er: are you looking for my review? [14:52] gusch, hmm, missing context. what review? [14:52] om26er: here the delete fix https://code.launchpad.net/~schwann/gallery-app/gallery-delete-last-photo/+merge/170077 [14:55] gusch, I am going to do the functional review to make sure if it fixes [14:55] om26er: fine === ckpringle_ is now known as ckpringle === salem_ is now known as _salem [16:19] gusch: MR approved [16:19] boiko: \o/ [16:47] om26er: please merge trunk and push again for https://code.launchpad.net/~om26er/gallery-app/add_test_for_auto_enhance/+merge/170054 [16:47] gusch, on it [16:47] :) === _salem is now known as salem_ [18:03] om26er, ping [18:03] renato_, pong [18:05] om26er, I am getting this error on my project when I try to run lcov: https://pastebin.canonical.com/93039/ [18:05] do you have any idea? [18:08] renato_, no, no clue :/ === salem_ is now known as _salem === _salem is now known as salem_ [18:40] renato_, there might be some compatibility problem? https://bugs.launchpad.net/ubuntu/+source/lcov/+bug/1163758 [18:40] Ubuntu bug 1163758 in lcov (Debian) "geninfo: Argument "=====" isn't numeric in numeric gt (>) at /usr/bin/geninfo line 1126." [Unknown,Confirmed] [19:10] om26er, but the real problem is: geninfo: ERROR: /home/renato/Projects/phablet/contact-service/fix-dbus-lost/build/src/CMakeFiles/address-book-service-lib.dir/qindividual.cpp.gcno: reached unexpected end of file [19:10] this crash gcov compilation, and I am not able to see any resuts [19:15] renato_, tried split_crc ? http://ltp.sourceforge.net/coverage/lcov/geninfo.1.php [19:15] renato_, that error is specified there [19:18] om26er, let me try === francisco is now known as Guest75516 [19:26] om26er, I am not using geninfo command [19:26] only lcov === boiko_ is now known as boiko === marlinc is now known as marlinc|away