/srv/irclogs.ubuntu.com/2014/09/16/#ubuntu-app-devel.txt

=== Laney is now known as Guest45203
=== _salem is now known as salem_
=== salem_ is now known as _salem
=== chihchun_afk is now known as chihchun
Akiva-Thinkpadhey all05:29
shyamskAkiva-Thinkpad: hey there05:31
Akiva-Thinkpadshyamsk, hey whats up!?05:31
shyamsknothing much reading up about bazaar. @ the office right now.05:32
shyamskwas here yesterday... you might've been offline then05:32
Akiva-Thinkpadshyamsk, definitely a possibility05:34
Akiva-ThinkpadI don't have internet, so I am only connected when I'm at the library or coffee shop05:34
shyamskAkiva-Thinkpad: oh... so what branches would I have to test. I had branched music-app yesterday. But bazaar says something about series.06:47
Akiva-Thinkpadshyamsk, good question06:47
shyamskso I presume that there is like a dev branch?06:48
shyamsk*dev series06:48
Akiva-Thinkpadshyamsk, yep; from bazaar, that is usually dev06:48
Akiva-Thinkpadthe core apps page links to all the currently developing versions06:48
Akiva-Thinkpadso for example06:48
Akiva-Thinkpadlp:ubuntu-calendar-app is the dev version06:49
Akiva-Thinkpadshyamsk, do you have a device?06:49
Akiva-Thinkpador are you running it on the desktop?06:49
shyamskso I would be testing on that(dev series). Got it. Also is there documentation for how to file a bug. So that I can reference that while you're offline06:50
shyamskdesktop.06:50
dholbachgood morning06:51
Akiva-Thinkpadshyamsk, hummmm06:53
Akiva-Thinkpadshyamsk, do you have team viewer?06:53
Akiva-ThinkpadI can show you how I file a bug06:53
shyamskAkiva-Thinkpad: hmm... the music app doesn't seem to have a dev series. Atleast its not visible on graph shown.06:54
Akiva-Thinkpadit shouldnt06:54
shyamsk???06:54
Akiva-Thinkpadthink of it this way06:54
Akiva-Thinkpadthe debian package is the release.06:54
Akiva-Thinkpadthe source code which you branched is the development version06:55
Akiva-Thinkpadbranched with bazaar*06:55
Akiva-Thinkpadshyamsk, as to filing bugs,06:55
shyamskya I've gt team viewer06:56
Akiva-Thinkpadshyamsk, okay cool; i'll let you connect to my machine06:56
Akiva-Thinkpadsec06:56
Akiva-Thinkpadshyamsk, 648 859 68206:57
Akiva-Thinkpadpw 666606:57
Akiva-Thinkpadah cool beans so this is how I file a bug06:58
Akiva-Thinkpadhmmmm I found a bug!07:01
Akiva-ThinkpadMake sense?07:06
shyamskyup got it. Thanks :D07:06
Akiva-Thinkpadshyamsk, so in cases of app development; that was more of a suggestion07:06
Akiva-Thinkpadits not really neccessary to state, "Running 14.04, running version 5.0.2.3 revision 13, here is my log"07:07
Akiva-Thinkpadbecause its already assumed you are using the latest branch07:07
Akiva-Thinkpaddoing a screenshot of the application makes it more visual, and easy to digest.07:08
shyamskyup.07:08
Akiva-Thinkpadshyamsk, in other cases though, well Linus Torvalds put it really well07:08
Akiva-ThinkpadLinus stated he is not a networking guy, and that sort of stuff just does not jive with him. So when it comes to dealing with network issues07:09
Akiva-Thinkpadhe doesnt really have any actual suggestions07:09
Akiva-Thinkpadinstead what he tries to do is just file a really really good quality bug report.07:10
shyamskLets say I wanna tackle bugs, you know fix them. Would the work flow be similar to filing bugs? except I would may be make a comment on the existing bug ticket, right?07:10
Akiva-Thinkpadso use your best discretion, and THANK YOU!!!!07:10
Akiva-Thinkpadshyamsk, well, do you want to do teamviewer again?07:10
shyamskwell bug reports with possible visual hints are always helpful :)07:11
Akiva-Thinkpadshyamsk, yessum!07:11
Akiva-Thinkpadshyamsk, to your question though07:11
shyamskno. its not necessary. just asking. I'll figure it out on my own.07:11
Akiva-Thinkpadwhat usually happens is that you take the branch you got07:11
Akiva-Thinkpadedit it07:11
Akiva-Thinkpadand when you are ready to push it, you will have to write a commit message07:12
Akiva-Thinkpadthe commit message usually suffices.07:12
Akiva-Thinkpadwhen the branch makes it onto launchpad, then from there, you will link your branch to the bug report07:13
Akiva-Thinkpadand then you will propose a merge. In the merge you can write a description of the change07:13
shyamskright. so similar workflow to Git... I can handle that07:13
mihirAkiva-Thinkpad: can you resolve the conflits in your banch ?07:14
Akiva-Thinkpadand from there, someone will review it, and either approve, reject, or offer suggestions to how to fix it.07:14
Akiva-Thinkpadmihir, working on it07:14
Akiva-Thinkpadsorry by the way!!!07:14
Akiva-ThinkpadI was doing it yesterday07:14
Akiva-Thinkpadand the bloody internet kafunked in the middle of the merge07:14
Akiva-Thinkpadand my computer crashed07:14
Akiva-ThinkpadLinux stability, pah!07:14
Akiva-Thinkpadworking on it right nwo07:15
shyamskAkiva-Thinkpad: so take care, catch ya l8r. I will get into this once I reach home.07:16
Akiva-Thinkpadshyamsk, thanks so much !07:16
Akiva-Thinkpadlook forward to your bugs!07:16
mihirnik90: ping07:33
fcole90balloons: ping07:36
mihirelopio_: ping07:53
Akiva-Thinkpad /home/akiva/Programming/build-ubuntu-calendar-app-Desktop-Default/po/CMakeFiles/com.ubuntu.calendar.pot.dir/build.make:138: *** multiple target patterns.  Stop.08:00
Akiva-Thinkpadwhat does that mean?08:00
Akiva-ThinkpadI am trying to run my project so I can resolve08:01
=== Guest45203 is now known as Laney
mihirAkiva-Thinkpad: can you go to po/CmakeList.txt08:20
mihirand build again..?08:20
mihirAkiva-Thinkpad: it is sometime random error08:20
Akiva-Thinkpadmihir, sure08:20
Akiva-Thinkpadgo and do what though; build that cmakelists?08:21
Akiva-Thinkpadhuh this is frustrating08:27
* Akiva-Thinkpad tries to restart08:27
Akiva-Thinkpadno dice08:30
mihirAkiva-Thinkpad: did that work ?08:31
Akiva-Thinkpadnope08:31
Akiva-Thinkpadrestarting didnt work08:31
mihirokay do one thing, go to root CmakeList.txt08:31
mihirand at the botoom , remove add_subDirectory(po)08:31
Akiva-Thinkpadah okay thanks08:31
mihirthis is again temoprary fix, but don't push this08:32
Akiva-Thinkpadokay08:32
mihirzsombi: ping08:33
Akiva-Thinkpadmihir, that worked08:33
mihirAkiva-Thinkpad: :) but don't push this :)08:33
Akiva-Thinkpadi'll undo it08:33
fcole90popey: ping08:41
popeyfcole90: pong08:41
Akiva-Thinkpadmihir, okay i think it was done right08:41
Akiva-Thinkpadmihir, will you get a chance to go through it today?08:41
mihiryou resolved conflicts?08:42
Akiva-Thinkpadlp:~akiva/ubuntu-calendar-app/akivaDayWeekViewMergeMihirNik into lp:~ubuntu-calendar-dev/ubuntu-calendar-app/ViewRedisign08:42
fcole90popey: hello, can you help me with the cmake branch? :)08:42
Akiva-Thinkpadmihir, yah I believe so08:42
popeyfcole90: what's up?08:42
Akiva-Thinkpadmihir, ah just realized I downgraded 2.3 to 2.008:44
fcole90popey: Jenkins doesn't find the tests even switching back to python2, I'm also checking for the folder placement and seems ok, maybe there is a step that moves the folders out of their place :/08:44
* Akiva-Thinkpad should go through all the bloody apps, and upgrade the qtquick libraries08:44
fcole90popey: http://91.189.93.70:8080/job/generic-mediumtests-utopic/2020/console08:44
* popey looks08:44
mihirAkiva-Thinkpad: we'll do it in saperate MR08:44
Akiva-Thinkpadmihir, yah sounds good08:45
popeyfcole90: let me test here, and then find someone to help08:45
fcole90popey: if you look at the "Installing... " part you'll notice that the tests go under their correct placement, but when the test time comes the test is not found under its location :/08:46
fcole90popey: ok, thanks :)08:46
JamesTaitGood morning all; happy Working Parents Day! :-D09:04
fcole90Good morning :D09:04
justCarakasgood morning :)09:04
mihirpopey: ping !!09:20
mihirpopey: nik90 Akiva-Thinkpad , i tried to do this with Day view design, http://i.imgur.com/IUljzHg.png09:21
mihirnow, all day events will be visible only when there are any all day events.09:22
popeyexcellent.09:22
Akiva-Thinkpadmihir, gah09:22
dholbachzbenjamin, setting up (and updating) an emulator and creating a click chroot is done by scripts in q-p-u, right? could I run them through an ssh session where I don't need to fire up the sdk?09:22
popeynice one, dpm ^^ what we discussed yesterday! mihir is a mind-reader09:22
Akiva-Thinkpadmihir, my patch solved that though09:22
popeyoh?09:22
zbenjamindholbach: basically the scripts use the ubuntu-emulator command09:23
Akiva-Thinkpadhere I'll screenshot it09:23
zbenjamindholbach: but there is no way to update a emulator09:23
dholbachzbenjamin, will I need to register the emulator or chroot somehow?09:23
dpmpopey, mihir, nice!09:23
zbenjamindholbach: no, "ubuntu-emulator list" knows all your emulators09:24
zbenjamindholbach: with "click chroot" you can handle your chroots09:24
dholbachzbenjamin, brilliant thanks, I'll look into it09:24
mihirpopey: dpm , now we won't have partial views of next/previous day in Dayview09:25
Akiva-Thinkpadhttp://imgur.com/GCsBfma09:25
mihirAkiva-Thinkpad: i still want something else on all events  bubbule :\09:26
mihirAkiva-Thinkpad: or change the color atleast, this is too eye catchy09:26
Akiva-Thinkpadmihir, what do you mean by the bubble?09:26
nerochiarooSoMoN: still in your queue ? https://code.launchpad.net/~phablet-team/gallery-app/desktop-i18n/+merge/23467909:27
oSoMoNnerochiaro, yes, still in my queue, getting near the top09:27
nerochiarooSoMoN: thanks09:27
mihirAkiva-Thinkpad: so now we can merge your branch here with weekview , http://i.imgur.com/Mg6HO0F.png09:28
mihirit solves both the problem.09:28
mihirpopey: dpm nik90 Weekview , http://i.imgur.com/Mg6HO0F.png09:28
=== ondra| is now known as ondra
Akiva-Thinkpadholy misunderstanding batman09:53
Akiva-Thinkpadanyways looks great mihir09:54
mihirAkiva-Thinkpad: hahah :D09:55
nik90mihir can you show a screenshot with a all day event in the new day and week view?09:59
nik90mihir: hi09:59
nik90;)10:00
mihirnik90: hey, how are you ?10:00
mihirnik90: i haven't change the all day envets view , in a day view it will display that top bar if there are any all day events..10:00
nik90mihir: good..I was working on importing some font color fixes from akiva branch10:00
nik90Into our branch yesterday nite10:00
mihirso all day events we need to merge from Akiva-Thinkpad10:00
Akiva-Thinkpadnik90, yah conflicts were resolved afaik10:01
nik90Akiva-Thinkpad: when did you resolve them? Cause I copied from your branch over yesterday10:02
mihirnik90: he did this morning10:02
nik90Not all but a lot of it10:02
nik90Ok10:02
mihirnik90: it looks same but for your info , http://i.imgur.com/agkfR2I.png10:02
mihirAkiva-Thinkpad: ^^10:02
nik90I will review the code and then merge10:02
mihirnik90: sorry for lil late , i finished day view stuffs and weekview..still need to do small fixs10:03
nik90No worries10:03
mihirnik90: now question on QML side hehe, "\n" is the only way to enter in next line in label ?10:03
nik90Yes10:03
Akiva-Thinkpadnik90, no10:04
Akiva-Thinkpadyou can set the label to html and use <br>10:04
Akiva-ThinkpadI THINK10:04
nik90mihir: or you can do word wrap and then limit the label width to force to the next line (I wouldn't recommend this)10:04
Akiva-Thinkpadha10:05
nik90Akiva-Thinkpad: same methodology ;) HTML tags10:05
Akiva-Thinkpadthe force wrap... sheesh now that is creative :P10:05
mihirnik90: okay...so i would go for two lables in row10:05
mihirnik90: any suggestion on Day view ?10:07
mihirnik90: btw , just make a pull before you start working on it10:07
mihirnik90: i have pushed the code :)10:07
nik90mihir: ok10:09
mihirAkiva-Thinkpad: can you try to change the orange color to something ligher and try how does it looks ?10:09
Akiva-Thinkpadmihir, what about grey gradient?10:09
mihirAkiva-Thinkpad: can you try, i didn't like the orange , nik90 what about you ?10:10
nik90mihir: ? I am a bit out of the loop here10:15
mihirnik90: talking about this http://i.imgur.com/GCsBfma.jpg10:15
mihirall day events10:15
Akiva-Thinkpadnik90,  look, this needs to be merged. You said you already handled it, but this request still says pending https://code.launchpad.net/~akiva/ubuntu-calendar-app/akivaDayWeekViewMergeMihirNik10:16
nik90personally I am fine with it, however if you want we can ask Akiva to use a Stroke button10:16
nik90Akiva-Thinkpad: I know, it had conflicts yesterday so I couldn't merge it10:16
Akiva-Thinkpadnik90, right10:17
Akiva-Thinkpadand now10:17
Akiva-Thinkpadconflicts gone10:17
nik90Akiva-Thinkpad: so what I did was manually copy some code over to the branch10:17
nik90Akiva-Thinkpad: ok10:17
Akiva-ThinkpadAll I want to do, is get this merged, and then any issues can be fixed later10:17
Akiva-Thinkpadbecause having a 500 + line  pending merge request is better to get out of the way first.10:18
Akiva-ThinkpadAny aesthetic issues that is, like button colours.10:18
mihirAkiva-Thinkpad: hehe10:20
Akiva-ThinkpadYawn... Anyways; I'm about to head out10:20
Akiva-Thinkpadits 320 am, and the donut shops chairs are uncomfortable10:20
mihirnik90: i have used in this way to split in a new line , http://paste.ubuntu.com/8356912/10:21
mihirlet me know if any other way..10:21
nik90mihir: that should work but with one small drawback10:21
nik90mihir: you cannot horizontalcenter the label on the bottom line with the one on the above.10:22
nik90so both will be left centered by default10:22
mihiryeah, i see that now :|10:23
nik90mihir: btw nice work :) More vertical space saving10:29
nik90mihir: it looks like https://imgur.com/EsrRAiP now10:29
nik90mihir: why is there a gap on the left of the events?10:29
nik90Akiva-Thinkpad: but I can't merge your branch. I think your resolved in an incorrect way since the code diff shows old code that is being added and new code being removed from the parent branch.10:31
nik90Akiva-Thinkpad: Let me try to manually copy your stuff out of it.10:32
Akiva-ThinkpadJust use your best discretion.10:33
nik90Akiva-Thinkpad: No worries I will take out all the important bits of your work.10:34
Akiva-Thinkpadyep; whatever you need to do. I'm not too concerned.10:34
mzanettinik90: hey, tried to set an alarm yesterday... Seems the time picker in there is broken10:49
nik90mzanetti: why? What happened?10:50
mzanettinik90: I select some hour, e.g. 9, it jumps to 7.10:50
mzanettinik90: when I change the minutes, it changes the hours too10:50
nik90mzanetti: I am using the standed SDK widget for this.10:51
nik90mzanetti: hmm I am checking this on the latest-develproposed and I dont see the issue.10:52
nik90mzanetti: could it be due to a unity8 branch that you may have installed on your phone for testing?10:52
nik90mzanetti: since it feels like the input is being shifted to other neighbouring elements10:53
mzanettinik90: strange... doesn't happen now any more either10:54
mzanettinik90: unlikely that its because of some change here10:54
mzanettinik90: no. input was fine. but when I released it, it changed once more10:55
nik90mzanetti: may be you were sleepy when setting the alarm in the nite :D10:55
mzanettiI'll watch it... lets see if I can repro10:56
nik90mzanetti: probably the timepicker is too sensitive, so when you released your finger you could have accidentally gave it a tiny nudge10:56
mzanettinik90: no... this was definitely some logic issue...10:56
nik90ok10:56
* nik90 checks clock app code to be sure10:57
nik90mzanetti: was this while creating a new alarm or editing an existing one?10:59
nik90mzanetti: in the clock app code, all I do (while creating a new alarm) is instantiate a time picker and then set it to the correct time rounded to the nearest 5. After that its all the SDK.11:00
mzanettinik90: creating a new one11:00
nik90mzanetti: after which the alarm time is saved only when you press the save button. So I don't think this bug is due to a logic mistake in the clock code.11:00
mzanettimight well be in the time picker... strange that it doesn't happen any more11:01
nik90mzanetti: can you perhaps create a sample app with a time picker in it and test if its input is flaky?11:01
nik90mzanetti: I will keep an eye on this11:01
mzanettinik90: yeah... if I see it again I will dig deepter11:01
mzanettideeper11:01
mihirnik90: i handled 24 hours case in so if it is undefined , it shouldn't generate new errors.11:02
nik90mihir: that doesn't work...for me now it only shows "AM" or "PM" with no time.11:02
nik90mihir: do we really need to split it into 2 lines?11:03
mihiri feel that should look great...11:03
nik90ok, let me fix that bug then11:03
mihirnik90: it worked in my case, so i handled exception for 24 hours case11:04
nik90mihir: you misspelled "undefined". So I get TimeLineBackground.qml:39: ReferenceError: undefidend is not defined11:05
nik90let me correct that, 1 sec11:05
mihirahh, my bad :(11:05
* mihir hates typo 11:05
nik90mihir: https://imgur.com/52JoCL111:05
mihirnik90: can you try to remove that condition and just put date[0] + "\n" + date[1]11:06
nik90mihir: then https://imgur.com/G2dyaKI11:07
mihirhmm okay,11:07
mihirif you think one line looks okay , lets go ahead with one line11:07
nik90I think it would remove unnecessary javascript operations like split in the delegate11:09
mihirnik90: okay :)11:09
mihirno issues , lets go ahead with one line :)11:09
mihirnik90: lets get it merged by ToNight :D11:10
* mihir excited for new view designs for calendar :D11:10
nik90mihir: :) we will try our best11:10
nik90mihir: I found a bug in the week view https://imgur.com/OUriA0N11:12
nik90mihir: you see that event "Test"? It is supposed to be on Sun 14. But it is shown shifted to the right.11:12
mihirnik90: ahhh , i couldn't  realized on desktop with wide screen11:13
nik90mihir: I think I know why..11:13
nik90mihir: I think the logic you used to not make event overlap over the dates should only be applied in the day view.11:14
nik90mihir: then week view will work as expect11:14
nik90mihir: let me push first11:14
mihirnik90: nope, this should be related to time change i believe11:14
nik90time change?11:14
mihirnik90: i meant the time label as we moved to left and margin changes we did11:15
nik90mihir: true but that's TimeLineBackground which is in the background.11:15
nik90mihir: the TimeLineBaseComponent is at fault here11:15
mihirHmmmmm yeah got it11:15
mihirnik90: i am planning to put this time in first two column and remove the background and put it front11:16
nik90mihir: ok I pushed my fixes for the time label. You can now pull11:17
mihirnik90: okay , thanks...also i'll run the test on machine11:17
nik90mihir: ok11:18
=== MacSlow is now known as MacSlow|lunch
* mihir really needs to fix his device to connect 11:18
nik90mihir: btw I don't see the all day events in the week view!11:18
nik90mihir: it correctly shows and hides in the day view.11:19
mihirnik90: ohh , i forgot to push that change , in last..it has to just check in AllDayEventComponent , if it is weekview root.visible:true ,11:20
mihirnik90: i will do that too and fix that:)11:20
nik90mihir: :P11:20
mihirnik90: btw , did you merge code of Akiva-Thinkpad  ?11:20
=== _salem is now known as salem_
nik90mihir: I cant..I checked it few minutes back and the conflict resolution went wrong11:21
nik90mihir: the branch has code that I removed (for sure) and is a bit old11:21
nik90mihir: I will have to manually copy stuff over11:21
mihirnik90: okay , because it has large changes...11:22
nik90mihir: actually not much ... the code diff look big because of the mistake I mentioned above11:22
mihirnik90: okay, i'll try once i reach home from work :)11:22
nik90mihir: for instance I removed the duplicate header actions code, but it is there in his branch11:22
mihirnik90: hmm , i see11:23
nik90mihir: so you will push the week views fixes when you get home?11:23
mihirnik90: sure i'll do that :)11:25
mivoligomzanetti: ping11:41
mzanettihi mivoligo11:41
mivoligomzanetti: I'm about to push enemies in svg11:42
mivoligoshould I go ahead and edit enemies.json as well?11:42
mivoligomzanetti: I mean remove .png extensions11:43
mzanettimivoligo: sure, yes11:43
=== MacSlow|lunch is now known as MacSlow
mivoligomzanetti: also I reduced spriteCount for Boing family, I didn't tested it though11:44
mzanettimivoligo: ok. should be fine I guess11:44
mzanettihope you're ok with the names :D11:44
mzanettiI just came up with random stuff11:44
mivoligomzanetti: they're great :D11:45
mzanettifeel free to rename enemies/towers if you have better ideas11:45
mivoligomzanetti: other thing: I have found sounds for some towers11:46
mzanettiawesome :D11:46
mivoligomzanetti: what format should they be and where to put them?11:47
mzanettiI'd say put them into the towers directory11:47
mzanettiwhat format... good question11:47
mzanettiI guess .ogg would be appropriate :D11:47
mzanettibut not sure what the CPU thinks of decoding multiple of them simultaneously11:48
mzanettimaybe we have to go with .wav11:48
=== chihchun is now known as chihchun_afk
mzanettimivoligo: what are they currently?11:48
mivoligomzanetti: mp311:48
mzanettilet me ask some people that did games before11:49
mivoligook11:49
mzanettimivoligo: hah. know what:11:54
mzanettimivoligo: add them as mp3 or ogg.11:54
mzanettimivoligo: if it turns out to be a problem, I'll add a decoding step to wav in the setupdata script11:54
mivoligomzanetti: ok11:55
ogra_so i asked that before ... but will ask again :) ...11:59
brendandnik90, why are clock tests skipped?11:59
ogra_if i go backwards in a page stack, is there any signal i can listen for to force a page refresh ?12:00
nik90brendand: ?12:00
mivoligomzanetti: also I think for some towers the sound should only appear when they shoot (like Cannon, Radioactive Cloud) and for some (like Electro) should be constant. What do you think?12:00
brendandnik90, http://paste.ubuntu.com/8357462/12:00
nik90brendand: ah that..they are skipped only on the device because datepicker doesn't work on the phone due to upstream autopilot issue12:01
nik90brendand: let me grab the bug for your12:01
nik90you*12:01
brendandnik90, grumble grumble :P12:02
nik90brendand: I believe balloons is working on a fix upstream in AP12:02
nik90brendand: https://code.launchpad.net/~nskaggs/autopilot/fix-1328600/+merge/22739912:02
mzanettimivoligo: isn't that the case already?12:03
mivoligomzanetti: don't know tbh12:03
mzanettimivoligo: so yeah... shot sounds are only played when the tower is shooting12:04
mzanettimivoligo: think we need other sounds too?12:04
mivoligomzanetti: I was thinking about some „buzzing” sound for example for Electro12:05
mzanettimhm... I guess we can add that too12:06
mzanettinot entirely sure yet how this will turn out12:06
mzanettimivoligo: do you think we'd be able to get some good music too? I figure most games come with some sound12:07
mivoligomzanetti: I hope we can do that but I have no experience in it12:08
mzanettime neither12:09
mivoligomzanetti: my friends made couple games for android, I'll ask them about it12:10
mzanettiah ok, cool12:10
mivoligomzanetti: ok, I've pushed these pesky enemies ;)12:11
mzanettimivoligo: great! will merge soonish12:13
mzanettimivoligo: I guess we can ask some people to test drive the first 5 levels then :)12:13
mivoligomzanetti: sure :)12:14
mivoligomzanetti: I hope I finish the next five today12:14
mzanettimivoligo: ah right... reminds me... not sure if you've seen the question yesterday still: is 30 levels ok for you or is that too much to ask?12:15
mivoligomzanetti: yes, I've seen it, sorry I could not answer12:16
mzanettino problem12:16
mivoligomzanetti: I have ideas for next 10 for now12:16
mzanettithat would make 20 then12:16
mivoligomzanetti: but I think I can make 3012:16
mzanettiawesome... I have already made a plan on how to distribute the enemies and towers across them12:17
mivoligoI've seen the concept.txt ;)12:17
Randy_Oany U-SDK emulators pro's around?12:27
=== salem_ is now known as _salem
nik90Randy_O: not a pro, but how can I help?13:04
nik90zbenjamin: hey, I noticed that when I run apps on the emulator for like 15-20 times, I then get an error http://paste.ubuntu.com/8357783/. If I close the emulator and reopen it and run the app, it works fine.13:05
Randy_Onik90, I was getting an error while trying to delete an emulator, I created it using the name local, but the SDK for some reason named it cat:13:06
nik90zbenjamin: Is this more a rsalvetti question?13:06
Randy_Onik90, I couldnt delete it, but I just renamed the directory from local to cat: and managed to delete it13:06
zbenjaminlet me check13:06
nik90Randy_O: what did it rename to?13:07
Randy_Onik90,  it's fine now, but have you ever seen that before?13:07
Randy_Onik90, cat:13:07
nik90Randy_O: also do you have the error log?13:07
zbenjaminnik90: uh, wtf13:07
nik90cat: ?13:07
zbenjaminnik90: its not caused by the launcher its something with the emulator i guess13:07
Randy_Onik90, This instance does not exist, use 'create cat:' to create it13:07
nik90zbenjamin: I think so too..it only happens after like 15-20 launches...may an error keeps adding up, but I will talk to rsalvetti about this13:08
nik90Randy_O: never got that error13:08
Randy_Onik90, it's fixed now, thanks for responding13:08
nik90np13:08
zbenjaminRandy_O: ok its fixed, just wanted to start helping you ;)13:08
Randy_Ozbenjamin, no worries, but has that every happened to you before? the SDK randomly renamed the emulator to "cat:"?13:10
Randy_Ozbenjamin, even though the local directory was called "local"13:10
zbenjaminRandy_O: nop13:11
Randy_Ozbenjamin, random glitch # 13232 for me I guess13:11
Randy_Ozbenjamin, thanks13:11
mzanettisergiusens: hey, about this one: https://bugs.launchpad.net/ubuntu/+source/media-hub/+bug/137001113:21
ubot5Launchpad bug 1370011 in media-hub (Ubuntu) "media-hub backwards compat with authenticator broken" [Undecided,New]13:21
sergiusensmzanetti: I saw you actually seemed to discuss something really similar right after I pasted that link ;-)13:22
mzanettisergiusens: who should I talk to? I don't think we should just add tons of permissions to apps in order to workaround issues in the backends13:22
sergiusensmzanetti: yeah, that's why I mentioned, broke backwards compat :)13:22
sergiusensmzanetti: it's jhodapp you need13:23
mzanettisergiusens: ack, thanks13:23
mzanettijhodapp: hello :)13:23
sergiusenshe might not be in yet13:23
jhodappmzanetti, hello13:23
sergiusensah, look at that :-)13:23
jhodapplike magic! :)13:23
mzanettijhodapp: heh13:24
mzanettiabout that bug: https://bugs.launchpad.net/ubuntu/+source/media-hub/+bug/137001113:24
ubot5Launchpad bug 1370011 in media-hub (Ubuntu) "media-hub backwards compat with authenticator broken" [Critical,New]13:24
jhodappmzanetti, I don't get what that bug is referring to...what's the authenticator referring to here?13:25
mzanettijhodapp: well, basically it happens with any app that uses the "camera" policy, but not "audio" or "video"13:25
mzanettijhodapp: Authenticator is just an example (search for it in the store)13:25
jhodappmzanetti, so wait, the app is using the apparmor camera policy group and then does what, tries to play a sound or video through media-hub?13:26
mzanettijhodapp: no13:26
mzanettijust opens the camera => *boom*13:26
ogra_seemingly th service thinks you also want to record audio if you open the camera nowadays :)13:27
jhodappmzanetti, shouldn't be any different than the camera-app though13:27
mzanettijhodapp: well, the camera does request the "audio" (or video) policy because it actually makes use of it13:28
mzanettibut those apps wouldn't need it.13:28
jhodappmzanetti, ah interesting, we need to get jdstrand in on this discussion13:28
jhodapphe wrote those policies13:28
jdstrandjhodapp: fyi, tagger used to work fine, but would have a denial for playing the camera click, but this was deemed acceptable (even desirable) by mzanetti13:29
nik90popey: When is the calendar app meeting? I like to join it :D13:29
mzanettijdstrand: yep, same for authenticator13:29
jdstrandjhodapp: now, it tries to connect to media-hub and gets a dbus denial and crashes13:29
mzanettijdstrand: which fails for the same reason actually. scanning a QR code13:29
jdstrandjhodapp: (the dbus denial is for connecting to media-hub)13:29
jdstrandjhodapp: so, it can be worked around by specifying the audio or video policy groups, cause both allow connecting to media-hub over dbus13:30
jhodappjdstrand, yeah indeed...the issue could be the camera click sound that is played by qtubuntu-camera13:30
jhodappjdstrand, sounds like other apps wouldn't have the policy permission to allow qtubuntu-camera to play this click sound13:30
jdstrandseems something down low is not failing gracefully13:31
jdstrandjhodapp: mzanetti just mentioned authenticator13:31
jhodappyeah, that's interesting13:31
jdstrandI think most apps would probably want to specify these policy groups when using 'camera', however, clearly there are some that legitimately do not13:32
jhodappso the way to reproduce this is to have an app that needs to use the camera and is in the camera policy group, but is not in the video or audio policy groups13:32
jdstrandnow, one could also argue that we should add the access to the camera policy group13:32
jhodappadd audio and video?13:32
popeynik90: 13:00 UTC Thursday13:33
jdstrandno, just the dbus rule(s) for talking to media-hub13:33
jhodappah13:33
jdstrandjhodapp: and yes, that is the reproducer. installer 'Tagger' and you'll see13:33
jhodappjdstrand, I think that makes sense, especially since in the somewhat near future I'll be moving all of the camera service functionality into media-hub13:33
nik90popey: thnx13:34
nik90renatu: hey, I just saw your replies to the MP13:34
jdstrandI can do that13:34
jdstrandlet me check that it works13:34
jhodappjdstrand, cool thanks13:34
beuno_dholbach, jdstrand, FYI, pindonga has started adding the API for frameworks13:42
=== beuno_ is now known as beuno
dholbachbeuno, jdstrand: will only click-reviewers-tools make use of the information then?13:42
beunodholbach, I'm hoping we switch everything over gradually13:43
dholbachbeuno, what else would be on the shopping list?13:43
beunogenerate the framework packages from it as well13:43
jdstrandjhodapp: it seems to work fine adding only the media-hub dbus call to camera policy13:43
jdstrandjhodapp: btw, has trust-store integration landed?13:43
jhodappjdstrand, oh good, let's do that then13:44
jhodappjdstrand, for media-hub specifically?13:44
dholbachbeuno, I was just asking so we can maybe use bug 1363096 for tracking this13:44
ubot5bug 1363096 in Canonical Click Reviewers tools "FRAMEWORKS_DATA_URL needs to replaced with a good solution" [Undecided,New] https://launchpad.net/bugs/136309613:44
beunodholbach, I also hope to then expose an automatically-generated website we can point people at13:44
beunodholbach, right. I'll assign that to pindonga, he will update the review tools with the URL as soon as it's on production13:45
jdstranddholbach: I could see the sdk consuming it. they may just rely on click-reviewers-tools output13:45
dholbachbeuno, fantastico - thanks a lot13:45
dholbachjdstrand, ah yes - that's right13:46
dholbachzbenjamin, do you know off hand where you get the information from which frameworks are currently available and up-to-date?13:48
beunodholbach, AFAICT, they were extracting it from the chroots the SDK created13:48
zbenjamindholbach: atm its hardcoded, but there is a tool for it as well13:48
beunoah13:48
dholbachzbenjamin, bug 1363096 is about having this information live in myapps13:49
ubot5bug 1363096 in Canonical Click Reviewers tools "FRAMEWORKS_DATA_URL needs to replaced with a good solution" [High,Triaged] https://launchpad.net/bugs/136309613:49
zbenjamindholbach: aa-clickquery can help you13:50
zbenjamindholbach: err that was for the policy version13:50
zbenjamindholbach: "click framwork list"13:51
dholbachbut some of them are obsolete13:51
dholbachthe idea was to move something like https://bazaar.launchpad.net/~ubuntu-core-dev/+junk/frameworks/view/head:/frameworks.json into myapps13:52
zbenjamindholbach: it just lists which frameworks are available, even if they are obsolete they are still there :)13:52
dholbachright :)13:52
dholbachI was just wondering and asking around who needs this info and if we have a canonical place for it, who all might move over to using it13:53
renatunik90, ok let know what you think13:54
nik90renatu: I think we might not go with the final animations since it is a bit distracting in the clock app.13:55
nik90renatu: as for the bottom hint being hidden on app startup, should I add Component.onCompleted { tip.forceActiveFocus() } to ensure it gets the focus ?13:55
nik90renatu: since I am not sure what could steal focus in the main clock app. It has no input fields to my recollection.13:56
=== greyback__ is now known as greyback
mzanettielopio_: hey, you good with this one too? https://code.launchpad.net/~unity-team/unity8/dash-geometry/+merge/23433313:58
=== _salem is now known as salem_
mzanettiwrong channel...13:58
fcole90ahayzen: ping14:10
ahayzenfcole90, pong14:10
fcole90ahayzen: Hello, can I ask you some help? I'm still trying to merge the same branch :/14:11
ahayzenfcole90, yeah whats up?14:11
* ahayzen reads through latest jenkins failure14:11
fcole90ahayzen: I've discovered that even if the log shows autopilot file being installed, they're maybe removed after some time :/14:12
balloonsmorning fcole9014:12
ahayzenhmm14:12
ahayzenand this still doesn't look right as a path for the app itself... "-- Installing: /tmp/buildd/ubuntu-docviewer-app-0.1.1+autopilot0/debian/tmp//ubuntu-docviewer-app.qml"14:13
ahayzenballoons, am i right in saying that ^^ ?14:13
fcole90ahayzen: If you look into autopilot deb files, you will see that mine have only doc folder and not lib :/14:13
ahayzenah14:13
fcole90balloons: hello :)14:13
fcole90ahayzen: music app has a "normal" path, I don't know if // could be valid :/14:14
ahayzenfcole90, music ends up in /usr/share/ ... -- Installing: /tmp/buildd/music-app-1.2ubuntu2/debian/tmp/usr/share/music-app/music-app.qml14:15
ahayzenbut i don't think that helps solving "could not import package ubuntu_docviewer_app: No module named ubuntu_docviewer_app" ? ... that will just be the next issue lol14:15
fcole90balloons: I'm still trying the same merge, unsuccessfully :/14:15
fcole90ahayzen: yes, I think the same XD14:16
fcole90ahayzen: having some issues to solve is not a problem, the problem is that I'm finding it really difficult to understand what's causing them :/14:17
ahayzen+114:17
elopio_mzanetti: yes I am.14:18
fcole90ahayzen: and I'm still learning debian packaging and cmake, so that makes the process quite  in trouble XD14:19
ahayzenfcole90, aren't we all hehe14:19
=== gatox_ultra is now known as gatox
fcole90ahayzen: ahahahah XD I feel less newbie now XD14:20
ahayzenfcole90, what is this file for http://bazaar.launchpad.net/~fcole90/ubuntu-docviewer-app/ubuntu-docviewer-app/view/head:/debian/ubuntu-docviewer-app.install ... we don't have that in music so how does ours work lol14:20
ahayzenoh we just have a file called "install"14:21
fcole90ahayzen, balloons: do you know what could be causing the "//" in the installing path? :)14:21
fcole90ahayzen, yes :)14:21
ahayzenfcole90, we have this http://bazaar.launchpad.net/~music-app-dev/music-app/trunk/view/head:/debian/install ..14:21
fcole90ahayzen: you have one only for autopilot :)14:22
ahayzenfcole90, yeah...don't know if that is related..i'm just playing "spot the difference" at the moment14:22
fcole90ahayzen, I have also tried your solution, with only install, but the issue is always the same XD14:22
ahayzenfcole90, hmm ok14:23
fcole90ahayzen, I'm doing spot the difference for about 2 or 3 days now XD but still can't find the ones that make it works :/14:24
fcole90ahayzen, do you know what's the purpose of the url-dispatcher? We don't have one ATM XD14:25
fcole90ahayzen, I was wondering if it could be related, but i feel no :/14:26
ahayzenfcole90, yep that is a way of sending things paths to the app... eg if you go to the music scope, click on an album and then click "play in music-app"...14:26
balloonsfcole90, so jenkins makes available the debs it installs. So atm, we can see it's still not finding the tests. Let's look at the debs and see how we need to fix the packaging to account for this14:26
ahayzenfcole90, this then sends a url via the dispatcher to the app which it then plays14:26
balloonshttp://91.189.93.70:8080/job/generic-mediumtests-utopic/2020/artifact/work/output/*zip*/output.zip14:27
ahayzenfcole90, so no probably not related it is just a feature ;)14:27
fcole90balloons, yes, the debs miss the lib folder with the tests :/14:28
balloonsfcole90, looking at the 2 generated debian files, heh, nothing is in them14:28
ahayzenfcole90, more info here if you were interested... http://developer.ubuntu.com/api/qml/sdk-14.10/Ubuntu.Components.UriHandler/14:28
balloonsjust /usr/share/doc.., I don't even see the qml14:28
fcole90ahayzen, mmh, maybe it can be nice for a future feature :) thank you :)14:29
ahayzenballoons, so you can see cmake installing the files in the log...but they don't appear in the deb?14:29
fcole90balloons, yes, it's like if the files get extracted in tmp and then just discarded :/14:29
balloonsI'm not sure what you mean, but I would point at the packaging as needing fixing14:31
=== artmello_ is now known as artmello
fcole90balloons, I mean: I see in the logs that says "installing" but then it cannot find them :14:33
fcole90balloons, I would like to get deeper in the matter to better understand the proccess of debian installation, cmake install and what does jenkins do when receives a branch. Do you know some good documentation about that, so that I can be better aware of the whole process? :)14:35
balloonsfcole90, jenkins just builds the debian package; you can try this locally: http://packaging.ubuntu.com/html/packaging-new-software.html14:36
fcole90balloons, ok, you mean with pbuilder? :)14:38
balloonsbzr builddeb -- -us -uc14:39
fcole90balloons, easier than :) Ok, I will try with that but I think that will need to dig a bit around debian packaging docs :)14:40
fcole90balloons, thank you very much :)14:41
balloonsfcole90, yea, sorry I'm not much of a packager14:43
fcole90balloons, don't worry, debian packaging is a hard job ;)14:46
fcole90balloons, do you know any packager? :)14:46
mihirnik90: would you mind, if we can merge reminder view change in existing branch ?14:48
nik90mihir: may I ask why?14:49
nik90mihir: I don't want to make our ViewReDesign more bigger. That's why I started of with a separate branch :P14:49
nik90mihir: but do you like that branch? :D14:49
mihirnik90: got it :D14:50
mihirnik90: i already commented on that :D14:50
nik90cool.. thnx...14:50
mihirnik90: it loos amazing :)14:50
nik90mihir: the whole Defines.js is a mess :/ I don't why there are so many Qt.createComponent() calls14:50
nik90mihir: I think that's the cause of the new event page lag14:51
mihirnik90: i know it is..i was new so couldn't decide which one is better and best, but now i can understand that :)14:51
nik90mihir: btw we have a bug in the EventRepition.qml page14:52
* mihir is surpirsed 14:52
nik90mihir: if I select repeat weekly and choose some days, I don't see that in the new event page14:52
mihirnik90: let me try14:53
mihirnik90: i believe when we did, i myself tested it properly14:53
nik90mihir: I know. I thought I tested it as well. But I see the bug in trunk14:53
nik90mihir: basically in the NewEvent page, it reads "Weekly on" instead of "Weekly on Mon,Tue"14:54
nik90and when I go into the EventRepition page, Mon and Tue is not selected14:54
mihirnik90: i debugged, somehow it is not storing data into Reccurence Object15:01
nik90mihir: I think it is important for us to write some manual tests which we should run before merging branches to avoid regressions like this.15:02
mihirnik90: agreed, currently we don't hve any AP for reminders, repetations , future past , that bug is in progress by Json.15:03
nik90mihir: we are making it really hard for jason by changing code too often :p15:04
mihirhehehe , true...15:06
mihiri haven't heard anything from him not seen on IRC atleast15:06
nik90mihir: I will try to tackle the week day issue.15:08
nik90mihir: you tackle the week view issue we discussed earlier in the day.15:08
mihirnik90: ohkie, i am almost done15:11
nik90mihir: you got home15:11
nik90?15:11
mihirthe issue is with , this condition if(limitOptions.selectedIndex > 0)15:11
mihiri came early due to traffic issue, china president is visitng Ahmedabad heheh15:12
nik90ooh :)15:12
mihirso all roads are almost blocked , still it took 2 hours for me to reach home15:12
nik90oh15:13
nik90can you create a branch with that fix and I will take it with there w.r.t to code clean ups like I did with the EventReminders.15:13
* nik90 hopes he doesn't break it after mihir fixes it :P15:13
mihirnik90: is there anything changed in OptionSelector ?15:15
nik90mihir: not that I know of15:15
mihirnik90: this is not working , if(limitOptions.selectedIndex > 0)15:15
mihiri mean selected index is always 0 even if i select 4th index15:15
mihirnik90: got the catch15:16
mihirlol15:16
nik90:)15:17
=== salem_ is now known as _salem
mihirnik90: https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/NewEventRepeatFix/+merge/23483115:19
elopio_mihir: pong.15:23
=== elopio_ is now known as elopio
elopiomihir: too late?15:23
nik90mihir: your branch exposes another bug..when you choose weekly->Fri and then press back button, it reads "Weekly on Sun" instead of "Weekly on Fri"15:27
* mihir tries 15:29
mihirelopio: done i guess :)15:30
=== chihchun_afk is now known as chihchun
=== gatox is now known as gatox_lunch
mihirnik90: i got that..15:35
mihirnik90:  i believe we tested all these stuffs, some MR just got changed it15:35
* mihir wonders why same bugs again :-s15:36
* nik90 looks at trunk15:36
nik90mihir: check out rev 433 in trunk. That could be it..not sure15:38
mihirnik90: anyway , can you try now ?15:42
nik90sure15:44
nik90mihir: yup that seems to fix it.15:45
nik90mihir: I am going to run some more manual tests before approving it.15:46
sturmflut-workpopey, mzanetti: I am quite surprised by how well the RSS feed works. Low amount of updates though :/15:50
mihirnik90: no issues, thanks for doing this..:)15:52
sturmflut-workrpadovani: What happened to your excellent blog post about Bacon2D from yesterday?15:52
mzanettisturmflut-work: indeed15:52
mihirnik90: subtracting units.gu(10) to adjust is good way to do or no ?15:52
nik90mihir: yw15:52
nik90mihir: personally no :P15:52
rpadovanisturmflut-work, it wasn't ready yet, but I forgot to postpone the publication date. You'll can read it next monday ;-)15:52
nik90mihir: I would recommend using anchors.left, right and margins15:52
sturmflut-workrpadovani: I cant't wait for so long! It was so interesting ;)15:53
nik90mihir: anchors are recommended by upstream to be used over manually specifying width or height in general15:53
rpadovanisturmflut-work, thanks, but there wasn't a lot of code :-)15:54
mihirnik90: http://paste.ubuntu.com/8358762/15:57
mihiri tried to remove that subjstraction and assigned leftMargin , but that didn't work well15:57
mihirsubtraction*15:57
nik90did you do anchors.left: parent.left?15:58
sturmflut-workpopey, dholbach__ : Nice interview on the Ubuntu UK Podcast. I didn't have the time to listen to it earlier.16:03
popeyoh super!16:03
* mihir_ brb after dinner 16:10
=== gatox_lunch is now known as gatox
=== _salem is now known as salem_
mihir_nik90: popey ping16:46
popeymihir_: yo16:46
nik90mihir_: I was just about to ping you :P16:46
popey\o/16:46
mihir_ahhh  , that was quick16:46
nik90:)16:46
mihir_popey: nik90 http://i.imgur.com/0oWI7h0.png16:46
mihir_now this is how weekview looks like16:46
mihir_nik90: it solves problem of time overlaps as well, it will never overlap on time, so time frame separate now :)16:47
popeycan you do me a set of screenshots, day, week, month, year?16:47
* mihir_ takes screenshots16:47
nik90mihir_: nice work!16:47
popeythanks16:47
popeyloving the improvements to calendar!16:47
nik90mihir_: would it be possible to center the week day labels like "Sun 7"..at the moment it is left centered w.r.t to the events themselves16:48
=== dholbach__ is now known as dholbach
popeyback in a bit, making curry16:52
nik90mihir_: btw I just commented in https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/NewEventRepeatFix/+merge/23483116:52
mihir_popey: nik90 here you go with all views screenshot , http://imgur.com/a/xaJog#017:09
mihir_popey: i haven't captured yearview as it hasn't been change.17:09
mihir_nik90: okay, thanks for regeression testing, i am able to reproduce it and working on that fix.17:10
nik90mihir_: thnx mate17:10
nik90mihir_: btw mind reviewing both MPs for bug 1370138.. its only 10 lines each17:12
ubot5bug 1370138 in Ubuntu Clock App "Replace 'X' with '<' in Edit Alarm" [Low,In progress] https://launchpad.net/bugs/137013817:12
nik90they change the back button icon..that's all17:12
mihir_nik90: will do it in next 5-10 mins.17:15
nik90no hurry17:15
popeymzanetti: thanks. can you make the day names consistent, Month view shows MON, rather than Mon (used elsewhere)17:21
popeyer, not mzanetti , mihir ^ ☻17:21
mzanettiphew...17:21
mihirpopey: will do that :)17:21
mihirin meanwhile , let us know any more improvements can be done.17:22
mzanettizbenjamin: hi, there's a conflict unfortunately: https://code.launchpad.net/~zeller-benjamin/reminders-app/sdkcompat/+merge/23374317:24
zbenjaminmzanetti: i'm going to fix it tomorrow, there was no conflict when i filed the MP though ;)17:25
popeymihir: nik90 not sure about the event going all the way to the edge of the window..17:25
mihirpopey: from right corner?17:26
popeymight be nice if the event bubble stoped short of the right edge. yes17:26
nik90popey: I was thinking of adding a right margin17:26
popeysame on week view17:26
popeyyes17:26
popeythey dont line up in week view17:26
mzanettizbenjamin: yeah... seems some other branch landed by now17:27
mihirnik90: I have pushed the change to that repeat MR.17:39
nik90popey: I am having trouble adding silo 002 to my phone. I did "sudo add-apt-repository ppa:ci-train-ppa-service/landing-002" but it says Cannot add PPA: 'ppa:ci-train-ppa-service/landing-002'.17:41
nik90Please check that the PPA name or format is correct.17:41
popeyis it an rtm silo?17:42
nik90no utopic17:42
nik90http://people.canonical.com/~platform/citrain_dashboard/#?distro=ubuntu&q=landing-00217:42
davmor2nik90: well for a start off some packages failed to build is not a good indicator ;)17:43
nik90okay we need to move this conversation to #ubuntu-touch since tvoss isn't in this channel and it was his silo that he wanted me to test.17:44
popeyprobably -ci-eng is better17:44
nik90ok17:45
mihirpopey: nik90 does this looks okay ,http://i.imgur.com/ScXvyLM.png ?17:45
popeymuch better17:45
nik90mihir: better17:46
mihirpopey: do you hav any suggestions for showing All Day Events, http://i.imgur.com/GCsBfma.jpg ?17:47
mihirdoes this looks okay, if yes then i'll Merge branch.17:47
renatupopey, can we get this MR released? https://bugs.launchpad.net/ubuntu-calculator-app/+bug/135798317:49
ubot5Launchpad bug 1357983 in Ubuntu Calculator App "Continuos calculation doesn't work with irrational results" [Critical,In progress]17:49
nik90mihir: which branch?17:50
mihirnik90:  akvia's branch in our branch17:50
nik90mihir: I told you we can't17:51
mihirnik90: ohh , manually too ?17:51
nik90mihir: well manually yes17:51
mihiror it would be long task ?17:51
nik90mihir: but you got to be careful17:51
nik90mihir: since you need to know which exactly to add and remove17:51
mihirokay ,let him resolve the conflicts.17:51
nik90he can't17:51
nik90let's just merge it manually17:51
mihirbut also , was getting inputs from popey17:52
nik90he already resolved the conflicts but incorrectly, I don't think we can go back in his branch17:52
nik90yeah yeah I was just commenting on akiva's branch17:52
mihirnik90: but i feel he is changing lots of stuffs,  looking at manually :|17:55
nik90mihir: then we cherry-pick only what we need17:55
mihirnik90: okay, becuase i just pulled his branch and tried to run , it is throwing errors.17:56
mihirnik90: btw, i have pushed all the changes to over branch17:56
nik90mihir: told ya17:56
mihirnik90: hahahah, i braced my self :P lol17:56
nik90mihir: if you look at http://paste.ubuntu.com/8359476/ (which I took from his branch), he basically removed what should be there and vice-versa17:57
mihirhehehe yeah , i guess so17:58
mihiri just commented on his branch17:58
mihirhe got lilttle frustrated17:58
nik90well bzr can be a little tricky sometimes17:59
mihiryup huge conflicts are little tedious to merge17:59
nik90first of all, are we inputting his changes or not? I remember you saying you weren't sure about the buttons18:00
mihiri liked, the two rows in weekview18:00
nik90let's make a decision since say he fixes his branch and then we decide not to go with it then he won't like it18:00
nik90then we have to also use the buttons for all day events18:00
nik90since otherwise the header space would like the old design18:01
nik90taking too much space18:01
mihirHmmm, I am fine whatever you say..:)18:01
mihiror lets take popey's input and close this :)18:01
nik90he is in a meeting18:01
mihirohkie :)18:02
mihiranything else you think we can take up with this MR?18:02
nik90I haven't had time to take a closer look yet18:03
mihirokay , let me know if you can spend sometime today so can work before i go to bed18:04
mihirnik90:18:06
nik90mihir: I am not sure I can18:06
mihirohkie no issues..:)18:06
mihirnik90: i'll take look at other things meanwhile.18:07
ahayzen_balloons, ping18:07
nik90mihir: ok, Were you able to center the labels in the week view?18:07
mihirbtw, can you look at Repeat branch , i am done with that , i guess left with merge.18:07
nik90mihir: the day names like "Sun 17" etc..18:07
nik90mihir: yeah I will test and approve when I am done18:07
balloonsahayzen_, pong18:10
mihirnik90: nope, the code says, it is already centered align ,             horizontalAlignment: Text.AlignHCenter18:10
mihir 18:10
popeyyo18:10
ahayzen_balloons, this bug 1369990 ... is the window simply not being found by autopilot?18:11
ubot5bug 1369990 in Ubuntu Music App "Music app is failing to start in some autopilot tests" [Undecided,New] https://launchpad.net/bugs/136999018:11
mihirpopey: inputs on this ,http://i.imgur.com/ScXvyLM.png18:12
shyamskAkiva-Thinkpad: So apparently there is a thing called a Kit that needs to be created b4 starting any actual testing. And with a slow-ass connection its taking ages to complete :( Is there any way to skip this? I keep timing out :( hitting 404s etc :'(18:13
popeymihir: i like18:14
=== gatox_ultra is now known as gatox
mihirpopey: okay :) nik90 we can get ask him to resolve it and we're good to goooo :)18:14
mihirpopey: sorry18:14
mihirwrong screenshots18:14
nik90mihir: wrong screenshot18:15
mihirpopey: http://i.imgur.com/GCsBfma.jpg18:15
mihirpopey: this one , sorry for trouble18:15
nik90popey: we are looking for feedback on the all event button that is shown and also the 2 row display of date and day names18:15
popeyWell, aside from not liking the "Ev." thing - I've never liked that ☻18:16
popeythat said, i prefer this than the big block we used to have18:16
mihirpopey: now the block visible only when there are any events.18:17
mihirbut we can take this  and can imporve it anytime :)18:17
nik90I will send an email to Akiva explaining this.18:18
mihirnik90: copy me ,so we all can be on same page :)18:18
nik90yeah ofc18:18
popeyyeah, i certainly prefer it18:19
shyamskSo this kit thing, what architechture and framework do I have to choose?18:19
mihirpopey: thanks for your inputs :)18:19
mihirnik90: one ques, nik90: what are tweaks changes that needs to be there in AP due to we cahange Actions ?18:19
nik90shyamsk: depends on which device you want to run it on18:20
nik90shyamsk: emulator or real hardward?18:20
nik90mihir: something like click_toolbarbutton() to click_headeraction()...18:20
nik90mihir: minor stuff18:20
shyamskemulator ryt now.18:21
mihirand FYI , we got 10 AP failurs18:21
nik90shyamsk: ok for an i386 emulator, you create a kit for 14.10 Framework i38618:21
nik90shyamsk: check out http://developer.ubuntu.com/apps/sdk/tutorials/18:22
nik90shyamsk: it is well explained there18:22
nik90mihir: ok18:22
shyamsknik90: I ran into a chroot error when I had tried that. Something about the chroot not being available or sumthing18:22
shyamskshall check the link18:22
nik90shyamsk: I created a kit just yesterday and it worked fine for me18:22
nik90shyamsk: are you running 14.04?18:22
shyamskyes18:23
nik90shyamsk: it should work afaik. kit creation takes a while though18:23
* popey wishes for a podcast app18:24
shyamsknik90: hmm... so then it could be connection issues. Ran into a couple of "W: Couldn't download package libc-dev-bin (ver 2.19-0ubuntu6 arch amd64)" and similar msgs18:24
balloonsahayzen_, the screenshots show the app is still attempting to load18:29
ahayzen_balloons, screenshots! ...how do i get to them from that console output?18:30
balloonsahayzen_, grab the subunit file18:30
balloonshttp://jenkins.qa.ubuntu.com/job/utopic-touch-mako-smoke-daily/901/artifact/clientlogs/music_app/test_results.subunit/*view*/18:31
balloonsahayzen_,  i have a tweak to the tests that should help this18:31
ahayzen_balloons, it appears as text...do i have to download and save as a .jpg or something?18:32
balloonssave the link yes18:32
mihirballoons: ping18:32
balloonsahayzen_, then use lp:trv to view it18:33
ahayzen_ah18:33
mihirballoons: if we need to get label from Header, which is override by the page, we can get value of that , is that correct?18:33
balloonsahayzen_, I'm looking at AP now, and adding AP as affected18:37
ahayzen_balloons, ok thanks....how do i use this tool to view the subunit?18:38
balloonsahayzen_, the tool is really rudimentary18:38
balloonsahayzen_, https://www.youtube.com/watch?v=jkLtbmQxXYc18:38
ahayzen_balloons, thanks18:39
shyamskWhat is Error 127? I tried running the project, after creating the kit. Also here can I find a listing of the error codes?18:43
* balloons proposes mp18:47
mihirnik90: is there any differnt librabry for using headers in AP ?18:56
nik90mihir: you will have to check the clock app AP tests or the autopilot documentation19:01
nik90mihir: i don't know of the top of my head19:01
mihirnik90: okay, i checked autopilot docs..let me refer to clock ap tests19:02
nik90mihir: doesn't calendar already press on the save header button from our new event design revamp?19:02
nik90mihir: why don't you just refer to that?19:02
nik90mihir: oh btw I think you posted both your comments in one Mp rather than this https://code.launchpad.net/~nik90/ubuntu-calendar-app/replace-x-icon/+merge/23485119:03
nik90:P19:03
mihirnik90: http://paste.ubuntu.com/8359857/19:04
balloonsmihir, I'm not sure what you are asking. But if it's in the app, you can get it :-)19:06
nik90mihir: its not exactly that..I said it is something like that :)19:07
nik90let me grab the line in the calendar app where we use it19:07
nik90balloons: btw may I merge https://code.launchpad.net/~nik90/ubuntu-clock-app/checkbox-manual-tests/+merge/234164 ?19:08
balloonsnik90, yep, I approved yesterday19:08
nik90cool thnx19:08
nik90mihir: https://bazaar.launchpad.net/~ubuntu-calendar-dev/ubuntu-calendar-app/trunk/view/head:/tests/autopilot/calendar_app/__init__.py#L12419:09
mihirnik90: hmm , i guess i missunderstood19:09
nik90mihir: no worries, I should have been more clear19:10
mihirnik90: it was same and i interpretted  , we need to change to something different19:10
* nik90 wonders if he missed today's landing email19:14
davmor2nik90: no19:15
nik90davmor2: cool19:18
balloonshey nik90, cleaning up old tabs and I see https://code.launchpad.net/~carla-sella/ubuntu-clock-app/new-add_world_city_test/+merge/231131. Can we land this? I know there are some bugs around it, but they are filed19:19
nik90balloons: yeah I didn't have time to review it. I will review and then merge it asap19:20
balloonsnik90, ack.. I leave it for you..19:20
* balloons closes tab ;-)19:20
nik90balloons: :-)19:20
balloonsdiff is getting big guys :-) https://code.launchpad.net/~ubuntu-calendar-dev/ubuntu-calendar-app/ViewRedisign/+merge/23468019:31
balloonscan't wait to see it completed19:31
mihirballoons: :D19:33
mihirballoons: now trying to fix AP19:33
ahayzen_balloons, that trv tool is awesome :) ...19:39
ahayzen_balloons, i find the test_reads_music_library failure screenshot interesting though why is it at the songs tab?19:39
ahayzen_balloons, looks like that test actually failed and the others were just it failing to load the app?19:40
mivoligomzanetti: hi again, I've pushed 5 more levels. Feel free to rearrange their order :)19:40
balloonsahayzen_, yes, seems one test did fail, but I didn't look at it19:41
mzanettimivoligo: awesome :)19:41
mzanettimivoligo: so... did you try to play the first 5 yet?19:41
ahayzen_balloons, it seems the now playing page didn't show after 10s after clicking a track...assuming the clicking worked...but that is using populate_queue() which pretty much every test uses?19:42
mivoligomzanetti: unfortunately no :(19:42
mivoligomzanetti: maybe tomorrow19:42
balloonsahayzen_, yes, quite possible. Welcome to devices, were things can happen. You'll have to look at things at bit closer to get a feel for what happened19:43
ahayzen_balloons, yey \o/19:43
mivoligomzanetti: I'll get back to you tomorrow, my kids need attention :D19:46
elopionik90: qml tests are suddenly running on jenkins. Did you or fginther do something?19:59
fgintherelopio, I haven't done anything. Can you provide an example of where you are seeing these tests?20:05
=== chihchun is now known as chihchun_afk
elopiofginther: http://91.189.93.70:8080/job/generic-mediumtests-utopic-python3/345/console20:05
elopioI've just added the missing dependency to the branch to see if it finishes the run20:06
mihirnik90: ping20:06
elopiohttps://code.launchpad.net/~canonical-platform-qa/ubuntu-clock-app/xvfb_and_qml_tests/20:06
mihirnik90: i have fixed some of APs and pushed the code.20:06
balloonsthat's part of the cmake build20:07
fgintherelopio, that's being done during the package build... er what balloons said20:07
* balloons waves to fginther 20:07
balloonsI thought you were incognito this week :-)20:07
ahoneybundoes anyone need someone with tiny C++ exp?20:07
fgintherballoons, I'm hit and miss. I'm at a conference and just in between sessions20:08
elopiofginther, balloons: I know, that's what I was trying to do. Last thing I saw on friday was a message saying that no tests were found.20:08
balloonselopio, ahh.. well, it's not useful in this case as the tests need xvfb and ui components20:08
=== gaspa_ is now known as gaspa
balloonsunless you have some magic for that20:08
elopioballoons: I just add xvfb as a build dep.20:09
* fginther goes to another sessioin20:10
elopionik90, fginther, balloons: http://91.189.93.70:8080/job/generic-mediumtests-utopic-python3/346/console20:12
elopiowoohoo, that's the test we expect to fail.20:12
elopioso these tests work on xvfb on a pbuilder.20:12
balloonselopio, hehe, you have opened pandora's box20:13
* balloons can't wait for folks whining about build failures in jenkins :-)20:14
elopioballoons: why? I think I'm missing something here.20:14
balloonselopio, you did good work. That's awesome qml tests can run20:14
elopiowell, I didn't do much, just copy things from the other apps.20:15
ahoneybunhey mihir20:16
ahoneybunhey mhall11920:16
mihirhey ahoneybun20:16
mihirahoneybun: by mistake ?20:17
mhall119hey ahoneybun20:17
balloonsahoneybun, howdy..20:18
nerochiarobfiller: i pushed the fixes for almost all the problems on the MR for sharing from selection. I'll pop in tomorrow morning to see if Tim can help with the header issue, as I could not find the problem and I'm using workarounds that he suggested (as there's no proper API to show/hide headers in the SDK right now)20:19
nerochiarobfiller: I fixed the AP test that I had broken but there are others failing which I don't think are related to my changes20:19
bfillernerochiaro: what api do we use to hide/show the header when clicking on the photo? wondering if we can use the same20:19
ahoneybunmihir: sorry it was a mistake by hi anyway :)20:20
ahoneybunhey balloons20:20
ahoneybunmhall119: I still missing the module20:21
ahoneybun*I;m20:21
ahoneybun*I'm20:21
mihirahoneybun: np :)20:21
nerochiarobfiller: basically to hide the header the way Tim suggested is changing the current Page title to "". Which is what we do in the photo viewer. But the same method doesn't work on the photo/event overview, since it's using Tabs. So he suggested to push a page with empty title with the ContentPeerPicker inside20:21
nerochiarobfiller: which works, but apparently not when the header was shown as a result of the picture viewer being up before20:22
mhall119ahoneybun: you can always install the app from the archive, that way the module is installed at hte system level and you can just use that20:22
nerochiarobfiller: and I have no idea why20:22
ahoneybunmhall119: true20:22
mhall119ahoneybun: but really, it would be nice to find the source for that and build it20:22
ahoneybunmhall119: source? I pulled it from bzr on lp20:22
ahoneybunoh the module20:22
bfillernerochiaro: ok20:22
mhall119ahoneybun: yeah20:23
ahoneybunmhall119: should I jump to +1>20:23
ahoneybun?20:23
mhall119ahoneybun: you mean utopic on your desktop?20:23
nerochiarobfiller: not sure about that new test failing, i'll have a look now and see if i can do something about it20:23
ahoneybunmhall119: yea I'm on trusty from that USB from fossetcon :)20:24
ahoneybunmhall119: I found the package page http://packages.ubuntu.com/trusty/notes-app that list the deps20:25
mhall119ahoneybun: it's probably safe to upgrade to utopic on your desktop, I've only stayed on Trusty to make sure our SDK support for Trusty users is working20:26
mhall119ahoneybun: the module binary is in the notes-app package, not a dep20:27
mhall119see http://packages.ubuntu.com/trusty/i386/notes-app/filelist20:27
ahoneybunoh so it should be in there mhall119 ?20:27
mhall119ahoneybun: yes20:27
ahoneybunoh odd20:28
* ahoneybun thinks he will upgrade to ubuntu+1 in a bit20:30
ahoneybunmhall119: where do you think I would be the most helpful?20:32
mhall119ahoneybun: any of the core apps, popey is trying to get more work done on the document viewer specifically20:32
ahoneybunmhall119: I'm not near a dev though20:33
mhall119ahoneybun: that didn't stop the other core apps contributors :)20:37
mhall119ahoneybun: where would you like to help? Obviously you know that documentation is always needed in any project20:38
mhall119testing too20:38
ahoneybunmhall119: true if I could get the notes-app to run I could test it lol20:38
* ahayzen_ wonders why the facebook app starts when he runs the url dispatcher test for music:// lol20:38
mhall119ahoneybun: I wouldn't put too much effort into the notes-app, contribute to Reminders if you want to work on a notekeeping app20:39
mhall119ahoneybun: poorly written URL regex perhaps?20:39
mhall119ahayzen_: ^^ that was for you20:39
ahoneybunok should I grab the code on lp or install it on the system from repos?20:40
ahayzen_mhall119, hah maybe...or i've just foobar'd the autopilot tests i'm working on lol20:40
mhall119ahayzen_: you can grab it from a PPA and get the code from LP20:40
ahayzen_mhall119, was that meant for ahoneybun ^^ ? lol20:40
ahoneybunmhall119: bzr branch lp:reminders-app?20:41
mhall119ahayzen_: yes, one of you is gonna have to change your nick :-P20:41
ahayzen_hehe lol20:41
mhall119ahoneybun: I believe so yes20:41
mhall119ahoneybun: there's an Online Account provider for Evernote that you'll need to run it20:42
* ahoneybun does not even have a evernote account lol20:43
ahoneybunmhall119: that is a API that should be included right20:45
ahoneybun?20:45
mhall119ahoneybun: the account provider might be a separate package20:46
mhall119popey could tell you more, but it's after hours for him now20:46
ahoneybunmhall119: ok I'm sick and took the day off tomorrow so I'll be on lol20:47
mhall119"I'm sick, let's upgrade the computer and start app development", lol20:48
nik90elopio: hey20:48
nik90elopio: yeah I saw that jenkins ran the qml tests :)20:48
nik90fginther: thnx ^^20:48
ahoneybunmhall119: when in rome, btw how do I open it in the SDK notes-app had a qml file I used20:48
nik90balloons: jenkins now runs qml tests :) :))20:48
nik90balloons: means I can complain more now about failing tests20:49
ahoneybunmhall119: I'm also working on the kubuntu teams new site lol20:49
mhall119ahoneybun: look for either a .pro, .qmlproject or CMakeLists.txt file and open that in the SDK20:50
ahoneybunok20:50
ahoneybunI saw that20:50
ahoneybunmhall119: bzr branch lp:account-plugin-evernote20:51
ahayzen_...could someone on #243 go to the music scope click on a track and then 'Play in music app' and check that the music-app starts and not facebook lol?20:53
nerochiarobfiller: the test that fails on make under jenkins works on my mako. i'm running the whole suite on the device again just to be sure.20:54
=== salem_ is now known as _salem
ahoneybunopenssl/ssl.h no such file or dic, I guess that must be on the plugins end20:55
* ahoneybun is going to start the upgrade now20:58
SturmFlutahayzen_: I'm on #243 now, the Music scope seems to do what it's supposed to do21:22
ahayzen_SturmFlut, ok i must have busted my device then thanks21:23
nerochiarobfiller: all tests run just fine on my phone now. not sure what is bothering jenkins21:36
ahoneybunmhall119: what places in fl still repair the nexus 4 screen?22:17
balloonsahoneybun, I did my own repair, it's not too difficult22:19
balloonsfinding a new screen would be the hard part22:19
* ahoneybun just damn bought a nexus 4 on ebay22:22
ahoneybunwell it should be here saturday22:23
ahoneybun$132.50 balloons22:23
ahoneybuncan't believe I did that with only 9 months into my 2 years with verizon lol22:24

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