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

=== Malsasa_ is now known as Malsasa
=== marcust__ is now known as marcustomlinson
dandradermzanetti, gdbus call --session --dest com.canonical.UnityGreeter --object-path / --method com.canonical.UnityGreeter.HideGreeter08:42
Saviqpstolowski, pete-woods, remote scopes crashing unity8: bug #1418176, actually hoping it's related to bug #1410385 that we don't have any trace on yet08:42
ubot5bug 1418176 in unity-scopes-shell (Ubuntu) "/usr/bin/unity8-dash:11:constData:operator:qMapLessThanKey:lowerBound:QMapData" [High,Confirmed] https://launchpad.net/bugs/141817608:42
ubot5Error: Launchpad bug 1410385 could not be found08:42
pete-woodsSaviq: thanks08:42
pete-woodsSaviq: have a fix already :)08:55
pstolowskiSaviq, we have a fix for that in silo 8, but currently fighting with test failures there08:55
Saviqpete-woods, :)08:55
tsdgeospstolowski: what do you think of my comment in https://code.launchpad.net/~stolowski/unity-scopes-shell/fix-temp-scopes/+merge/248235 ?09:22
pstolowskitsdgeos, yeah, i've a branch09:22
tsdgeosCimi: and you from https://code.launchpad.net/~cimi/unity8/fix-open-new-scope-from-tmp/+merge/248538 ?09:22
tsdgeos:D09:22
pstolowskitsdgeos, tbh i'm not totally sure why we need that, i guess it's for you to be able to mock stuff?09:23
tsdgeospstolowski: you mean having the api in a separate place?09:24
tsdgeospstolowski: yeah so that we have an expectation of what the api is going to be09:24
tsdgeosand we can mock it09:24
pstolowskitsdgeos, having the abstract scopes interface... yeah, ok09:24
tsdgeosand it doesn't happen that you change it behind our feet without us realizing09:24
tsdgeoswhich may be easier to happen if it was all part of your project alone09:25
tsdgeospstolowski:09:29
tsdgeos+ libunity-api-dev (>= 7.95),09:29
tsdgeoslibunity-api-dev (>= 7.94),09:29
tsdgeossecond needs to be a -09:29
pstolowskitsdgeos, oh, a mistake in conflict resolution09:30
pstolowskitsdgeos, pushed09:31
tsdgeospstolowski: https://code.launchpad.net/~stolowski/unity-scopes-shell/fix-temp-scopes/+merge/248235 looks good to me, want me to top approve or prefer someone else to review too?09:47
pstolowskitsdgeos, top approve pls, thanks!09:51
tsdgeospstolowski: btw closing from scopes makes so much sense09:52
tsdgeos\o/09:52
tsdgeospstolowski: was wondering if we even wanted to kill the possibility of closing from scope09:52
tsdgeossince basically now it's just callign scopes09:52
tsdgeosit's not even checking if the scope was opened though that scope09:53
tsdgeosbut i guess it's fine09:53
Saviqtsdgeos, pstolowski, there's one reason (I think) why we had it in Scope, and that's a stack of scopes09:56
Saviqi.e. you go Apps > Store > NewScope > YouTube > Store09:57
Saviqyou have two Store scopes, and you want to retain its history09:57
pstolowskitsdgeos, yeah. i left it exposed in Scope not to break any of your code.09:57
tsdgeosSaviq: that won't work atm09:57
tsdgeosyou'll get a gotoscope on the last one09:58
tsdgeoswell we dont' support stacks anyway :D09:58
Saviqtsdgeos, don't think so?09:58
Saviqtsdgeos, you only  get gotoScope for favs09:58
tsdgeosSaviq: yes with the code pstolowski just did09:58
tsdgeosi think09:58
Saviqtsdgeos, but anyway, yeah, we have no stack, and it doesn't actually seem like we will09:58
Saviqanyone seen MacSlow today?09:59
tsdgeosnope09:59
Saviqseb128, http://people.canonical.com/~platform/citrain_dashboard/#?distro=ubuntu&q=landing-00810:06
seb128Saviq, that fixes it10:10
seb128thanks10:10
Saviqcoolz10:10
seb128now what about qtmir/gtk? ;-)10:10
Saviqseb128, they're testing the silo10:16
seb128great10:17
Saviqkgunn, I removed the sync now to prevent accidental rebuilds10:18
=== alan_g is now known as alan_g|reboot
=== Malsasa_ is now known as Malsasa
Cimitsdgeos, but Scope::activate(QVariant const& result) is ever being called?11:54
tsdgeosCimi: of course11:55
Cimitsdgeos, by what?11:55
Cimitsdgeos, confused about those mocks11:56
tsdgeostsdgeos_work@xps:~/phablet/unity8/unity8/qml/Dash$ wcgrep "activate("11:56
tsdgeos./GenericScopeView.qml:72:            // TODO Technically it is possible that calling activate() will make the scope emit11:56
tsdgeos./GenericScopeView.qml:75:            scope.activate(result)11:56
tsdgeos./ScopesList.qml:126:                        onRequestActivate: root.scope.activate(result);11:56
Cimiah I see11:56
=== Malsasa is now known as Guest73507
=== Malsasa_ is now known as Malsasa
Cimitsdgeos, because the previous  close scope was checking m_openScope which changed only from the activate call12:12
Cimitsdgeos, this activate call only seem to happen when someone clicks an item inside the opened scope12:12
Cimithat's why I removed that code inside the mock, it looked confusing12:12
tsdgeoswell you're aware there's test code using it, no?12:13
tsdgeosi mean you just removed part of the test12:13
tsdgeosi suggest you just stop using the scopeToCallCloseOn12:14
tsdgeosor wathever weird name i used12:14
tsdgeoscan't remember now12:14
tsdgeosand just call closeScope on scopes12:14
Cimiyeah12:14
tsdgeosand put an assert on Scope::closeScope mock to make it clear we don't want to use it12:14
Cimitsdgeos, shall I add something inside the new close inside scopes?12:15
tsdgeosand make Scopes::closeScope actually work (and assert if the scope you want to close is not the one that was activated)12:15
Cimifor the mock12:15
Cimiso I need to create this temp scopes set too?12:15
CimiI don't understand the boundaries of mocks and what is needed and what not12:15
tsdgeoswell you need to make it work, otherwise how are you going to test that clicking in that button that is supposed to bring you to another scope actually brought you to another scope?12:17
tsdgeosthere's no need to create/destroy them12:18
tsdgeoswe have a few scopes in the mock that are not there by default12:18
tsdgeosso you can just do what the old closeScope did, store the pointer in a variable/list and make sure the close you're getting is in that variable/list12:18
=== MacSlow is now known as MacSlow|lunch
=== alan_g is now known as alan_g|lunch
=== alan_g|lunch is now known as alan_g
Saviqtsdgeos, hmm, our history popup in the dash appears offscreen, have we been touching that code recently?13:52
tsdgeosSaviq: not that i remember13:52
* Saviq goes to #sdk13:53
mzanettiDaekdroom: https://code.launchpad.net/~mzanetti/unity8/launcher-fixes/+merge/24876114:05
mzanettidandrader: https://code.launchpad.net/~mzanetti/unity8/launcher-fixes/+merge/24876114:05
dandradermzanetti, cool. reviewing it now14:06
DaekdroomHm... do I have to review it as well? :)14:07
=== marcusto_ is now known as marcustomlinson
mterryIs anyone familiar with what differences exist between tryTest and testTest?  I've got this problem where a DirectionalDragArea isn't working in testTest but does in tryTest...  :(14:26
mterrydandrader, ^?14:26
=== MacSlow|lunch is now known as MacSlow
MacSlowmterry, tryTest is interactive... while testTest is automatic14:31
mterryMacSlow, yeah, but why would drags be affected?14:31
mterryMacSlow, something seems to be stopping all touch/mouse interactions in test mode14:32
MacSlowmterry, hm... not sure... I've not run into such issues with drag-related QML-tests (for notifications)14:33
MacSlowmterry, can you be more specific?14:33
mterry:(  It also affects dragging down the indicators and the launcher14:33
mterryMacSlow, in test mode, clicks don't seem to register.  I've removed all code in init() and use a sample test with wait(100000).  So no test code is messing things up.  But for some reason, testTest fails to take input while tryTest is fine14:34
mterryThis is only with a branch I have that is trying to merge in another branch.  Both branches are fine by themselves.  But the size of the branches makes it hard to narrow down the change14:35
mterryAnd I'm not even sure where to look because this is such a weird bug14:35
mterryI don't know where we execute code that is specific to testTest runs vs tryTest runs14:35
dandradermterry, "make testFoo" uses qmltestrunner whereas "make tryFoo" uses uqmlscene14:36
mterrydandrader, huh...14:37
MacSlowmterry, you can have the commandlines for those different make-targets dumped and then copy&paste them to play around...14:37
MacSlowmterry, maybe that'll help to better narrow down your issue14:37
mterryyeah14:37
MacSlowmterry, I did just that for the new mocked NotificationModel in one of the notification-related QML-tests14:38
dandradermterry, with "make test" the tst_Foo.qml file is run by qmltestrunner binary, with "make try", the qml test file is run by the uqmlscene binary. clearer now?14:38
mterrydandrader, yes no it's clear.  I was just trying to remember what I knew about those two and what they did different14:39
mterryI wonder if there is also an environment var diff14:39
dandradermterry, and in uqmlscene we convert mouse events to touch events before sending them to the qml scene14:39
mterryhmmm14:39
dandradermterry, in order to emulate a touchscreen usage in the desktop14:40
dandradermterry, also I don't think you can properly interact with a test in "make testFoo" while it's hanging in a wait()...14:41
MacSlowmterry, btw... I only use mouseClick() and mouseDrag() in the notification QML-tests.14:42
dandraderMacSlow, which is a bad habit as it does not simulate well the usage on touchscreen devices14:42
mterrydandrader, I'm pretty sure you can...  let me test on trunk14:42
facubatistaI just upgraded to Vivid r89 and unity8 seems to have a weird problem, it repeats this every ~10 seconds, in its logs: http://linkode.org/TMgjPipYdh9URrootGEwv314:42
facubatistadoes anybody know what it could be? thanks!14:42
dandradermterry, I recall I had inconsistent and weird results wherever I tries to interact with a test while on wait()..... but maybe I'm wrong14:43
dandrader*whenever14:43
MacSlowdandrader, hm... that was never pointed out in MR-reviews... I'll keep it in mind for the future.14:44
tsdgeosdandrader: https://code.launchpad.net/~dandrader/unity8/fixSurfaceActiveFocus/+merge/24783614:46
dandradertsdgeos, ok14:46
mterrydandrader, you are so right!   wait() blocks me from interacting.  Phew!  So something else is wrong because I have failing tests, but I thought I was seeing something much worse.  I could have sworn I've done that in the past. Thanks for the heads up14:49
MacSlowmterry, mixup between wait() and waitForRendering() maybe?14:49
mterryMaybe...  it would be nice if there *was* a "stop and let user futz with stuff" command that I could insert in a test for testing purposes14:50
dandradertsdgeos, fixed14:51
tsdgeosdandrader: ok, reapproved14:52
dandradertsdgeos, thanks14:52
=== marcusto_ is now known as marcustomlinson_
=== marcustomlinson_ is now known as marcustomlinson
SaviqMacSlow, try running the test under xvfb, loop it, there's probably just some timing issue, I was able to repro quite reliably15:43
MacSlowSaviq, ah ok... will try that15:43
MacSlow Saviq, I removed some wait()s I wasn't fond of... guess I overdone it :)15:44
SaviqMacSlow, let me know if you need some testing done, maybe add debugging etc.15:45
MacSlowSaviq, do we have a specifc make-target for xvfb-run or do I need to do this manually?15:48
SaviqMacSlow, xvfbtestNotifications15:48
MacSlowSaviq, hm... still passes here15:49
MacSlowSaviq, I'll try my slower laptop15:50
SaviqMacSlow, yup, that might trigger it too15:50
MacSlowSaviq, I tend to forget the GeForce-monster that's in this machine15:50
=== marcusto_ is now known as marcustomlinson_
greyback_dandrader: hey, I've taken over https://code.launchpad.net/~mir-team/qtmir/port-to-event-2.0/+merge/248067 - if you have time, could you give it a review pass please?16:56
greyback_dandrader: tomorrow obviously16:56
dandradergreyback_, I'm fine with you taking it over16:59
dandradergreyback_, oh, you took over the branch, not the review17:00
greyback_dandrader: yeah17:00
dandraderso lazy dandrader still has to review it17:01
josharensonAnyone who has build ubuntu-system-settings-online-accounts? It seems that the online-accounts-service isn't finding mirclient and its compiling in mir-helper-stub.cpp instead of mir-helper.cpp. Is this expected? Its giving me a headache w/ trusted sessions.19:13
gQuigswho maintains https://unity.ubuntu.com?19:16
gQuigsI'm looking for an updated version of this page - https://unity.ubuntu.com/getinvolved/development/unity8/#developing-unity19:16
gQuigsvivid current instructions for developing unity8 (specifically want to do desktop, but on an nvidia card if possible)19:16
josharensongQuigs: I'm not sure there is an updated version... Do you have any specific questions?19:17
josharensonah19:17
* josharenson looks for a branch that could help19:18
gQuigsdo I need any PPAs on vivid?  and will it even work with the nvidia proprietary driver?19:18
gQuigsseems it's a wordpress site...19:20
josharensongQuigs: still looking, I actually haven't ran it on the desktop in a while... mostly been working on mobile. As for the nvidia driver, I cannot answer that, maybe try #ubuntu-mir19:20
josharensongQuigs: I'd try running http://cdimage.ubuntu.com/ubuntu-desktop-next/daily-live/current/ from a usb drive19:33
josharensongQuigs: I just tried making it work a hacky way, and broke my computer for a bit19:34
gQuigsjosharenson: oh, I know that doesn't work right now..  so even to develop it only works on open source right now19:35
gQuigsfor some reason I thought it was just not stable for that yet..19:35
gQuigsjosharenson: thanks!19:35
josharensongQuigs: yeah I just saw the docs stating that... np, sorry I couldn't be of more help19:35

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