/srv/irclogs.ubuntu.com/2014/11/07/#ubuntu-app-devel.txt

justCarakasqml question: can you dynamicly add or delete menu items in your app ? or is that not a good idea for having user managed tabs00:27
justCarakasnm found ut00:29
justCarakasit00:29
=== chihchun_afk is now known as chihchun
akiva-thinkpaderm07:42
dslulhello07:44
dsluli have some problems running an app on the emulator, it shows me this error: :-1: error: Could not connect to host: SSH Protocol error: Server and client capabilities don't match. Client list was: aes128-cbc,3des-cbc. Server list was aes128-ctr,aes192-ctr,aes256-ctr,aes128-gcm@openssh.com,aes256-gcm@openssh.com,chacha20-poly1305@openssh.com. Did the emulator fail to start?07:45
dslulthe emulator is running and the code is the default "App with tabbed UI" as shown in the video tutorial on the ubuntu wiki07:46
dslulis there anyone online?08:03
=== rmescandon is now known as rmescandon-brb
akiva-thinkpaddpm, ah so I can go ahead with the two sessions.08:42
dpmakiva-thinkpad, cool. We talked about the Go one, which was the other one?08:43
akiva-thinkpaddpm, roundtable on creating a theming standard08:44
dpmok, great. Then we'll approve them08:44
akiva-thinkpadand proposals on how we can create better incentives to contribute to ubuntu for developers.08:44
akiva-thinkpaddpm, well the Go one is still on hiatus08:44
akiva-thinkpadIts just one I wanted to see, but I know nothing about go, and have no one to speak on it.08:45
dpmakiva-thinkpad, I'm trying to confirm a go advocate - if he can drive the session, and there are some attendees interested, then we could run it08:46
akiva-thinkpaddpm, I would definitely join that :P08:46
akiva-thinkpadanyways let me know if you need anything from me, or have any expectations that you want to make sure are emphathized08:47
dpmakiva-thinkpad, cool. For the "creating a theming standard" I'd suggest to ask folks from the SDK team to join, as they're the experts08:49
akiva-thinkpaddpm, yah sure. Will do.08:50
akiva-thinkpad!ping08:57
ubot5pong!08:57
=== rmescandon-brb is now known as rmescandon
mzanettirpadovani: hey, you around?09:13
mzanettidpm: o/09:13
mzanettiI'd need some help getting some stuff merged09:14
dpmmzanetti, hey. Weird, that's generally the other way around me asking for help :)09:15
mzanettidpm: well, https://code.launchpad.net/~reminders-app-dev/reminders-app/trunk/+activereviews09:15
mzanetti:D09:15
mzanettiand there is one branch on my had disk here which I know you'll like a lot09:15
dpmah, yeah, man, I saw last night that you were on fire!09:16
mzanettiI tried to keep them in small easy to review portions09:16
dpmmzanetti, sure, will have a look. I tested 2 features from trunk last night: multiple accounts and sorting by edit date. But I could not get either to work. Do they work for you?09:18
mzanettimultiple accounts works perfect for me09:18
mzanettiI approved/merged it yesterday09:18
mzanettiI guess one prerequisite of that is to actually have 2 accounts correctly set up in OA09:19
dpmmzanetti, I've got both my personal account and evernote300 defined in system settings and I see no "Accounts" header action at all09:19
dpmpenk, rpadovani, re: the latest comment on https://code.launchpad.net/~reminders-app-dev/reminders-app/uriHandler/+merge/239961 - is this something that needs to be addressed in this branch, or somewhere else? Can the branch be approved?09:20
mzanettidpm: we need to fix that before landing09:21
popeyakiva-thinkpad: what time slot did you want for your go session?09:21
mzanettidpm: I do have the accounts button09:21
mzanettidpm: and at application startup it prompts me for which account to use09:21
penkdpm: in terms of uriHandler, that branch works09:21
penkdpm: I'm just wondering if the onOpened won't be called if application hasn't started is normal09:22
dpmmzanetti, weird, no prompt, no accounts header for me :/09:23
mzanettipenk: that worked fine for me09:23
popeynik90: what time / day do you want your component store session?09:23
penkmzanetti: really, do you mind to test it with scope?09:23
popeynik90: http://summit.ubuntu.com/uos-1411/2014-11-12/display?edit link to schedule09:23
mzanettidpm: you sure the accounts are enabled and the app has access to it?09:23
dpmmzanetti, re: urihandler, what exactly needs to be fixed?09:23
nik90popey: one sec09:24
mzanettidpm: what penk said. he says it won't work if the app is not running09:24
mzanettidpm: but I know it did work for me at some point09:24
mzanettilet me test it09:24
penkmzanetti: dpm: I'll send you guys the evernote scope09:25
dpmmzanetti, they are: I had already my personal account enabled (i.e. no changes there) and then I added evernote300. It should have already seen it even without granting access to reminders. It didn't, so I granted the access in u-s-s, but still no prompt, no header action09:25
dpmpenk, cool. Could you send a MP adding the urihandler branch as a prerequisite as mzanetti mentioned yesterday?09:25
nik90popey: I scheduled it to friday 18:00 UTC09:26
penkdpm: prerequisite of what?09:26
nik90popey: that should give me sufficient time to prepare for it09:26
popeyah yes, you can do that ☻09:26
popeysorry, forgot you had the rights to schedule09:26
popeyawesome09:26
nik90popey: I was still contemplating on if I should host the session. I just made the decision09:27
dpmpenk, I assume when merging the scope to reminders you used the urihandler branch, which hasn't landed in trunk yet. When you send the MP with the scope, you can add "~reminders-app-dev/reminders-app/uriHandler/" as a prerequisite, so that only the diff for the scope changes are shown in the MP09:28
popeyheh09:28
mzanettilol... now I get "Authentication failed" from OA, without touching the accounts at all09:29
penkdpm: no I'm not09:29
mzanettiit was working the whole day yesterday09:29
penkdpm: I'll push another branch09:29
dpmmzanetti, did you perhaps stare too hard at online accounts? :)09:30
penkmzanetti: dpm: scope sent09:30
mzanettidpm: yeah... I was thinking what could be wrong at your setup... seems to have broken mine :D09:30
mzanettipenk: should the department selection work already?09:53
penkmzanetti: yes it should09:53
mzanettioh, it does... I'm just confused by it...09:53
mzanettiit says "Notebooks"09:53
mzanettiwhich means no notebook is selected... in the app if the heading says "Notebooks" you only see notebooks09:53
penkmzanetti: would change it to "All Notebooks" better?09:54
mzanettinot sure if the reason I'm confused is the evernote scope... I kinda get confused by departments in the app store too tbh09:54
mzanettipenk: yeah... I guess in this case "All Notebooks" would be better09:56
penkmzanetti: will do09:56
popeymihir: those two merges failed AP ☹09:59
JamesTaitGood morning all; happy Friday, and happy Hug a Bear Day! :-D10:01
dpmmzanetti, penk, so to make sure I understand - the reason why the app does not start when invoked from the scope if it's not already running is a bug in the app, or in the platform? If it's the app, does it need to be fixed in a separate branch...?10:02
penkdpm: I'm not sure if it's a bug, because mzanetti said it works, if it's a bug, it will be in the platform I assume10:02
penkmzanetti: dpm: I'll be EOD soon, feel free to reply your test result via mail, thanks!10:05
mihirpopey: yes i saw that , i tried to rebuild , but it didn't can you re-run ?10:10
popeyi did already10:10
dpmpenk, installed the scope, building reminders with urihandler support now10:10
penkdpm: thanks10:10
dpmpenk, I like the notebooks department thing, quite handy10:11
penkdpm: Joe requested that ;-)10:11
* mihir fingers crossed for Jenkins10:16
=== chihchun is now known as chihchun_afk
popeymihir: https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/CalendarChoiceDesignFix-4/+merge/239648 passed!10:29
rpadovanimzanetti, o/10:32
mihirpopey: Good to go :D10:39
mihirpopey: you can top approve this :)10:39
mihirpopey: why i am not getting emails :P i relay on emails10:41
mzanettirpadovani: hey ho10:43
rpadovani\o10:43
mzanettirpadovani: just tested the uriHandler branch.. not good enough yet10:44
mzanettisorry10:44
rpadovanimzanetti, it's not your fault, it's mine :P10:45
rpadovanimzanetti, so, what's up?10:45
mzanettirpadovani: I left a comment on the MP10:47
rpadovanimzanetti, ok, thanks. You did a lot of work yesterday O.o10:47
dpmmzanetti, so 'url-dispatcher evernote://newnote' launches the app, but it doesn't show the new note dialog10:52
popeymihir: sorry, was in a meeting11:00
popeymihir: top approved the first one, second one still failed https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/DesignFixDefaultWeekView-5/+merge/24086711:00
mihirpopey: i'll look into that.11:00
mihirpopey: thanks11:00
mihirpopey: i am worried , is the Mail queue  , i am not able to receive emails11:01
popeyhmm, odd.11:02
rpadovanigventuri, mhh, I'm not able to join the hangout, when I press join the button disappears for a second then appears again :/ Could you please query me a direct link?11:03
gventurirpadovani: sorry, I'm in a conf call with Kaleo11:05
gventurirpadovani: I'll ping you when I'm done11:05
rpadovanisure, thanks11:05
popeyzbenjamin: you about?11:11
=== oSoMoN is now known as oSoMoN|afk
zbenjaminpopey: about what? ;D11:11
popey\o/11:11
popeycan you join #ubuntu-touch-meeting pls11:11
popeyt1mp: bug 1383042 could you take a look pls? http://imgur.com/xPD6Sml12:30
ubot5bug 1383042 in Ubuntu Weather App "Newly added locations display strange text rather than city name" [Critical,Confirmed] https://launchpad.net/bugs/138304212:30
t1mppopey: that is weird12:33
t1mp^is that a bug introduced recently, or did it exist all the time?12:33
popeyi only saw it recently12:34
popeym-b-o: ^12:35
m-b-ot1mp wasn't presented when I've moved the LocationManager from Sheet to Page12:35
t1mpthe header builds the menu using a Repeater on the Tabs and uses the title of each tab, and the menu contents is not created until the tabs button is clicked12:36
t1mpso somehow I suspect the Tabs to have wrong titles12:36
t1mpm-b-o: is it possible to refresh the data (and wait for it to finish) before popping the page?12:37
m-b-ohmm, I will try. Would it be a possible solution, to go back, wait unitl the pagestack is done and then triggering the rebuild?12:39
nik90rpadovani: hey, answered your questions in the mp12:41
justcarakashow can I set my app to use the SuruDark theme ?12:52
=== oSoMoN|afk is now known as oSoMoN
m-b-ot1mp: indeed, the Tabs in the weather app have at Component.OnCompleted the wrong title when file://... appears13:37
m-b-ot1mp but when building the Tabs in http://bazaar.launchpad.net/~ubuntu-weather-dev/ubuntu-weather-app/trunk/view/head:/ubuntu-weather-app.qml#L164 definitely the right title is set13:38
m-b-ot1mp and when after rebuilding the Tabs after deleting locations, some old tabs aren't destroyed http://paste.ubuntu.com/8867778/13:46
t1mpm-b-o: so if you print the qml strring, it appears correct, but when you create a qml object from it that is wrong?13:52
m-b-ot1mp: sometimes the error described in the bug report happens13:52
m-b-ot1mp: and after that it seems the Tabs were cached or something.13:52
m-b-ot1mp: and deleted locations are still shown, where as some locations are missing. like in the pastebin link13:53
m-b-ot1mp; so yes, the string is always right13:54
t1mpm-b-o: since when is this bug occurring?13:54
t1mpI wonder what may have changed to cause this13:54
m-b-osometimes early october, before 19.13:56
m-b-ot1mp: after 25.9. I'm pretty sure13:56
davidcalleIf anyone is interested in using it or improving it, I've just made a quick phablet-screencast script : http://bazaar.launchpad.net/~davidc3/+junk/phablet-screencast/view/head:/phablet-screencast13:59
m-b-ot1mp: the full file:/// string: "file:///opt/click.ubuntu.com/com.ubuntu.weather/1.1.latest/tabs"13:59
nik90davidcalle: what's the ratio for?14:00
davidcallenik90, 1 is actual screen size, 2 half the size, 3 a third of the size. The raw file can get pretty big (30 seconds ~= 1gb), so the ratio can be used to reduce that.14:01
=== rmescandon is now known as rmescandon|AFK
nik90davidcalle: ah14:01
nik90pretty cool thnx14:02
davidcallenp14:02
t1mpm-b-o: it is quite difficult to see what is going on. Do you think you can create a simpler qml program to reproduce the issues?14:04
* t1mp lunch, brb14:05
m-b-ot1mp: I 'll try. Would it be possible to use a repater for the Tabs?14:05
t1mpm-b-o: we had support for that once, but we had to revert some changes14:05
t1mpzsombi: ^ do you remember if we can have a Repeater for Tabs now?14:05
m-b-ot1mp: it happens only on the device, btw14:06
t1mpohh.. that makes debugging even harder14:06
t1mpI'll be back after I eat something14:06
=== rmescandon|AFK is now known as rmescandon
kalikianakarni: you'll have to review this one, whether you want or not :-D https://code.launchpad.net/~kalikiana/u1db-qt/parentPaths/+merge/24108514:34
karnihehe14:35
karnikalikiana: what's "-p" "-o" "-p" "-,txt") in the dbus-test-runner command at the end?14:39
karnikalikiana: approved, thank you. please see my approval comment before landing :)14:41
m-b-ot1mp once the descriped failure occurs, the recreation of the tabs after adding/removing locations fail too,  since the old tabs are shown14:43
kalikianakarni: -p means "pass this on to the command that dbus-test-runner runs"14:44
karniah14:44
kalikianakarni: the "changes" in the test are essentially refactoring so it's not all in one14:45
kalikianaso don't worry about Query there for now14:46
kalikianathe real tests are actually mostly qml, there's just the few to see if c++ api works as well14:46
kalikianaie. the functionality isn't really tested there14:46
karnikalikiana: yes, I noticed these are mostly refactors. the fix looks good though14:46
kalikianabut obviously in qml we can't create folders for testing14:46
karnikalikiana: by the way, who maintains u1db-qt? is there more people or just you?14:46
kalikianakarni: just me; had some reviews from benjamin at one point but then it was considered "low prio". I'm hoping to get to work on it again soon, side from critical bugs14:48
karnikalikiana: I see. It's nice that you're trying to maintain it. And important, for that matter.14:48
karnikalikiana: it's very good fit for Telegram per-user settings :)14:49
kalikianakarni: very happy to hear that!14:50
karnirmescandon:14:54
karnihttps://code.launchpad.net/~libqtelegram-team/libqtelegram/libqtelegram-dev-group-profile/+merge/24108814:54
karnihttps://code.launchpad.net/~libqtelegram-team/libqtelegram/telegram-app-dev-group-profile/+merge/24108714:54
karnirmescandon: both are work in progress, but most of group profile basic feature is there14:55
karnirmescandon: I'd like to get some feedback on the library bits from you, when you have time to look at it.14:55
rmescandonkarni, today i think i won't be able14:55
rmescandonkarni, i'm trying to upload sth useful to penk's branch14:56
m-b-opopey t1mp: sorry, I'm overasked with https://bugs.launchpad.net/ubuntu-weather-app/+bug/138304215:29
ubot5Ubuntu bug 1383042 in Ubuntu Weather App "Newly added locations display strange text rather than city name" [Critical,Confirmed]15:29
popeym-b-o: need help?15:30
m-b-opopey: yes15:31
popeyok. lets see what we can do. thanks m-b-o !15:31
sverzegnassidpm, could you please review this branch? https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/fix-1387651-1387023/+merge/24030415:31
popeyt1mp: is this something you can help us with ?15:34
dpmsverzegnassi, sure, thanks! It might have to be in a few hours time15:36
mhall119any QML gurus around here that might be able to answer http://askubuntu.com/questions/546649/ubuntu-sdk-how-do-i-create-a-tree-view-in-qml15:37
mhall119I'd be interested to know too :)15:37
sverzegnassidpm, thanks! As that branch is approved, all the work on contenthub will be finally able to land15:38
dpm\o/15:38
t1mppopey, m-b-o I am finishing some other work and then it is eow for me15:39
t1mpmaybe beginning of next week I can see if I can do something about it, but a simpler app to reproduce the issue would be very helpful to have15:40
t1mpwednesday-friday next week I'll be quite busy with the ubuntu online summit15:40
m-b-ot1mp popey I'll try to replace the dynmaic createQmlObject part with a repeater based solution like in shorts. perhaps that will help15:40
m-b-ot1mp okay, I'll try to be online15:41
bzoltanmhall119:  I answered that question... sadly one does not need to be a big qml guru to know the answer. But having text bubbles with different indentation (depending on depth in the tree) wouldn't be very complex to get.15:45
bzoltanmhall119:  The bubble text idea came from t1mp :)15:46
mhall119hmmm, that's what I did for my reddit app15:48
mhall119I assumed that was just a cheap hack on my part :)15:49
popeysverzegnassi: heya, I get this when building that branch... http://paste.ubuntu.com/8869161/15:49
popeysverzegnassi: this too... http://paste.ubuntu.com/8869171/15:51
* popey updates his chroot15:52
sverzegnassipopey: while building, it returns two warning. I copy-paste some code I wrote some time ago for an app-drawer. it works anyway. I'd like to rewrite the whole plugin, since I expose to QML some information about PDF pages that I send back to C++ again. I'd like to see the whole thing written in C++15:53
popeyok15:53
=== ara is now known as Guest54796
mhall119t1mp: hey, I'm trying to make a ListView-like component with a delegate, but I'm having some toruble getting it to work, think you can help me?17:03
mhall119http://bazaar.launchpad.net/~mhall119/ureadit/2.0/view/head:/ui/Frontpage.qml line 20 is how I want it to be used17:03
mhall119http://bazaar.launchpad.net/~mhall119/ureadit/2.0/view/head:/components/MultiColumnListView.qml is the implementation (notice it's not a subclass of ListView), line 39 is where I'm creating the delegate components17:04
t1mpmhall119: sorry, I have to leave now, it will have to wait till Monday17:04
mhall119t1mp: ok, I'll keep posting my problem here, maybe some other appdev can help me17:04
mhall119so my delegate creation "works" in that it makes the components and the layout code places them properly17:05
mhall119but they're not being initialized by the "model" variable, and I get the following error message for each one:17:05
mhall119file:///home/mhall/projects/uReadIt/ureadit/ui/Frontpage.qml:44: ReferenceError: model is not defined17:05
mhall119"model" is one of the properties I'm passing to delegate.createObject(display.contentItem, properties)17:06
mhall119so something is getting lost somewhere17:06
dpmany other qml expert around to answer http://askubuntu.com/q/544355/9781 ?17:11
mihirpopey: ping17:17
mihirgventuri: hi17:17
nik90dpm: I think that's a sdk bug.17:18
popeymihir: hey17:18
mihirpopey: saw email ?17:18
nik90dpm: also one shouldn't use a dialog for that purpose. It would best if he used a pagestack to show those options17:18
rpadovanimzanetti, ping17:19
nik90popey: let17:19
dpmnik90, thanks! If you use ask ubuntu, you might want to mention that to him17:19
gventurimihir: hi17:19
nik90popey: let's release a new clock to the store tonight, once https://code.launchpad.net/~nik90/ubuntu-clock-app/implement-location-finding/+merge/231793 gets merged17:19
nik90dpm: yeah will do17:19
popeymihir: ooh17:19
mihirgventuri: hi , saw email , and it looks great :)17:20
popeynik90: ok i am dependent on balloons for that.17:20
gventurimihir: thanks17:20
balloonsnik90, just ping when it's landed and we'll push the click17:20
popeygventuri: that was a nice surprise email. thanks!17:20
bzoltannik90: dpm: I will bring this issue up on Monday. But i agree with nik90 that dialogs are suboptimal for this use case17:20
gventuripopey: yeah, I didn't want to let you down guys17:21
gventuripopey: I'm going to be focusing on the browser form next week17:21
nik90balloons: sure will do17:21
gventurifrom17:21
popeymuch appreciated.17:21
mihirgventuri: this is an amazing surprise17:22
popeythanks nik90, you going to retire now? ☻17:22
ogra_are you offering him a pension ?17:22
nik90popey: retire?17:23
dpmthanks bzoltan. Now that I have you here... :) Not sure if you saw the question earlier, but is it possible to hook up an Ubuntu tutorial so that it appears in the "Tutorials" tab in Qt Creator?17:23
mihirgventuri: I guess as per our discussion we have something new in day view and bottom edge , if i am not mistaken.17:25
popeynik90: joke ☻17:25
gventurimihir: we can talk about it on Monday if you want17:25
gventurimihir: on hangout17:25
gventurimihir: actually, we should17:26
nik90popey: lol, sry my brain is a bit fried atm to get it :P17:26
mihirgventuri: excellent :) we can do that defiantly, but need to check kunal's availablity17:26
gventurimihir: yes17:26
mihirpopey: could you help us ?17:26
gventuriand popey of course17:26
popeyof course17:26
popeyjust tell us when17:26
mzanettirpadovani: pong17:27
popeymihir: gventuri how about 2pm UK / UTC?17:27
mihirpopey: you take call , either we can take this in UOS and gventuri can also join , i am not sure whether we can do that discussion in UOS or not.17:27
bzoltandpm:  yes, it is possible17:28
rpadovanimzanetti, I haven't installed urlHandler yet on the phone, do you have a log about the fail in creation of the new note?17:28
* dpm files some bugs17:28
dpmthanks bzoltan17:28
mzanettirpadovani: no, didn't really investigate17:29
mzanettirpadovani: is there a problem with installing url-handler?17:29
rpadovanimzanetti, don't think so, didn't have time yet17:30
rpadovanilast week I focused on other things17:30
mihirgventuri: small question , the bottom edge will be for creating new event correct?17:30
mzanettirpadovani: want me to debug it?17:31
rpadovanimzanetti, thanks, but no worries, I do it :-)17:32
mihirpopey: the good thing is we're very close to design :D17:32
mihirpopey: btw , did you try to push latest version to Store?17:37
popeymihir: no, because https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/DesignFixDefaultWeekView-5/+merge/240867 didnt land yet17:37
popeyi asked balloons to take a look.17:37
nik90balloons, popey: you are green. clock app rev 165. thn17:37
nik90thnx*17:37
mihirpopey: that's not big MP..we can release it.17:38
balloonsmihir, I'm going to propose an mp in a few to workaround the test setup issues (that cause the tests to fail in jenkins)17:39
popeyballoons: have kicked a build of clock 164 in jenkins17:39
mihirballoons: ahhh , great :) thanks.17:39
mihirballoons: there were some failures of not finding coordinates , but it just passed after re-run.17:40
popeycalendar also building17:40
mihirballoons: i am not sure :)17:40
* mihir hugs popey :D17:40
mihirpopey: did you miss my message , regarding hangout discussion ?17:40
popeymihir: i scheduled a meeting - you should have an invite...17:40
mihirahh , i was asking , whether we can discuss this in our UOS session or it is not advisable during that session.17:41
popeyyeah, we can discuss there too17:41
popeybe good to get the discussion with gventuri asap - so scheduled on monday17:42
nik90popey: hope you meant 16517:42
justcarakasI have a QML question: is there a way to something like destinct in the query of a XmlListModel17:42
nik90Otherwise we might not have location feature17:42
mihirpopey: no issues :) that's fine17:42
popeynik90: i did17:42
nik90Cool17:42
popey164 already built17:42
mihirpopey: just sent out email as well , as it doesn't send email invitation so we all are on same page.17:43
mihirsend*17:43
* nik90 goes to prep dinnet17:43
* nik90 meant dinner17:43
popeymihir: i think your email is broken ☻17:45
popeyam I on your spam list or something ☻17:45
mihirpopey: no i don't know whether i have done some setting by mistake in calendar or not , but i never get an email when i have new event onmy calendar.17:46
popeyballoons: calendar 534 failed.17:46
popeyERROR: Failed to clean the workspace17:47
popeyfginther: http://s-jenkins.ubuntu-ci:8080/job/calendar-app-click/lastFailedBuild/console  any chance someone can take a look?17:47
balloonspopey, oO17:51
popeyballoons: http://s-jenkins.ubuntu-ci:8080/job/clock-app-click/lastSuccessfulBuild/artifact/out/com.ubuntu.clock_3.2.165_armhf.click  passed...17:52
balloonspopey, probably worth a quick retry17:54
balloonsI kicked one17:55
popeyok17:55
popeylets get clock in the store before nik90 finishes his dinner! :D17:55
balloonsrenatu, ping. I have a question about eds17:55
renatuballoons, hi17:57
balloonsmihir, I noticed something odd about new event btw.. if you have a sec to confirm17:57
mihirballoons: sure if i can confirm.17:57
balloonsrenatu, so the issue with having a clean slate in which to test calendar is rearing it's head again. Is there a way (on the desktop) to delete all the events so calendar will load fresh?.17:58
balloonsI don't need to worry about the device for the moment, as we get a clean env during dashboard runs because it's freshly flashed17:59
balloonsmihir, if you start the app and select new event, the default times are rightnow. If you go into any view first and create a new event, the default time is 12:3017:59
renatuballoons, this is very trick, you can take a look on my CMakeFile used for qtorganizer test18:00
renatuits run eds in a temporary directory18:00
renatufor tests18:00
mihirballoons: i noticed the same thing , but thought it is somewhat my timezone/machine issue.18:00
balloonsrenatu, that sounds like a nice workaround. lp branch?18:00
renatuballoons, http://bazaar.launchpad.net/~phablet-team/qtorganizer5-eds/trunk/files18:01
renatuhttp://bazaar.launchpad.net/~phablet-team/qtorganizer5-eds/trunk/view/head:/tests/unittest/CMakeLists.txt18:01
renatuhttp://bazaar.launchpad.net/~phablet-team/qtorganizer5-eds/trunk/view/head:/tests/unittest/run-eds-test.sh18:01
mihirballoons: i'll confirm this and try to fix this.18:02
balloonsmihir, awesome thanks18:02
balloonsrenatu, just fyi there's work going on to avoid the need to do any of this: https://blueprints.launchpad.net/ubuntu/+spec/qa-v-testability18:03
dpmjdstrand, when you've got a minute, could you have a look at bug 1390570 ?18:04
ubot5bug 1390570 in Canonical Click Reviewers tools "Errors and warnings on Reminders app" [Undecided,New] https://launchpad.net/bugs/139057018:04
renatuballoons, how this will be applied for developers that run the tests on thy machines?18:05
renatuballoons, for example with my CMake I just run: ctest -V, and got all tests running18:05
balloonsrenatu, the idea is to launch everything under a quasi new temporary user18:05
renatuballoons, ok but this does not fit for developers18:06
balloonsrenatu, for your unit tests I don't think anything would need to change18:06
renatusince they could not have root access to the machines18:06
balloonsrenatu, right, this is more geared towards acceptance tests18:06
balloonsbut hopefully the desktop and device can be taken care of in the same way18:07
renatuballoons, yes this will be nice18:07
balloonsrenatu, so where do the evolution* args come from in that script?18:10
renatuballoons, from the CMAKE18:10
balloonsI see stuff in cmake, I guess I'm missing it18:10
renatuballoons,18:10
renatu  add_test(${TESTNAME}18:10
renatu             ${CMAKE_CURRENT_SOURCE_DIR}/run-eds-test.sh ${DBUS_RUNNER} ${CMAKE_CURRENT_BINARY_DIR}/${TESTNAME} ${TESTNAME}18:10
renatu             ${EVOLUTION_CALENDAR_FACTORY} ${EVOLUTION_SOURCE_SERVICE_NAME}18:10
renatu             ${EVOLUTION_SOURCE_REGISTRY}18:10
renatu             ${GVFSD})18:10
jdstranddpm: done18:15
dpmjdstrand, thanks! So for the errors, I get it's still being discussed. But for the warnings, was the other bug you linked not about _removing_ the checks? I.e. there shouldn't be any warings in the output, do I get this right?18:18
jdstranddpm: mardy asked me to add the checks and warn if apps provided the id18:18
jdstranddpm: let me get you the commit18:18
jdstranddpm: context: before the checks were requiring that the id be present18:19
jdstranddpm: mardy asked me to stop requiring the id and to warn if apps provided it, since they shouldn't normally18:20
jdstranddpm: so the check was semi-inverted for the bug fix18:20
dpmjdstrand, weird, in the app we added the files in the way mardy had suggested. I'll clarify with him, thanks!18:20
jdstranddpm: well, these things happened at different times18:21
jdstranddpm: iirc, checks were added to require the id. then you get the error, so fixed reminders to work with that test. then mardy said to not require the id, but to warn instead so the checks were updated18:22
jdstranddpm: and now you have this new warning18:22
jdstranddpm: yes, please point him at that bug and have him tell me if something needs to change18:22
dpmjdstrand, ack. I'll ask mardy about it, thank you18:22
dpmsent e-mail18:25
balloonspopey, seems the second build failed too18:25
popeyclock worked though ㋛18:25
balloonswhat do you mean?18:26
mihirpopey: pushed ?18:27
popeyballoons: clock r165 built in jenkins, can you upload it please? http://s-jenkins.ubuntu-ci:8080/job/clock-app-click/lastSuccessfulBuild/artifact/out/com.ubuntu.clock_3.2.165_armhf.click18:28
popey(is what I meant)18:28
popeymihir: no, it is failing in jenkins18:28
* balloons blindly clicks popey's link18:29
* popey pinged ci18:29
popeygoo dman18:29
popeystupid keyboard18:29
ogra_balloons, because it says .click in the end, right ?18:31
balloonsogra_, yes indeed. I always click clicks18:32
ogra_:D18:32
dpmmzanetti, what was the actual bug for https://code.launchpad.net/~mzanetti/reminders-app/gif-support/+merge/240842 ? That gif images were not cached?18:36
mzanettidpm: no, they weren't showing up in the notes list (the small preview in the delegates)18:36
mzanettidpm: because downscaling failed18:37
dpmmzanetti, they seem to show up for me. Let me send you a screenshot...18:37
dpmargh, the branch I'm testing changed the ordering, can't find the note with the gif file anymore18:41
mzanetti:D18:41
mzanettiI guess searching for gif would work18:41
mzanettias it has the filename in the title18:41
mzanettialthough I guess in those 100MB of text the string "gif" will show up quite often too18:42
rpadovanidpm, do you have evernote on android?18:43
dpmrpadovani, yeah18:43
rpadovanidpm, do you like how they manage to add tags to a note? It's a popup, I find it very cool18:43
dpmmzanetti, yeah, did that, but apparently 'gif' is very common in the about 300 notes of project gutenberg files we've got :)18:43
dpmI'll just create a new note18:44
dpmrpadovani, I've not tried it yet, I'll have a look18:47
rpadovanidpm, I think is the best solution, because are listed already existing tags and there is a text area to add new ones18:48
dpmrpadovani, ack, cool18:50
dpmrpadovani, mzanetti, argh, I cannot test gifs, as it seems we don't have any (I could have sworn I saw one) and we only have 16.3 KB left of our upload quota in the evernote300 account this month18:51
rpadovanidpm, ok, tomorrow I'll focus on it, it's the only thing the tagsUi branch misses: lot of work, hope to finish it for sunday18:51
* dpm looks for a 15KB gif...18:51
rpadovanidpm http://www.gifanimategratis.eu/img/cibo/frutta/frutta18.gif18:52
dpm:-)18:53
popeyballoons: http://s-jenkins.ubuntu-ci:8080/job/calendar-app-click/lastSuccessfulBuild/artifact/out/com.ubuntu.calendar_0.4.538_all.click  success! Can you upload pls?18:55
mzanettidpm: I'm sure we have some gifs in there18:55
mzanetti218:55
balloonspopey, it should be sitting ohh.. you mean calendar :-)18:58
popeyETOOMANYAPPS18:58
dpmmzanetti, not sure if it's just my perception, or a real issue: https://code.launchpad.net/~mzanetti/reminders-app/load-chunks/+merge/24087718:58
mzanettifrutta ftw18:59
dpmindeed :)18:59
balloonspopey, pushed19:00
popey\o/ thanks19:00
mzanettidpm: well... the thing is we load all 350 notes now instead of only 250 of them :D19:00
mzanettiwhich pulls in a lot image notes19:00
mzanettifor some reason it was always the same ones that got lost19:01
dpmmzanetti, ah, I see. But why is the UI blocking that much?19:01
mzanettiso now you have all those huge pictures on top...19:01
mzanettibecuase every time you scroll a delegate into view it downloads the 5MB image and scales it down into the cache19:01
mzanettiit will recover and get better once you have all images cached19:02
dpmoh, so it's the file system acces19:02
mzanettihowever, I do have improvements upcoming for this19:02
dpmmzanetti, ah, awesome. I think that'd be a big win19:02
mzanettiyep19:02
mzanettidpm: would you be ok with me doing a translation update only commit when those branches are merged? otherwise they will conflict a lot19:04
dpmmzanetti, ah, yeah, sure19:04
sethjdpm, do you have a minute? This is probably slightly off-topic (was there a better place to ask?), but there is an on-going discussion here http://askubuntu.com/questions/376633/why-cant-i-find-any-tutorial-for-unity-scope-and-python-for-ubuntu-13-10/376895#376895 about whether scopes were ported to C or C++ and whether you were generalizing when you said C here http://askubuntu.com/questions/310669/what-programming-languages-are19:05
sethj-scopes-written-in/310670#31067019:05
dpmhi sethj, sure, here is a good place to ask19:06
sethjhm, last link appears to have been truncated: http://askubuntu.com/questions/310669/what-programming-languages-are-scopes-written-in/310670#31067019:06
sethjdpm, ok cool :)19:07
mzanettidpm: you might want to mention Qt there too, no?19:07
popeynik90: mihir clock and calendar approved into the store. Thanks so much guys!19:08
dpmmzanetti, indeed, the answer is simply outdated19:08
balloonsmihir, still playing with evolution, but I have a version 3.0 of the plan now.. third times the charm19:08
dpmsethj, so I've added a comment to the first link, and I'll update the answer on the second19:11
sethjdpm, thanks much!19:11
sethjwe're trying to keep all the information accurate and up to date, hence the discussion :)19:12
akiva-thinkpad!ping19:17
ubot5pong!19:17
dpmsethj, let me know if that helps: http://askubuntu.com/a/310670/978119:20
sethjdpm, that's great, thank you!19:22
sethjI really hope that JS support gets added myself :)19:22
dpmyeah, you're not the only one :)19:24
dpmmzanetti, one thing that seems to block the UI in particular is opening a note and going back to the list of notes. IIRC we do a refresh when we come back from a single note view to the list of notes, but it seems we're then pulling more notes than the ones that are in view? I.e. I often am in a state where all the images I've got in view have been downloaded. Then the app is responsive. Then I tap on a note and load it, all good. Then I tap on the19:31
dpm"Back" icon and the app freezes for a while - it seems it's downloading more stuff19:31
mzanettidpm: yep... consider it fixed :D19:36
dpm\o/19:36
dpmmzanetti, I'll file a bug in any case for my own sanity. I'm testing so many branches right now that bugs and features are starting to be a big blur :)19:37
mzanettigive me the number so I can link it :D19:38
mzanettiproposing the merge as we atm19:38
mzanetti-as we19:38
dpmmzanetti, bug 139059119:39
ubot5bug 1390591 in Ubuntu Reminders app "App blocking when coming back from the single note view" [Undecided,New] https://launchpad.net/bugs/139059119:39
mzanettidpm: https://code.launchpad.net/~mzanetti/reminders-app/caching/+merge/24113619:40
* dpm high-fives mzanetti19:41
mzanettidpm: there's one issue though... with accounts like the evernote300 account you'll run into quota issues on the first sync19:41
dpmmzanetti, can you clarify the quota issue on first sync? I'm afraid the evernote300 type of accounts are more the rule than the exception19:43
mzanettidpm: well, it'll download the list19:44
mzanettiyou can start using it19:44
mzanettiin the background it starts syncing all notes19:44
mzanettiat some point you'll run into the rate limit19:44
mzanettithen you wait an hour and then you'll never run into the rate limit again (unless you close the app and add another 300notes and then start the app)19:45
mzanettiI can try to add some waiting between the background sync calls19:45
mzanettior I could never automatically sync but only download each note when requested19:46
mzanettiwhich causes slightly worse performance because it'll fetch and downscale those huge images when you want to see them19:47
dpmyeah, I think we need to think of a way to not hit the rate limit, as it could be a potentially bad user experience if the user has to wait for 1h to use the app, even if it's a one-off19:48
mzanettiI agree19:52
dpmmzanetti, you might want to revert the changes on the .pot file (I assume it's the one creating conflicts) or merge trunk on https://code.launchpad.net/~mzanetti/reminders-app/only-resources/+merge/239605 - it failed autolanding due to conflicts19:54
mzanettiyeah... I just merged it... lets see how it is going19:54
mzanettidpm: ^19:55
dpmok, cool19:55
dpmmzanetti, could you give me some more context on https://code.launchpad.net/~mzanetti/reminders-app/improve-sorting/+merge/240848 ? I'm not sure I can follow the description19:56
mzanettiso right now you probably have noticed that the notes are jumping up and down while they load19:59
mzanettithat's because they all have the same timestamps19:59
dpmah, yeah19:59
mzanettithis commit falls back on another field title when the primary sort role is the same19:59
mzanettiI think the default is to compare "left < right"20:00
mzanettithen it sorts it and because left and right swaps, also the result of this evaluation swaps20:00
mzanettidpm: makes sense?20:03
* dpm is still digesting it20:05
mzanettidpm: so it compares "item1 < item2" on the timestamp. but the timestamp is the same20:08
dpmmzanetti, I think I'm simply missing some context about how the API works and about the structures in the code, so I'm not sure I follow the details. But I think I get the big picture: essentially, if the timestamp is the same, then we compare and sort on title, correct?20:08
mzanettiso no matter if you do "item1 < item2" or "item2 < item1", it'll never be true20:09
mzanettiwhich makes it swap around over and over again20:09
mzanettiso I've overridden the sorting function to add a check for == and sort on title instead of timestamp in that case20:10
mzanettiyes... exactly... what you said is the result :)20:11
dpmok, cool :)20:11
mzanettiI tried to explain why that weird jumping happened20:11
dpmoh, discovered a new feature on the phone: switch off wifi, set flight mode, switch wifi. There you go, wifi working in plane mode :)20:12
mzanettidpm: yeah. it's the same on all phones. except on nokia phones you additionally get a question if you're sure you want to enable wifi|bluetooth in flight mode20:13
dpmoh, I never knew that!20:13
mzanettihad quite a discussion about this at the sprint :D20:13
mzanettiI've reapproved the branch that just failed for merge conflicts20:16
mzanettidpm: for the connection errors I plan some better looking way to display them... just haven't really made up my mind yet how20:17
mzanettiI imagine something sliding in from the top (below the header) to make it more visible but less intrusive20:18
mzanettihiding again after a timeout20:18
dpmmzanetti, ah, ok, I had already added a comment on that regard. I had no such a nice idea as yours, just mentioning that it'd be nice to improve it :)20:19
mzanetti+120:19
dpmmzanetti, probably not related to this branch, but I never got the app to recover from network disconnect: https://code.launchpad.net/~mzanetti/reminders-app/connection-errors/+merge/24093720:19
mzanettibut atm getting it usable with the evernote300 account is higher priority20:19
dpmyeah20:19
mzanettidpm: yeah... was thinking about whether to show something like "transport connection" or not20:21
mzanettiI think all of them say "Error connecting to evernote" and then some additional stuff20:21
mzanettiI hoped that the first part would be understandable enough for everyone. but then they report bugs that just say "Doesn't work" if we don't give a little more information20:22
mzanettiso I added the second part20:22
mzanettibut not sure if its really good... can drop it I guess20:22
dpmmzanetti, I think we can spend some time reviewing and making them more user friendly, but for now at least we're showing them to users. But that particular one: will it always be shown when there is no network? Could we say something along the lines of: "Could not connect to Evernote. Is your phone connected to the Internet?"20:25
mzanettidpm: yep... also when you hit the rate limit the api gives us "Generic TException"20:27
mzanettibecause of that I wanted to collect a bit more info when they happen before placing more generic strings there20:27
dpmok, gotcha20:27
mzanettibut I'm totally aware of this issue and agree with you20:28
mzanettihave to improve a lot in this area20:28
dpmmzanetti, do you want to fix the "transport connection" message on this branch or on a separate one? Otherwise, I'm happy to approve if you can comment on the connection recovery issue20:31
mzanettilet me see what exactly happens with that activityindicator20:32
dpmok20:33
mzanettiI guess I can fix that easily20:33
mzanettinot adding proper roaming support here atm20:34
mzanettibut I can make the spinner stop20:34
dpmmzanetti, I think for this branch it'd be enough to make the spinner stop. But is it possible at all to recover the connection without having to restart the app? Or to use an API to check connectivity before hitting the Evernote servers (or the net at all)?20:35
mzanettidpm: right now I don't20:36
mzanettibut would be something for the todo list20:36
mzanettiI guess that comes with offline mode and sync support20:36
mzanettiatm you can't do much withoug connection20:36
dpmmzanetti, ok, in that case, in addition to stopping the spinner, we should probably say in that message "Please restart the app" or something like that20:37
dpmAnd I think with that, these were all the branches I could review today20:38
karniYo guys, I need to use CMAKE_SOURCE_DIR, but click-buddy seems to break it/hide it, any pointers?20:39
=== jhodapp is now known as jhodapp|sick
karnifound it.20:45
mzanettidpm: pushed20:46
mzanettidpm: thanks a lot20:46
dpmmzanetti, well, thank _you_!20:47
mzanettidpm: I'll try to improve the caching part a bit...20:47
mzanettiprevent it from hitting the rate limit20:47
mzanettidpm: one small question:20:47
mzanettihow important would you say is it that notes immediately fetch the preview?20:47
mzanettilike if there are 300 notes and you connect for the first time20:48
mzanettiwould it be an issue if the previews only show up when you a) open the note or b) wait for like 10 minutes20:48
mzanettiah... never mind... I think I can fix it...20:48
mzanettijust explaining an issue sometimes helps20:49
dpmmzanetti, ok, but to answer the question, I think it's important to fetch the preview soon, at least for the notes that are in view20:49
mzanettidpm: only speaking about the first sync here20:49
dpmas in soon being < 1 min as opposed to 10 mins20:49
mzanettionce that is done all previews and images are instant right now20:50
mzanettiwith the caching branch that is20:50
mzanettithe problem with that is however, if the user opens the full list and does a quick flick through all 30020:50
mzanettihe'll schedule 300 high priority jobs20:50
mzanettiwhich then work while he's using the ui20:51
dpmmzanetti, yeah, gotcha20:51
balloonsmihir, I assume you are off, but the mp is finally ready.20:58
balloonsajalkane, did you get anywhere on running file manager in 'phone' mode (aka smaller size) on the desktop?21:27
balloonsdid you file bugs?21:27
balloonsakiva-thinkpad, howdy! I was looking at this branch:21:31
balloonshttps://code.launchpad.net/~akiva/sudoku-app/upgrade-ubuntu-components-1.1/+merge/232987. Do you think you could finish it?21:31
akiva-thinkpadballoons, yah sure21:35
balloonsakiva-thinkpad, awesome..21:47
davidcallemzanetti, popey, I'm looking for an evernote scope branch, does that exists somewhere, and is it using the Online Accounts API?22:02
mzanettidavidcalle: yes, it is using OA22:16
mzanettidavidcalle: the plugin can be found in lp:reminders-app22:17
mzanettithe scope itself is quite experimental still22:17
mzanettibut the OA plugin is used by the reminders app too22:17
davidcalleThanks mzanetti, I'm gathering code samples of scopes using OA, I'm starting to get questions I'm unable to answer for the scopes contest :)22:19

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!