=== salem_ is now known as _salem [03:40] Hey.... any one there [03:42] How to detect filemanager data in qml [03:42] ??? [07:07] Good Morning guys :) [07:38] good morning [07:41] dpm: Good morning :) [07:41] Anyone with device can test this MR on device ? https://code.launchpad.net/~ravirdv/ubuntu-calculator-app/calculator_label_width_fix [07:55] good morning [07:58] morning dholbach [07:59] hey dpm === seb128_ is now known as seb128 [07:59] hey dpm dholbach [08:00] morning seb128, how's it going today? [08:01] good work with the system settings summary e-mails btw, they're very useful to see what's going on [08:01] dpm, thanks ;-) [08:36] dpm: hey , how are you doing ? [08:41] hi mihir_, very well, thanks, and you? [08:41] yes doing good :) thank you :) [08:42] dpm: I have translated calculator app in my native lanungage..the application will automatically take the translation ? [08:43] mihir_, if you translated it in Launchpad, yes. Translations are picked up from there and automatically committed to trunk daily :) [08:43] dpm: okay thank you :) [08:48] gusch: good morning [08:48] oSoMoN: hi [08:48] gusch: first MR of the day: https://code.launchpad.net/~osomon/webbrowser-app/lastVisit/+merge/178790 [08:48] oSoMoN: you start early ;) - I'm on it ... [08:54] gusch: I’m cheating, I submitted it yesterday :) [09:01] oSoMoN: is there something to test in the UI? [09:02] gusch: it’s not 100% reproducible, but I’ve observed that sometimes the list of domains doesn’t remain sorted chronologically, so if you open a number of different websites while keeping the activity view open, eventually you will see that some domains move to where they shouldn’t be, and this MR fixes it [09:03] gusch: basically, iterating over the list of entries outside of the model itself was unreliable, as there was no guarantee the indexes had been fully updated (most of the time they were half updated, which caused weird issues) [09:04] gusch: doing it inside the model guarantees that when m_lastVisit is assigned, the indexes have been fully updated, and so the first entry is guaranteed to be the one we’re interested in [09:14] Good morning all, happy Lighthouse Day! :-D [09:15] oSoMoN: approved [09:15] gusch: thanks! === chriadam is now known as chriadam|away [09:57] dpm: could you please help me to review this https://code.launchpad.net/~mihirsoni-123/ubuntu-calculator-app/RemovedOnclick-1207685/+merge/178526 if you get time :) [10:01] mihir_, sure, looking at it now... [10:09] mihir_, done, thanks for the branch! [10:11] dpm: Thank you :) I'll look into it :) [10:14] cool [10:18] dpm: One question I found another scenario , [10:18] 2+2 = 4 now what if user tap on the previous result ?? what should be the exepected result ?? [10:20] oSoMoN: zsombi: Kaleo: I have been tasked to transform the share-app in a component. do you think it should live inside ubuntu-ui-extras or in a new separate repo ? [10:20] oSoMoN: zsombi: Kaleo: or maybe split share-app in app and component and leave everything in there, and the app will be used only as example or for tests ? [10:25] mihir_, let me get christina in here to ask her [10:25] dpm: sure :) [10:26] dpm: I am unable to review translation done by someother guy in calendar application https://translations.launchpad.net/ubuntu-calendar-app/trunk/+pots/calendar-app/gu/+translate?memo=10&start=10 [10:27] nerochiaro: can you do that? https://code.launchpad.net/~schwann/qtubuntu-camera/qtcamera-vp-flash/+merge/178784 [10:30] gusch: are you sure that there is no possibility of phones haveing a screen-facing camera and a light ? [10:30] hi christina, thanks for joining! mihir_ had a question on what to do when tapping on the result from a previous calculation. mihir_ could you expand on that? [10:31] hi dpm, hi mihir_ - i am just going into a meeting now, but if you leave me some notes i can answer when im back :) [10:32] nerochiaro :Kaleo: my opinion is to be placed in extras [10:32] christina: Hi Christina thank you for joining I am working on bug https://code.launchpad.net/~mihirsoni-123/ubuntu-calculator-app/RemovedOnclick-1207685/+merge/178526 [10:32] nerochiaro: re-the split of share app, I think it would be a good candidate for ubuntu-ui-extras, but the other option you’re proposing sounds good too [10:33] It copy result from previous result [10:33] In following scenario , what should be the exepcted result [10:33] 2+2 = 4 now what if user tap on the previous result (Without starting new converstation) ?? what should be the exepected result ?? [10:33] oSoMoN: zsombi: Kaleo: i'll go with extras then. Will ask to remove the share-app repo later when it's done [10:34] christina: No issues :) this is a scenario i came across let me know the expected result of this scenario :) [10:34] mihir_, my understanding is that nothing should happen as I described on the merge proposal, but let's see if design has got any other view [10:35] dpm: so that means untill user start new converstation it shouldn't react right once equal is tapped? [10:36] mihir_, that's what I understand, yeah. Until the user tears off the calculation, tapping on operands or on the result of the current calculation shouldn't do anything [10:37] basically, the same behaviour that the app has at the moment [10:37] (except for the bug) [10:38] dpm: Okay :) let me try to implement that :) [10:38] cool, thanks! [10:39] nerochiaro, oSoMoN, gusch, if you're on the latest build and have got a device with you, could you help me and christina determine if the weather app toolbar works as expected? It's basically about loading the weather app on the device and pull the toolbar to see if it's empty or if it contains the toolbar actions as expected [10:39] it shouldn't take more than a few seconds, and would be really helpful :) [10:40] dpm: sure, will do that in a moment once the tests that I’m running on the device are done [10:40] great, thanks! [10:46] nerochiaro: thi si not the to properly detect if the light is available (from android) [10:46] gusch: next MR of the day: https://code.launchpad.net/~osomon/webbrowser-app/fix-crash-collapsing-domain/+merge/178919 [10:46] dpm: i need to reflash to the latest. do you want me to test on a regular update or with --pending ? [10:47] gusch: sorry, didn't undersand your last comment [10:47] nerochiaro, a regular update would do [10:47] dpm: I’m not seeing any toolbar in the weather app, when I swipe from the bottom edge I’m getting the HUD icon only [10:47] dpm: ok, flashing, will test in a bit [10:47] dpm: I’ve got ubuntu-weather-app 0.2bzr84saucy0 [10:48] oSoMoN, thanks. And you had a location defined in the app, too? [10:48] nerochiaro: about your question "are you sure that there is no possibility of phones haveing a screen-facing camera and a light ?" [10:48] oSoMoN: looking at it [10:49] gusch: ok, but then i'm not sure what the MR is about. you basically remove all the code for reporting the flash/torch modes and replace it with hardcoded results [10:49] dpm: it says "Current location | Lookup location (Dummy)" [10:50] dpm: wait, I was on the "add location" screen which showed up by default when I started the app [10:50] nerochiaro: well - most important here is, that photo and video share the same flash modes [10:50] oSoMoN, ah, you're on the initial screen with the location settings. Could you try to add a location and press confirm, and then see if you can pull the toolbar on the next screen? [10:50] yeah [10:50] nerochiaro: that's needed for flash [10:50] nerochiaro: that's needed for torch [10:50] dpm: now that I chose a city, I’m seeing the toolbar work, it has three actions "refresh", "edit", "settings" [10:50] oSoMoN, perfect, so it seems to work as expected. Thanks! [10:51] yw [10:51] dpm: the app looks pretty neat, first time I’m running it :) [10:51] nice :) [10:52] oSoMoN, if you notice any bugs, feel free to report them on https://bugs.launchpad.net/ubuntu-weather-app/ :) === MacSlow is now known as MacSlow|lunch [10:56] gusch: so basically it should be possible to enable also torch while in picture mode ? [10:56] dpm: sure, will do [10:57] dpm: would "the weather is weird around here today" be considered a valid bug? ;) [10:57] nerochiaro: yes, not in the camera, but for the Qt QML Torch component [10:57] oSoMoN, perhaps, but don't expect the developers to fix the weather :-) [10:58] * oSoMoN knew it… [10:58] :) [10:58] gusch: well, i'm gonna approve it because it doesn't seem to have any real effect on camera. [10:58] gusch: but i'm still not sure i see the point [10:59] nerochiaro: hmmm - we don't have a torch app, to see that it currently is broken ... [10:59] nerochiaro: but thx === marcoceppi_ is now known as marcoceppi [11:26] oSoMoN: approved the crash [11:27] gusch: thanks [12:02] dpm: i got no toolbar on the weather app as well [12:03] greyback: on the latest image have you noticed how the shell isn't showing icons for some apps ? [12:03] greyback: and the converter app appears twice === MacSlow|lunch is now known as MacSlow [12:05] nerochiaro: this in apps lens, or launcher? [12:05] greyback: app lens [12:06] greyback: reflashing with --pending as i need to test something else, will let you know if it's still happening [12:06] nerochiaro: that data is coming from an actual apps lens, it's not faked any more. Lens got confused?? [12:07] greyback: i guess. let's see if it's still there, if it is we can try to figure out what's happening [12:07] nerochiaro: agreed [12:09] nerochiaro, thanks for testing. And this is after you've added a location? [12:09] dpm: no, didn't try that [12:09] nerochiaro, ah, you'll need to add a location first. The toolbar is only shown on the weather screen [12:10] dpm: i find it confusing that you can't add a location from the toolbar [12:10] dpm: i'm currently reflashing the phone again, will try adding a location when it's finished and let you know [12:11] nerochiaro, that's probably a question for design, but essentially, the "new location" screen is shown if you don't have any locations defined upon start, so there is no need for a toolbar there. Once you've added a location, the weather screen is shown and from there you can manage your locations from the toolbar [12:11] Kaleo: zsombi: timp: oSoMoN: ubuntu-ui-extras is still pending the initial commit, can anyone take care of approving it now that Jenkins has all test passing on the MR: https://code.launchpad.net/~amanzi-team/ubuntu-ui-extras/ubuntu-ui-extras-initial/+merge/171086 [12:12] dpm: ok, if that's the way it's been designed let's roll with it [12:12] :) [12:15] nerochiaro: uhh... that's not that fast to review... hasn't this been reviewed by someone of us months ago? [12:15] zsombi: that's fast because after the last commit there's only a skeleton example left in there [12:16] zsombi: and yeah, you guys started reviewing but never finished [12:16] nerochiaro: so a proper review has never been done then... [12:17] zsombi: not a full one I guess [12:17] s/proper/complete [12:19] zsombi: i'm also not sure who should be in charge of that [12:19] nerochiaro: :) "you are not alone..." [12:23] gusch, ping [12:23] om26er: pong [12:23] hey, this test is specifically failing on mako camera_app.tests.test_focus.TestFocus.test_focus_valid_and_disappear [12:24] gusch, i have seen in a video that the camera goes blank [12:24] gusch, it does not happen always but in most cases (given you have the very latest image) running the whole suite will end up with a failure of that test [12:25] om26er: hmmm - might be https://bugs.launchpad.net/camera-app/+bug/1204260 [12:25] Ubuntu bug 1204260 in touch-preview-images "[camera-app][mako] switching cameras crashes mm-qcamera-daemon" [High,Confirmed] [12:25] om26er: but I don't have mako, so I can't check it ... [12:25] gusch, its probable, the issue happened while it was switching cameras [12:26] gusch, you should get a nexus 4 :) [12:27] om26er: and a 7, and ... ;) [12:39] zsombi: so any idea what should I do about it ? [12:39] nerochiaro: checking the MR... [12:41] nerochiaro: I guess it should be us who should own it, but discuss this with bzoltan [12:45] bzoltan: zsombi: Kaleo was of the idea that the sdk team should own it too, if i recall correctly [12:45] bzoltan: talking about lp:ubuntu-ui-extras [12:46] greyback: still broken (missing icons) in latest image with --pending [12:46] nerochiaro: let me see it [12:47] bzoltan: the main branch is empty, there's a branch pending merge into it that contains the current initial code [12:47] bzoltan: it's only a skeleton example for adding a new component in there, at the moment [12:47] bzoltan: nerochiaro: if we get it under SDK, then my only question is whether we should have a separate project for it, or should we put under the same code base as the current SDK is. [12:48] bzoltan: nerochiaro: the current branch (the initial one) is based on cmake and our one is on qmake... [12:48] bzoltan: zsombi: separate project, as it doesn't have to match the same standards as the sdk (as evidenced by the description that Kaleo put on the lp project) [12:48] zsombi: i'm fine with either, i just used cmake as it was what i was used to from other projects [12:49] nerochiaro: then it can live in its own way, with whatever build system it has now [12:49] dpm: with latest image and --pending the weather app shows the toolbar after adding a location. toolbar has refresh, edit and settings buttons [12:49] zsombi: great [12:49] perfect, thanks a lot nerochiaro for testing! [12:49] nerochiaro: I have nothing else to comment on the MR then :) [12:49] nerochiaro: let me talk with Kaleo about it. [12:50] dpm: you're welcome [12:50] zsombi: then give your comment-approval at least [12:50] bzoltan: ok, thank you [12:51] nerochiaro: you have it! [12:52] nerochiaro: My inner sense says that if something belongs to the SDK team than the same quality standard applies as to any other project... If that project would be a home for components and APIs we do not officially support or do not consider mature enough then I would suggest to create a Ubuntu.Components-Extras under the UI TK project. === _salem is now known as salem_ [12:56] bzoltan: +1 [12:59] bzoltan: the current organization and repo was discussed with Kaleo and AFAIK approved by him [12:59] nerochiaro: Is there a mail, irc log or anything about it? [12:59] nerochiaro: I would like to hear Kaleo's idea in details [13:00] nerochiaro: do you have the link to the google doc? [13:00] Kaleo: bzoltan: looking for the doc [13:01] Kaleo: bzoltan: https://docs.google.com/a/canonical.com/document/d/1xGmQd1qRMFIHIupybpzFmIZHRGUmRhSJanX62ElG5EA/edit === greyback is now known as greyback|lunch [13:02] Guys that document starts with perfect explanation why I would never touch that project :) [13:02] QML components that either: - won’t necessarily see much reuse - does not have the necessary level of quality ... [13:03] nerochiaro: Kaleo: Why to move our finger if a component (1)is useless crap (2) [13:05] bzoltan: Kaleo: i don't necessarily agree with that definition. I just want to have a place where to put some components that don't have any other place to go. for example now i need to put the sharing component somewhere, and that seems a pretty good place [13:05] nerochiaro: go for it [13:06] nerochiaro: however, review wise I would suggest to do what you would do for your app [13:06] nerochiaro: we need a sharing component. It is on the list.... [13:07] Kaleo: yes, but before that I would like to have the pending merge request merged into the trunk of that repo. i'll then have whoever would be responsible to review the share-app review the share component when i propose it for review into the ubuntu-ui-tookit repo [13:07] bzoltan: i was tasked by bfiller to transform share-app into a component. i need a place to put it. that's all i know. if you guys are planning to make a different one, we need to talk about it [13:08] nerochiaro: let's do it the right way than... but I would not suggest to create a shadow UITK where different rules apply than in the real one....I agree that if the community needs components and want to create projects-packages they are free to do. [13:11] bzoltan: i'm not sure what you're suggesting I do [13:12] nerochiaro: I see the MR is approved [13:12] Kaleo: thank you [13:12] nerochiaro: 20 minutes ago :) [13:12] Kaleo: yeah :) [13:30] nik90, when you get the time https://code.launchpad.net/~om26er/ubuntu-clock-app/fix_autopilot_on_touch/+merge/178443 === greyback|lunch is now known as greyback [13:31] om26er: definitely [13:32] nerochiaro: I suggest that if you plan to work on a component what is on the roadmap of the UITK project then just contribute it to the UITK [13:37] bzoltan: i had no idea it was on the roadmap, and i suppose that i can start working on it there and then if you guys think it's ok we can move it to the main UITK ? [13:38] om26er: 3 tests are failing. You can find the output at http://paste.ubuntu.com/5958920/ [13:38] zsombi: are the changes in IMA we were discussing yesterday already in the SDK that goes into today's image smoke tests ? [13:38] om26er: you used timer_page and then returned time_page instead [13:39] hah [13:39] nik90, fixing [13:39] om26er: :) [13:39] nerochiaro: haven't checked, however it got merged into the SDK trunk... [13:40] nik90, done [13:40] nerochiaro: Good that it came out then :) Yes, let's shoot for normal and full integration to the UI TK [13:40] nerochiaro: this one also sais it's in: https://code.launchpad.net/~ps-jenkins/ubuntu-ui-toolkit/latestsnapshot-0.1.46+13.10.20130807-0ubuntu1/+merge/178920 [13:42] zsombi: it seems that we have one test failing and i suspect this to the cause [13:42] ? [13:45] oSoMoN: to make gallery crop test work with the latest SDK https://code.launchpad.net/~schwann/gallery-app/gallery-close-toolbar-on-edit/+merge/178961 [13:45] nerochiaro: which one? [13:47] nerochiaro: in the tests where you wait for the note to be expanded, are you also verifying that the 'state' property of the note is 'expanded' (or something like that)? [13:48] zsombi: https://jenkins.qa.ubuntu.com/job/saucy-touch-mako-smoke-notes-app-autopilot/47/consoleText [13:49] oSoMoN: i'm not in this specific case, but the property change is what actually triggers the height change, so i thought there was no need to do it [13:51] nerochiaro: agreed, in theory there shouldn’t be any need, but in practice having more checks allows easier debugging of failures, which is why I was asking [13:51] oSoMoN: makes sense, [14:08] oSoMoN: to make gallery crop test work with the latest SDK https://code.launchpad.net/~schwann/gallery-app/gallery-close-toolbar-on-edit/+merge/178961 [14:18] gusch: is there a bug associated to your MR? [14:18] oSoMoN: no - I only came across that issue, because I wanted to get this merged... https://code.launchpad.net/~schwann/gallery-app/gallery-uitk-emulators/+merge/178330 [14:19] gusch: ok, thanks === mpt_ is now known as mpt [14:43] oSoMoN: i'm not really sure what to do to fix that failing test, i doubt adding the extra check or isExpanded will solve the problem, but I can certainl try [14:44] nerochiaro: can you point me to the code of the failing test? I’ll have a look too [14:49] gusch: approved, please see my comment [14:49] oSoMoN: sure, hold on [14:51] oSoMoN: http://bazaar.launchpad.net/~phablet-team/notes-app/trunk/view/head:/tests/autopilot/notes_app/tests/test_parts.py#L55 this is the function that fails in that specific case, but as you can see it is used other cases that don't fail [14:56] oSoMoN: thx [15:00] nerochiaro: what is the height of a collapsed note? [15:01] oSoMoN: 11 gu [15:02] nerochiaro: ok, so in the failing test the note wasn’t expanded at all, it remained collapsed (as opposed to being expanded halfway or something like that) [15:03] oSoMoN: ok, that means that the click on the note wasn't registered for some reson. but it's a standard click so i'm not sure what can go wrong. self.pointing_device.click_object(note) [15:34] http://developer.ubuntu.com/2013/08/announcing-the-2013-ubuntu-app-showdown-contest/ [15:34] a new contest for Ubuntu Touch app developers! [15:34] mhall119, \o/ [15:39] mhall119: awesome! [15:45] dpm, about bug 1204702 you saw that the music app team as wrote some code, but I'm not able to find it, can you help me? [15:45] Launchpad bug 1204702 in Ubuntu Calculator App "ubuntu-calculator-app does no db upgrade to 0.1.1 on first start" [High,In progress] https://launchpad.net/bugs/1204702 [15:45] dpm, if you have time, I'll say you what problem I'm finding to fix it [15:46] WebbyIT, let me see [15:47] dpm, I'm searching a way to have more upgrade, one after the other, so I think to a recursive function but... [15:48] ... dpm, db.changeversion doesn't work as I expected, see http://askubuntu.com/questions/327594/db-changeversion-doesnt-work-as-expected [15:49] dpm, I thinking something like http://paste.ubuntu.com/5959364/ [15:50] dpm, I can do more cases to avoid intermediate version of db, but in case of future upgrade will be a mess [15:53] WebbyIT, I've just found how they did it at the bottom of the diff for https://code.launchpad.net/~danielholm/music-app/show-tracks-in-playlists/+merge/175333 [15:53] however, I'm not sure it's the best way [15:54] dpm, that is a wrong way, because at the first start the db will be create but NOT update to v1.1 [15:55] yeah, I didn't look into it in detail, but somehow it didn't look good to me [15:56] dpm, is the same code we are in calc, we have to find a way to have something that can allow to have more than one upgrade [16:00] dpm, I also tried http://stackoverflow.com/questions/11834739/phonegap-android-database-changeversion-not-functioning dut unfortunately doesn't work :( === greyback__ is now known as greyback|food === greyback|food is now known as greyback [16:47] om26er: ping [16:47] boiko, pong [16:47] om26er: hi, do you know why on this MR jenkins didn't post the URL for the deb packages? https://code.launchpad.net/~tiagosh/telepathy-ofono/audioflinger-speakermode/+merge/178873 [16:49] boiko, the job needs to be changed to post debs [16:49] boiko, I can enable that if you want [16:50] om26er: that would be awesome [16:55] om26er: thanks [17:06] boiko, its good to go, you can re-build to have the debs now [17:07] om26er: thanks! [17:08] boiko, np :) [17:09] oSoMoN, ping [17:10] om26er: pong [17:10] oSoMoN, what was it about in #phablet ? I mean no one even knew the bug number ? [17:10] om26er: looks like there is no bug report, there should be one but I can’t find it [17:10] oSoMoN, so probably we cannot be blamed ;) [17:11] no indeed [17:32] m-b-o: hey the new flip animation looks cool! Nice one! [17:38] nik90: yes, neokore did some really nice work [17:39] nik90, perhaps the visual design will land this week, so more to come :) [17:39] thank you nik90 m-b-o [17:39] m-b-o: oh wow..looking forward to them [17:39] neokore: nice work :) [19:20] nik90, ping [19:27] seb128: is the fix for libgsettings-qt-dev already merged? (if CI is failing, autolanding is going to fail too for the sound settings MR) [19:28] boiko, yes, that landed on friday [19:29] seb128: ah nice, let me trigger CI again then, thanks [19:29] boiko, thanks ;-) [19:40] How do I run just the ubuntu touch os in the sdk [19:40] ? === n00b is now known as Guest63986 [19:41] How do I run just the ubuntu touch os in the sdk? [20:08] hi, i have a question regarding the u1db-qt plugin.... y want to create a db for my app... but i can't use the: "~/.config/db" notation in path.... which would be the proper place to create the db?? i assuming that i don't have write permissions where my app was installed [21:14] Is it allowed to use a different library for a Showdown application? I want to write my app in Python and Kivy, which is very good for mobile/touchscreen apps. === salem_ is now known as _salem