/srv/irclogs.ubuntu.com/2015/02/11/#ubuntu-unity.txt

=== duflu_ is now known as duflu
liuxgalexabreu, ping07:51
tsdgeosdednick: ping09:15
tsdgeosunping09:16
tsdgeosSaviq: ping09:16
Saviqtsdgeos, wassup?09:16
tsdgeosyou sure that's the duplicate?09:16
tsdgeosregarding https://bugs.launchpad.net/ubuntu/+source/indicator-messages/+bug/141850509:17
ubot5Launchpad bug 1385331 in Canonical System Image "duplicate for #1418505 Notification LED stays on when no messages in notification center" [High,In progress]09:17
tsdgeosi mean the duplicate is marked as fix released09:17
tsdgeosbut this still happens09:17
Saviqtsdgeos, there's a indicator-messages task that's not fix released09:17
tsdgeosaahh09:18
tsdgeosi see09:18
Saviqwhich is basically bug #141277909:18
ubot5bug 1385331 in Canonical System Image "duplicate for #1412779 Notification LED stays on when no messages in notification center" [High,In progress] https://launchpad.net/bugs/138533109:18
Saviqwhich is a duplicate09:18
tsdgeosoki09:19
tsdgeosoh man09:54
tsdgeoswhy does platform-api doesn't build parallel?09:55
tsdgeosgreyback: ping10:27
greybacktsdgeos: pong10:28
tsdgeosgreyback: i built https://code.launchpad.net/~mir-team/qtubuntu/port-to-mirclient/+merge/245164 and https://code.launchpad.net/~mir-team/platform-api/expose-mir-connection/+merge/245054 on the pohne and now everything is crashing like mad10:28
tsdgeosanything i may be missing?10:28
greybacktsdgeos: you're building the packages I guess. Maybe merge their trunks? Nothing obvious coming to mind10:30
tsdgeosi think i merged everything10:30
tsdgeoslet me check10:30
greybackhasn't been any qtubuntu change this year anyway10:31
greybackplatform got a few fixes, nothing I would expect impacts input10:31
greybackDoes every app crash on startup? Or only if you interact with it?10:32
tsdgeosyeah everything is merged10:32
tsdgeoswell unity8 is crashing10:32
greybackah10:32
tsdgeosso ca'n't see anything10:32
tsdgeosand i guess the compositor is crashing too10:32
tsdgeossince i can't even see the spinning ubuntu10:33
greybackunity8 doesn't use qtubuntu10:33
greybackbut it does use part of platform-api - the sensors bit10:33
greybackit could be the ABI change requiring qtmir rebuild10:33
tsdgeoslet me see if a qtmir rebuild helps10:34
dandradermzanetti, did you see those failures? https://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-vivid/438/?10:58
dandradermzanetti, could that be related to the changes in lp:~mzanetti/unity8/saveRestoreWindowSizePosition?10:59
mzanettidandrader: no, haven't seem them yet. but yes, I believe it's the same issue as you reported11:00
mzanettidandrader: which I just fixed a minute ago11:00
mzanettinormally on a device ~/.cache/unity8 exists because main.cpp (or something triggered by that) creates it11:01
mzanettihowever, the test alone didn't do that and failed if you ran it on a clean device (or wiped ~/.cache first11:01
Cimitsdgeos, how do I create a qvariantmap with a qvariantmap inside?11:15
tsdgeosjust create it?11:15
tsdgeosQVariantMap a; QVariantMap b; a["moo"] = b;11:15
Cimitsdgeos, I was trying http://paste.ubuntu.com/10171978/11:22
tsdgeossecond one doesn't have to be a qvariantmap btw11:23
tsdgeosit has to be a qvariantlist11:23
tsdgeossee https://code.launchpad.net/~aacid/unity8/testFor131666011:23
tsdgeoshttps://code.launchpad.net/~aacid/unity8/testFor1316660/+merge/24921011:23
Cimitsdgeos, well, I can just reuse that button then11:31
Cimias I need one to click11:31
tsdgeosyou'll need to give it some more info though, i didn't fill in the other fields11:31
tsdgeossince my code is just good enough to have them on screeen11:32
tsdgeosi mean there's no id for example11:32
tsdgeosbut yeah you can reuse that11:32
tsdgeosand if you can review the MR it'd be even awesomer :)11:32
Cimitsdgeos, can you add buttonData["id"] = "open_click" ?11:42
Cimior well, I can since I will need other stuff...11:42
Ciminevermind...11:43
tsdgeosok :)11:43
=== MacSlow is now known as MacSlow|lunch
mzanettidandrader: come one... now approve it already12:07
=== marcusto_ is now known as marcustomlinson
tsdgeosgreyback: i rebuilt qtmir too12:09
tsdgeosnothing :/12:09
greybacktsdgeos: that's v strange12:14
tsdgeosmaybe i messed up somewhere12:14
tsdgeoswill try again12:14
greybackdon't spend too much time at it, the MR needs fixing anyway12:15
tsdgeosgreyback: is UbuntuWindow::moveResize being gone in https://code.launchpad.net/~mir-team/qtubuntu/port-to-mirclient/+merge/245164 very bad? or the code that was there basically did nothing and the result is the same?12:15
greybacktsdgeos: it never did anything. I'd keep the stub in though, as it needs to be implemented once mir supports it12:16
tsdgeosthe stub is there12:16
tsdgeosjust no code12:16
tsdgeosand a todo12:16
greybackno change needed there then so12:16
greybackit's to allow clients resize their own windows12:17
tsdgeosok12:17
greybackwhich we've never needed on phone/tablet so far12:17
tsdgeosso from a "i know nothing about this" pov the code looks ok-ish12:17
greybacksounds good to me12:17
tsdgeosi've done some comments that are more from the "let's make the code nicer"12:17
tsdgeoslike he has some casts he doesn't need and stuff12:18
dandradermzanetti, approved, thanks12:18
greybacktsdgeos: cool. the functional testing will be the more important part, to ensure gestures and stuff all work as before12:18
tsdgeosyeah12:20
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
Cimitsdgeos, so something like that is correct? http://paste.ubuntu.com/10172979/13:14
CimiI see "no such signal", but should be in the interface... :/13:20
=== MacSlow|lunch is now known as MacSlow
tsdgeosgreyback: now i have it running13:57
greybacktsdgeos: what changed?13:57
tsdgeosi installed too many packagers before and something broke when tried to fix it13:57
tsdgeoslike i installed *.deb13:58
greybackah13:58
tsdgeosand it includes qtmir-desktop13:58
tsdgeosand kaboom13:58
greybackah yeah13:58
greybackhave done that myself manys a time13:58
greybacksorry, should've thought of that as a possible shoot-yourself-in-the-foot13:59
mterryWoah.  is launchpad down?14:05
greybackmterry: works for me14:05
mterryHuh, my browsers can't get to it14:05
MacSlowmterry, lp works for me too14:05
mterryLet me try resetting my internet14:06
greybacklarsu: hey, would you have any idea, with the GTK mir backend, is there a way to stop it drawing client-side window decorations?14:06
mterryYup, that fixed it14:07
larsugreyback: tell the app you're running unity. Many apps don't care, though14:07
greybacklarsu: I want to run it with Mir though - but just to play with server-side decoration14:08
larsugreyback: hm, what do you mean?14:09
larsugtk's client side decorations are totally up to the app. It's just another widget you pack14:09
larsuso there's really no way to disable them for all apps14:10
greybacklarsu: right now, if I run a GTK app on Mir, the app draws it's own decorations. As if the backend decided to do it for the app14:10
larsuoh wait, really?14:10
* larsu is confused14:10
* larsu asks attente14:10
greybackthat's what I'm seeing anyway14:10
dandradermterry, hey, in greeter-refactor, I noticed you removed the behavior that animated the LoginList resize. So when you show the vkb in the test the LoginList jumps up immediately. Is it also like that on a tablet or does it appear to be animated (becasue maybe the reported kbd rect grows and shrinks along with the internal ubuntu-keyboard animation)14:20
dandradermterry, I'm asking that because I'm still flashing my tablet :)14:20
Cimitsdgeos, did you se before? ^^14:29
tsdgeosCimi: hmmm see what?14:29
tsdgeosi guess not14:29
Cimitsdgeos, so something like that is correct? http://paste.ubuntu.com/10172979/14:30
CimiI see "no such signal", but should be in the interface... :/14:30
tsdgeossomething like tha tyes14:30
tsdgeosCimi: you're getting that error?14:30
Cimiyes14:31
tsdgeosvoid triggered(QString const&, QString const&, QVariantMap const&);14:31
tsdgeosfix your signature14:31
tsdgeosin the connection14:31
mterrydandrader, hrm that may not have been an intentional drop, let me look14:37
Cimitsdgeos, yes works, thanks!14:38
Cimilearning bit of c++ a day keeps saviq away14:38
Saviqlol14:38
Cimi:))14:39
tsdgeosgreyback: so anything in particular you'd want me to manual test on that MR?14:40
Cimitsdgeos, to get scopes from previewmodel, can I use Scopes *scopes = dynamic_cast<Scopes*>(parent()); ?14:42
tsdgeosCimi: no, see who is the parent14:43
tsdgeosCimi: pass it down as another param in the constructor and that's it14:43
mterrydandrader, restored the behavior -- that was just a merge mistake14:44
mterrydandrader, thanks for the catch  :-/14:44
Cimitsdgeos, ok14:44
dandradernow my N10 won't turn on...14:45
Saviqdandrader, if it drained, it will take a good half hour on a high-current (2A) charger to recover14:47
greybacktsdgeos: main things I'b be concerned with are gestures, and keyboard input on desktop. Tapping around should just work as before14:48
Cimitsdgeos, in scope.ccp we have this return new PreviewStack; which then creates the previewmodel14:48
dandraderSaviq, I was flashing it and the flash tool tried to reboot it after pushing the images. that's when the N10 dropped dead. but who knows, maybe it has no battery left indeed. it's been ages since I last played with it14:48
Cimitsdgeos, so your advice is calling PreviewStack(this) then moving the parent down the line?14:49
dandraderSaviq, something is very wrong. while charging, all of a sudden display turns on, big full battery  icon shows up, then big empty battery icon, then rotating ubuntu logo (like from usc), then display turns off14:55
dandraderSaviq, then the whole things repeats itself after a minute. seem to be in some sort of reboot loop14:56
tsdgeosgreyback: don't have the desktop setup to try14:56
tsdgeosCimi: yeah14:56
Saviqdandrader, the "full" batter is just charging indication probably, it doesn't actually convey the charge level14:56
greybacktsdgeos: ok, leave that14:57
Saviqdandrader, try getting it into fastboot (power+vol...left?) and leave it there for a while14:57
dandraderSaviq, yeah, at least now it's responding to power button presses. got into fastboot14:58
Saviqdandrader, if you go 'start' it should boot into ubuntu14:58
Saviqdandrader, it's in a reboot loop because nothing clears the "go into recovery" flag, 'cause recovery dies14:58
dandraderSaviq, is says "Downloading...." under the robot. is that right?14:59
dandraderSaviq,  s/right/expected14:59
Saviqdandrader, no, it went into download mode, I think you might've gotten into download mode instead of fastboot (which are really similar)14:59
Saviqdandrader, use the other vol button when powering up ;)14:59
dandraderSaviq, ok, booted into ubuntu successfully now. thanks for the tip15:00
MacSlowdoes anyone here know which package provides the python-import-module "ubuntuuitoolkit"?15:01
Saviqdandrader, think it'd make sense to file a bug, we should not get into a reboot loop like that, our recovery should probably clear the recovery flag as soon as it got it15:01
SaviqMacSlow, ubuntu-ui-toolkit-autopilot15:02
MacSlowSaviq, hm... in that case dpkg and grep failed me... thanks15:02
MacSlowwhat is "Binder_2" and why would it create a 99% load on the device?15:24
SaviqMacSlow, it's android's kernel IPC15:25
MacSlowhm15:25
SaviqMacSlow, shouldn't hog CPU unless it's actually chugging data around15:26
Saviqdandrader, there's a conflict on lp:~phablet-team/ubuntu-keyboard/shellRotation15:27
dandraderSaviq, when merging it with trunk, you mean?15:27
Saviqdandrader, yes15:27
dandraderSaviq, ok, if I can't solve it I will ask Eleo15:28
Cimitsdgeos, chaining down doesn't seem to work well for me... having issues with parents http://paste.ubuntu.com/10174489/15:38
tsdgeosCimi: hmm15:40
tsdgeospass it as additional parametero15:40
tsdgeosotherwise when previewstack creates previewmodel15:40
tsdgeosthis is is not a scopes15:40
tsdgeos-s15:42
tsdgeosmterry: tbh i don't think it really matters much, i mean we're swiping the greeter, it's not like it's the most performance critical part of the phone15:43
=== dandrader is now known as dandrader|lunch
tsdgeosSaviq: greyback: performance discussion around clipping in https://code.launchpad.net/~mterry/unity8/bleeding-infographic/+merge/249223 if you want to jump in15:44
tsdgeospaulliu: there?15:51
paulliutsdgeos: yes15:52
tsdgeospaulliu: did you mention you wanted https://code.launchpad.net/~paulliu/unity8/notification_helper/+merge/249211 to be reviewed by some people from QA? or can i just approve it?15:52
paulliutsdgeos: Because they will use it. So I'd like to see if they like it.15:53
tsdgeosok15:53
tsdgeosi'll give my approval but not top approve letting them comment15:53
tsdgeospaulliu: maybe you should subscribe them15:53
tsdgeosso they know there's stuff for them to do15:53
paulliutsdgeos: yes. ok. I'll do.15:53
Cimitsdgeos, changed something, but other c++ errors http://paste.ubuntu.com/10174764/15:54
Cimioh no15:55
Cimihold on15:55
* tsdgeos holds15:55
SaviqCimi, the actual errors are usually more useful than patches, this way tsdgeos doesn't need to compile with his eyes :P15:55
greybacktsdgeos: I replied to the MR. A single clip shouldn't have any big impact, so I don't object to that change. If doing the same task without clipping is much more complex, it's probably not worth the code complexity15:56
CimiSaviq, I want at least give it a try for myself :)15:57
Cimibut yeah15:57
tsdgeosgreyback: fwiw i wasn't objecting to clipping, i was just arguing to leave the clipping on all the time :D15:58
Saviqgreyback, so, it's probably better to just clip: true15:58
tsdgeosbut it's just arguing for the arguing15:58
Saviqgreyback, no point in turning it on/off?15:58
greybackSaviq: probably yeah15:58
Saviqmterry, ↑15:58
greybackif Greeter offscreen, it should be marked invisible, in which case the whole thing has zero impact15:59
greybackone thing I keep meaning to do is set everything under the greeter as invisible while greeter is occluding it, would stop that jank in the infographics animation on startup16:00
mterrySaviq, OK16:00
mterrygreyback, if greeter is offscreen, it is invisible16:00
greybackmterry: good16:00
greybackwould be even nicer if we unloaded Greeter, to release its resources16:01
greybackbut not a huge deal16:01
Saviqgreyback, yeah, we tried that, but it's not worth it on phone, the greeter is up too often16:01
greybacksounds likely yeah16:02
Saviqgreyback, destroying it and recreating all the time16:02
tsdgeosmterry: your greeter-refactor branch is a tag-hell, can you run ./strip-tags.py on it?16:10
Saviqtsdgeos, when free, you could have a look at https://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-vivid/452/?16:16
Saviqtsdgeos, that seems to be our last flaky qml test16:16
Saviqmzanetti, https://www.youtube.com/watch?v=3J4TWDQDuhU :D16:18
tsdgeosSaviq: i added a tryCompare in one of my branches16:19
tsdgeosshould maybe hopefully fix that16:19
tsdgeosgievn that i don't have anything slow enought o reproduce that error16:19
Saviqtsdgeos, lemme loop16:20
tsdgeosSaviq: the branch i mean is https://code.launchpad.net/~aacid/unity8/testFor1316660/+merge/24921016:20
Saviqkks16:20
tsdgeosSaviq: btw the rtm branch has a few dead tags, not sure we care16:21
Saviqtsdgeos, they're not dead according to strip_tags actually, bzr weird as usual16:22
Saviqtsdgeos, meaning that they actually refer to a valid revision16:22
tsdgeosok16:23
mzanettiSaviq: yeah, saw that video :D16:26
Saviqdandrader|lunch, mterry, lp:~dandrader/unity8/fixSurfaceActiveFocus and lp:~mterry/unity8/tutorial-new-screens conflict, please either rebase one on the other or let me know which should I leave out from the silo16:27
mterrySaviq, dandrader|lunch: I'll try rebasing mine16:28
Saviqtx16:29
mterrySaviq, done16:32
Cimitsdgeos, so I sent scopes down, but I also need to emit openScope signal from scope16:40
Cimitsdgeos, I might send Scope instead, but I have then the problem of getting Scopes from it16:41
seb128can I run a qt5 app from a vt and get it to display under the mir demo server?16:41
tsdgeosCimi: add a scopes() getter to scope16:42
tsdgeosreturn dynamic_cast<Scopes*>(parent());16:42
Cimitsdgeos, ok16:42
Cimitsdgeos, I was doing dynamic_cast<Scopes*>(m_scope->parent()); inside the previewmodel16:42
tsdgeosit's not nice, but it's a mock :D16:42
Cimitsdgeos, but this was not working, why?16:43
tsdgeosCimi: it should16:43
tsdgeosall the scope objects are forced to have Scopes as parent16:43
tsdgeoswhat problem did you have?16:43
Cimicannot dynamic_cast ‘((PreviewModel*)this)->PreviewModel::m_scope->Scope::<anonymous>.unity::shell::scopes::ScopeInterface::<anonymous>.QObject::parent()’ (of type ‘class QObject*’) to type ‘class Scopes*’ (target is not pointer or reference to complete type)16:43
Cimi     Scopes *scopes = dynamic_cast<Scopes*>(m_scope->parent());16:43
CimiI suspect I need cast something16:44
tsdgeos#include "fake_scopes.h"?16:44
Cimitsdgeos, this compiles16:45
Cimitsdgeos, how did you get it was missing an include?16:45
tsdgeosCimi: because it was complaining it can't cast qobject to scopes16:45
* Cimi trying to learn c++ here16:45
tsdgeosbecause it didn't know scopes *is* a qobject16:46
tsdgeossince the include was missing and thus it couldn't16:46
Cimitsdgeos, but fake_scope.h was included16:46
tsdgeossure fake_scope only defines Scope, not Scopes16:46
CimiI see16:46
Cimiok16:46
Cimigracias16:46
tsdgeoswell there's a forward declaration saying16:46
tsdgeos"there will be eventually a class named Scopes"16:46
tsdgeosbut that's all16:46
Cimitsdgeos, now can I emit openScope from the previewModel?16:48
tsdgeosQ_EMIT myscope->openScope(something);16:49
tsdgeossomething is probably another scope, no?16:49
tsdgeosso Q_EMIT myscope->openScope(myscope->scopes()->getScopeFromAll("SomeotherScope"))16:50
tsdgeosor something like that16:50
tsdgeosthat actually compiles D:16:50
Saviqmterry, resubmit the MP please16:54
Saviqwith the prerequisite16:54
Cimitsdgeos, ok works! tomorrow the test finally :)16:56
tsdgeos:)16:58
mterrySaviq, guh right16:59
mterrySaviq, https://code.launchpad.net/~mterry/unity8/bleeding-infographic/+merge/249223 could also be squeezed in maybe16:59
mterrySaviq, try now17:00
=== dandrader|lunch is now known as dandrader
=== alan_g is now known as alan_g|EOD
=== seb128_ is now known as seb128
=== willcooke_ is now known as willcooke
=== dandrader is now known as dandrader|afk
EncryptHello there o/19:35
EncryptHi tedg o/19:35
EncryptIm coming to bother you people with the GMainLoop and the MessagingMenu :D19:35
EncryptI thought about what tedg_ told me last time19:36
EncryptSO, I think I understood what I have to do to add a gsource to the GMainLoop using an idle_source_add (or something like that)19:36
EncryptBut, I'm still wondering how to deal with the messaging menu19:37
EncryptThat is to say how to deal with "messaging_menu_app_append_source()"19:37
EncryptBecause I noticed that I have to stop the loop to add the event19:37
tedgYou don't have to stop the loop as much as append the source in the same thread.19:38
tedgSo if you do the idle_source_add(), you can append in that function.19:38
tedgThat will execute in the same thread as the mainloop.19:38
EncryptOh!19:39
EncryptOk!19:39
EncryptI hadn't understood it like that19:39
EncryptLet's give it a try :p19:39
EncryptBy the way...19:47
EncryptI discovered the editor "Atom" yesterday19:47
EncryptIt works pretty well! :)19:47
EncryptFor those who want to have a look19:47
Encrypthttps://atom.io/19:47
mterryDoes anyone here run xmir on their desktop?20:48
ChrisTownsendmterry: Do you mean to run an X app on the Unity 8 desktop?20:50
mterryChrisTownsend, I'm just trying to start with running normal unity7 desktop20:50
mterryChrisTownsend, I can run that mostly fine20:50
mterryChrisTownsend, but it crashes when I start chromium20:50
mterryChrisTownsend, and am trying to see if I'm alone or if there are workarounds or what20:50
Encrypttedg, Am I doing it right now?20:51
Encrypttedg, http://pastebin.com/ZqcLTm3120:51
EncryptThe only thing I'm not sure about is line 4420:52
ChrisTownsendmterry: Oh, ok.  I don't do that.  Where are you getting xmir from?20:52
EncryptIt seems that I can't really give an array of gpointer to this kind of function20:52
EncryptBTW, notifications don't work anymore :/20:53
ChrisTownsendmterry: I run Xmir to run X apps in Unity 8, so it's a different use case than what you are doing.20:53
EncryptBut that might be related to the data[2] thing20:53
mterryChrisTownsend, just getting it from vivid20:57
ChrisTownsendmterry: Ok, not sure about that then.20:57
mterryChrisTownsend, that's fair, thanks anyway!  :)20:57
ChrisTownsendmterry: Sorry I couldn't be more help:)20:58
* Encrypt should use a structure21:00
tedgEncrypt, You probably want to create everything on the thread.21:01
tedgEncrypt, So instead of creating, and then running it. You want the function on the thread to create everything.21:01
EncryptHum21:01
EncryptApparently it work21:01
EncryptFine then21:01
tedgAnd yes, a structure would be good. You also need to free some of the memory.21:02
Encrypttedg, Then, what I don't understand is how to do that21:04
tedgAlso, you can't pass memory that is allocated in the stack between threads. I needs to be on the heap.21:04
EncryptHow could an external function call the thread?21:04
EncryptI'm confused21:04
tedgYour mm_notify function would, in theory, be called by the non-Glib thread.21:05
tedgSo then it needs to pass of the data to the GLib thread.21:05
Encrypt<tedg> Also, you can't pass memory that is allocated in the stack between threads. // Are you talking about run_mmloop?21:06
tedgNo mm_notify()21:06
EncryptOk21:06
tedgAn mm_rm_entry()21:06
=== dandrader|afk is now known as dandrader
Encrypttedg, It works _o/22:40
tedgWoot!22:43
EncryptYep :p22:43
Encrypttedg, Thanks for your help! :)22:44

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