=== MacSlow is now known as MacSlow|lunch === dandrader is now known as dandrader|afk === dandrader|afk is now known as dandrader [14:56] dandrader, can you review my grail merge proposals soon? [14:57] we need to get them in, and your fix in today [14:57] before the beta 2 freeze tomorrow [14:57] cnd, sure. Right after I finish the refactoring I'm doing on my merge proposal as per your comment [14:57] comments [14:57] cool [14:57] thanks :) [15:15] Spending some quality time with Jenkins and Google Test/Google Mock, Prepared packaging for patched chromium [15:16] I got utouch-geis up-to-speed on libxorg-gtest-dev, now back to the grind on #904731 and #944822, pending merge-request-reviews [15:17] I'm refactoring the test in my merge proposal for utouch-grail according to comments received from Chase [15:18] I'm fixing up utouch-frame for the new xorg-gtest, and then I'll be releasing frame and grail upstream and for precise [15:18] Satoris won't be around for standups on wednesdays for a while [15:18] just fyi [15:19] Wednesdays have always been special to his people [15:26] bregma, just reviewed your branch [15:53] I'm stumped by this failure: http://paste.ubuntu.com/893791/ [15:53] it attempts to link a program [15:53] it complains about not finding the evemu symbols in libxorg-gtest.a [15:54] but I have -lutouch-evemu before libxorg-gtest.a on the command line [15:54] link order is significant [15:54] yeah, and I think I have it in the right order [15:54] no, it's in the reverse order [15:55] argh [15:55] yeah [15:55] I got switched up [15:55] now fight with libtool to get things in the right order [15:56] although that bug ni libtool was probably fixed years ago === dandrader is now known as dandrader|lunch [16:29] dandrader|lunch, I just reviewed your branch [16:29] looking good :) [16:36] alright, I've got all the merge proposals out that I wanted to get into beta 2 [16:37] as soon as they are all merged into grail and frame I'll be making releases [16:37] biab === dandrader|lunch is now known as dandrader [18:11] cnd: ping [18:11] Satoris, pong [18:11] oh right [18:11] sorry! [18:11] I should have replied [18:21] cnd, about https://code.launchpad.net/~chasedouglas/utouch-grail/fix-tap-touch-accept/+merge/98478 [18:22] what's the effect of accepting of rejecting a touch that has already ended? [18:22] s/of/or [18:32] dandrader, all touches must be accepted or rejected no matter what [18:32] if a touch has ended, then when you reject, the entire touch sequence will be replayed to further X clients [18:32] so imagine you have a finger paint app open [18:33] you tap on the screen with one finger [18:33] the touch ends before grail rejects the tap [18:33] grail still needs to reject it, and then the X server will resend the touch event sequence to the finger painting app [18:35] ChickenCutlass, you mean it will resend the touch event sequence to a X client other than the finger painting app, right? [18:35] cnd, ^ [18:36] damn auto-completion [18:37] dandrader, no, it will resent it to the finger painting app [18:37] here, I'm assuming grail is being used through unity [18:37] so unity's grail touch grab is rejecting the touch [18:37] and then the finger paint application will receive the touch sequence [18:38] ah, ok === dandrader is now known as dandrader|afk [20:27] dandrader|afk, bregma: I need one last review: https://code.launchpad.net/~chasedouglas/utouch-frame/fix-visibility2/+merge/98725 [20:27] it's necessary before we make a release of utouch-frame === dandrader|afk is now known as dandrader [20:29] dandrader, btw, you don't need to fully resubmit a merge proposal [20:29] merely updating the branch will suffice :) [20:30] you can signify that you've fixed things and it is ready for review again by leaving a comment with a review type of "Resubmit" [20:33] ok, that will probably make the review simpler as you can just look at the new commits containing the changes caused by the review comments [20:33] than I can squash them all before pushing to trunk [20:33] s/than/then [20:39] dandrader, you don't really need to squash them even [20:39] you just merge them in [20:40] and they'll appear to be one commit [20:40] then you can still drill down into the history if you really want [20:45] but those commits that are just fixes coming from review comment just clutter the history [20:51] dandrader, if you don't want to see the clutter, then just diff from the beginning to the end of the rview [20:51] which will normally be the diff of the merge itself [20:51] dandrader, your testcase is failing [20:51] it's in the jenkins instance [20:56] dandrader, I can confirm the failure here [20:56] I need to grab a sandwich [20:56] then I'll try to figure out what's going wrong [20:56] it looks like bad events are being fed into frame though... [20:56] that would cause the "Warning: failed to get previous touch value" [21:12] damn, I'm not getting any failure here with current trunk versions of utouch-frame and utouch-grail... [21:13] dandrader, I don't have the current frame installed, let me double check [21:15] from the error it seems that nothing was played [21:16] as the slice checker was still on its first state, which check for a Begin slice from the touch gesture [21:17] cnd, have you run that ParallelAtomicGestures test before on your machine? [21:17] dandrader, not until just now [21:17] I assumed it would work [21:17] :) [21:18] * cnd can't wait until we have proper jenkins support [21:18] could you check if the recording plays correctly on your machine? [21:18] yeah, I'm just setting up frame right now [21:18] from scratch [21:18] to be sure [21:19] dandrader, with the latest utouch-frame it fails in the same way here [21:23] dandrader, I just noticed that you copied in the asynchronous playing of the recording [21:23] which I just removed from all the other tests :) [21:23] I should have caught that in review [21:24] but sadly it's not the cause of the failure [21:24] dandrader, here's my output with grail debugging: http://paste.ubuntu.com/894298/ [21:25] let me know if you can make anything out [21:25] * dandrader reads... [21:28] that touch 4 is coming way too soon [21:28] with the same timestamp as touch 3 [21:29] causing all gestures to be canceled before they have a chance to do anything [21:29] when I did the recording the 4th touch should come just by the end of the recording [21:30] * cnd tries to read the recording [21:32] the fourth touch starts at 1331926701.180596 [21:32] the beginning is 1331926700.835657 [21:32] the end is 1331926701.548991 [21:32] so it's about half way between the beginning and the end [21:33] but the whole recording is less than one second long [21:33] does that seem right? [21:34] in the debug output, gestures 0, 1, and 2 start at time 691389450 [21:34] the first touch began at time 691389450 [21:34] same time [21:35] the fourth touch begins at time 691389450 [21:35] which is the same time again... [21:35] that doesn't seem right [21:38] could you try to play it with evemu-play and watch it with mtview? [21:40] yeah, [21:41] cnd, http://ubuntuone.com/3zGHZA1ILFUm6LgFrhEYOZ [21:41] this is what I get in the end [21:41] the pink blot is the fourth touch that comes late [21:44] dandrader, using evtest, I see the three touches begin at time 1332366154.522432, and then the fourth touch begins at time 1332366154.879781, [21:44] so it is coming out of evdev about .35 seconds later than the first three touches [21:45] arthritis slows you down [21:45] which is accurate, according to the recording [21:45] so the touch events are getting coallesced in the x servre [21:45] server* [21:46] 0.35 s is a long time for the server to be sitting around... [21:51] since grail now deals with timestamps, it would be nice if in the tests we could feed the events directly to grail instead of sending via an evdevice, through xserver, in realtime. Then all tests could be run in a split second and there would be less external interferences in the tests (if any) [21:52] I wonder if we are running the same version of xserver... [21:53] dandrader, then we would be wondering why gesture recognition isn't occurring properly in real life :( [21:53] we could do what you suggest, but we would still need integration tests [21:53] I'm looking at the X code [21:54] and it gets its time from either clock_gettime() or from gettimeofday() [21:54] and events are processed in sigio context, so they should be getting their timestamps immediately [21:54] so what's going on... [21:54] * cnd goes to build a debug server with timestamp printouts [21:58] I think it would be good to have both kinds of tests. [21:59] yeah, no doubt [21:59] but I think if we have only one, they should be integration [21:59] I agree [21:59] we need to find the resources for more testing :) [22:01] actually, something doesn't seem right... [22:02] wait, nm [22:03] I'm wondering if the first events aren't being posted [22:03] they are just getting dropped until the fourth touch comes along [22:03] dandrader, is this on a trackpad? [22:03] I know what's going on... [22:03] yes, the apple wireless one [22:04] dandrader, the default X synaptics properties are filtering touch events until 4 touches are present [22:04] ah! one thing is that I don't have the synaptics x driver installed on my machine! [22:04] aha [22:05] dandrader, if you redo the recording with 4 and 5 touches instead of 3 and 4, it should pass the test just fine [22:06] though this raises the question of why in ubuntu X synaptics is withholding 3 touch gestures [22:06] it shouldn't be doing that [22:09] hmm, I'm getting touch events [22:10] oh wait [22:11] no, I'm not [22:11] why... [22:16] ok, there's a bad default in xf86-input-synaptics [22:18] aha [22:18] ok, I know what has happened [22:18] I need to upload a fix to xf86-input-synaptics [22:19] dandrader, when I fix xserver-xorg-input-synaptics in an upload I'm making today, it should start working again [22:19] great! [22:19] no need to make any changes, though the test needs to be made non-async [22:19] so we can run it through valgrind [22:19] if you're at your EOD, it can wait [22:20] phew, I was worried X timestamps were so unreliable they were coallescing events .35 seconds apart :) [22:21] dandrader, make sure you have synaptics installed from now on :) [22:21] just installed it and rebooted [22:21] because I can't seem to be able to connect to my trackpad still [22:22] hmm? you can connect when using evdev but not synaptics? [22:23] before I installed the synaptics driver I was able to pair with it effortlessly. but after I installed I cannot pair. might just be a coincidence... [22:24] well, will get back to it tomorrow. but hey, in the end you found a new bug! :) [23:03] bregma, will you be uploading utouch-geis soon? [23:03] we need to get the tap fixes into precise for beta 2, at the very least