tsdgeos | cimi: what's the status of card-social, want me to review? | 09:46 |
---|---|---|
cimi | tsdgeos, debugging why the scope does not work for me :) | 09:46 |
tsdgeos | ok | 09:46 |
cimi | tsdgeos, https://code.launchpad.net/~stolowski/+junk/scope-social_actions | 09:46 |
tsdgeos | cimi: want me to try to help you debug what's wrong or you have it? | 10:21 |
cimi | tsdgeos, for now let me see | 10:22 |
tsdgeos | oki | 10:24 |
tsdgeos | did a quick review of the code and looks reasonablre | 10:26 |
=== dpm_ is now known as dpm | ||
cimi | tsdgeos, I tried putting an onCardDataChanged jsonstringify cardData inside cardCreator (when !isCardTool) but it is not printing anything | 10:46 |
tsdgeos | cimi: phone? desktop? | 10:46 |
cimi | desktop | 10:47 |
tsdgeos | did you run make? | 10:47 |
tsdgeos | cardcreator needs make | 10:47 |
cimi | tsdgeos, ok I changed a bit into onTitleChanged: console.log("UCD inside card", JSON.stringify(cardData["socialActions"])); | 10:49 |
cimi | tsdgeos, but is undefined | 10:49 |
cimi | tsdgeos, same is social-actions | 10:49 |
tsdgeos | print cardDAta | 10:49 |
cimi | tsdgeos, it doesnt print | 10:50 |
tsdgeos | what code do you have exactly? | 10:50 |
cimi | tsdgeos, but I can print cardData["attributes"] fine | 10:51 |
cimi | tsdgeos, http://paste.ubuntu.com/15471189/ | 10:51 |
cimi | tsdgeos, print http://paste.ubuntu.com/15471192/ | 10:52 |
cimi | the other UCD print is from cardtool fake template | 10:53 |
cimi | components["social-actions"] is indeed defined for both | 10:53 |
cimi | I guess we have a bug somewhere in pawel branches no? | 10:53 |
tsdgeos | cimi: http://paste.ubuntu.com/15471216/ works fine to print cardData for me | 10:57 |
tsdgeos | runnign make tryCard i get | 10:58 |
tsdgeos | http://paste.ubuntu.com/15471219/ | 10:58 |
cimi | tsdgeos, for cardTool? | 10:59 |
tsdgeos | i don't understand why cardData would be undefined for you | 10:59 |
cimi | tsdgeos, I run with unity scope tool | 10:59 |
tsdgeos | cimi: what do you mean "for cardTool"? | 10:59 |
cimi | tsdgeos, the template card with fake data from cardtool? | 10:59 |
cimi | tsdgeos, when isCardTool is true | 11:00 |
tsdgeos | yes, what about it? are you asking if it prints there? | 11:00 |
cimi | or the one with false | 11:00 |
tsdgeos | well, you can see it is there too | 11:00 |
tsdgeos | second like | 11:00 |
tsdgeos | line | 11:00 |
cimi | tsdgeos, works fine too here for tryCard | 11:01 |
cimi | tsdgeos, not scope tool | 11:01 |
cimi | mmm maybe I need to run smart proxy something | 11:02 |
cimi | nah always not working with that | 11:03 |
tsdgeos | yeah | 11:06 |
tsdgeos | actually it seems json.stringify doesn't know how to stringify the real cardData since it's a QQmlDMAbstractItemModelData | 11:06 |
tsdgeos | wonder how i did print that | 11:06 |
cimi | tsdgeos, you told me to use stringify :D | 11:09 |
tsdgeos | i did | 11:09 |
cimi | but yeah I understand | 11:09 |
tsdgeos | i've been known to lie :D | 11:09 |
cimi | tsdgeos, it worked for the fake ones because we were treating them as json | 11:09 |
tsdgeos | cimi: ok, this is what you want | 11:16 |
tsdgeos | + onCardDataChanged: { console.log("root", root); for (var k in cardData) console.log(k, cardData[k]); } \n\ | 11:16 |
tsdgeos | gives me something like http://paste.ubuntu.com/15471434/ | 11:17 |
tsdgeos | that betteR? | 11:18 |
cimi | tsdgeos, http://paste.ubuntu.com/15471733/ | 12:32 |
tsdgeos | cimi: ask pstolowski | 12:33 |
cimi | pstolowski, I updated my card social branch, but the model doesnt have socialActions defined | 12:34 |
=== maclin1 is now known as maclin | ||
=== alan_g is now known as alan_g|lunch | ||
greyback | Saviq: http://people.canonical.com/~gerboland/perf/ | 13:36 |
greyback | perf needs symbols files to backtrace properly | 13:36 |
greyback | it'll have same limitations as gdb probably | 13:37 |
Saviq | greyback, yeah got those | 13:37 |
greyback | Saviq: what are you seeing? Can you paste me a report | 13:37 |
greyback | Saviq: in general, everything I know, I got from http://www.brendangregg.com/perf.html | 13:37 |
greyback | # Sample CPU stack traces for the PID, using dwarf to unwind stacks, at 99 Hertz, for 10 seconds: | 13:38 |
greyback | perf record -F 99 -p PID -g dwarf sleep 10 | 13:38 |
greyback | is probably the one I use most | 13:38 |
Saviq | greyback, http://pastebin.ubuntu.com/15472057/ | 13:39 |
greyback | Saviq: huh it saved nothing | 13:39 |
greyback | try use the -F flag to increase the frequency of the perf snapshots | 13:40 |
greyback | does "perf top" give you much? | 13:40 |
Saviq | greyback, http://pastebin.ubuntu.com/15472067/ | 13:41 |
greyback | Saviq: wtf | 13:42 |
greyback | as if that function is stuck in infinite loop | 13:42 |
Saviq | greyback, what's "dwarf"? | 13:43 |
greyback | Saviq: attempts to unwind the stack using libdwarf, instead of trying to follow the frame pointer | 13:44 |
Saviq | dwarf: No such file or directory | 13:44 |
greyback | hmm, maybe older perf doesn't support it | 13:45 |
Saviq | greyback, same thing in perf report :/ | 13:46 |
greyback | Saviq: :( We need to learn what that symbol corresponds to. I don't really know what else can be done | 13:52 |
Saviq | greyback, at least for now we seem to have found why the freeze - we were never changing timezone so early before | 13:53 |
Saviq | and with new OOBE tz change is part of it | 13:53 |
greyback | Saviq: but you were thinking rotation might also have something to do with it? | 13:54 |
Saviq | greyback, yeah, tz change + rotation triggers it | 13:54 |
Saviq | it's real dumb | 13:54 |
greyback | bizarre | 13:56 |
greyback | if you want me to look into it, say the word | 13:56 |
Saviq | greyback, I think we'll let Mir folk look into it, assuming we get clear steps to repro | 13:56 |
greyback | ok | 13:56 |
pstolowski | cimi, will take a look | 13:58 |
tsdgeos | dandrader_: will you review https://code.launchpad.net/~aacid/unity8/rootObjectCanBeNull/+merge/289800 ? It's on some code that seems you made (or copied from some other file) | 14:18 |
dandrader_ | tsdgeos, sure | 14:19 |
=== dandrader_ is now known as dandrader | ||
pstolowski | cimi, are you sure you're using shell plugin from the silo? | 14:46 |
cimi | pstolowski, which branch? | 14:50 |
cimi | pstolowski, yes | 14:53 |
cimi | I built locally | 14:53 |
cimi | pstolowski, does it work for you? | 14:53 |
pstolowski | cimi, i need to flash my phone to check. only looked at my code (which is exactly the same as for inline playback) | 14:58 |
pstolowski | cimi, have you pushed all your changes? i'll rebuild the silo | 14:58 |
cimi | pstolowski, I did | 14:58 |
pstolowski | k | 14:58 |
cimi | mmm looks like lp is annoyed | 15:07 |
cimi | pstolowski, no pawel we need this merge https://code.launchpad.net/~cimi/unity8/card-social/+merge/289663 | 15:08 |
pstolowski | Saviq, greyback I reproduce the freeze once by following the steps from the bug report. and i've just got another freeze after activating app-switcher | 15:08 |
Saviq | pstolowski, I think we found reliable steps to repro - and it's a preexisting issue triggered by the silo | 15:09 |
Saviq | pstolowski, I just updated the bug report a moment ago | 15:09 |
pstolowski | Saviq, ack | 15:09 |
Saviq | related to timezone change, which we didn't do in OOBE so far | 15:10 |
Saviq | and triggered reliably by device rotation | 15:10 |
pstolowski | Saviq, the freeze I just got with app switcher is probably something else. unity8 eats very little cpu | 15:10 |
Saviq | mhm | 15:11 |
pstolowski | Saviq, in the original case i couldn't even adb | 15:11 |
Saviq | pstolowski, yeah, that's when adb asks unity8 whether it's unlocked | 15:11 |
Saviq | and it times out | 15:11 |
pstolowski | Saviq, is this only on 1st timezone setup? i couldn't reproduce it when changing timezones later | 15:14 |
Saviq | pstolowski, yeah, needs a wipe | 15:15 |
=== dandrader is now known as dandrader|lunch | ||
tsdgeos | dandrader|lunch: ping when you're back | 16:09 |
=== dpm is now known as dpm-afk | ||
tsdgeos | dandrader|lunch: commented on the homekeypage | 16:32 |
cimi | tsdgeos, pstolowski, josharenson seems like the rebuild of silo 71 is going well | 16:50 |
josharenson | cimi: I'll make sure to dist-upgrade when its done | 16:51 |
=== dandrader|lunch is now known as dandrader | ||
dandrader | tsdgeos, ping | 16:54 |
tsdgeos | dandrader: the question i made on the MR | 16:54 |
josharenson | cimi: How do you install that test scope? I built/installed it w/ click/pkcon but its not showing up in the scope list | 17:00 |
pstolowski | cimi, oh, i need all the filters stuff in silo 71 :( | 17:04 |
pstolowski | cimi, and yes, i remember you mentioned rebasing our stuff on top of filters.. | 17:04 |
cimi | josharenson, you installed the silo? | 17:27 |
josharenson | cimi: yes but haven't updated it in ~ an hour | 17:28 |
cimi | josharenson, we need to wait for unity-api stuff | 17:29 |
josharenson | ok | 17:29 |
cimi | josharenson, build didnt go well I think | 17:29 |
dandrader | Saviq, ping | 18:36 |
Saviq | dandrader, pong | 18:47 |
dandrader | Saviq, for animated cursors, I wanna use the AnimatedImage item. But for that I've to write a QImageIOPlugin to read xcursor files. It's a Qt GUI plugin. Ideally it should live in a separate package and project. But that's probably a lot of bureaucracy, so at last at first I guess we should just put int in unity8/plugins (along with those qml plugins)? | 18:51 |
dandrader | s/last/least | 18:52 |
Saviq | dandrader, maybe it makes sense upstream, simply? | 18:57 |
Saviq | dandrader, we can carry it as a distro patch until it lands upstream in Qt and we have their version | 18:58 |
dandrader | Saviq, using xcursos is quite a niche. Only compositors written in qt would be interested in that | 19:07 |
Saviq | dandrader, yeah, maybe - no opposition on including it in unity8 from me | 19:08 |
dandrader | ok | 19:08 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!