=== duflu_ is now known as duflu | ||
=== 21WACCEWU is now known as Wug | ||
=== Wug is now known as Wug[bonk] | ||
=== Wug[bonk] is now known as 21WACCEWU | ||
mhr3_ | tsdgeos, ping? | 08:43 |
---|---|---|
tsdgeos | mhr3_: hi | 08:44 |
mhr3_ | tsdgeos, hey, last week saviq had an issue where the scope results were not showing up cause there was something broken in the renderer | 08:44 |
mhr3_ | tsdgeos, and we're seeing the same issue in new-scopes branch | 08:44 |
mhr3_ | tsdgeos, wonder if you know what the issue was | 08:45 |
tsdgeos | no :/ | 08:45 |
tsdgeos | mhr3_: is it white text vs white background? | 08:45 |
tsdgeos | or is it something more "broken"? | 08:45 |
mhr3_ | tsdgeos, no, cause there's no icons either | 08:45 |
tsdgeos | no idea then | 08:46 |
mhr3_ | tsdgeos, only the category header shows up and nothing else | 08:46 |
tsdgeos | but if you give me a way to reproduce it i can have a look | 08:46 |
mhr3_ | tsdgeos, just branch new-scopes and run the unity-scope-tool | 08:46 |
mhr3_ | perhaps run `restart scope-registry` in between | 08:46 |
mhr3_ | tsdgeos, oh plus the onlinemusic scope from the demo ppa | 08:47 |
tsdgeos | ok | 08:49 |
tsdgeos | mhr3_: "unity-scope-onlinemusic is already the newest version." ? | 08:51 |
tsdgeos | 0.1+14.04.20131029.1-0ubuntu1 ¿? | 08:51 |
tsdgeos | ah | 08:51 |
tsdgeos | silly me | 08:51 |
tsdgeos | missing an update | 08:51 |
mhr3_ | tsdgeos, you need this one https://launchpad.net/~unity-team/+archive/demo-stuff/+packages | 08:51 |
tsdgeos | yes yes | 08:51 |
tsdgeos | i always forget to update after add-apt-repository | 08:51 |
mhr3_ | tsdgeos, if you don't even see the category header then something else is wrong :) | 08:57 |
tsdgeos | so i see "Popular online" | 08:57 |
tsdgeos | and that's all | 08:57 |
tsdgeos | should i see more? | 08:57 |
mhr3_ | cool, yea that's the problem | 08:57 |
mhr3_ | you should also see the results | 08:57 |
tsdgeos | ok fix is easy | 08:59 |
tsdgeos | don't know if it's the correct fix yet though | 08:59 |
tsdgeos | http://paste.ubuntu.com/6908000/ | 08:59 |
tsdgeos | mhr3_: ↑↑↑ if it's blocking you | 08:59 |
mhr3_ | tsdgeos, yea, we mostly need to fix new-scopes | 08:59 |
mhr3_ | tsdgeos, can you explain why that messes stuff up? | 09:00 |
tsdgeos | because you are assigning to something that is readonly | 09:00 |
tsdgeos | and qml says | 09:00 |
tsdgeos | no | 09:00 |
tsdgeos | won't do that | 09:00 |
tsdgeos | and then all breaks | 09:00 |
tsdgeos | need to find out if it should not be readonly | 09:01 |
tsdgeos | or the assignment isn't neede | 09:01 |
tsdgeos | d | 09:01 |
mhr3_ | tsdgeos, so why is it there in the first place? isn't trunk's cardtool broken too then? | 09:01 |
tsdgeos | sure | 09:01 |
tsdgeos | i mean | 09:01 |
tsdgeos | no | 09:01 |
tsdgeos | because trunk isn't merged since ten years ago | 09:01 |
tsdgeos | because we have unreproducible autopilot errors in a phone we are not supposed to support anymore blocking stuff | 09:02 |
tsdgeos | yeah \o/ | 09:02 |
tsdgeos | mhr3_: but yes it is a mistake in some branch that we want to eventually merge to trunk | 09:02 |
tsdgeos | i'll hunt it down | 09:02 |
mhr3_ | tsdgeos, thx | 09:03 |
tsdgeos | karni: you there? | 09:05 |
tsdgeos | mhr3_: i'm going to propose a branch for merging to new-scopes, can you review it? Do you know if we have automerging in there or we manually merge stuff in it? | 09:07 |
mhr3_ | tsdgeos, manual merging afaik | 09:07 |
tsdgeos | mhr3_: https://code.launchpad.net/~aacid/unity8/no_readonly_headerheight/+merge/205538 | 09:14 |
tsdgeos | if you approve it i'll manually merge it | 09:14 |
mhr3_ | tsdgeos, acked | 09:15 |
tsdgeos | ok, merged | 09:18 |
mhr3_ | tsdgeos, thx | 09:21 |
mhr3_ | tsdgeos, do you know which category layouts are properly supported these days? | 09:21 |
tsdgeos | mhr3_: grid, carousel | 09:21 |
tsdgeos | i have coded all the rest, but they are not wired | 09:21 |
mhr3_ | tsdgeos, i'm looking at the designs and wondering if we can even do them :) | 09:21 |
tsdgeos | i.e. lots of qml glue is missing | 09:22 |
tsdgeos | mhr3_: Saviq told me about the addSpecialCategory thing you're working on, is it done, what's your timeframe if not | 09:22 |
mhr3_ | tsdgeos, can we do all of this for example? https://docs.google.com/a/canonical.com/file/d/0B-a_7E3tDxOgZDJvd3l4U2tKYnM/edit | 09:22 |
mhr3_ | tsdgeos, yea, there's mp for it | 09:22 |
mhr3_ | tsdgeos, feel free to review :) | 09:23 |
tsdgeos | we can't do the seemore/less | 09:23 |
tsdgeos | i mean we can | 09:23 |
tsdgeos | but it's the old expansion pattern | 09:23 |
tsdgeos | not that one | 09:23 |
mhr3_ | tsdgeos, https://code.launchpad.net/~mhr3/unity-scopes-shell/special-categories/+merge/205427 | 09:23 |
tsdgeos | besides that, it's just grid and carousel no? | 09:23 |
mhr3_ | tsdgeos, i think cimi is working on the more/less | 09:23 |
mhr3_ | tsdgeos, the carousel is overlaid though | 09:23 |
tsdgeos | just for the text summary | 09:23 |
tsdgeos | we do support overlay | 09:24 |
mhr3_ | ah, good | 09:24 |
tsdgeos | mhr3_: make tryCard and select the overlaid one (the last) | 09:28 |
mhr3_ | tsdgeos, missing title when using { | 09:32 |
mhr3_ | "template": { | 09:32 |
mhr3_ | "overlay": true, | 09:32 |
mhr3_ | "category-layout": "carousel" | 09:32 |
mhr3_ | }, | 09:32 |
mhr3_ | "components": { | 09:32 |
mhr3_ | "title": "title", | 09:32 |
mhr3_ | "art": "art", | 09:32 |
mhr3_ | "subtitle": "subtitle" | 09:32 |
mhr3_ | } | 09:32 |
mhr3_ | } | 09:32 |
mhr3_ | (and subtitle) | 09:32 |
tsdgeos | trueth | 09:34 |
tsdgeos | and if you leave the mascot there it looks out of position | 09:34 |
mhr3_ | but at least it's there :) | 09:35 |
tsdgeos | so i guess someone needs to look at it and that someone is me? | 09:35 |
mhr3_ | tsdgeos, well, i'm not your team lead :) but would be nice if you did | 09:37 |
tsdgeos | there is no team lead :D | 09:38 |
tsdgeos | someone hinted i was acting one this week | 09:38 |
tsdgeos | don't believe it | 09:38 |
tsdgeos | j/k | 09:40 |
tsdgeos | i'll have a look | 09:40 |
tsdgeos | sil2100: do you have any idea what happened to the unity8 silo/landing/train thing? still blocked by autopilot failures? | 09:41 |
tsdgeos | and any idea where i can check the status of that? | 09:41 |
tsdgeos | mhr3_: it works | 09:54 |
tsdgeos | it's just the tryCard thing that is not as good as it should | 09:54 |
tsdgeos | and doesn't set all the stuff needed for carousel to work | 09:55 |
tsdgeos | mhr3_: but if you use tryCardTool | 09:55 |
mhr3_ | oh? | 09:55 |
tsdgeos | you'll see it work | 09:55 |
mhr3_ | great then | 09:55 |
tsdgeos | even a bit crappy since it's just wireframes either | 09:55 |
tsdgeos | i can fix tryCard to work if you want | 09:55 |
mhr3_ | tsdgeos, working tests are always good, primary thing is that it will work | 09:56 |
tsdgeos | mhr3_: i pushed the fix of the test to new-scopes | 10:01 |
mhr3_ | cheers | 10:03 |
sil2100 | tsdgeos: hi! Not sure if anything changed, on Friday Didier wanted the maguro failures on unity8 resolved, right? | 10:09 |
mhr3_ | sil2100, ping, ping, libunity landing? | 10:10 |
tsdgeos | sil2100: i guess, why are we still caring about galaxy nexus if we don't support it and almost noone has one anymore to try to reproduce the problems? | 10:10 |
sil2100 | tsdgeos: the rationale is that maguro is slow, and we'll start using the emulator soon which is even slower - so making stuff work on maguro will help things working on the emulator | 10:16 |
sil2100 | mhr3_: ping! Give us a few moments | 10:17 |
mhr3_ | sil2100, i did, i asked last week :P | 10:17 |
mhr3_ | sil2100, it's a unity7 pkg, so it better land before ff | 10:18 |
sil2100 | Aaaah, libunity! | 10:18 |
tsdgeos | sil2100: the rationale is fine, but if it's just blocking our builds and noone can reproduce/fix it, not really helping the flow | 10:18 |
tsdgeos | anyway i know i'm not going to win | 10:18 |
tsdgeos | so i go to my cave | 10:19 |
sil2100 | Mirv: ^ could you give it a shot maybe? | 10:19 |
sil2100 | Mirv: could you see if you could use the good-old cu2d to land the new libunity? (stand-alone) | 10:19 |
sil2100 | Mirv: I'll add a landing for that in the old spreadsheet | 10:19 |
mhr3_ | +1, there probably won't be anymore changes there, so no need to use train for it | 10:20 |
Mirv | sil2100: sure in theory, but it's known that the tests don't pass at the moment for unity stack so I'm not sure how to validate it | 10:30 |
sil2100 | Mirv: I guess mhr3_ can help out with finding the right set of tests to run/do manually in the meantime :) | 10:32 |
Mirv | ok. and on device I guess unity8 AP etc. | 10:34 |
Mirv | my device is as usual in Qt 5.2 mode, but I'll see if I can do that still today or tomorrow morning (provided mhr3_ will instruct how to test on desktop) | 10:35 |
Mirv | the changelog is essentially the dropping of gee | 10:35 |
sil2100 | Sure, thanks! | 10:38 |
mhr3_ | Mirv, indeed, it's just dropping gee, so what could break is scopes themselves (especially aggregation) and then sound indicator (the play/pause buttons in there)... if that works it should be good | 10:40 |
Cimi | tsdgeos, when will we have branches merged? | 10:44 |
tsdgeos | Cimi: when some tests we can't reproduce failing don't fail | 10:44 |
Mirv | mhr3_: ok, but no problems known? should it work fine with the unity itself from october? (since nux+compiz+unity haven't been updated in trusty at all) | 10:45 |
tsdgeos | Cimi: meanwhile, can you review https://code.launchpad.net/~aacid/unity8/preview_header | 10:45 |
mhr3_ | Mirv, sure | 10:45 |
Cimi | tsdgeos, in your code seems all of them are optional | 10:46 |
Cimi | tsdgeos, shall we have title required? | 10:47 |
tsdgeos | it's not required afaics | 10:47 |
tsdgeos | at least the tryCard doesn't make it mandatory either | 10:47 |
Mirv | mhr3_: hmm, I updated libunity (the packages that come from the source package) from daily-build and now my windows don't have decorations for some reason | 10:47 |
Cimi | tsdgeos, but if you set to use it, you might have the title no? | 10:47 |
Cimi | tsdgeos, what's the point of using the header if you have all the json null? | 10:48 |
mhr3_ | Mirv, ?? compiz weirdness? | 10:48 |
tsdgeos | Cimi: makes no sense | 10:49 |
tsdgeos | Cimi: but that's the guy that created the json fault | 10:49 |
tsdgeos | Cimi: what do you suggest us to do? | 10:49 |
Mirv | mhr3_: hmm ok I guess compiz weirdness, not seeing the problem on guest account | 10:49 |
Mirv | and yes I was testing compiz+nux+unity for fun before that, probably related | 10:50 |
Cimi | tsdgeos, fix the json | 10:50 |
tsdgeos | Cimi: right, so if the solution is "fix the json", what change do you want me to do in the qml code of that branch? | 10:53 |
Mirv | mhr3_: ok thanks, I should be able to test the desktop side | 10:53 |
Cimi | tsdgeos, 38- title: root.widgetData["title"] || "" | 10:53 |
Cimi | + title: root.widgetData["title"] | 10:53 |
tsdgeos | Cimi: but that makes no sense | 10:54 |
Cimi | why? | 10:54 |
tsdgeos | why are you forcing me to have a title? | 10:54 |
Cimi | tsdgeos, because if I use a header, I must provide a title | 10:54 |
tsdgeos | where does it say that? | 10:54 |
Cimi | tsdgeos, it's like having a button without a label and an icon | 10:55 |
Cimi | it makes no sense | 10:55 |
mhr3_ | Mirv, and if desktop didn't break, unity8 won't either :) | 10:55 |
tsdgeos | Cimi: but i can have a header with an emblem and no title | 10:55 |
Cimi | tsdgeos, title is always important | 10:55 |
Cimi | it's the title of the app, no? | 10:56 |
tsdgeos | sure | 10:56 |
tsdgeos | it's the title of something | 10:56 |
tsdgeos | this is a preview | 10:56 |
tsdgeos | not a preview of an app | 10:56 |
tsdgeos | title is important | 10:56 |
Cimi | tsdgeos, well you got my point | 10:56 |
Cimi | tsdgeos, I'd kinda force developers to keep things right | 10:56 |
tsdgeos | but since we provide stuff for randomguy to do a preview, i don't see why we should force title on him | 10:56 |
Cimi | because otherwise he uses no title making it not consistent? | 10:57 |
tsdgeos | Cimi: honestly your change won't force anything | 10:57 |
tsdgeos | only will give us a console warning | 10:57 |
tsdgeos | and the developer will still don't care | 10:57 |
Cimi | tsdgeos, following your thought we should have those checks for empty strings all around unity | 10:57 |
tsdgeos | and we mostly do | 10:58 |
tsdgeos | and it's not a check for empty string | 10:58 |
tsdgeos | is a check for undefined data in a map | 10:58 |
tsdgeos | ./plugins/Unity/Indicators/qml/MenuItemFactory.qml:229: text: menuData && menuData.label || "" | 10:59 |
tsdgeos | ./qml/Dash/Card.qml:71: title: cardData && cardData["title"] || "" | 10:59 |
tsdgeos | mhr3_: about the addSpecialCategory thing, can you explain why are we doing that? | 11:02 |
mhr3_ | tsdgeos, because of the running apps | 11:02 |
tsdgeos | mhr3_: but it works as we have it now, no? | 11:02 |
tsdgeos | so, why are we doing it? | 11:02 |
mhr3_ | tsdgeos, i'm not sure i understand, the new plugin didn't allow shell to "override" a category | 11:03 |
mhr3_ | the old one did yes, so we're mostly bringing it back | 11:03 |
mhr3_ | in a slightly different shape | 11:04 |
tsdgeos | mhr3_: ok, so overrideResults is gone | 11:04 |
mhr3_ | right | 11:04 |
tsdgeos | hmmm | 11:05 |
tsdgeos | so we need addSpecialCategory for running apps and for the dashhome then | 11:05 |
mhr3_ | tsdgeos, there's no dashhome with new scope | 11:06 |
mhr3_ | s | 11:06 |
tsdgeos | ah right | 11:06 |
tsdgeos | forgot about that | 11:06 |
mhr3_ | tsdgeos, so far we don't have apps scope, so try to test it with music or something, but fwiw apps scope id will most probably be "clickscope" | 11:07 |
Cimi | tsdgeos, this is not always | 11:12 |
Cimi | tsdgeos, I'm scared of developers adding empty headers | 11:12 |
tsdgeos | Cimi: ok | 11:12 |
tsdgeos | Cimi: do you understand that your change won't fix anything? | 11:13 |
Cimi | I do | 11:13 |
tsdgeos | ok | 11:13 |
Cimi | because just gives a warning | 11:13 |
tsdgeos | that noone will see anyway | 11:13 |
Cimi | ok | 11:13 |
tsdgeos | if you want to force CardHeader to have a title, discuss with design and propose a code change that actually breaks the thing if it's not there | 11:13 |
Cimi | tsdgeos, but warning is better than nothing, no? | 11:13 |
tsdgeos | but later please | 11:14 |
tsdgeos | we need to get all this done by friday | 11:14 |
mhr3_ | if i do `system-image-cli -b 0` i'm getting the same thing as if i did a full reflash, right? | 11:23 |
mhr3_ | ie i can do it on dirty device? (one with ppa pkgs installed) | 11:25 |
mhr3_ | also, is it faster than phablet-flash? | 11:27 |
sil2100 | tsdgeos: anyway, I know it's all a very risky topic, but did anyone of you have any success with those maguro unity8 failures in overall? I see one, test_hide_hud_click, seems to happen on almost every maguro test in smoketesting | 11:28 |
sil2100 | tsdgeos: you think it would be easy to identify and/or fix? | 11:29 |
tsdgeos | sil2100: i don't have a maguro | 11:29 |
tsdgeos | i've been running the test for 1 hour in a loop | 11:29 |
tsdgeos | and nothing failed | 11:29 |
tsdgeos | in my nexus4 | 11:29 |
tsdgeos | if we are fine being blocked, fine | 11:30 |
greyback | sil2100: is a test app crashing occasionally on maguro? | 11:30 |
sil2100 | Do you know if unity8 tests are going fine on the emulator now? | 11:30 |
sil2100 | greyback: yes, but this failure seems to be unrelared to the qmlscene-crash | 11:30 |
tsdgeos | sil2100: i have not touched the emulator yet, and honestly don't feel like doing | 11:31 |
tsdgeos | sil2100: we have one target which is nexus4, and stuff works there | 11:31 |
tsdgeos | if people feels we need to get stalled fine | 11:31 |
sil2100 | tsdgeos: I'm thinking that if the problem here is the slowness of maguro, maybe if the timeout could be bumped for the failing assert in that test? | 11:32 |
tsdgeos | but i'm not going to take any blame for stuff not working on stuff we don't support and stuff i can't reproduce | 11:32 |
greyback | sil2100: ok | 11:32 |
sil2100 | A hacky way around it, but hmmm | 11:32 |
tsdgeos | sil2100: hey, maybe, if i had a maguro, i would get a fix, but i don't | 11:32 |
tsdgeos | sil2100: i doubt very much the problem is slowness | 11:36 |
tsdgeos | i've run the test on nexus4 | 11:36 |
tsdgeos | while running "stress --cpu 8 --io 4 --vm 2 --vm-bytes 128M " also | 11:36 |
tsdgeos | and it's just fine | 11:36 |
mhr3_ | tsdgeos, if you want i could give you ssh tunnel to a maguro :) | 11:36 |
tsdgeos | mhr3_: i guess that could help | 11:37 |
sil2100 | \o/ | 11:37 |
mhr3_ | tsdgeos, you sure? you won't see much :) | 11:38 |
tsdgeos | mhr3_: i don't need to see anything | 11:38 |
tsdgeos | mhr3_: just need to run the test | 11:38 |
tsdgeos | and make it succeed | 11:38 |
tsdgeos | well | 11:38 |
tsdgeos | see if it fails first | 11:38 |
tsdgeos | and then make it succeed | 11:38 |
tsdgeos | seeing would help | 11:38 |
tsdgeos | but it's not critical | 11:38 |
tsdgeos | i can just add billions of prints | 11:39 |
tsdgeos | mhr3_: can you phablet flash it with latest stuff? | 11:39 |
mhr3_ | ok, let me try to setup everything | 11:39 |
mhr3_ | first i need to charge it a bit though | 11:39 |
tsdgeos | mhr3_: thanks | 11:39 |
sil2100 | mhr3_: thanks! | 11:40 |
=== mhr3__ is now known as mhr3 | ||
mhr3 | tsdgeos, :/ someone changed our network setup, can't open ports, so.. not good | 11:49 |
tsdgeos | :/ | 11:49 |
mhr3 | tsdgeos, i can do it at the office tomorrow, if that's good enough for you | 11:49 |
tsdgeos | unity: anyone has a galaxy nexus ? | 11:50 |
tsdgeos | @unity: anyone has a galaxy nexus ? | 11:50 |
tsdgeos | i mean | 11:50 |
Cimi | tsdgeos, I have | 11:50 |
* mzanetti too | 11:50 | |
tsdgeos | Cimi: mzanetti: you have two options, fix the autopilot failures we have or give me ssh so i can try to | 11:51 |
sil2100 | Cimi, mzanetti: can anyone of you flash it to the latest version and provide ssh access for tsdgeos ? :) | 11:51 |
tsdgeos | Cimi: mzanetti: you choose :D | 11:51 |
sil2100 | (or that) | 11:51 |
sil2100 | ;D | 11:51 |
sil2100 | It seems to be something easily reproducible on maguro from what I see | 11:51 |
mzanetti | ok... need to charge and flash tho | 11:52 |
Cimi | tsdgeos, I can do both | 11:52 |
Cimi | tsdgeos, I have to run to the doctor though now :) | 11:52 |
Cimi | tsdgeos, maybe i can setup ssh before | 11:53 |
tsdgeos | Cimi: that'd be great | 11:53 |
Cimi | oh yeah | 11:53 |
Cimi | tsdgeos, so what want me to do? | 11:53 |
tsdgeos | Cimi: setup ssh so i can login from here, if you have time to phablet-flash it first the better | 11:53 |
tsdgeos | if not i guess i can manage from here | 11:54 |
Cimi | tsdgeos, you want a linux pc or ssh to the phone? | 11:54 |
tsdgeos | Cimi: ssh to the phone | 11:54 |
tsdgeos | or ssh to a pc where i can adb to the phone | 11:54 |
tsdgeos | you choose | 11:54 |
tsdgeos | but i guess the first is easier | 11:54 |
Cimi | tsdgeos, ok if I set static ip | 11:55 |
Cimi | tsdgeos, battery is dead | 11:55 |
Cimi | tsdgeos, hah | 11:55 |
mhr3 | tsdgeos, don't forget that flashing would remove the ssh server | 11:55 |
Cimi | tsdgeos, charging then flashing | 11:55 |
Cimi | tsdgeos, 30 mins+ | 11:55 |
tsdgeos | Cimi: ok | 11:55 |
sil2100 | Bad luck! | 11:55 |
sil2100 | :\ | 11:55 |
mzanetti | yeah... mine won't turn on even with power attached | 11:56 |
mzanetti | will take a little here too | 11:56 |
mzanetti | Cimi: if you gotta run to the doctor I can set it up | 11:56 |
Cimi | mzanetti, I can do it | 11:57 |
mzanetti | ok | 11:57 |
Cimi | mzanetti, I don't have appointment, just grab medicine | 11:57 |
mzanetti | ah ok | 11:57 |
tsdgeos | mhr3: about the progress preview type | 11:58 |
tsdgeos | mhr3: i don't understand this | 11:58 |
tsdgeos | "id": null, // mandatory here! action $id triggered on completion/error(/cancel) with argument for status | 11:58 |
mhr3 | tsdgeos, there should be just a regular widget_id | 11:59 |
mhr3 | tsdgeos, when triggering actions, we're sending widget_id + action_id | 11:59 |
tsdgeos | is this something i care at all from the UI POV? | 12:00 |
mhr3 | tsdgeos, for progress it should be the widget_id as specified by the scope author + action_id specified by us | 12:00 |
tsdgeos | i mean for progress i just show stuff, don't send anything back, no? | 12:00 |
mhr3 | tsdgeos, the action should be auto-triggered when stuff happens | 12:01 |
mhr3 | http://developer.ubuntu.com/api/devel/ubuntu-14.04/cplusplus/libunity-scopes/classunity_1_1scopes_1_1_preview_widget.html#progress | 12:01 |
tsdgeos | mhr3: what do you mean with auto-triggered? | 12:02 |
mhr3 | tsdgeos, download finishes - shell triggers "finished" action on the progress widget | 12:02 |
tsdgeos | can i say that doesn't make any sense? | 12:04 |
mhr3 | no | 12:05 |
tsdgeos | i knew it :) | 12:05 |
mhr3 | tsdgeos, the entire progress widget really needs to tied to the progress provider service | 12:06 |
tsdgeos | mhr3: why is that on the frontend and not the backend? | 12:06 |
tsdgeos | imho the frontend all it should care is "show 50", "show 75", "done" | 12:06 |
mhr3 | tsdgeos, it is, but shell is the bridge | 12:07 |
tsdgeos | ok, i don't think it makes sense | 12:07 |
mhr3 | tsdgeos, scope might not be running to receive those notifications | 12:07 |
tsdgeos | but we're of course not going to change it | 12:07 |
mhr3 | tsdgeos, all it wants is result of the operation | 12:07 |
tsdgeos | so let's pretend i think it makes sense | 12:07 |
mhr3 | tsdgeos, and since shell does know both the progress and the result, it makes sense to notify the scope when the result is available | 12:08 |
mhr3 | tsdgeos, plus the completion will change the preview | 12:08 |
tsdgeos | mhr3: the contract we have for previews right now is http://paste.ubuntu.com/6908739/ does it mean that besides triggered your model is also going to expose finished and failed? | 12:14 |
tsdgeos | does it do that already? | 12:14 |
mhr3 | tsdgeos, no, that's the progress provider's job | 12:15 |
mhr3 | (the thing that gives you progress == 60%) | 12:15 |
tsdgeos | what is its job? | 12:16 |
=== MacSlow is now known as MacSlow|lunch | ||
mhr3 | to notify you of the progress and whether the operation was successful/failed | 12:16 |
tsdgeos | you're killing me :D | 12:17 |
tsdgeos | you told me the shell has to notify the scope | 12:17 |
mhr3 | wait, let me try to find something | 12:17 |
tsdgeos | yes? | 12:17 |
mhr3 | yes, as in you're supposed to call .triggered(progress_widget_id, "finished", meta) | 12:18 |
mhr3 | but you do that in response to progress indicator telling you that is what happened | 12:18 |
mhr3 | *provider | 12:18 |
tsdgeos | ok | 12:19 |
tsdgeos | so that is what i was asking | 12:19 |
tsdgeos | if we use triggered | 12:19 |
tsdgeos | or something else | 12:19 |
tsdgeos | but the docu you pointed says | 12:19 |
tsdgeos | "Upon completion the scope will receive preview action activation with id "finished" or "failed" depending on the outcome of the operation." | 12:20 |
tsdgeos | will you take care of converting that .triggered(progress_widget_id, ) to that? | 12:20 |
mhr3 | sure | 12:20 |
mhr3 | that is what's happenning with any "triggered" method call | 12:20 |
mhr3 | tsdgeos, this might help to see the entire flow https://docs.google.com/a/canonical.com/drawings/d/1AKbddm35Tw9IoxZ8T33HBO30qO8c35B_IwEzacmf3bI/edit | 12:21 |
mhr3 | the 3rd part especially | 12:21 |
tsdgeos | mhr3: so can you tell me what triggered call exactly you want for success and failure? | 12:21 |
sil2100 | mhr3: the unity-scope-scopes landing - will this have some impact on the current image? i.e. is this scope enabled by default now? | 12:22 |
mhr3 | sil2100, no | 12:22 |
mhr3 | .triggered(progress_widget_id, "finished", progress_widget_properties_map) | 12:22 |
mhr3 | tsdgeos, ^ + same for "failed" | 12:23 |
dednick | mh. my nexus 10 keeps showing device out of space when installing packages... | 12:23 |
tsdgeos | mhr3: but s/finished/failed | 12:23 |
mhr3 | right | 12:23 |
tsdgeos | mhr3: what's in progress_widget_properties_map ? the same thing that you gave us? | 12:23 |
mhr3 | tsdgeos, yep | 12:24 |
tsdgeos | mhr3: we don't do that on buttons, saviq told me not to, do you expect it too for buttons? | 12:24 |
mhr3 | yea, it should be there | 12:24 |
tsdgeos | why? | 12:24 |
mhr3 | tsdgeos, but needs to per-action | 12:25 |
mhr3 | to be* | 12:25 |
tsdgeos | i mean you gave it to us, you sure know its contents, no? | 12:25 |
mhr3 | tsdgeos, cause scope can push there extra data that you don't care about, but it does | 12:25 |
tsdgeos | ok | 12:26 |
tsdgeos | so that needs changing in the regular actions too | 12:26 |
sil2100 | mhr3: did this package get pre-NEWed by an archive admin already? | 12:26 |
mhr3 | tsdgeos, ie if actions = [{"id": "foo", "label": "Foo", "session-id": "0x1234ef"}, {...}], i expect triggered(widget_id, "foo", {"session-id": "0x1234ef", ...}) | 12:27 |
mhr3 | sil2100, no idea, robru was dealing with that | 12:27 |
tsdgeos | mhr3: you'll get back id and label too | 12:27 |
mhr3 | tsdgeos, yea, fine | 12:27 |
tsdgeos | ok, i added it to my todo | 12:28 |
tsdgeos | thing is growing too much :D | 12:28 |
mhr3 | tsdgeos, btw i saw designs where there is an action combo + regular button on the right side, do we support that? | 12:29 |
tsdgeos | kind of | 12:30 |
tsdgeos | we don't support "combos" | 12:30 |
mhr3 | tsdgeos, https://docs.google.com/a/canonical.com/file/d/0B-a_7E3tDxOgUUY0MjFTQThBZFE/edit | 12:30 |
tsdgeos | we support a button that when you click on it | 12:30 |
tsdgeos | it grows | 12:30 |
tsdgeos | so it's an "ugly combo" | 12:30 |
tsdgeos | but yes | 12:30 |
tsdgeos | if you give us 3 actions | 12:30 |
tsdgeos | you'll end up with the first action in the right and then on the left a More/Less buttons that shows the other 2 actions | 12:30 |
mhr3 | ah, ok | 12:31 |
mhr3 | so the right one is "primary" action? | 12:31 |
tsdgeos | so yes but if looks a bit ugly because there's no proper combo in the SDK yet and we didn't feel like coding one :D | 12:31 |
tsdgeos | yep | 12:31 |
mhr3 | ok | 12:31 |
tsdgeos | also if you give us two actions | 12:31 |
tsdgeos | they are laid right to left | 12:32 |
tsdgeos | so first is on right and second on left | 12:32 |
mhr3 | makes sense | 12:32 |
tsdgeos | see make tryPreviewActions | 12:32 |
mhr3 | tsdgeos, how hard is it to get support for "image" and "video" widget types? | 12:35 |
mhr3 | we kinda... use those | 12:36 |
tsdgeos | mhr3: saviq doesn't want to do video | 12:36 |
tsdgeos | until it's out of process | 12:36 |
tsdgeos | so basically never :D | 12:36 |
mhr3 | tsdgeos, let's just make it same as image, but on click it will open the uri? | 12:36 |
tsdgeos | non zoomable? | 12:37 |
tsdgeos | the uri would be a new param in the json? | 12:37 |
mhr3 | tsdgeos, it's already defined | 12:37 |
mhr3 | see the json def | 12:37 |
tsdgeos | is it? | 12:37 |
tsdgeos | i'm looking at it :D | 12:37 |
tsdgeos | type, source, zoomable | 12:38 |
tsdgeos | whiere's uri? | 12:38 |
mhr3 | tsdgeos, oh you meant image itself, no, it's there only for video | 12:38 |
mhr3 | tsdgeos, but the source is a uri | 12:38 |
mhr3 | in both cases | 12:38 |
tsdgeos | sure | 12:38 |
tsdgeos | the source of the image :D | 12:39 |
mhr3 | but in video's it's video/* | 12:39 |
tsdgeos | but ok, in video we have screenshot | 12:39 |
mhr3 | indeed | 12:39 |
tsdgeos | so you want video to show the screenshot and when the guy presses launch the video | 12:39 |
mhr3 | tsdgeos, if not specified, just use the thumbnailer | 12:39 |
=== alan_g is now known as alan_g|lunch | ||
mhr3 | tsdgeos, yep | 12:40 |
dednick | anyone know if it's possible to redistribute freespace on touch device? i keep running out of root partition space! | 12:40 |
tsdgeos | dednick: i clean the apt cache manually | 12:41 |
tsdgeos | but not anything else | 12:41 |
tsdgeos | if you know how to do it, 'll be interested too! | 12:41 |
tsdgeos | mhr3: well, it should not be "hard", but we need more work in getting stuff to have feature parity with what we had | 12:42 |
dednick | only 540MB free on root partition on clean install. it's a bit useless | 12:42 |
mhr3 | tsdgeos, that's what this week is about :) | 12:42 |
tsdgeos | mhr3: right, so that's why i'm saying "no" to video and image | 12:42 |
tsdgeos | since we didn't have that | 12:42 |
tsdgeos | so let's achieve parity first | 12:42 |
tsdgeos | and then talk about those? | 12:43 |
mhr3 | tsdgeos, what do you mean we didn't have those? isn't opening an album preview using an image? | 12:43 |
tsdgeos | and where did we do that in previews? | 12:44 |
mhr3 | tsdgeos, i don't mind if some props will get ignored, i just don't want some of the widget types to be ignored | 12:44 |
mhr3 | tsdgeos, so if "image" is an alias for gallery with 1 image for now... i can live with it | 12:45 |
mhr3 | but ignoring all "image" widget types sucks | 12:45 |
mhr3 | and same for video really, if it's aliased image.. i'm happy | 12:47 |
tsdgeos | ok | 12:49 |
tsdgeos | we'll see if we can get anyone to do it | 12:49 |
mhr3 | cool | 12:49 |
=== MacSlow|lunch is now known as MacSlow | ||
=== _salem is now known as salem_ | ||
=== tvoss is now known as tvoss|test | ||
=== alan_g|lunch is now known as alan_g | ||
kgunn | mzanetti: ping | 14:06 |
mzanetti | hi kgunn | 14:06 |
kgunn | mzanetti: hey...did Saviq leave landing responsibility with you ? | 14:07 |
kgunn | or someone else ? | 14:07 |
mzanetti | kgunn: yeah, he asked me if I could do if something needs to be landed | 14:07 |
kgunn | mainly just needs a shepherd to watch for reasons builds might fail and/or failures on AP's | 14:07 |
mzanetti | but no list of things to be landed | 14:07 |
tsdgeos | mzanetti: i can give you a list, we have a list :D | 14:07 |
mzanetti | I guess so :D | 14:07 |
mzanetti | 53 branches | 14:08 |
tsdgeos | kgunn: i'm trying to get a maguro setup for me to find out how we fix those two failing autopilot tests | 14:08 |
tsdgeos | mzanetti: well no, look at the document, it's 13 branches | 14:08 |
kgunn | tsdgeos: is that for this silo ? | 14:08 |
kgunn | https://docs.google.com/a/canonical.com/spreadsheet/ccc?key=0AuDk72Lpx8U5dFlCc1VzeVZzWmdBZS11WERjdVc3dmc&usp=drive_web#gid=22 | 14:08 |
tsdgeos | 3 unity-mir and 10 unity8 | 14:08 |
tsdgeos | yes | 14:08 |
kgunn | ok | 14:08 |
mzanetti | tsdgeos: yeah... but in total of branches we have was 53 on friday | 14:08 |
mzanetti | tsdgeos: so those 13 should be released this week? | 14:10 |
tsdgeos | mzanetti: no, those 13 need to be released today | 14:10 |
tsdgeos | well | 14:10 |
tsdgeos | on friday | 14:10 |
tsdgeos | last friday | 14:10 |
tsdgeos | this week we need to release *much* more stuff | 14:10 |
=== dandrader is now known as dandrader|afk | ||
=== dandrader|afk is now known as dandrader | ||
tsdgeos | elopio: karni: you joining? | 14:32 |
elopio | hectic morning, sorry. | 14:32 |
mhr3 | tsdgeos, we'll need someone to fix the issues with previews in new-scopes | 14:37 |
tsdgeos | mhr3: karni, Cimi or me | 14:37 |
tsdgeos | mhr3: write them somewhere, or tell me and i'll write them down and try to get them passed around | 14:37 |
mhr3 | tsdgeos, k, just reminding that it's another thing for this week :) | 14:37 |
tsdgeos | i know i know | 14:37 |
tsdgeos | mhr3: if you know what exactly fails, please tell me | 14:38 |
tsdgeos | my todo item says "new preview needs work" | 14:38 |
mhr3 | tsdgeos, 1) preview doesn't open when clicking on non-first result | 14:38 |
mhr3 | 2) progress indicator never hides | 14:38 |
mhr3 | 3) i'm hungry | 14:39 |
mhr3 | 3 might seems unrelated, but it's because i need to check previews :P | 14:39 |
Cimi | hah | 14:40 |
tsdgeos | @unity: so the news scopes branch is https://code.launchpad.net/~unity-team/unity8/new-scopes/ | 14:42 |
tsdgeos | we can merge stuff in there | 14:42 |
tsdgeos | but we should be as strict as we are with landing to trunk | 14:42 |
paulliu | Is there any way to test the pinch event if I don't have multiple mouses? Is there any emulators for that? | 14:45 |
greyback | dandrader: so here's my notepad of nexus7 jobbies to do: http://studio.sketchpad.cc/IOYP51EBZk | 14:46 |
dandrader | greyback, I'm currently stuck with wifi | 14:47 |
dandrader | greyback, no wifi device is listed | 14:47 |
tsdgeos | dandrader: i've heard you need to flash some special android radio firmware or something | 14:49 |
tsdgeos | greyback probably knwos which one, but if not ask in #ubuntu-touch | 14:49 |
greyback | dandrader: that issue was mentioned in the ML thread. Which image did you flash? I flashed ogra's image | 14:50 |
dandrader | tsdgeos, I'm currently poking people on #phablet | 14:50 |
tsdgeos | okidoki | 14:50 |
dandrader | greyback, I flashed the regular http://cdimage.ubuntu.com/ubuntu-touch/daily-preinstalled/pending/trusty-preinstalled-touch-armhf.zip | 14:51 |
dandrader | greyback, ok, I will dig into this e-mail thread.. | 14:51 |
greyback | dandrader: I used http://people.canonical.com/~ogra/ubuntu-touch/ubuntu-touch-4.4.2-mir.rootfs-armhf.zip | 14:52 |
mzanetti | greyback: kgunn: just to be sure. don't expect the current branch to perfectly work. the question is more if we should focus on fixing mine or fixing trunk during this week | 14:55 |
greyback | mzanetti: yep, understood | 14:56 |
kgunn | mzanetti: do you feel like we could ? get it fixed? | 14:59 |
=== tvoss|test is now known as tvoss | ||
mzanetti | kgunn: not 100% I'm afraid. but afaik that wouldn't be the case for trunk too, right? | 15:01 |
kgunn | mzanetti: possibly...that's why i was asking greyback what he had up his sleeve... | 15:01 |
mzanetti | if we can make trunk perfectly fine for MWC, I'd vote for trunk | 15:02 |
kgunn | in fact, greyback why don't you send me a list of any non MP'd branches | 15:02 |
kgunn | i can do a more complete compare | 15:02 |
dandrader | tsdgeos, greyback: rebooting solved my wifi issue. | 15:03 |
* dandrader shrugs | 15:03 | |
tsdgeos | all the computer stuff is fixed that way! | 15:03 |
tsdgeos | elopio: can we change wait_for timeout? do you know how one does that? | 15:03 |
tsdgeos | @unity: ↑ do you remember? | 15:04 |
greyback | kgunn: lp:~gerboland/unity8/sidestage-smoothness-tweaks/ and lp:~gerboland/unity8/sidestage-unlock-sidestage-appears/ and lp:/~gerboland/unity8/sidestage-handle-input-area/ will help a bit | 15:05 |
kgunn | greyback: ta | 15:05 |
tsdgeos | timeout=something | 15:06 |
* tsdgeos tries | 15:06 | |
dandrader | tsdgeos, missing context. | 15:06 |
tsdgeos | dandrader: autopilot wait_for | 15:06 |
dandrader | tsdgeos, ah, no idea | 15:06 |
elopio | tsdgeos: yes, this is the method signature: | 15:07 |
elopio | def wait_for(self, expected_value, timeout=10) | 15:07 |
tsdgeos | elopio: so i do | 15:07 |
tsdgeos | wait_for(bla, timeout=5)? | 15:08 |
* tsdgeos knows not much python :D | 15:08 | |
tsdgeos | seems to work | 15:08 |
elopio | tsdgeos: where are you? | 15:08 |
elopio | it's like self.main_window.visible.wait_for(True, timeout=60) | 15:08 |
tsdgeos | ok | 15:08 |
elopio | the timeout is in seconds. | 15:08 |
tsdgeos | tx | 15:09 |
=== salem_ is now known as _salem | ||
=== alan_g is now known as alan_g|tea | ||
tsdgeos | elopio: | 15:37 |
tsdgeos | ping | 15:37 |
tsdgeos | :D | 15:37 |
elopio | tsdgeos: pong. | 15:37 |
tsdgeos | so i've found i seems to be a "load" thing | 15:37 |
tsdgeos | if you make the galaxy nexus be busy | 15:37 |
tsdgeos | it fails more oten than when it's not busy | 15:37 |
tsdgeos | i thought i had something that made the test more robust | 15:37 |
tsdgeos | but just failed | 15:38 |
tsdgeos | so i'm back to the ddrawing board | 15:38 |
tsdgeos | sorry :D | 15:38 |
elopio | tsdgeos: but why is the device busy? in jenkins, isn't running one single test at a time? | 15:38 |
tsdgeos | well it also fails when it's not busy | 15:38 |
tsdgeos | but less often | 15:38 |
tsdgeos | for me we could just skip that failure, but i understand we need to have reliable tests | 15:39 |
elopio | but it's still not clear to me. | 15:40 |
elopio | tsdgeos: when the device is busy, does it take more than 10 seconds to open the scope? | 15:40 |
tsdgeos | takes a lot to start | 15:41 |
tsdgeos | so you swipe | 15:41 |
tsdgeos | and stuff is not settled yet | 15:41 |
tsdgeos | so the swipe is lost | 15:41 |
elopio | hum, ok, so we need to wait for something before swiping, that makes a lot of sense. | 15:41 |
=== _salem is now known as salem_ | ||
tsdgeos | so my current "fix" is http://paste.ubuntu.com/6909754/ | 15:42 |
tsdgeos | it makes it quite a bit more stable | 15:42 |
tsdgeos | but still got it to fail once | 15:42 |
elopio | tsdgeos: where are you reproducing the errors? | 15:44 |
elopio | I can't get it to fail. | 15:44 |
=== alan_g|tea is now known as alan_g | ||
tsdgeos | elopio: Cimi's galaxy nexus | 15:44 |
tsdgeos | while [ true ]; do autopilot run unity8.shell.tests.test_emulators.DashAppsEmulatorTestCase.test_get_details; if [ $? -ne 0 ]; then break; fi; done | 15:44 |
elopio | tsdgeos: and do you have the patch from https://code.launchpad.net/~elopio/unity8/fix1277591-open_scope_scrolling_waiting_for_wrong_property/+merge/205423 applied? | 15:45 |
elopio | that will help a little too. | 15:45 |
tsdgeos | actually no | 15:45 |
tsdgeos | i don't ^_^ | 15:45 |
tsdgeos | let me apply it | 15:45 |
elopio | I'm leaving the loop running while I walk the dog. | 15:55 |
elopio | bbs. | 15:55 |
om26er | MacSlow, ping | 16:01 |
MacSlow | om26er, yup? | 16:01 |
om26er | MacSlow, is there a way to disable the OSD notification on touch ? | 16:01 |
MacSlow | om26er, no | 16:01 |
om26er | MacSlow, they come during my autopilot test runs over the app's tabbar which results in failing test | 16:02 |
om26er | since the app cannot switch tabs | 16:02 |
MacSlow | om26er, unless you change unity8's sources of course | 16:02 |
om26er | MacSlow, is there a plan to make them click through like on desktop ? | 16:02 |
MacSlow | om26er, no... they were made explicitly non-click-through for UbuntuTouch. | 16:03 |
MacSlow | om26er, do you run your AP-tests on a live device/session... just wondering | 16:03 |
om26er | MacSlow, yes, we run it on mako | 16:04 |
MacSlow | om26er, what kind of notifications do you get in your case? I assume you're not getting incoming calls on a test-device :) | 16:07 |
om26er | MacSlow, its '<number> call ended' non-interactive | 16:08 |
MacSlow | om26er, how odd... I wonder what app triggers that notification... doesn't make much sense to me. | 16:10 |
om26er | MacSlow, its the dialer-app, when the call ends | 16:10 |
MacSlow | Why would the user need to be notified about that? | 16:11 |
MacSlow | om26er, he/she certainly knows that the moment the call was hung up :) | 16:12 |
om26er | MacSlow, I was just told by the dialer-app dev that that notification is going away :) | 16:12 |
MacSlow | om26er, :) | 16:12 |
=== dandrader is now known as dandrader|lunch | ||
Cimi | tsdgeos, which is the signal I should trigger in rating when I change the rating? | 16:37 |
Cimi | triggered(string widgetId, string actionId, var data) | 16:37 |
tsdgeos | yes | 16:37 |
elopio | tsdgeos: can you capture a video of the failure? | 16:46 |
tsdgeos | elopio: the thing is running on cimi's phone | 16:47 |
tsdgeos | not sure how to do that tbh | 16:47 |
elopio | I am seeing it like 1/30 runs on this maguro, but that doesn't help a lot to understand where should I add a wait. | 16:47 |
Cimi | tsdgeos, sorry was more a question, which data I sent? | 16:47 |
tsdgeos | mhr3: which data do you want sent when someone clicks on the ratings? ↑↑↑ | 16:48 |
elopio | tsdgeos: you convince Cimi to hold a camera in front of the maguro until there's an error :p | 16:48 |
Cimi | hah | 16:48 |
tsdgeos | elopio: doesn't seem cool | 16:49 |
mhr3 | tsdgeos, Cimi, first of all, want a signal only when both rating and review are set | 16:50 |
mhr3 | well.. if "required"=="both" | 16:50 |
mhr3 | then keep the review widget format | 16:51 |
mhr3 | merge the input property dict with "rating", "author" & "review" keys | 16:51 |
mhr3 | see http://developer.ubuntu.com/api/devel/ubuntu-14.04/cplusplus/libunity-scopes/classunity_1_1scopes_1_1_preview_widget.html#rating-input | 16:52 |
mhr3 | and http://developer.ubuntu.com/api/devel/ubuntu-14.04/cplusplus/libunity-scopes/classunity_1_1scopes_1_1_preview_widget.html#reviews | 16:52 |
Cimi | mhr3, rating *and* review? | 16:54 |
Cimi | mhr3, why that? | 16:54 |
Cimi | mhr3, a rating is fine | 16:54 |
mhr3 | Cimi, if required==both, both are needed | 16:54 |
Cimi | I don't usually put a comment | 16:54 |
Cimi | ah ok | 16:54 |
mhr3 | hm, not sure where you'd be getting "author" from... i guess that's server thing, so not needed from the shell | 16:56 |
elopio | kgunn: how far are we from having the screen casting API to record autopilot tests executions? | 16:58 |
kgunn | elopio: desktop closer, touch devices further | 16:59 |
karni | tsdgeos: mhr3: I'm reading scrollback. I haven't looked into it yet, but I know the problem exactly. I believe tsdgeos suggested Saviq to make the property readonly during a code review, while it is actually used to *set* the header size. If you guys haven't fixed this yet, I'll take this on as priority once I finish reading scrollback. | 17:01 |
mhr3 | karni, good morning :) | 17:01 |
mhr3 | fixed | 17:01 |
karni | Thank you. Yes, I had the fix ready, I thought I pushed it already to that MR :/ | 17:02 |
karni | Guess I was distracted with the sprint. | 17:02 |
karni | mhr3: Morning. FYI It's 9AM here, so I won't be appearing much earlier than 8AM my TZ for next 2 weeks. | 17:02 |
mhr3 | someone's in california | 17:03 |
karni | Yes, sir. | 17:03 |
mhr3 | is that what it takes to hide from us? :) | 17:03 |
karni | mhr3: lol! | 17:04 |
=== dandrader|lunch is now known as dandrader | ||
mhr3 | karni, you need to go to hawaii, i still get 1 hour overlap with you | 17:04 |
karni | mhr3: hahahh | 17:04 |
karni | I wouldn't complain ;) | 17:04 |
mhr3 | actually anywhere between hawaii and central australia | 17:05 |
karni | The more overlap we have, the better ;P | 17:05 |
tsdgeos | mzanetti: so the blocking to fix unstable tests has been lifted, can you take care of the landing that is on the ci train document? | 17:06 |
tsdgeos | elopio: do you know where the qml output go when running an autopilot test? | 17:12 |
mhr3 | karni, if you're looking for things to do, we'd want "image" widget type (even if it'd be just an alias for gallery with just one source) | 17:16 |
elopio | tsdgeos: it's added to the test as details, and printed to stdout when it fails. | 17:16 |
tsdgeos | elopio: you sure? | 17:21 |
tsdgeos | i only get | 17:21 |
tsdgeos | http://paste.ubuntu.com/6910208/ | 17:22 |
tsdgeos | and not the billions of qml output i've added | 17:22 |
tsdgeos | elopio: which command line do you use? | 17:22 |
mzanetti | tsdgeos: ok | 17:25 |
mhr3 | tsdgeos, i suppose you didn't have time to look at the special-category branch? | 17:57 |
tsdgeos | mhr3: i didn't | 17:57 |
tsdgeos | mhr3: are you expecting me to review it? | 17:58 |
tsdgeos | or? | 17:58 |
mhr3 | tsdgeos, yes | 17:58 |
mhr3 | :) | 17:58 |
tsdgeos | mhr3: i know close to nothing of unity-scopes-shell honestly | 17:58 |
tsdgeos | mhr3: isn't there anyone else that can? | 17:59 |
mhr3 | tsdgeos, mzanetti saw it a few times | 17:59 |
mhr3 | but i thought that as long as you implement the shell side of it and it works you can greenlight it | 17:59 |
tsdgeos | mhr3: ok, i'll have a look | 18:01 |
mhr3 | afterall it has good enough tests :) | 18:01 |
=== alan_g is now known as alan_g|EOD | ||
karni | mzanetti: There's an e-mail which perhaps you could address on ubuntu-phone mailing list: [SDK] Expandable items | 18:18 |
mzanetti | karni: hey... I started to reply but not sure what I should write tbh | 18:19 |
karni | mzanetti: the interesting part is that I am almost positive I saw an expandable list view in action during the sprint. do we have some older component with similar functionality? | 18:20 |
mzanetti | karni: not in the sdk... some apps do their own thing. and unity we do it ourselves too | 18:20 |
karni | I see | 18:20 |
=== dandrader is now known as dandrader|bbl | ||
karni | I have something that worries me. Not havin Saviq and tsdgeos around, who would I raise this with? | 19:12 |
karni | A branch has been merged to new-scopes without the prerequisite being merged to unity8 trunk first. https://code.launchpad.net/~unity-team/unity8/newscopes-card-overlay/+merge/205359 | 19:13 |
karni | mhr3: by the way, what did you mean by saying everything will break tomorrow? we expect some minor changes? | 19:13 |
mhr3 | yea, we want to land scopes-api 0.3, and it changes things, see the email | 19:14 |
karni | mhr3: noted the "image widget". I'll see if I can get to it today (I doubt today, though) | 19:14 |
karni | mhr3: ok, I'll make sure to shut up now and read my mail :P | 19:14 |
mhr3 | karni, we're already using it in the previews, and shell just ignores it... so not nice | 19:15 |
karni | ack | 19:15 |
mhr3 | karni, second major thing is that opening previews for stuff is mostly not working | 19:22 |
mhr3 | just sometimes it does | 19:22 |
mhr3 | and only when clicking on the first result | 19:23 |
karni | sometimes? haha ;) | 19:23 |
karni | mhr3: ack. is someone looking into this, or perhaps something I could take a look at? I guess I should get in sync with folks here how's the preview work coming along. | 19:23 |
mhr3 | karni, i don't think anyone is handling this yet | 19:24 |
karni | noted | 19:24 |
karni | mhr3: has the -readonly branch landed? can't find it | 19:40 |
cwayne | 380939 | 19:42 |
cwayne | 431178 | 19:42 |
cwayne | agh crap, sorry | 19:42 |
karni | mhr3: ok, found it. and its merged, great. thank you. | 19:47 |
=== salem_ is now known as _salem | ||
=== dandrader|bbl is now known as dandrader | ||
kgunn | its so quiet w.o saviq :( | 21:59 |
mhr3 | karni, could you send off a mail by your eod to sync up? (if you manage to do something dash/preview related) | 22:24 |
karni | mhr3: Yes, although I'm test driving custom scopes atm, so I haven't even gotten to that yet. I'll try to get to the preview bug on items != 1st | 22:25 |
karni | mhr3: sure | 22:25 |
=== jhodapp is now known as jhodapp|afk | ||
karni | I'll actually consider starting early tomorrow to make sure I wrap my stuff before Wed | 22:26 |
cwayne | mhr3, hey, does unity8 no longer look in XDG_DATA_DIRS for scopes? | 22:41 |
mhr3 | cwayne, nope | 22:41 |
mhr3 | cwayne, you might want to comment on https://bugs.launchpad.net/unity-scopes-api/+bug/1274421 | 22:42 |
ubot5 | Launchpad bug 1274421 in unity-scopes-api "Deployment directories for scopes conflict with packaging" [High,Triaged] | 22:42 |
cwayne | mhr3, hm, that's bad for us | 22:43 |
cwayne | mhr3, commented | 22:43 |
cwayne | mhr3, just FYI, having unity look in XDG_DATA_DIRS is an MWC blocker at the moment | 22:46 |
mhr3 | cwayne, noted, will bring up tomorrow | 23:03 |
mhr3 | cwayne, we might want to change it to look in a specific dir instead of XDG_DATA_DIRS, but i guess you'd be fine with that | 23:04 |
mhr3 | of course it'd be a dir somewhere in /custom | 23:04 |
cwayne | mhr3, i think we'd prefer XDG_DATA_DIRS, but i think ssweeny feels more strongly about that than I do :) | 23:04 |
* ssweeny does :) | 23:08 | |
ssweeny | though really we should focus on where click scopes will go, since that will be how we do it eventually right? | 23:09 |
ssweeny | the /custom XDG_DATA_DIRS thing is just a workaround waiting for that IIRC | 23:09 |
ssweeny | cwayne, mhr3 ^^ | 23:10 |
cwayne | ssweeny, yeah, that was my understanding as well | 23:31 |
cwayne | so maybe just adding /custom/xdg/data is fine as a stopgap then | 23:34 |
ssweeny | maybe, but what about the shared libraries? can they go there as well? | 23:37 |
ssweeny | the bug chatter seems to say they should be split out | 23:37 |
ssweeny | but i don't think they would be in a click pkg | 23:38 |
cwayne | hm, good point | 23:38 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!