Merge lp:~sylvain-pineau/checkbox/camera_test_cheese_introspection into lp:checkbox

[Sylvain Pineau]

This MR removes all gstreamer pipelines from the camera_test script. This new version relies on libcheese as creating a camera bin is not something we want to maintain in the checkbox provider.

WARNING:

DO not merge without the packaging branch update first.

[Taihsiang Ho (tai271828)]

"./camera still"
works well on dino2 (CID 201508-18005) besides one small inline comment (please see below)

-------------

"./camera display"
crashed. The error message is:

http://paste.ubuntu.com/15760725/

./camera_test:570: PyGIWarning: Cheese was imported without specifying a version first. Use gi.require_version('Cheese', '3.0') before import to ensure that the right version gets loaded.
  from gi.repository import Cheese
Traceback (most recent call last):
  File "./camera_test", line 575, in <module>
    sys.exit(getattr(camera, args.test)())
  File "./camera_test", line 248, in display
    camera.set_device_by_device_node(self.args.device)
AttributeError: 'Camera' object has no attribute 'set_device_by_device_node'

review: Needs Fixing

[Sylvain Pineau (sylvain-pineau)]

Thnaks for the review Tai. I made a few changes to support Xenial, resubmitting for review

review: Needs Resubmitting

[Taihsiang Ho (tai271828)]

on dino2 (CID 201508-18005)

* "./camera still" works well
* "./camera display" pops up a window and there is nothing (all white panel) in the window together with the following message in the console:

ubuntu@201508-18805:~/camera_test_cheese_introspection/providers/plainbox-provider-checkbox/bin$ ./camera_test display

(python3:2218): cheese-WARNING **: Internal GStreamer error: code not implemented. Please file a bug at http://bugzilla.gnome.org/enter_bug.cgi?product=GStreamer.: gstvideofilter.c(292): gst_video_filter_transform (): /GstCameraBin:camerabin/GstViewfinderBin:vf-bin/GstVideoConvert:vfbin-csp:
invalid video buffer received

(python3:2218): cheese-WARNING **: Internal GStreamer error: code not implemented. Please file a bug at http://bugzilla.gnome.org/enter_bug.cgi?product=GStreamer.: gstvideofilter.c(292): gst_video_filter_transform (): /GstCameraBin:camerabin/GstViewfinderBin:vf-bin/GstVideoConvert:vfbin-csp:
invalid video buffer received

[Taihsiang Ho (tai271828)]

test on another XPS (CID 201309-14238)

both of still and display works.

[Sylvain Pineau (sylvain-pineau)]

Another xenial api change... the camera constructor expects a Clutter.Actor instead of a Clutter.Texture. I changed the code to handle both cases. Tested the daily build it works for me. Could you do another round with the dino hw please?

review: Needs Resubmitting

[Pierre Equoy (pieq)]

Tested successfully on:

- One Dino 2 device with Trusty
- One Dino 2 device with Xenial
- One Chiclet 15 device with Xenial

In all cases, I got the expected results for camera/detect, camera/still, camera/display and camera/multiple=resolution-images tests (from the Certification Test Plan).

review: Approve

[Taihsiang Ho (tai271828)]

more info:

my above test on a XPS (CID 201309-14238) was tested on Trusty. Sorry for unclear statement.

https://code.launchpad.net/~sylvain-pineau/checkbox/camera_test_cheese_introspection/+merge/291499/comments/747099

[Taihsiang Ho (tai271828)]

tested on a XPS (CID 201309-14238) with Xenial daily build (built on Apr. 7th)
both of camera still and display looks good.

review: Approve

[Sylvain Pineau (sylvain-pineau)]

No proposals found for merge of https://code.launchpad.net/~sylvain-pineau/checkbox/ppa-packaging-plainbox-provider-checkbox-fix-1568664 into https://code.launchpad.net/~checkbox-dev/checkbox/trunk.

[Sylvain Pineau (sylvain-pineau)]

I'm creating a new version of the same MR in order to remove the prerequisite branch. It was a mistake a the targeted branch was not trunk but the ppa packaging branch. Tarmac didn’t like that

review: Approve

=== modified file 'providers/plainbox-provider-checkbox/bin/camera_test'
--- providers/plainbox-provider-checkbox/bin/camera_test	2016-02-13 13:21:18 +0000
+++ providers/plainbox-provider-checkbox/bin/camera_test	2016-04-13 08:13:58 +0000
@@ -164,13 +164,10 @@
     """
     A simple class that displays a test image via GStreamer.
     """
-    def __init__(self, args, gst_plugin=None, gst_video_type=None):
+    def __init__(self, args):
         self.args = args
-        self._mainloop = GObject.MainLoop()
         self._width = 640
         self._height = 480
-        self._gst_plugin = gst_plugin
-        self._gst_video_type = gst_video_type
 
     def detect(self):
         """
@@ -204,51 +201,62 @@
                   else '',
                   ' ]', sep="")
 
-            resolutions = self._get_supported_resolutions(device)
-            print('    ',
-                  self._supported_resolutions_to_string(resolutions).replace(
-                      "\n", " "),
-                  sep="")
+            resolutions = self._supported_resolutions_to_string(
+                self._get_supported_resolutions(device))
+            resolutions = resolutions.replace(
+                "Resolutions:", "    Resolutions:")
+            resolutions = resolutions.replace("Format:", "    Format:")
+            print(resolutions)
 
             if cp.capabilities & V4L2_CAP_VIDEO_CAPTURE:
                 cap_status = 0
         return dev_status | cap_status
 
+    def _on_destroy(self, *args):
+        Clutter.main_quit()
+
+    def _take_photo(self, *args):
+        Cheese.Camera.take_photo(self.camera, self.filename)
+
     def led(self):
         """
         Activate camera (switch on led), but don't display any output
         """
-        pipespec = ("v4l2src device=%(device)s "
-                    "! %(type)s "
-                    "! %(plugin)s "
-                    "! testsink"
-                    % {'device': self.args.device,
-                       'type': self._gst_video_type,
-                       'plugin': self._gst_plugin})
-        logging.debug("LED test with pipeline %s", pipespec)
-        self._pipeline = Gst.parse_launch(pipespec)
-        self._pipeline.set_state(Gst.State.PLAYING)
-        time.sleep(3)
-        self._pipeline.set_state(Gst.State.NULL)
+        Clutter.threads_add_timeout(0, 3000, self._on_destroy, None, None)
+        video_texture = Clutter.Actor()
+        try:
+            camera = Cheese.Camera.new(
+                video_texture, self.args.device, self._width, self._height)
+        except TypeError: # libcheese < 3.18 still use Clutter.Texture
+            video_texture = Clutter.Texture()
+            camera = Cheese.Camera.new(
+                video_texture, self.args.device, self._width, self._height)
+        Cheese.Camera.setup(camera, None)
+        Cheese.Camera.play(camera)
+        Clutter.main()
 
     def display(self):
         """
         Displays the preview window
         """
-        pipespec = ("v4l2src device=%(device)s "
-                    "! %(type)s,width=%(width)d,height=%(height)d "
-                    "! %(plugin)s "
-                    "! autovideosink"
-                    % {'device': self.args.device,
-                       'type': self._gst_video_type,
-                       'width': self._width,
-                       'height': self._height,
-                       'plugin': self._gst_plugin})
-        logging.debug("display test with pipeline %s", pipespec)
-        self._pipeline = Gst.parse_launch(pipespec)
-        self._pipeline.set_state(Gst.State.PLAYING)
-        time.sleep(10)
-        self._pipeline.set_state(Gst.State.NULL)
+        stage = Clutter.Stage()
+        stage.set_title('Camera test')
+        stage.set_size(self._width, self._height)
+        stage.connect('destroy', self._on_destroy)
+        Clutter.threads_add_timeout(0, 10000, self._on_destroy, None, None)
+        video_texture = Clutter.Actor()
+        try:
+            camera = Cheese.Camera.new(
+                video_texture, self.args.device, self._width, self._height)
+        except TypeError: # libcheese < 3.18 still use Clutter.Texture
+            video_texture = Clutter.Texture()
+            camera = Cheese.Camera.new(
+                video_texture, self.args.device, self._width, self._height)
+        stage.add_actor(video_texture)
+        Cheese.Camera.setup(camera, None)
+        Cheese.Camera.play(camera)
+        stage.show()
+        Clutter.main()
 
     def still(self):
         """
@@ -272,7 +280,7 @@
                    "-d", self.args.device,
                    "-r", "%dx%d"
                    % (width, height), filename]
-        use_gstreamer = False
+        use_cheese = False
         if pixelformat:
             if 'MJPG' == pixelformat:  # special tweak for fswebcam
                 pixelformat = 'MJPEG'
@@ -281,40 +289,37 @@
         try:
             check_call(command, stdout=open(os.devnull, 'w'), stderr=STDOUT)
         except (CalledProcessError, OSError):
-            use_gstreamer = True
+            use_cheese = True
 
-        if use_gstreamer:
-            pipespec = ("v4l2src device=%(device)s "
-                        "! %(type)s,width=%(width)d,height=%(height)d "
-                        "! %(plugin)s "
-                        "! jpegenc "
-                        "! filesink location=%(filename)s"
-                        % {'device': self.args.device,
-                           'type': self._gst_video_type,
-                           'width': width,
-                           'height': height,
-                           'plugin': self._gst_plugin,
-                           'filename': filename})
-            logging.debug("still test with gstreamer and "
-                          "pipeline %s", pipespec)
-            self._pipeline = Gst.parse_launch(pipespec)
-            self._pipeline.set_state(Gst.State.PLAYING)
-            time.sleep(3)
-            self._pipeline.set_state(Gst.State.NULL)
+        if use_cheese:
+            stage = Clutter.Stage()
+            stage.connect('destroy', self._on_destroy)
+            video_texture = Clutter.Actor()
+            try:
+                self.camera = Cheese.Camera.new(
+                    video_texture, self.args.device, self._width, self._height)
+            except TypeError: # libcheese < 3.18 still use Clutter.Texture
+                video_texture = Clutter.Texture()
+                self.camera = Cheese.Camera.new(
+                    video_texture, self.args.device, self._width, self._height)
+            Cheese.Camera.setup(self.camera, None)
+            Cheese.Camera.play(self.camera)
+            self.filename = filename
+            Clutter.threads_add_timeout(0, 3000, self._take_photo , None, None)
+            Clutter.threads_add_timeout(0, 4000, self._on_destroy, None, None)
+            Clutter.main()
+            Cheese.Camera.stop(self.camera)
 
         if not quiet:
-            import imghdr
-            image_type = imghdr.what(filename)
-            pipespec = ("filesrc location=%(filename)s ! "
-                        "%(type)sdec ! "
-                        "videoscale ! "
-                        "imagefreeze ! autovideosink"
-                        % {'filename': filename,
-                           'type': image_type})
-            self._pipeline = Gst.parse_launch(pipespec)
-            self._pipeline.set_state(Gst.State.PLAYING)
-            time.sleep(10)
-            self._pipeline.set_state(Gst.State.NULL)
+            stage = Clutter.Stage()
+            stage.set_title('Camera still picture test')
+            stage.set_size(width, height)
+            stage.connect('destroy', self._on_destroy)
+            Clutter.threads_add_timeout(0, 10000, self._on_destroy, None, None)
+            still_texture = Clutter.Texture.new_from_file(filename)
+            stage.add_actor(still_texture)
+            stage.show()
+            Clutter.main()
 
     def _supported_resolutions_to_string(self, supported_resolutions):
         """
@@ -563,16 +568,20 @@
 
     # Import Gst only for the test cases that will need it
     if args.test in ['display', 'still', 'led', 'resolutions']:
+        import contextlib
+        # Workaround to avoid "cluttervideosink missing"
+        # See https://bugzilla.gnome.org/show_bug.cgi?id=721277
+        with contextlib.suppress(FileNotFoundError):
+            gst_registry = '~/.cache/gstreamer-1.0/registry.x86_64.bin'
+            os.remove(os.path.expanduser(gst_registry))
+        import gi
+        gi.require_version('Gst', '1.0')
         from gi.repository import Gst
-        if Gst.version()[0] > 0:
-            gst_plugin = 'videoconvert'
-            gst_video_type = 'video/x-raw'
-        else:
-            gst_plugin = 'ffmpegcolorspace'
-            gst_video_type = 'video/x-raw-yuv'
+        gi.require_version('Cheese', '3.0')
+        from gi.repository import Cheese
+        gi.require_version('Clutter', '1.0')
+        from gi.repository import Clutter
         Gst.init(None)
-        camera = CameraTest(args, gst_plugin, gst_video_type)
-    else:
-        camera = CameraTest(args)
-
+        Clutter.init()
+    camera = CameraTest(args)
     sys.exit(getattr(camera, args.test)())

=== modified file 'providers/plainbox-provider-checkbox/jobs/camera.txt.in'
--- providers/plainbox-provider-checkbox/jobs/camera.txt.in	2015-09-08 22:04:46 +0000
+++ providers/plainbox-provider-checkbox/jobs/camera.txt.in	2016-04-13 08:13:58 +0000
@@ -1,3 +1,7 @@
+unit: packaging meta-data
+os-id: debian
+Depends: gir1.2-cheese-3.0
+
 plugin: shell
 category_id: 2013.com.canonical.plainbox::camera
 id: camera/detect