16:00 #startmeeting Calculator app meeting 16:00 Meeting started Thu Jan 15 16:00:03 2015 UTC. The chair is popey. Information about MeetBot at http://wiki.ubuntu.com/meetingology. 16:00 16:00 Available commands: action commands idea info link nick 16:00 Hows it going guys? 16:00 Great thanks 16:00 We still waiting for UI design team for approval our calculator design 16:00 Busy but good, thanks :-) 16:01 We proposed some innovation with brackets. 16:01 So, since this week we have gcollura in our team \o/ 16:01 (and I can focus on study because gcollura reviews gang66 branches :P) 16:02 hello everybody :) 16:02 Hah! I hear gcollura is being distracted from his own studies too! 16:02 rpadovani, (I have to study too :P) 16:02 We are terrible people. 16:03 No, you're not :) 16:03 We have one issue: 16:03 Just one? 16:03 this is quite amazing :D 16:04 We reverted BigNumber to Number, due to autopilot failures 16:04 and temporary result not working 16:04 Welcome gcollura 16:04 oh, right, didn't investigate that yet, but will be hard to fix because fails are random 16:05 yes, failure occurs 4 / 5 runs 16:06 In my opinion it is something connected with math.js 16:06 This isn't the same issue we had previously? 16:07 Where we had different results for when on battery or powered? 16:08 popey, I'm scared this is a similar issue 16:08 popey, and this means we will have to debug ~20000 lines of code. Let me finish my exams first, please :D 16:08 (brb in 5 minutes) 16:09 :) 16:09 got a log of the failure? 16:10 Do you have ticket number? 16:10 there is no error 16:10 or failure 16:11 it just gets the answer wrong? 16:11 here I'm 16:11 Just sometimes calculator do not reposne for pressing "=" 16:11 popey, error is math.js var is undefined 16:11 so it cannot use eval function 16:11 that is called by pressing "=" 16:12 but only with bignumber and not simple number 16:12 aaa, yes 16:12 ugh 16:12 but it happens also on pc 16:12 and we're the first to find this? 16:12 well, sometimes 16:13 popey, we modified math.js importer function to have it working in qt - so yes 16:13 mhhh 16:13 are we sure it is related to how we import the library in qml? 16:13 gcollura, nope 16:13 Any way we can simplify this? 16:14 I have no idea on what causes the issue, thanks to gang66 we know there is only with bignumber, and it is random 16:14 I didn't investigate more than this at the moment 16:15 anyway, you can reproducing this changing number in bignumber in math.js (~line 60) and running autopilot 16:15 doesn't happen with qmlscene on desktop 16:15 but happens sometimes on arm 16:16 and also with pc and autopilot 16:16 indeed 16:17 we can do nothing bug lot of debug - maybe changing how math.js is included? 16:17 other thing I have no time to do this week is publish the app in the store - I didn't understand why, when I create click package it doesn't work 16:18 if some of you guys want to try to do it, feel free 16:18 which branch did you try to create a click package for? 16:18 I'll try here 16:18 what happen when you run click package 16:18 ? 16:20 popey, I tried from reboot itself 16:20 gang66, one time I had the same problem we have with autopilot: = doesn't work. Other times the app doesn't start at all (and no log) 16:21 ok, let me try 16:21 rpadovani, pkg generation with click-buddy works for me, maybe it was related to the click arch? 16:21 I don't remember the issue you had 16:21 popey gcollura , it works when I use com.ubuntu.calculator as name, but I cannot publish it, I have to use com.ubuntu.developer.rpadovani.calculator 16:21 ah 16:21 yes, you need to modify the manifest etc 16:21 so I think is CMake related or something like this 16:22 let me have a play with it 16:22 I'm sure we can sort that one. 16:22 I think it is related to how the pkgs must be, isn't it? 16:22 The calculations going wrong, I'm less helpful with, sorry. 16:22 I mean on the click store 16:22 yeah, I'll take a look, don't worry. 16:23 popey gcollura this is changes I made: http://paste.ubuntu.com/9756555/ 16:23 *these are 16:24 you have two names 16:24 +project(com.ubuntu.developer.rpadovani.calculator) 16:24 + applicationName: "com.ubuntu.developer.rpadovani.calculator.reboot"; 16:24 that has reboot in it 16:24 /o\ 16:24 they should be the same? 16:24 I can actually picture you doing /o\ now :) 16:24 actually, I don't know, but could worth a try 16:24 :D 16:24 yeah. 16:25 I'd remove '.reboot' 16:25 and just call it com.ubuntu.developer.rpadovani.calculator 16:26 rpadovani, I thought a little more about the timer thing btw, the way it is now, it's the correct way 16:28 gcollura, of course it is, I approved it :P 16:28 ahah 16:30 now click starts but = button doesn't work, it is anyway an improvement 16:30 I'll take a look 16:30 ok 16:30 thanks popey 16:30 np 16:31 Ok. So any ETA on when we can expect a version in the store? :) 16:31 I'd like to get some wider testing 16:31 there's another bug btw, when the app starts the view is scrolled few pixels down and the keyboard isn't completely shown 16:32 I've not seen that... 16:32 this happens only on my phone (mako) it doesn't happen on desktop 16:32 ok, i have just made a click from trunk and installed on krillin 16:33 it may be related that now listitems are loaded asynchronously 16:33 What is "krillin" ? 16:33 uh 16:33 bq phone :) 16:34 you need to have several results entries 16:34 yes 16:34 http://people.canonical.com/~alan/screenshots/device-2015-01-15-163349.png 16:34 i see that 16:35 yep 16:36 file:///usr/lib/arm-linux-gnueabihf/qt5/qml/Ubuntu/Components/TextField.qml:979:9: QML TextInput: Binding loop detected for property "" 16:36 I created a working click \o/ (first start is so slooooooow) 16:36 seeing that a lot too 16:36 popey, yes, it's due we manage cursor 16:36 http://people.canonical.com/~alan/screenshots/device-2015-01-15-163552.png 16:37 tricky to see results on krillin ㋛ 16:37 we may need a smaller font? 16:37 Technical explanation on popey's bug here -> https://bugs.launchpad.net/ubuntu-calculator-app/+bug/1400899 16:37 ok 16:38 popey, gcollura try it please http://people.ubuntu.com/~rpadovani/com.ubuntu.developer.rpadovani.calculator_0.1.4_all.click 16:38 if it works, I publish it :-) 16:39 We need also translations 16:39 well, it should, i just made the same click :) 16:39 we do. 16:39 https://code.launchpad.net/~gang65/ubuntu-calculator-app/reboot_fix_translation_generation 16:40 rpadovani: publish and I'll test from the store :) - make sure you add "reboot" to the description. 16:41 rpadovani, it works! :) 16:43 Do we have a bug tracking the calculation error issue? 16:45 and here we are, app is on the store, congrats to all :-) 16:45 popey, nope 16:45 Testing time! 16:45 Thanks rpadovani 16:45 Ok, anything else? 16:45 \o/ 16:46 it's a good calculator, I use it, advanced mode is so useful! 16:46 I'll file some bugs. :) 16:46 \o/ 16:46 Ok, lets wrap up then. Thanks all! 16:47 #endmeeting