/srv/irclogs.ubuntu.com/2015/03/26/#ubuntu-unity.txt

=== duflu_ is now known as duflu
Mirvtsdgeos: the AP is in the PPA. the test case seemed fixed for me too, but it'd look like running full UITK test suite just hangs at some point - in the sense the test app is loaded but it stays there.08:37
MirvI killed now one process and will see if the current run finishes eventually08:50
tsdgeosok08:55
tsdgeosi'm also a bit concerned about richard's report of the addressboox app hanging08:56
tsdgeostrying to reproduce08:57
tsdgeosi can't reproduce richard's issue09:20
tsdgeosCimi: can you do https://code.launchpad.net/~aacid/unity8/urldispatcher_hideeverything/+merge/253527 ?09:22
tsdgeosand someone https://code.launchpad.net/~aacid/unity8/passphrase_kewboard/+merge/253657 ?09:22
=== davidcalle_ is now known as davidcalle
tsdgeospstolowski: commented on https://code.launchpad.net/~stolowski/unity-api/filters-iface/+merge/25289009:51
pstolowskitsdgeos, thanks, looking09:52
pstolowskitsdgeos, is this also a problem with roles, do I need to avoid "id" in role names?10:01
tsdgeospstolowski: roles are fine since they're not objects per se10:02
tsdgeosafaik we have have some other id roles10:02
tsdgeosin there10:02
tsdgeosyeah10:02
tsdgeos./include/unity/shell/scopes/ScopesInterface.h:109:        roles[RoleId] = "id";10:03
pstolowskiyeah, you're right10:03
pstolowskithanks10:03
pstolowskitsdgeos, about your second comment, i can added optionsChanged signal, but it will never be triggered since any changes are changes to options rows (model changes)10:14
pstolowskitsdgeos, makes sense?10:14
tsdgeospstolowski: is the options varaible set correctly on creation alrady?10:14
pstolowskitsdgeos, yes, it's going to be set on creation of the filter only10:15
tsdgeosthen it's fine10:15
pstolowskitsdgeos, fine to add but never trigger?10:15
tsdgeosyes10:15
pstolowskiok10:16
=== greyback_ is now known as greyback
Mirvhmm. so slow.10:54
tsdgeosdednick: i think we should do the same thing i did for all the drags and see fi that helps11:03
tsdgeossince it seems it may very well be that11:03
tsdgeoslooking at the logs the last movement is very few pixels11:03
=== JMulholland_ is now known as JMulholland
alan_ggreyback: does MirSurfaceManager have any reason to hold m_mirServer? I don't see it being used.11:47
greybackalan_g: quite right, it is unused11:52
* greyback expcted compiler to notice that11:52
greybackno actually, that would be presumptuous of it11:52
alan_gThe compiler knows there's a non-trivial dtor11:53
=== MacSlow is now known as MacSlow|lunch
alan_gI just wanted to double check there isn't a weird lifetime issue.11:54
greybacknone I'm aware of, this it an oversight11:56
buggattiHello. I am trying to boot a 15.04 next live cd. Whenever i enter the username ubuntu-desktop-next and password, its stuck at login. i can move the cursor around but it wont let me in the gui12:15
=== MacSlow|lunch is now known as MacSlow
=== Pici` is now known as Pici
alan_ggreyback: not urgent, but would something like this make sense to you? https://code.launchpad.net/~alan-griffiths/qtmir/spike-using-WindowManager/+merge/25423512:54
greybackalan_g: looks reasonable at first glance. You've got work to do to replace our custom input dispatcher with the handle_*_event bits, but nothing huge. Direction looks good12:58
alan_ggreyback: I don't *have* to change that. (At least not for a stable form.)12:59
greybackalan_g: true13:00
greybackalan_g: question, would "handle_*_event" supposed to communicate things like close event or focus/unfocus? A nested compositor would probable need to know such things13:01
alan_gBTW how do you go about building the stack and deploying on a phone? Every time I try I can't help feeling there's a better way...13:02
greybackalan_g: either building on device, or an sbuild chroot13:03
greybackif you disable tests building, dev on device isn't so bad13:03
alan_gHow do you get space on the device to install the build tools?13:04
greybackI don't hit that issue usually13:04
greybackif you're changing mir too, then yeah, that's a problem13:05
alan_gWell, it is up to the WM to set focus etc, so yes13:05
alan_gI can probably work around that.13:06
alan_gGotta go13:06
=== alan_g is now known as alan_g|lunch
greybackenjoy13:06
larsu*giggle* "13:30
larsuPlease note that Antti's "Disapprove" review is a joke.13:30
tsdgeos@unity: guys my unity8 is not accepting any touch at all13:50
tsdgeoshttp://paste.ubuntu.com/10683845/13:50
tsdgeosany clue?13:52
tsdgeosgreyback: thread 22 suspicious?13:53
dandradertsdgeos, no13:53
* tsdgeos gets more debug symbols13:53
greybacktsdgeos: thread 22 in that state means the main loop is blocked13:53
tsdgeosgreyback: that sounds bad13:54
tsdgeos:D13:54
greybackyeah13:54
greybackit is mir asking qt to authorize an application connection13:54
greybackand is waiting for a blockingqueuedconnection to return13:54
tsdgeosyet13:54
tsdgeoson screen13:54
tsdgeosi have soemthing that used to be a sdk test application13:55
tsdgeosthat is not even running anymore13:55
greybackrenderer is blocked too then13:55
tsdgeosgreyback: anything else more i can do than filing a bug?13:57
greybacktsdgeos: is anything using lots of cpu? is dbus busy?13:58
greybackI don't see a good clue in that BT sadly13:58
tsdgeoscpu is idle13:58
tsdgeosdbus-monitor says nothing13:58
greybackok13:58
greybackcan you get the thread names13:59
tsdgeossure13:59
tsdgeosgreyback: http://paste.ubuntu.com/10683891/14:00
greybacktsdgeos: any ide what those pool threads are?14:00
greybackidea14:00
tsdgeosgreyback: gio14:00
tsdgeoswhich we don't use but somehow we're stuck with afaik14:01
greybackhmm ok14:01
greybacktsdgeos: oh hang on, thread 9 is weird14:02
greybackthat's the render thread14:02
greybackit's blocked waiting for a new buffer from mir14:03
tsdgeosmir is still unity8 here, no?14:03
tsdgeosor is it usc?14:03
greybackyeah, mir is a lib unity8 uses14:03
tsdgeosany clue which thread shuould supply that buffer?14:03
greybacknot sure, I suspect thread 23 but am not sure14:05
greybackasking mir folk...14:05
=== alan_g|lunch is now known as alan_g
greybacktsdgeos: please log a bug anyway, including unity8.log14:05
tsdgeosk14:06
tsdgeosi'll get some more debug symbols intsalled14:06
tsdgeosgreyback: this is not "hard" to get it seems14:06
tsdgeoshappens if you run the sdk suite14:06
greybackhmm14:07
greybacknot what I like to hear :(14:07
tsdgeosseems more easily if you have the silo we have that makes qt dbus locking less14:07
tsdgeosbut there's nothing that would seem to blame qtbus in that bt14:08
tsdgeosi'll note it in the bug anyway14:08
tsdgeosjust in case14:08
greybacktsdgeos: what mir version?14:11
=== dandrader is now known as dandrader|afk
tsdgeosQQuickAsyncImageProvider merged \o/14:29
=== davidcalle_ is now known as davidcalle
greybacktsdgeos: nice one14:38
greybacktsdgeos: link? We backported to 5.4?14:38
tsdgeosgreyback: not yet14:40
tsdgeosgreyback: https://codereview.qt-project.org/#/c/108540/14:40
tsdgeosonly needed 17 patchsets :D14:40
tsdgeosand https://code.launchpad.net/~aacid/thumbnailer/asyncprovider is our counterpart14:41
tsdgeoswhich i think doesn't compile now14:41
tsdgeossince i had to change stuff upstream to make them happy14:41
greybacktsdgeos: no worries. just wanna be familiar with the api14:42
tsdgeosthere's not much14:42
tsdgeosbasically you get a job14:42
tsdgeosand then you signal done()14:42
tsdgeosit's up to you to spawn threads and stuff14:42
tsdgeosi think it's not done() but finished()14:42
tsdgeos:D14:42
tsdgeosbut you know what i mean14:42
greybackgotcha14:48
=== dandrader|afk is now known as dandrader
=== lool- is now known as lool
tsdgeoslol15:19
tsdgeosmake testDialogs was doing nothing15:19
tsdgeos:D15:19
tsdgeosMirv: \o/15:20
tsdgeosLoading tests from: /usr/lib/python3/dist-packages15:20
tsdgeosTests running...15:20
tsdgeosRan 365 tests in 2579.659s15:20
tsdgeosOK15:20
tsdgeosgreyback: that silo seems ot really help15:20
* tsdgeos launches again15:20
greybacktsdgeos: sweet15:21
Mirvtsdgeos: good that you caught me still, I can still launch a 018+013 full suite then now15:21
Mirvand, \o/ if a working combo of Mir + Qt + Autopilot is found15:23
tsdgeoshe he15:23
tsdgeosmzanetti: test updated https://code.launchpad.net/~aacid/unity8/passphrase_kewboard/+merge/25365715:24
tsdgeosMirv: you probably need my autopilot change too, do you have that one?15:26
mzanettitsdgeos, ack15:27
Mirvtsdgeos: hmm? you mean the one I submitted to ap1.5 and added to the PPA, or something new?15:27
tsdgeosMirv: that one15:28
tsdgeosah didn't know you had it in the ppa already15:28
tsdgeosgood then15:28
Mirvtsdgeos: right, the MP is there in the PPA.15:28
dandradertsdgeos, yeah, it's there only for its "try" version15:28
tsdgeosdandrader: lol right i broke the try now15:29
tsdgeosmzanetti: there was no test for this functionality https://code.launchpad.net/~aacid/unity8/urldispatcher_hideeverything/+merge/253527 originally15:57
tsdgeosbut i guess i can write one for my fix15:58
tsdgeosand for the feature itself15:58
tsdgeoswill take a bit but it's good to have another test15:58
faeniltsdgeos: can any of the properties of a Result be used as a unique id?16:22
faenilto look for an item and similar tasks16:22
tsdgeospstolowski: ↑↑16:22
faeniluri is probably not unique enough16:22
tsdgeosfaenil: result is16:24
tsdgeossince it's basically the pointer16:24
tsdgeos        case RoleResult:16:24
tsdgeos            return QVariant::fromValue(std::static_pointer_cast<unity::scopes::Result>(m_results.at(index.row())));16:24
tsdgeosnot sure the mock implements it that way16:24
tsdgeosif it doesn't it should :D16:24
tsdgeosor at least return something that is unique16:25
faeniltsdgeos: in the mocks it's a string16:26
faenilbut ok, thanks!16:26
tsdgeosfaenil: but is it unique?16:26
faenilyeah,16:26
tsdgeosthen it's "ok216:27
tsdgeos"16:27
pstolowskifaenil, none of the results' attributes should be treated as a key, we do not impose any constraints on any values to be unique.. as tsdgeos says you may relay on the pointers, but this is of course a hack/workaround. note, when you do a new search or refresh, you get new result instances even though the data looks the same16:31
faenilpstolowski: mmm ok...16:33
* tsdgeos missed something16:33
tsdgeoswhat's the ok for?16:33
pstolowskifaenil, on new search we basically wipe the model and create new result instances16:34
faenilpstolowski: I see16:34
faeniltsdgeos: in reply to pstolowski16:34
tsdgeosoki16:34
pstolowskitsdgeos, fyi "faenil, none of the results' attributes should be treated as a key, we do not impose any constraints on any values to be unique.. as tsdgeos says you may relay on the pointers, but this is of course a hack/workaround. note, when you do a new search or refresh, you get new result instances even though the data looks the same"16:36
tsdgeossure16:37
tsdgeosi meant that for a given set of results16:37
tsdgeosyou can use the pointer as key16:37
tsdgeosbut won't be stable or regresh or something16:37
pstolowskitsdgeos, ok.. i didn't know what is it for16:37
tsdgeospstolowski: me neither :D16:38
=== dandrader is now known as dandrader|lunch
om26ermzanetti, Hi! who is the best person for bug 1436982 ?17:58
ubot5bug 1436982 in unity8 (Ubuntu) "Unity freezes for a few seconds when 'clear messages' is tapped in Notifications" [High,New] https://launchpad.net/bugs/143698217:58
om26eractually it should be dednick ^17:59
mzanettiprobably17:59
mzanettilet me try to reproduce17:59
om26er'Clear all' button.18:02
mzanettiyeah... so far not happened. but only had telegram notifications in it. let me try with a missed call18:02
=== alan_g is now known as alan_g|EOD
om26ermzanetti, same happens when you change volume from sound Menu18:05
om26erwhen notification appears, try to flick close the menu up18:06
mzanettihmm... no problem here...18:06
mzanettiom26er, maybe try checking if unity8.log prints something suspicious18:06
om26ermzanetti, on rtm right now, will flash and update you/bug_report.18:09
=== dandrader|lunch is now known as dandrader
dednickmzanetti, om26er: yeah, that's probably me.18:16
mzanettiack18:16
om26erdednick, mzanetti try to clear while the notification is still on screen.18:22
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
=== greyback__ is now known as greyback
=== tmpRAOF is now known as RAOF

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