| === chriadam is now known as chriadam|away | ||
| JamesTait | Good morning all; happy Friday and happy Belly Laugh Day! :-D | 07:45 |
|---|---|---|
| dholbach | good morning | 08:03 |
| === luke is now known as Guest43582 | ||
| dpm | mzanetti, morning! For the hack days starting today, I'm trying to make sure the po/reminders-app.pot file is up to date, but I'm not very familiar with cmake, tbh. Is there a way I can run cmake to update that file only, instead of having to build the whole source tree? | 08:56 |
| mzanetti | dpm: no, there isn't atm | 09:20 |
| dpm | mzanetti, ok, thanks. It seems that the current rule to build the .pot file has no effect as it's missing the list of files to extract translations from. Would something along these lines help? -> http://pastebin.ubuntu.com/6807439/ (I need to check if the glob is recursive and if there is a better way to specify the path to the QML files) | 09:23 |
| dpm | ok, seems GLOB_RECURSE is my friend | 09:28 |
| === mpt_ is now known as mpt | ||
| Guest___ | Hi, I am new to core-app. Any place to start work on? | 09:44 |
| popey | Guest___: hi there! | 09:46 |
| popey | Guest___: what's your area of interest? | 09:46 |
| Guest___ | app or language? | 09:47 |
| popey | either/both | 09:48 |
| Guest___ | I can do both qml and c++. I want to work on music app today. | 09:49 |
| popey | ok, do you have a development environment setup? | 09:49 |
| Guest___ | Do we do it on phone or desktop | 09:50 |
| popey | You can develop on the desktop and test both on desktop and device | 09:50 |
| Guest___ | I have the environment setup already. | 09:51 |
| stefano92100 | Hi everybody! Just a few question: I need to sign the Contributor Agreement, who's the Canonical Project Manager that I nedd to add in the form? Is there any roadmap for the Music App today? | 09:52 |
| popey | stefano92100: me, Alan Pope. | 09:52 |
| stefano92100 | Thanks, so do I just need to add your name? | 09:53 |
| popey | yeah | 09:53 |
| popey | they know who i am ☻ | 09:53 |
| popey | stefano92100 / Guest___: dpm and I are having a meeting in a moment to make sure we have things for you. will ping you shortly. | 09:54 |
| popey | in the meantime, there's always the bugs already filed.. https://bugs.launchpad.net/music-app/+bugs?orderby=-id&start=0 | 09:54 |
| stefano92100 | I'm sure they know :D Anyway, yesterday I was testing the music app, I found some problems that aren't filed | 09:56 |
| dpm | morning stefano92100! Please make sure you file them at https://bugs.launchpad.net/music-app/+filebug | 09:58 |
| popey | oh, awesome, get them filed and ping me and I can confirm them | 09:58 |
| dpm | Guest___, a good place to start is by picking a small bug and trying to fix it. Do you want to choose one from this list https://bugs.launchpad.net/music-app/+bugs?field.tag=bitesize and start working on it? | 10:00 |
| Guest___ | dpm: Sure | 10:00 |
| dpm | awesome :) | 10:02 |
| Guest___ | How to import songs on desktop? | 10:02 |
| dpm | Guest___, it should work automatically for you if you are in Ubuntu 13.10 or Ubuntu 14.04 - if it doesn't work, let me know | 10:03 |
| dpm | Guest___, there is a service called mediascanner that imports your music in the background | 10:03 |
| WebbyIT | Good Hack Days to all :) | 10:04 |
| Guest___ | dpm: Operation failed Cannot open media index params at ""/home/luke/.cache/mediascanner"": g-file-error-quark(4): No such file or directory | 10:04 |
| stefano92100 | popey: did it. I've just send one for now, because others were just related to the design that seems not to fit well on a tablet. | 10:16 |
| popey | stefano92100: thanks! | 10:17 |
| dpm | stefano92100, awesome, thanks! Feel free to file the other design bugs too | 10:23 |
| dpm | hey WebbyIT, happy Hack Days to you too! :-) | 10:23 |
| stefano92100 | ok | 10:28 |
| dpm | mzanetti, when you've got a minute, could you review https://code.launchpad.net/~dpm/reminders-app/update-translations/+merge/203030 - it's a trivial one to update translations | 10:48 |
| mzanetti | dpm: left 2 comments here: https://code.launchpad.net/~dpm/reminders-app/update-translations/+merge/203030 | 11:29 |
| dpm | mzanetti, thanks, looking at it now | 11:37 |
| === _salem is now known as salem_ | ||
| mzanetti | dpm: ah one more note: The "attach" button does only pictures right now, but the code (and evernote) would allow to attach any file | 11:44 |
| mzanetti | dpm: so I'm not sure it should be named "Attach picture" | 11:44 |
| dpm | mzanetti, yeah, I agree with you, although perhaps we should leave it as picture until we have support for other types of files? Btw, what would be needed to attach other files? Content Hub support for File Manager? | 11:46 |
| mzanetti | dpm: I'd say just content hub in general | 11:47 |
| mzanetti | dpm: we could also attach music files from the music app for example | 11:47 |
| dpm | mzanetti, yeah, so we'd have to make each app a provider, right? (not sure about the terminology there) | 11:48 |
| dpm | so music app for media files, doc viewer for PDFs, File Manager for everything else... | 11:49 |
| mzanetti | dpm: don't know exactly what it takes... but yeah. I assume once an app registers something to export, it'll show up in there. | 11:49 |
| mzanetti | dpm: from the reminders app we'd just need to change ContentType from Pictures to Any or the like | 11:49 |
| mzanetti | apart from a few more fixes needed in the plugin (e.g. mime type detection) | 11:49 |
| mzanetti | so far I only check for jpeg and png | 11:50 |
| dpm | ok | 11:50 |
| dpm | mzanetti, the "too long strings" for actions should rather be an SDK bug, but I'll work around it by calling them "Attach" and "Shot" | 11:51 |
| dpm | thanks for the review! | 11:52 |
| === MacSlow is now known as MacSlow|lunch | ||
| dpm | mzanetti, updated the MP with the fixes. When you've got a minute, could you re-review? https://code.launchpad.net/~dpm/reminders-app/update-translations/+merge/203030 | 12:25 |
| mzanetti | dpm: bzr push ;) | 12:26 |
| dpm | mzanetti, hm, I did, it should be up to date on r33, let me check... | 12:27 |
| mzanetti | dpm: still on r31 here and bzr pull does nothing | 12:28 |
| dpm | mzanetti, argh, bzr push :parent pushed to trunk! | 12:29 |
| dpm | bollocks | 12:29 |
| dpm | mzanetti, the MP should now be up to date at least | 12:30 |
| mzanetti | dpm: :D | 12:30 |
| mzanetti | dpm: I was going to approve anyways... | 12:31 |
| mzanetti | dpm: so consider it approved and merged | 12:31 |
| dpm | mzanetti, thanks :) | 12:31 |
| dpm | ok, we're all set for translations now: https://translations.launchpad.net/reminders-app | 12:33 |
| zsombi | nik90: hi, so, I have ~20 mins before my meeting starts, do you have the Qs for the AlarmAPI? | 12:35 |
| nik90 | zsombi: yes | 12:35 |
| zsombi | nik90: shoot then | 12:36 |
| nik90 | zsombi: me and popey noticed that when we save an alarm, it doesn't appear in the alarm list until 10-15 mins later or only after we restart the phone. Is this an issue with the EDS - Alarms API communication? | 12:36 |
| zsombi | nik90: I see it as a problem of EDS: the Alarm API sends it straight, so if it would be the problem of the API, then a consecutive restart of the app (not the phone) should bring it in. | 12:37 |
| nik90 | zsombi: okay, in that case I will report a bug against EDS and ask renato to take a look at it. | 12:38 |
| zsombi | nik90: yep | 12:39 |
| nik90 | zsombi: I heard that you are working on the snooze feature for Alarms API v2.0. Considering that the indicator-datetime patch is planned to land next week, would it be possible to focus the efforts first on clearing up any issues with alarms API so that the basic functioning of Alarms is perfect. | 12:39 |
| zsombi | nik90: my network is just crap... | 12:41 |
| zsombi | nik90: anything else? | 12:41 |
| nik90 | zsombi: 1 last question | 12:42 |
| nik90 | I heard that you are working on the snooze feature for Alarms API v2.0. Considering that the indicator-datetime patch is planned to land next week, would it be possible to focus the efforts first on clearing up any issues with alarms API so that the basic functioning of Alarms is perfect. | 12:42 |
| nik90 | some issues were like I mentioned before, 1. Switching an alarm on/off results in it being immediately reverted. This only happens sometimes but requires testing and needs a fix. | 12:42 |
| nik90 | 2. Deleting an alarm does not update the alarm manager count | 12:42 |
| zsombi | nik90: Alarm API 2.0 is on hold till we get all Qt5.2 issues cleared | 12:43 |
| nik90 | zsombi: ok. that's fine by me | 12:43 |
| zsombi | nik90: these seme to be issues with the EDS, so we need to get together with renato to solve these | 12:43 |
| zsombi | nik90: as it works smoothly with Memory backend | 12:43 |
| nik90 | zsombi: I already reported a bug about the delete alarm issue. But I am unable to reproduce the switch issue reliably | 12:44 |
| zsombi | nik90: and renato suggested that I should concentrate on that as it should be completely the same with EDS... but it is apparently not | 12:44 |
| zsombi | nik90: can you repro it unreliably on desktop? | 12:44 |
| nik90 | zsombi: ok, in that case, I will try to contact renato and see where it leads | 12:44 |
| nik90 | zsombi: the delete issue I can reliably reproduce on the phone and the desktop | 12:45 |
| nik90 | zsombi: the switch issue happens sometimes on the phone and desktop. I do not have a 100% sure way of reproducing it | 12:45 |
| zsombi | nik90: if yes, try to use Memory and not the EDS, and if still behaves badly, it's in Alarm | 12:45 |
| nik90 | zsombi: Memory? | 12:45 |
| zsombi | nik90: you can force Memory backend either by removing EDS support from your desktop, or by altering Alarm backend | 12:46 |
| nik90 | zsombi: is there any special package I need to install or would removing EDS plugin alone suffice? | 12:47 |
| zsombi | nik90: removing EDS plugin would be enough | 12:48 |
| nik90 | zsombi: alrite, I will give that a try | 12:48 |
| zsombi | nik90: ok.. and if you see it happening, file a bug to Alarm API, otherwise it should be synced with EDS | 12:48 |
| nik90 | zsombi: ok | 12:48 |
| zsombi | need to move now to a place where I have more reliable connection | 12:48 |
| nik90 | zsombi: that was it. thnx for your time. | 12:49 |
| zsombi | nik90: you're most welcome :) | 12:49 |
| zsombi | nik90: any time :) | 12:49 |
| Guest___ | I am fixing Bug #1272113 of music-app and got confused with the description. Does that mean every time we enter the songsList, it should show the top instead of the currently played file. | 12:59 |
| ubot2` | Launchpad bug 1272113 in Ubuntu Music App "Genre and Playlist sheets skip currently playing/paused song if it's in the list" [Low,Triaged] https://launchpad.net/bugs/1272113 | 12:59 |
| === MacSlow|lunch is now known as MacSlow | ||
| dpm | Guest___, let me have a look... | 13:11 |
| dpm | Guest___, yes, that's it. When you click on a genre, the list shown should start at the top | 13:16 |
| Guest___ | dpm, removing three lines of coode will fix the problem. Is this a new design decision | 13:19 |
| dpm | Guest___, I think we're fine to go ahead: one of the developers proposed and I agree with that too, so we've got consensus :) | 13:20 |
| nik90 | renato: how do I report bugs on EDS? I noticed that https://launchpad.net/qtorganizer5-eds doesn't allow reporting bugs. | 13:21 |
| nik90 | renato: Is there any other launchpad project for it? | 13:22 |
| dpm | nik90, perhaps https://bugs.launchpad.net/ubuntu/+source/qtorganizer5-eds ? | 13:22 |
| dpm | (i.e. reporting against the source package) | 13:22 |
| nik90 | dpm: ah that works | 13:23 |
| dpm | nik90, ok cool, but renato should probably confirm where he looks at the bugs | 13:23 |
| nik90 | dpm: I will just report them for now and ping him the bug links | 13:24 |
| dpm | sounds good! | 13:24 |
| ahayzen | hackday \o/ | 13:25 |
| nik90 | ahayzen: :) | 13:26 |
| dpm | hey ahayzen o/ | 13:29 |
| ahayzen | dpm, o/ | 13:29 |
| dpm | Guest___, ahayzen and danielholm are music app developers, so if you've got any further questions about the bug you're fixing, they can probably help you | 13:31 |
| ahayzen | vthompson, danielholm, o/ | 13:32 |
| danielholm | ahayzen: hey! | 13:32 |
| ahayzen | danielholm, hows things? | 13:32 |
| danielholm | ahayzen: absolutely lovely. how about you? | 13:32 |
| ahayzen | danielholm, yeah good thanks, just trying to figure out binding loops :/ | 13:33 |
| danielholm | ahayzen: sounds interesting ;) I'm in class :) | 13:33 |
| ahayzen | danielholm, hehe | 13:33 |
| danielholm | ahayzen: so what are you trying to achieve? | 13:36 |
| ahayzen | danielholm, i'm trying to get the U1DB code to be in a state to be pushed up, obviously not fully working due to issues upstream but at least visible to others | 13:36 |
| danielholm | ahayzen: okey. what is the issue upstream? | 13:37 |
| ahayzen | danielholm, me and nik90 found some bugs/missing features tht we'll need | 13:38 |
| nik90 | ahayzen: btw, I noticed that stuart reported some more bugs upstream | 13:38 |
| PaoloRotolo | Hi all :) | 13:38 |
| nik90 | PaoloRotolo: hey paolo | 13:38 |
| PaoloRotolo | Hi nik90 :) | 13:38 |
| ahayzen | nik90, u got any links? i saw his blog post | 13:38 |
| danielholm | ahayzen: aha ok. nice to see those changes land then, I can imagine? | 13:38 |
| nik90 | ahayzen: hopefully we can expect fixes soon | 13:38 |
| nik90 | ahayzen: 1 sec | 13:38 |
| ahayzen | nik90, awesome | 13:38 |
| ahayzen | danielholm, yh it should help people solving the issues if i can get the code posted as well | 13:39 |
| nik90 | ahayzen: https://bugs.launchpad.net/u1db-qt/+bug/1271973 | 13:39 |
| nik90 | ahayzen: https://bugs.launchpad.net/u1db-qt/+bug/1271972 | 13:39 |
| ubot2` | Launchpad bug 1271973 in U1DB Qt/ QML "Results from U1db.Query contain only indexed fields, not the whole document" [Medium,Confirmed] | 13:39 |
| ubot2` | Launchpad bug 1271972 in U1DB Qt/ QML "Top-level fields in document cannot be indexed" [High,Confirmed] | 13:39 |
| mzanetti | dpm: https://code.launchpad.net/~mzanetti/reminders-app/enable-attaching-images/+merge/202817/comments/473595 | 13:39 |
| ahayzen | nik90, thanks | 13:39 |
| * dpm looks | 13:40 | |
| vthompson | ahayzen, danielholm, good day! | 13:40 |
| danielholm | vthompson: hey mate! how are you? | 13:40 |
| ahayzen | nik90, looks like some of them may be related to our issues or assist us in resolving | 13:40 |
| ahayzen | vthompson, morning :) | 13:40 |
| vthompson | danielholm, Good, yourself? | 13:40 |
| nik90 | ahayzen: yup :) | 13:40 |
| dpm | mzanetti, hm... will try again, updating the image on the phone now (I'm using dual-boot, so it takes a while) | 13:41 |
| ahayzen | nik90, u any good with resolving binding loops? | 13:41 |
| danielholm | vthompson: I am great, thank you. Long time since we spoke now | 13:41 |
| nik90 | ahayzen: with the height? | 13:41 |
| nik90 | ahayzen: let me give a shot at it :) | 13:41 |
| ahayzen | nik90, no i've been building the U1DB stuff for the music-app and getting a binding loop :/ | 13:41 |
| vthompson | ahayzen, danielholm, nik90 The music app has had a height binding loop error/warning for a while now | 13:41 |
| ahayzen | nik90, oh does tht one with the height still exist? | 13:42 |
| mzanetti | dpm: that said, I believe you that there might still be an issue somewhere... just not sure where as your steps to make it fail don't work here | 13:42 |
| nik90 | ahayzen: what vthompson said ^^ | 13:42 |
| vthompson | ahayzen, what does the U1DB stuff say is binding too what? | 13:42 |
| ahayzen | vthompson, well erm its probably easier to post the code then post links hang on i'll dump it in +junk... | 13:43 |
| vthompson | nik90, ahayzen maybe the music app got rid of the binding loop issue. I think a lot of it had to do with anchoring listviews in a pagestack | 13:43 |
| dpm | mzanetti, ack. Will try to reproduce again, this time with the latest image | 13:43 |
| nik90 | ahayzen: it seems that there are branches in https://code.launchpad.net/~uonedb-qt/u1db-qt/trunk/+activereviews that need reviewing...hopefully if they get quickly reviewed we can expect good fixes | 13:43 |
| dpm | nik90, ahayzen, you might want to ping kalikiana for the status of reviews in U1DB | 13:44 |
| ahayzen | dpm, thanks | 13:44 |
| nik90 | dpm: actually they are his branches | 13:44 |
| nik90 | dpm: so he needs someone else to review them :) | 13:44 |
| ahayzen | dpm, who can i talk to about getting UITK merges landed? | 13:44 |
| ahayzen | dpm, eg this one https://code.launchpad.net/~andrew-hayzen/ubuntu-ui-toolkit/fix-swipe-delete-002/+merge/202171 | 13:45 |
| dpm | ahayzen, you can try the SDK guys that are around - zsombi, timp, kalikiana, Kaleo... ^^ | 13:45 |
| ahayzen | vthompson, nik90, right this binding loops http://bazaar.launchpad.net/~andrew-hayzen/+junk/music-app-u1db-conversion/view/head:/MusicPlaylists.qml#L164 | 13:45 |
| ahayzen | dpm, cool | 13:46 |
| nik90 | ahayzen: wanted to ask how you test your fixes in UITK? As in are you able to build and test uitk? | 13:46 |
| ahayzen | nik90, i just overwrite the files with my modifications... | 13:46 |
| nik90 | ahayzen: ah | 13:46 |
| ahayzen | nik90, vthompson, then this is the output in the console MusicPlaylists.qml:155:17: QML CoverRow: Binding loop detected for property "covers" | 13:47 |
| vthompson | ahayzen, off the top of your head do you know what pkg I need to use U1DB-qml | 13:49 |
| ahayzen | vthompson, errr hang on | 13:50 |
| nik90 | vthompson: qtdeclarative5-u1db1.0 | 13:50 |
| ahayzen | vthompson, try qtdeclarative5-u1db1.0 | 13:50 |
| ahayzen | vthompson, note tht bits of it don't work due to the bugs | 13:50 |
| vthompson | nik90, ahayzen thx | 13:50 |
| ahayzen | vthompson, so u won't be able to see the tracks of a playlist | 13:50 |
| ahayzen | vthompson, the recent list isn't sorted by time | 13:51 |
| ahayzen | vthompson, and probably other bugs lol but u get the idea | 13:51 |
| kalikiana | nik90: those branches need reviewers in fact, as in people stepping up to do those reviews ^_~ | 13:51 |
| nik90 | kalikiana: I know..unfortunately I can help test the patches but not really the code itself since I am not good with c++ :/ | 13:52 |
| nik90 | ahayzen: perhaps there is something wrong with the playlistStorage.getPlaylistCovers(name) function? Since CoverRow component is used in the exact same way in other files | 13:53 |
| kalikiana | nik90: how about https://code.launchpad.net/~kalikiana/u1db-qt/newstylelog/+merge/183887 there's no c++ there | 13:53 |
| ahayzen | nik90, yeah thts wht i suspect but don't know wht yet lol | 13:53 |
| nik90 | may be try doing, component.onCompleted: console.log(playlistStorage.getPlaylistCovers(name)) | 13:54 |
| ahayzen | nik90, gd idea :) | 13:54 |
| * kalikiana waits for aquarius to come online to talk him into reviewing the other branches | 13:55 | |
| ahayzen | nik90, it comes back with a cover.. [file:///home/andy/.cache/media-art/album-ddbaf6bd108836d4afe269d86532a844-3847274bce1afa48001cb1e0d877a17c.jpg] | 13:55 |
| nik90 | ahayzen: hmm | 13:57 |
| ahayzen | nik90, its not something stupid like having two queries running on the same index at the same time? | 13:57 |
| vthompson | Could the binding loop be because of accessing playlistStorage? | 13:57 |
| ahayzen | vthompson, tht was my other thought | 13:57 |
| vthompson | Maybe comment out "playlist"'s count string | 13:58 |
| nik90 | vthompson: but ahayzen seems to have accessed playlistStorage in other places as well though | 13:58 |
| vthompson | nik90, yh just wondering if that's what's causing the loop | 13:58 |
| vthompson | ahayzen, I can't run the branch so I won't be of much help :) | 13:59 |
| ahayzen | vthompson, nik90, playlistStorage is where everything for it is stored http://bazaar.launchpad.net/~andrew-hayzen/+junk/music-app-u1db-conversion/view/head:/storage/PlaylistStorage.qml | 13:59 |
| ahayzen | vthompson, no worries we're still waiting for stuff upstream so probably shouldn't spend too long on it | 13:59 |
| nik90 | ahayzen: just a thought, the coverRow component assigns covers to Image source. So shouldn't the file path be "/home/andy/.cache/media-art/album-ddbaf6bd108836d4afe269d86532a844-3847274bce1afa48001cb1e0d877a17c.jpg" instead of it starting with file:/// ? | 14:00 |
| nik90 | ahayzen: or is this used in other places as well where that works | 14:01 |
| ahayzen | nik90, possibly....don't we just put Qt.ResolvedUrl() around it or something? | 14:01 |
| nik90 | ahayzen: nope we dont use Qt.resolvedurl() there | 14:01 |
| ahayzen | oh no we don't | 14:01 |
| nik90 | http://bazaar.launchpad.net/~andrew-hayzen/+junk/music-app-u1db-conversion/view/head:/common/CoverRow.qml | 14:01 |
| * ahayzen checks wht is happening in trunk | 14:02 | |
| vthompson | popey, just saw your blog post. Good taste in music! :) | 14:02 |
| popey | :D | 14:03 |
| ahayzen | nik90, i get this back in trunk [file:///home/andy/.cache/media-art/album-57bab982bcba3e65c33206f442a015db-ec0c33dc9346a3e53339f4bdcb13ee35.jpg] | 14:03 |
| popey | I was careful with the one I picked for that screenshot ㋛ | 14:03 |
| nik90 | popey: lol...btw nice new blog theme | 14:04 |
| nik90 | seems like everyone is changing their blog theme recently | 14:04 |
| ahayzen | popey, hehe...wait until the scrobbling works ;) then u'll have to be real careful wht u choose | 14:04 |
| nik90 | webupd8, iloveubuntu.net, popey etc etc | 14:04 |
| nik90 | ahayzen: ah ok... | 14:05 |
| ahayzen | vthompson, did u get a chance to look at https://code.launchpad.net/~andrew-hayzen/music-app/convergence-prototype-a | 14:05 |
| ahayzen | nik90, i'll come back to it another time i've been staring at it for too long lol | 14:05 |
| nik90 | +1 | 14:05 |
| popey | haha ahayzen | 14:05 |
| popey | nik90: i saw someone else use it, it's just the default 2014 wordpress one :D | 14:06 |
| vthompson | ahayzen, yep. Good start. The real hard part will be introducing panels. Filemanger might have a good example for us to use however | 14:06 |
| ahayzen | vthompson, yeah tht'll be me next branch hopefully :) | 14:07 |
| ahayzen | vthompson, do u think i need to do anything else to https://code.launchpad.net/~andrew-hayzen/music-app/convergence-prototype-a before merging? | 14:07 |
| ahayzen | vthompson, i'm trying to do the transition in small steps to make it easier | 14:07 |
| vthompson | ahayzen, not sure. Maybe propose a merge but leave it as a WIP so we can get a quick read of the diffs | 14:08 |
| ahayzen | vthompson, cool | 14:08 |
| vthompson | ahayzen, might also be good to get Jenkins to see it too | 14:09 |
| ahayzen | vthompson, hehe | 14:09 |
| ahayzen | vthompson, hmmm more to the fact how would we test convergence in jenkins? | 14:09 |
| ahayzen | vthompson, can we have a test resize the window? then check the wideAspect bool? | 14:09 |
| WebbyIT | popey, on calc blueprint you wrote "Ensure keypad does not get disappear when labels being entered". What does it mean? | 14:10 |
| vthompson | ahayzen, yea, that's a fair point. I think jenkins starts the app in the size you specify, ie how we'd start in the desktop. I don't know what Filemanager tests--it starts in tablet/desktop mode | 14:10 |
| mzanetti | dpm: sorry. I have to revert you commits | 14:10 |
| popey | WebbyIT: lemme see | 14:10 |
| mzanetti | dpm: seeing that sergio's commit broke lots of stuff | 14:10 |
| dpm | mzanetti, no worries, committing directly was a mistake anyway | 14:11 |
| ahayzen | vthompson, will i need to wait for Jenkins to run before marking as WIP? https://code.launchpad.net/~andrew-hayzen/music-app/convergence-prototype-a/+merge/203070 | 14:11 |
| vthompson | ahayzen, not sure. you can leave it proposed for now | 14:12 |
| ahayzen | vthompson, cool, its actually quite a small diff :) | 14:12 |
| popey | WebbyIT: http://imgur.com/q209FFc - when you click a label, the OSK/keypad disappears, which looks odd. | 14:12 |
| popey | (on desktop) | 14:12 |
| vthompson | ahayzen, it might be interesting to let it pass, then to specify a wide width to see what would fail at the moment | 14:13 |
| WebbyIT | popey, so, on desktop has to not disappear, but on phone yes? | 14:13 |
| popey | WebbyIT: well, on phone it switches to OSK, on desktop we don't have OSX, so need to keep keypad on | 14:14 |
| ahayzen | vthompson, it would fail with trying to get to anything on the smaller toolbar at a guess | 14:14 |
| WebbyIT | there is a simple way to know if app runs on desktop on or phone? | 14:14 |
| ahayzen | heh OSX | 14:14 |
| vthompson | ahayzen, would assume the same | 14:14 |
| vthompson | ahayzen, I think this should go in as well | 14:15 |
| vthompson | https://code.launchpad.net/~vthompson/music-app/sheet-size-on-desktop | 14:15 |
| ahayzen | vthompson, yep :) | 14:15 |
| ahayzen | vthompson, u want me to top-approve? | 14:15 |
| vthompson | ahayzen, sure thing | 14:15 |
| popey | WebbyIT: not really that I'm aware yet | 14:18 |
| WebbyIT | popey, mhh, I'll try to find something | 14:19 |
| vthompson | popey, dpm, mhall119, etc. I think one nice thing would be to assure that all fixes currently in launchpad for core apps are available in the click store and/or the Trusty builds. Music app already has one dup bug that's been fixed--I expect we'll get a few | 14:19 |
| popey | I was about to say the same thing | 14:20 |
| dpm | +1 | 14:20 |
| popey | Just let us know when you want us to crank a build out | 14:20 |
| vthompson | ahayzen, what do you think? Push to the store now? we don't really have any merges waiting for phone UI. | 14:21 |
| ahayzen | vthompson, when was the last click build? | 14:21 |
| vthompson | Just in case ppl want to hack on our app next week as well | 14:21 |
| vthompson | LONG time ago | 14:21 |
| ahayzen | vthompson, do it :) | 14:21 |
| vthompson | Last was r273 from launchpad... so not that long ago Dec 11.. but long enough ago | 14:22 |
| popey | 245 is the one on my phone | 14:24 |
| dpm | vthompson, popey, I agree, but I'd just wait for a few minutes for Jenkins to merge this one: https://code.launchpad.net/~vthompson/music-app/sheet-size-on-desktop/+merge/202001 | 14:24 |
| vthompson | dpm, sounds good. That mp should be transparent to the N4, but I agree. | 14:25 |
| popey | phablet@ubuntu-phablet:~$ click list music | grep com.ubuntu.music | 14:25 |
| popey | com.ubuntu.music 1.1.245 | 14:25 |
| dpm | vthompson, yeah, if it were to fail the tests, then we go for the click store update straight away and fix that branch later | 14:26 |
| vthompson | popey, odd. root@ubuntu-phablet:/usr/share/click/preinstalled/com.ubuntu.music# click list music | grep com.ubuntu.music | 14:26 |
| vthompson | com.ubuntu.music1.1.273 | 14:26 |
| ahayzen | vthompson, my uni internet just crashed so if u said anything i would have missed it :P | 14:26 |
| popey | haha | 14:26 |
| popey | look at all the updates on my phone pending | 14:26 |
| popey | my bad | 14:26 |
| vthompson | ahayzen, saw you drop. I didn't say anything. Your campus internet is worse that ours was in the states.. that's saying something :) | 14:27 |
| dpm | :) | 14:27 |
| popey | http://popey.com/~alan/phablet/device-2014-01-24-142705.png | 14:27 |
| vthompson | popey, for shame! | 14:27 |
| popey | check that out! | 14:27 |
| dpm | what a bad person you are... | 14:27 |
| ahayzen | vthompson, hehe it is 10 up/down when it works in halls :) and 100 up/down on campus | 14:27 |
| popey | its because I flashed clean then pushed a load of clicks from my laptop then did "pkcon install-local *.click" | 14:27 |
| popey | clearly had old clicks on my laptop | 14:27 |
| popey | good way to test updates though *ahem* | 14:28 |
| ahayzen | vthompson, we have a pass from Jenkins https://code.launchpad.net/~andrew-hayzen/music-app/convergence-prototype-a/+merge/203070 | 14:29 |
| vthompson | ahayzen, now the question is do we want to use your branch to default to a wide mode on the desktop? I use the app on my desktop all the time, so it might be nice... BUT it'd probably also require you to make AP exceptions for the "wideAspect" property? | 14:32 |
| ahayzen | vthompson, hmm yeah how would we do tht? | 14:32 |
| vthompson | ahayzen, specifying a width in MainView IIRC | 14:33 |
| ahayzen | vthompson, do we have the API to figure out if ur on a touch/pointer device | 14:33 |
| ahayzen | vthompson, oh cause on the mobile it'll just fill the screen | 14:33 |
| vthompson | ahayzen, yep, and on the tablet... I assume | 14:34 |
| dpm | popey, when you ping sergiusens for a store update for music, can you check with him if there is a way to upload the reminders app without having to wait for a conclusion on the frameworks discussion? I've got a hunch it's going to take a while otherwise | 14:34 |
| popey | dpm: its already uploaded | 14:34 |
| popey | dpm: its blocked on me, i can let it through any time | 14:34 |
| popey | if you want | 14:34 |
| ahayzen | vthompson, but this is definitely gonna cause issues with autopilot...how has the file manager done it? | 14:35 |
| dpm | popey, ah, yeah, let's upload it. Or were you waiting on anything else? | 14:35 |
| popey | how about we get music updated and push both together, nice end to the day today ☻ | 14:35 |
| popey | no, just testing | 14:35 |
| dpm | popey, sounds like a plan | 14:35 |
| popey | kk | 14:35 |
| popey | will do | 14:35 |
| dpm | thanks \o/ | 14:35 |
| popey | once we get the nod from vthompson about when to push music | 14:35 |
| vthompson | ahayzen, not sure. I would assume they check a property | 14:35 |
| ahayzen | vthompson, i've just pushed changes to the size so we'll see wht fails | 14:36 |
| vthompson | popey, I think pushing would be good | 14:36 |
| popey | ok | 14:37 |
| vthompson | ahayzen, just tested. I like this ratio: width: units.gu(125) | 14:37 |
| vthompson | height: units.gu(80) | 14:37 |
| ahayzen | vthompson, i just did w100/h80 | 14:38 |
| vthompson | ahayzen, that's probably good too | 14:38 |
| ahayzen | vthompson, wht resolution is ur screen? | 14:39 |
| vthompson | 1680/1200ish | 14:39 |
| ahayzen | vthompson, mine is 1366x768 so tht nicely fills the centre :) | 14:40 |
| vthompson | ahayzen, I lied 1680x1050 | 14:40 |
| vthompson | ahayzen, not bad! only 1 AP test to do some patchwork to! | 14:54 |
| ahayzen | vthompson, \o/ it is the one tht tests the play/pause on the small toolbar | 14:55 |
| ahayzen | vthompson, but surely we want to test both cases? desktop and mobile mode? | 14:56 |
| vthompson | ahayzen, Probably. But I'm not sure how we'd do so. | 14:58 |
| vthompson | ahayzen, I've seen AP accidentally resize the window... so maybe we could intentionally do so | 14:59 |
| ahayzen | vthompson, hmmm i think we need to talk to some autopilot people about wht we should do about this | 15:00 |
| dpm | mzanetti, what's the status of the reminders-app branch right now? Are you working on cmake changes? Reverting the .pot file made the translations in LP disappear, and if it's going to take a while for the rest of the fixes to land I'd send a simple MP that only touches the .pot file to have translations back | 15:02 |
| vthompson | dpm, popey, what do you think? I'd rather we start in wide mode on the desktop--because I'm self-serving and use the app frequently--and then work on getting an AP test to get Jenkins to test phone mode. The builds will still test phone mode stuff... | 15:02 |
| mzanetti | dpm: on it | 15:02 |
| dpm | mzanetti, ok, thanks! | 15:03 |
| vthompson | dpm, popey, addendum to above. I mean wait to get an AP that Jenkins can use to test both modes | 15:05 |
| dpm | vthompson, what do you exactly mean with "we start", though? (sorry if I'm asking something obvious, I want to make sure I understand) | 15:05 |
| popey | i agree | 15:05 |
| popey | when the app is opened on desktop it should be wider than tall, or full screen by default | 15:05 |
| vthompson | dpm, I mean launch the app in wide mode on the desktop. This means that Jenkins will also be testing the app in this mode | 15:05 |
| popey | full screen is probably the easy/dirty way to do it | 15:05 |
| vthompson | popey, ahayzen has found a nice ratio. | 15:06 |
| ahayzen | popey, but how do we test the different buttons/labels that are present in the different modes? | 15:06 |
| popey | ahayzen: ---> balloons is that way | 15:06 |
| ahayzen | popey, we can only test the default size unless we can resize the window somehow | 15:06 |
| popey | right | 15:06 |
| dpm | ahayzen, that's a really good question this might be a good one for fginther and balloons | 15:06 |
| vthompson | popey, ahayzen, balloons, I think the proper way would be to check one or more properties that specify which mode is currently being used | 15:06 |
| balloons | ahayzen, we can open the different sizes | 15:06 |
| popey | happy for it to be a fixed size of some kind. | 15:06 |
| balloons | yes, we do that for filemanager for instance | 15:07 |
| james_w | hello, it appears that there's no field in the manifest of a click package for the application name. Should that be taken from a desktop file instead? | 15:07 |
| balloons | the test changes depending on if we're in wide mode or not | 15:07 |
| vthompson | balloons, so Jenkins tests both modes somehow? | 15:07 |
| balloons | vthompson, atm jenkins merge will test wide mode, the lab tests phone mode | 15:08 |
| vthompson | balloons, ok, ahayzen let's do the same | 15:08 |
| ahayzen | balloons, vthompson how would we implement this? | 15:08 |
| vthompson | balloons, This scares me a bit because we can break the phone and jenkins wont know and we won't know until an image is built | 15:08 |
| balloons | once running in the lab it should run fine on both.. We simply check the flag in filemanager to see what UI it is running | 15:08 |
| balloons | vthompson, yes, that is possible. But if it happens we'll revert | 15:09 |
| vthompson | ahayzen, in the test you'd do an if wideMode else | 15:09 |
| balloons | we do test on the phone before pushing manually | 15:09 |
| ahayzen | balloons, ah so does Jenkins force the window size? | 15:09 |
| popey | james_w: not "title"? | 15:09 |
| balloons | ahayzen, jenkins tests in desktop mode.. We could ask fginther if we could also have it run a second set of tests (maybe only during autolanding) on a small window size | 15:10 |
| popey | james_w: and "name" | 15:10 |
| james_w | popey: name is package name right? Not user-friendly display name? | 15:10 |
| ahayzen | balloons, tht would be useful | 15:10 |
| popey | james_w: my manifest.json has both name and title., yes, name is com.popey.appname, title is Appname | 15:10 |
| fginther | balloons, can't the test itself launch the application in a specific mode? | 15:10 |
| james_w | popey: hmm | 15:10 |
| popey | james_w: also, description | 15:10 |
| james_w | popey: then my question becomes where I should get the "tagline" from :) | 15:10 |
| popey | james_w: http://paste.ubuntu.com/6808794/ for example | 15:11 |
| ahayzen | fginther, how would we do tht? because tht would be the best way | 15:11 |
| vthompson | fginther, balloons the other idea would be to have jenkins resize | 15:11 |
| popey | ooh | 15:11 |
| james_w | "title: short (one-line) synopsis of the application" | 15:11 |
| vthompson | fginther, balloons, or rather AP resize | 15:11 |
| james_w | which to me is the "tagline" not the "app name" | 15:11 |
| james_w | I have these holes to fill for submission: package name, app name, tagline, description | 15:11 |
| balloons | fginther, in theory it could.. but if we could limit the xserver screen size that would force all the apps to behave in the smaller size. Which is probably better | 15:11 |
| james_w | the first and last are easy | 15:12 |
| james_w | I only have one element to put in the middle two, and I'm not sure which it is | 15:12 |
| popey | i dont think I have seen tagline anywhere other than the website | 15:12 |
| popey | so app name. | 15:12 |
| fginther | balloons, is the application forced to fit within the xserver window? Do we know if this will work? :-) | 15:13 |
| balloons | fginther, I don't know for sure, I'm curious :-) | 15:14 |
| james_w | popey: ok, thanks | 15:14 |
| popey | np | 15:14 |
| fginther | balloons, it's an interesting idea. If it works, we should just need to duplicate the existing jenkins job and start the xserver with the limited size. Then run both jobs for the apps that want to do the small window testing | 15:15 |
| mzanetti | dpm: ok. ready to approve your translation branch after you push the last fixes | 15:17 |
| balloons | fginther, right.. I would suggest we do it for all apps. But consider only running the tests twice as part of autolanding, or maybe that would be confusing. I wouldn't want to slow down the jobs needlessly | 15:17 |
| mzanetti | dpm: DO NOT use :parent :P | 15:18 |
| fginther | balloons, I would not worry about slowing down the jobs, that problem is solved by scaling out | 15:18 |
| balloons | true :-) | 15:18 |
| balloons | then yea, let's do it | 15:18 |
| dpm | mzanetti, I'll stay well away from it :) | 15:19 |
| ahayzen | vthompson, lunch brb | 15:21 |
| nik90 | renato: I have some EDS bugs for you :) | 15:44 |
| renato | I saw it | 15:44 |
| renato | thanks for report it | 15:44 |
| nik90 | popey: hey I got a design question (usability) for you. | 15:44 |
| PaoloRotolo | Guys, how to import music in the music-app? | 15:44 |
| nik90 | renato: let me know if you need any logs for the bug reports | 15:44 |
| nik90 | vthompson: ^^ | 15:45 |
| popey | nik90: go | 15:45 |
| nik90 | popey: In the add alarms page, we have options to set an alarm daily, weekly and one time. But isn't daily a special case of weeky? | 15:45 |
| popey | PaoloRotolo: you have a phone connected via USB? if so, just drag it over via mtp in nautilus | 15:45 |
| nik90 | popey: since when you choose weekly and then select all days of the week, it is actually a daily alarm | 15:46 |
| nik90 | perhaps that can be removed | 15:46 |
| nik90 | popey: stefano made a small gif mockup https://launchpadlibrarian.net/162723769/proposal2.gif | 15:47 |
| popey | nik90: ios has just one repeat option | 15:47 |
| popey | you pick the days and thats it | 15:47 |
| PaoloRotolo | Hi popey! No, unfortunately I've only a Nexus 7 (2013), still not supported by Ubuntu Touch... | 15:47 |
| popey | http://imgur.com/PkUXIxs | 15:47 |
| popey | nik90: ^ | 15:48 |
| nik90 | popey: how do you know if it is repeating for 1 week or for the entire year? | 15:48 |
| popey | you dont get that option | 15:48 |
| popey | its on or off on the previous screen | 15:48 |
| === gatox is now known as gatox_lunch | ||
| popey | nik90: http://imgur.com/PqOHQcu,WDuWWtW | 15:49 |
| nik90 | popey: ok..so technically that mockup by stefano is good | 15:50 |
| popey | yeah. | 15:50 |
| nik90 | popey: In the repeat alarm page, you can choose several values? | 15:50 |
| popey | yes | 15:54 |
| popey | nik90: same as yours, individually flip them on and off | 15:55 |
| * popey cancells before he sets himself some ludicrous alarms | 15:55 | |
| nik90 | ok | 15:56 |
| PaoloRotolo | popey, can I do that on desktop too? | 15:57 |
| popey | PaoloRotolo: if you put music in ~/Music and have mediascanner installed, yes, it should | 15:58 |
| popey | vthompson: thats right isn't it? | 15:58 |
| PaoloRotolo | popey, I've just installed mediascanner but it doesn't work... | 16:00 |
| popey | is it running? | 16:01 |
| vthompson | Hey PabloRubianes | 16:04 |
| PabloRubianes | vthompson: hello | 16:04 |
| vthompson | PabloRubianes, do a "ps -ef|grep mediascanner-service" | 16:04 |
| PabloRubianes | vthompson: I am in my work in windows :S | 16:04 |
| PaoloRotolo | popey, oh, right, I've to run mediascanner-service. Thanks :D | 16:04 |
| vthompson | If you reboot it should run | 16:05 |
| PabloRubianes | and yes I know I need to change of work vthompson | 16:05 |
| vthompson | Otherwise you can do "mediascanner-service ~/Music" | 16:05 |
| vthompson | PabloRubianes, ha. My work's workstation is a Windows PC as well | 16:05 |
| renato | nik90, I added some comments on your bug | 16:08 |
| renato | nik90, would be nice if you can isolate the bug | 16:10 |
| renato | nik90, btw I have added the SDK project into the affected projects since this affect the SDK alarm components | 16:10 |
| renato | and the problem could be on the component too | 16:11 |
| Guest___ | What's next if my merge request is approved? | 16:30 |
| popey | Guest___: it gets tested by jenkins, and lands in trunk if all is well. | 16:33 |
| popey | then we break out the champagne and celebrate | 16:33 |
| nik90 | renato: I will try to reproduce this from a sample app. I remember creating one sometime back. | 16:44 |
| mzanetti | popey: what do you think? http://i.imgur.com/mk2YEqN.png | 16:51 |
| === gatox_lunch is now known as gatox | ||
| rottinrob | morning everyone | 16:54 |
| popey | mzanetti: wow, thats quite green! | 17:00 |
| mzanetti | :D | 17:00 |
| popey | is it their green? | 17:00 |
| mzanetti | yeah | 17:00 |
| popey | Well it's delightful then ☻ | 17:00 |
| mzanetti | you think I should file a merge request for it? | 17:00 |
| * mzanetti likes | 17:00 | |
| mzanetti | and it makes the right edge animation cooler if there is some color in there :D | 17:01 |
| popey | I am not sure I can speak authoritatively on matters of design. | 17:02 |
| popey | but propose a merge and I will wave it at design on monday | 17:03 |
| dpm | fginther, would you happen to know why this branch hasn't landed despite being approved? -> https://code.launchpad.net/~dpm/reminders-app/update-translations/+merge/203030 | 17:17 |
| fginther | dpm, the merge is failing with: Server sent an unexpected error: ('error', 'GhostRevisionsHaveNoRevno', 'Could not determine revno for {david.planella@ubuntu.com-20140124121210-n7rr1k4ezcznmdgw} because its ancestry shows a ghost at {david.planella@ubuntu.com-20140124121210-n7rr1k4ezcznmdgw}') | 17:19 |
| fginther | Build step 'Execute shell' marked build as failure | 17:19 |
| dpm | "an ancestry shows a ghost" | 17:19 |
| dpm | that's scary stuff | 17:19 |
| fginther | http://wiki.bazaar.canonical.com/GhostRevision | 17:20 |
| WebbyIT | There is no castle for this ghost? Poor ghost! | 17:20 |
| dpm | mzanetti, how did you revert the changes in trunk? Did you delete revisions or did you overwrite revisions? ^^ | 17:20 |
| dpm | "revision ghosts are just like real ghosts (such as they are) in that their discorporeal existence typically causes little, if any, harm. " I'm relieved now | 17:21 |
| mzanetti | lol | 17:21 |
| mzanetti | right... yeah, I did uncommit and push overwrite | 17:22 |
| fginther | dpm, there still exists the problem that this branch won't merge automatically. You might need to create a new branch and resubmit | 17:22 |
| dpm | yeah, will do that, thanks fginther | 17:22 |
| mzanetti | thanks fginther | 17:22 |
| fginther | dpm, mzanetti, no problem, may I change https://code.launchpad.net/~dpm/reminders-app/update-translations/+merge/203030 to something other than approved? | 17:23 |
| fginther | dpm, jenkins is stuck in a loop trying to merge this in | 17:23 |
| dpm | fginther, sure | 17:23 |
| mzanetti | fginther: sure. disapprove it, we'll resubmit | 17:23 |
| mzanetti | sorry dpm | 17:23 |
| fginther | I marked as rejected | 17:23 |
| dpm | mzanetti, no worries, it's fixable | 17:24 |
| rottinrob | good morning all | 17:24 |
| rottinrob | mzanetti...david mentioned a emulator to run the reminders app...where can I find this emulator | 17:26 |
| samertm | rottinrob: there's a guide here http://davidplanella.org/ubuntu-emulator-quickstart-guide/ | 17:26 |
| samertm | it depends on which version of ubuntu you have | 17:26 |
| rottinrob | thank you samertm | 17:27 |
| samertm | and speaking of that, if we want to work on the Core apps, is it sufficient to have the emulator or should we be running Trusty Tahr? | 17:27 |
| === Ursinha is now known as Ursinha-afk | ||
| popey | samertm: personally i run on trusty | 17:31 |
| samertm | popey: hey! I saw you on the ubuntu engineering webcast last wednesday! | 17:31 |
| samertm | that was a lot of fun | 17:31 |
| samertm | is there a way I can install trusty tahr beside 13.10 without dual-booting? | 17:32 |
| popey | well, you can run 13.10 | 17:32 |
| dpm | rottinrob, I did mention it, sorry I didn't follow up, it's been a bit hectic this last couple of days | 17:32 |
| popey | that should work fine | 17:32 |
| dpm | rottinrob, the guide above should get you started, it's what I wanted to send you | 17:32 |
| WebbyIT | samertm, I'm on 13.10 and have no problems, but remembert to add the right PPA ;) | 17:33 |
| popey | samertm: thanks for the nice comments ☻ | 17:34 |
| rottinrob | no worries...I was just wanting to set this up and try to get back up to speed | 17:36 |
| nik90 | popey: I was following your policy of staying on the stable release before upgrading :) ... you upgrade too early | 17:38 |
| popey | hah | 17:38 |
| popey | I upgraded through peer pressure | 17:38 |
| popey | I'd have been happy staying on 12.04! :D | 17:38 |
| nik90 | popey: oh great let's blame mhall119 and dpm :) | 17:38 |
| mhall119 | no no no, always blame popey | 17:39 |
| dpm | I'm innocent: I'm using both 13.10 and trusty! :) | 17:39 |
| samertm | How do active developers set up 14.04 for testing? Do they use a chroot? | 17:39 |
| samertm | what is the most convenient way for me to run the latest 14.04 build :P | 17:40 |
| popey | samertm: most just upgrade from 13.10 to 14.04 tbh | 17:40 |
| dpm | yeah | 17:40 |
| popey | 14.04 is pretty stable | 17:40 |
| samertm | ah kk | 17:40 |
| vthompson | samertm, I run a VM | 17:40 |
| popey | he says with a 4GB xorg process | 17:40 |
| dpm | I was going to say that too, a VM is another option | 17:40 |
| samertm | I'll try a VM first | 17:40 |
| samertm | ty | 17:40 |
| vthompson | samertm, np | 17:41 |
| nik90 | mhall119, dpm: :) | 17:41 |
| rottinrob | which is the best way to upgrade from 13.10 to Trusty? | 17:44 |
| mhall119 | update-manager -d | 17:44 |
| rottinrob | kk...ty | 17:44 |
| dpm | mzanetti, would you mind having a look to https://code.launchpad.net/~dpm/reminders-app/add-pot/+merge/203136 - I've kept the changes to the minimum and will wait until the cmake cleanup is done before doing more i18n-related changes | 17:44 |
| mzanetti | dpm: approved | 17:45 |
| dpm | awesome, thanks | 17:45 |
| mzanetti | dpm: https://code.launchpad.net/~mzanetti/reminders-app/change-edit-page/+merge/203139 | 17:50 |
| dpm | mzanetti, on it | 17:51 |
| dpm | btw guys, if someone is looking for an app to help with, reviewing reminders-app is an area where we need help with: https://code.launchpad.net/reminders-app/+activereviews | 17:52 |
| * dpm likes cmake's indication of percentage built | 17:55 | |
| jumper | hi all | 17:55 |
| popey | hi | 17:56 |
| mzanetti | dpm: hey, I think its not possible yet to open the camera in order to take a picture. I will implement a minimalistic camera view myself, ok? | 18:13 |
| dpm | mzanetti, ack, makes sense then | 18:14 |
| balloons | ahayzen, did the autopilot bug you found for your merge from a bit ago get solved yet? | 18:15 |
| samertm | hey all | 18:29 |
| nik90 | samertm: hi | 18:29 |
| samertm | how long does it take for the ubuntu-emulator to work? It's been on a black screen for a while. | 18:29 |
| samertm | and it's launched the ubuntu-phablet terminal | 18:30 |
| nik90 | ooh I havent tried the emulator yet to be helpful.sry | 18:30 |
| samertm | haha that's k | 18:30 |
| dpm | samertm, it's in the order of minutes. Just check that you don't have other apps consuming a lot of memory or CPU, which tends either to kill or slow down the emulator | 18:33 |
| dpm | if after 10 mins the emulator still shows the black screens it's a good signal it's better to restart it :) | 18:33 |
| samertm | dpm: haha it's been about 15, but my cpu is still at 100% | 18:34 |
| samertm | so I assume something is happening | 18:34 |
| dpm | yeah, the PC is probably good as heating :) | 18:35 |
| samertm | the bright side :P | 18:35 |
| ahayzen | vthompson, why hasn't Jenkins run on https://code.launchpad.net/~bobo-324/music-app/fix1272151and1272113/+merge/203074 | 18:36 |
| vthompson | ahayzen, I think it only does so when top approved if you are not on the team | 18:37 |
| ahayzen | balloons, it was solved but i don't think the merge has been approved | 18:37 |
| ahayzen | vthompson, ah ok | 18:37 |
| ahayzen | vthompson, shall i top approve? | 18:37 |
| balloons | ahayzen, what vthompson said is correct | 18:37 |
| vthompson | ahayzen, I just did | 18:37 |
| ahayzen | vthompson, cool | 18:37 |
| ahayzen | balloons, i'll find the autopilot MP... | 18:38 |
| ahayzen | balloons, this one https://code.launchpad.net/~thomir/autopilot-qt/export-qobject/+merge/201865 | 18:38 |
| ahayzen | vthompson, wht was the result of the discussions about screen size? | 18:38 |
| vthompson | ahayzen, from a testing perspective? | 18:39 |
| ahayzen | vthompson, yeah | 18:39 |
| vthompson | ahayzen, I assume we just want to make the test detect wideMode and handle both | 18:39 |
| ahayzen | vthompson, ok i'll add tht in | 18:39 |
| vthompson | ahayzen, Jenkins and desktop (by default) will run in wide mode and the image builds will test phone mode | 18:39 |
| vthompson | ahayzen, anything more than that can be worked out soon here | 18:40 |
| ahayzen | vthompson, cool and we can do manual autopilot on device | 18:40 |
| ahayzen | vthompson, u have the script to do tht? | 18:40 |
| vthompson | yep | 18:40 |
| vthompson | ah | 18:40 |
| vthompson | ahayzen, do a "debuild -uc -us" from the music-app root directory | 18:41 |
| ahayzen | vthompson, no there is one which runs autopilot on the device from a bzr branch for u | 18:41 |
| ahayzen | vthompson, http://paste.ubuntu.com/6721815/ | 18:41 |
| ahayzen | vthompson, then just run for example ... script.sh lp:music-app music_app | 18:42 |
| ahayzen | vthompson, it doesn't even need a writeable image which is cool | 18:42 |
| vthompson | ahayzen, I see and then you can also do something like " music_app.tests.test_music.TestMainWindow.test_create_playlist_from_songs_tab" as the second param to run just one test | 18:42 |
| vthompson | ahayzen, I've never done it that way | 18:43 |
| ahayzen | vthompson, yep | 18:43 |
| samertm | dpm: I waited way too long the first time | 18:43 |
| vthompson | ahayzen, I know you've shown me the script before... just havent used it/saved it away | 18:43 |
| samertm | I restarted the emulator and it took ~5 minutes to launch | 18:43 |
| ahayzen | vthompson, :) | 18:44 |
| ahayzen | vthompson, also we found if u comment out the lines that run the tests.... if u then restart the device the music-app is then replaced with tht one ...so it can be used to deploy apps | 18:44 |
| ahayzen | vthompson, i usually do script.sh lp:music-app && adb reboot to test the app now | 18:45 |
| vthompson | ahayzen, neat. I might still do it the old way as I don't want to deploy if I'm testing. But this is a neat way to deploy | 18:46 |
| ahayzen | vthompson, yep | 18:46 |
| ahayzen | vthompson, it just means u don't have to make the image writeable | 18:47 |
| dpm | samertm, ah, cool (as "cool" as 5 minutes can be :) | 18:47 |
| vthompson | ahayzen, speaking of deploying: http://i.imgur.com/0kRBfFu.png | 18:48 |
| dpm | I need to call it a day, nice work everyone, good hack day, and it's still not over! :-) | 18:48 |
| vthompson | ahayzen, woot | 18:48 |
| samertm | dpm: it was nice to meet you! | 18:48 |
| samertm | cya | 18:48 |
| ahayzen | vthompson, 321! | 18:49 |
| nik90 | dpm: enjoy your evening | 18:49 |
| dpm | see you, have fun :) | 18:49 |
| dpm | thanks nik90, samertm | 18:49 |
| doflah | are there any hack day tasks I can help out on? | 18:50 |
| ahayzen | vthompson, we need a commit msg https://code.launchpad.net/~bobo-324/music-app/fix1272151and1272113/+merge/203074 | 18:52 |
| vthompson | ahayzen, just added one | 18:54 |
| ahayzen | doflah, there are a few links on these blog posts http://popey.com/blog/2014/01/24/january-2014-core-apps-hack-day-one-reminders-and-music/ http://davidplanella.org/ubuntu-core-apps-hack-days-back/ | 18:57 |
| ahayzen | doflah, also searching for bugs which have the tag hackday or bitesize within the coreapps | 18:57 |
| ahayzen | vthompson, passes on the Nexus 4 now for Jenkins ... https://code.launchpad.net/~andrew-hayzen/music-app/convergence-prototype-a/+merge/203070 :) | 19:04 |
| ahayzen | vthompson, passed on Jenkins as well https://code.launchpad.net/~andrew-hayzen/music-app/convergence-prototype-a/+merge/203070 | 19:22 |
| ahayzen | vthompson, brb | 19:22 |
| vthompson | ahayzen, I get one failure, but it might be the same playlists test fluke seen earlier in the #ubuntu-touch room | 19:26 |
| vthompson | ahayzen, I need to step out. I'll be back in an hour or so | 19:27 |
| samertm | this might be a dumb question | 19:52 |
| samertm | but how do I run the music-app on the ubuntu phone emulator after I grab the code from launchpad? | 19:53 |
| balloons | samertm, you want to run code you've written?> | 19:56 |
| balloons | samertm, in general it's easier to run it locally on the desktop if you are trying to test things. Otherwise you can simply ssh in and run it as normal | 19:56 |
| balloons | bzr branch, qmlscene music-app.qml | 19:56 |
| samertm | ah, so I should run it from the ubuntu-phone terminal? | 19:58 |
| balloons | samertm, yep that should work out fine | 19:59 |
| samertm | thanks! | 19:59 |
| balloons | ahayzen, re: https://code.launchpad.net/~thomir/autopilot-qt/export-qobject/+merge/201865.. Cool, looks like it might be ready to go. I wonder if I can get someone to review outside of thomi | 20:00 |
| popey | balloons: do you have reminders-app on your device? | 20:12 |
| popey | or indeed mhall119 | 20:12 |
| balloons | popey, my device is down for the count atm | 20:14 |
| balloons | well, I do have the n10, I could try it on that | 20:14 |
| balloons | why do you ask? | 20:14 |
| popey | its in the store | 20:17 |
| popey | wanted someone to test install | 20:17 |
| popey | because i already have it | 20:17 |
| ahayzen | balloons, cool would be good to get tht landed | 20:32 |
| ahayzen | vthompson, back.... yh looks like we have a flaky test...which one is it in particular i may look to see if i can improve it | 20:32 |
| === Ursinha-afk is now known as Ursinha | ||
| samertm | where does music-app look for music? | 20:39 |
| ahayzen | samertm, it uses the mediascanner are u on desktop/device? | 20:49 |
| samertm | ahayzen, desktop | 20:49 |
| ahayzen | samertm, if u say ran the following it would add ~/Music to the index... mediascanner-service ~/Music | 20:50 |
| ahayzen | samertm, assuming the package 'mediascanner' has been installed (we did have issues with this not being installed before) | 20:51 |
| samertm | ahayzen, it worked! | 20:51 |
| ahayzen | samertm, \o/ | 20:51 |
| samertm | and mediascanner wasn't installed | 20:51 |
| samertm | thanks! | 20:51 |
| ahayzen | samertm, np | 20:51 |
| samertm | should mediascanner be added to the touch-coreapps package? | 20:55 |
| ahayzen | samertm, there is/was a bug for it | 20:55 |
| * ahayzen tries to find | 20:55 | |
| ahayzen | samertm, https://bugs.launchpad.net/music-app/+bug/1241403 | 20:56 |
| ubot2` | Launchpad bug 1241403 in mediascanner "Music app's grilo plugin package needs to depend on mediascanner" [Undecided,New] | 20:56 |
| ahayzen | samertm, also we will be moving away from Grilo soon (hopefully) so this won't be an issue | 20:57 |
| doflah | ahayzen, what's replacing it? | 20:57 |
| ahayzen | doflah, there will be a media scanner service that we will talk directly to | 20:57 |
| ahayzen | doflah, and a media-hub for playing tracks | 20:58 |
| ahayzen | danielholm, o/ | 20:58 |
| samertm | ah, kk | 20:58 |
| samertm | about this bug: https://bugs.launchpad.net/music-app/+bug/1272155 | 21:12 |
| ubot2` | Launchpad bug 1272155 in Ubuntu Music App "Clicking a track from a sheet when it is currently playing pauses the track instead of taking the user to the Now Playing page" [Low,Triaged] | 21:12 |
| samertm | what is the correct behavior for clicking a track that is already playing | 21:12 |
| ahayzen | samertm, either for it to restart the track and refill the queue or to do nothing...but in both cases show the now playing | 21:13 |
| samertm | alright | 21:13 |
| samertm | thanks | 21:13 |
| ahayzen | samertm, could be a one liner ... | 21:13 |
| ahayzen | samertm, however i guess if it clicked in the now playing page it would need to pause | 21:14 |
| ahayzen | samertm, i suspect changing this else if statement to only be true if on the now playing page might work ... http://bazaar.launchpad.net/~music-app-dev/music-app/trunk/view/head:/music-app.qml#L447 | 21:15 |
| ahayzen | samertm, thts where i would start | 21:15 |
| samertm | thanks a bunch! | 21:16 |
| samertm | this is a brand new code base to me :P | 21:16 |
| === snwh_ is now known as snwh | ||
| ahayzen | samertm, hehe np just ping me, vthompson or danielholm if u need help | 21:16 |
| ahayzen | samertm, anyway i gtg, good luck i'll be around tomorrow if u need any help | 21:26 |
| === salem_ is now known as _salem | ||
| danielholm | anyone else having issues with logging in to Evernote from Online Accounts on Touch device? | 21:53 |
| vthompson | danielholm, I had tried a bit ago and recall that it was difficult. I'll try to give it ago again. | 22:00 |
| samertm | vthompson, I'm a new contributor and I think I've fixed a bug | 22:01 |
| samertm | how do I run the testing suite? | 22:01 |
| samertm | and do you want to look at my patch | 22:01 |
| popey | danielholm: you have a sandbox.evernote.com account? | 22:01 |
| popey | thats what it ties to | 22:01 |
| vthompson | samertm, push your code to launchpad when you want. do you have a device? | 22:01 |
| popey | (i.e. not a standard evernote.com account) | 22:02 |
| samertm | vthompson, I have the emulator | 22:02 |
| vthompson | samertm, I'm not sure how to test against the emulator, actually. If you push your code I'll run it through my device | 22:02 |
| mhall119 | popey: I don't see the reminders app for download | 22:02 |
| samertm | alright! | 22:03 |
| vthompson | mhall119, popey I thought you still had to push it from the project branch | 22:03 |
| mhall119 | 15:17 < popey> its in the store | 22:03 |
| vthompson | as I've looked for it in the click store recently as well | 22:04 |
| mhall119 | I assumed he was talking about reminders | 22:04 |
| popey | hmm, well, it should be | 22:10 |
| * popey pokes sergio | 22:11 | |
| popey | ok, should be live soon | 22:12 |
| vthompson | samertm, if you run this on the desktop or emulator one thing you'll notice is that the user can no longer pause a track by clicking on it in the Now Playing page. | 22:22 |
| daker | popey: responded, i can't make it :'( | 22:23 |
| samertm | vthompson, so it needs to be paused when you click on it on the now playing page? | 22:25 |
| popey | daker: ☹ | 22:26 |
| vthompson | samertm, that's my opinion. There probably isn't a spec saying we need to do so. Let's take a poll | 22:27 |
| samertm | I know on some media players, like the iphone, clicking the album art doesn't change whether or not the track is being played | 22:28 |
| samertm | it brings up the tool bar | 22:28 |
| samertm | which is the behavior it has right now | 22:28 |
| vthompson | mhall119, popey: samertm is fixing a bug I reported. When a user navigates an album, genre, playlist, etc, if he/she clicks on a track that happens to be the one playing in the list it pauses--even though they are not in the now playing page. I think this should bring up the Now Playing page. Now the question is we don't really have a spec for this, but we assumed that clicking on the large now playing cover art wo | 22:29 |
| vthompson | uld play/pause when clicked. What do you think? | 22:29 |
| popey | hmm, i can see clicking the currently playing track is an interesting use case. how often would you do that? once you do, and it pauses, would you be pleasantly surprised at discovering that feature | 22:32 |
| vthompson | samertm, IMO the main thing is the behavior for the sheet should be as you have it. However, the user should still be able to click and toggle play/pause in the now playing page | 22:32 |
| popey | I'm always a fan of having a quick way to get a phone to STFU | 22:32 |
| samertm | vthompson, alright, I can work on that | 22:33 |
| samertm | thanks for the input | 22:33 |
| popey | if I happen to have a list of tracks on screen and one of them is playing and i want to shut the phone up because some other thing needs my attention, it could be useful, yes. | 22:33 |
| vthompson | Well, let's talk about this... it was just our assumption | 22:33 |
| popey | are there any other music apps which do this? | 22:34 |
| samertm | hmm! | 22:34 |
| popey | its a novel behaviour | 22:34 |
| samertm | wait, how about this | 22:34 |
| vthompson | So, the google music app restarts the song if clicked from the queue | 22:34 |
| vthompson | the google play music app does nothing it seems | 22:35 |
| samertm | if you're using the music-app from the Now Playing screen and you click a song to start it, the song starts and the toolbar comes up | 22:35 |
| samertm | but if you drag the screen to look at the other songs, the tool bar goes down | 22:35 |
| samertm | so if the user wanted to get back the toolbar without pausing the song, they'll need to swipe up from the bottom | 22:36 |
| samertm | (if pressing the song being played pauses the track) | 22:36 |
| vthompson | Actually let me check on something. I think one reason I did it this way is due to someone's original desire to have an easy way to pause, etc, when driving | 22:36 |
| samertm | otherwise, clicking the currently playing song will bring up the toolbar | 22:36 |
| vthompson | samertm, yea, the swipe up to get to the currently playing track is a design decision (and a good one). It is a neat feature that clicking on the now playing item pops the toolbar | 22:38 |
| vthompson | samertm, I think it's a bit of an unprecedented feature we currently have, but I've come to depend upon it. | 22:41 |
| vthompson | danielholm, have you been reading along? | 22:41 |
| danielholm | vthompson: reading what? | 22:42 |
| danielholm | ah, | 22:42 |
| danielholm | danielholm: I can read up | 22:42 |
| vthompson | danielholm, do you click on the currently playing large cover art to play/pause at all? | 22:42 |
| danielholm | vthompson: nope. Not usually. | 22:42 |
| popey | tbh I didnt know that happened before you just mentioned it | 22:42 |
| danielholm | vthompson: maybe when I am in a hurry to stop the music. but usually I use the buttons for it | 22:43 |
| vthompson | Well, I might be outvoted. I use it all the time. (possibly because I introduced the feature :X) | 22:44 |
| popey | heh | 22:45 |
| danielholm | vthompson: ok. WOuld there be another use of clicking the large cover instead? | 22:45 |
| popey | zoom | 22:45 |
| popey | ? | 22:45 |
| vthompson | danielholm, maybe restarting the song from the beginning. That's what my Google Music app does | 22:45 |
| samertm | danielholm: right now it just brings up the tool bar. | 22:45 |
| vthompson | I think simply bringing up the toolbar is less useful | 22:46 |
| samertm | clicking on the album art on the default android music app does nothing | 22:47 |
| danielholm | vthompson, samertm hmm.. ok. Someone was asking for this? It might be an idea. But then again, I always drag the duration back to 0, or press back | 22:47 |
| vthompson | But given that no one else knew you could pause the song this way... only very slightly less useful :) | 22:47 |
| danielholm | vthompson: I am ok with it pausing. I dont think it should restart the track. back button should do that (right?) | 22:48 |
| vthompson | danielholm, yep after the time limit (5 or 10 secs IIRC) | 22:48 |
| danielholm | vthompson: exactly. so either keep the play/pause or make it do nothing? | 22:49 |
| vthompson | It'd just be extra work for samertm to retain this feature just in the Now Playing queue. | 22:49 |
| danielholm | aha | 22:49 |
| danielholm | do whats less work then, I suppose | 22:49 |
| vthompson | I think the consensus is that it could be useful, but isn't obvious to use. | 22:49 |
| vthompson | samertm, You could check what mode the app's in. There's a mode for the now playing page | 22:51 |
| samertm | vthompson, what do you mean? | 22:54 |
| vthompson | samertm, Easiest way to check for the now playing page being active might be to check if "currentPage == nowPlaying" | 22:54 |
| danielholm | vthompson, samertm: that the large cover can do different things based in page? | 22:55 |
| vthompson | If so then you'd allow the play/pause toggle | 22:55 |
| vthompson | danielholm, no, the mode is more for the state of the toolbar | 22:55 |
| danielholm | aha ok | 22:56 |
| samertm | Okay, so I'll add the play/pause toggle | 22:56 |
| samertm | thanks for the help! | 22:56 |
| vthompson | No worries, thanks for *your* help! | 22:57 |
| samertm | vthompson, do you know what player.playbackState represents? | 23:00 |
| vthompson | samertm, should be the play/paused state of the MediaPlayer component. | 23:00 |
| vthompson | this is the main object that plays the tunes | 23:00 |
| samertm | okay | 23:00 |
| === _salem is now known as salem_ | ||
| samertm | vthompson, I just pushed a branch which has the behavior you wanted | 23:36 |
| samertm | :D | 23:36 |
| === salem_ is now known as _salem | ||
| vthompson | samertm, good work! | 23:45 |
| samertm | vthompson, thanks! | 23:46 |
| vthompson | samertm, we have no current tests for this area. So there's no benefit to directly testing with the autopilot infrastructure. | 23:46 |
| vthompson | samertm, Assigned the bug to you and formally reviewing now | 23:48 |
| samertm | vthompson, awesome! | 23:48 |
| samertm | this is my first free software patch haha | 23:48 |
| vthompson | samertm, Ok. So what I'll let you do next is click on your branch and "Propose for merging" you'll then add a short description (which can be the same as the commit message) click on "Extra options" and then fill in the "Commit message" with something like "* Prevent pausing from elements that do not directly control playback" and click the "Propose Merge" button. | 23:51 |
| samertm | vthompson, should I include the bug number in the description or the commit? | 23:53 |
| vthompson | No need, it will be assigned when merged. We have a nice sophisticated bot that does that for us | 23:54 |
| samertm | alright, I just proposed the merge | 23:54 |
| vthompson | as you can see the bug is still related to the merge since it was related to the branch itself. | 23:55 |
| samertm | yup | 23:55 |
| vthompson | samertm, approved for merge! Feel free to take the time to look at the other apps as well in the oncoming days as they also could use some love! Thanks again! | 23:59 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
