After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 723364 - deinterlace: add RGB16 and BRG16 support
deinterlace: add RGB16 and BRG16 support
Status: RESOLVED WONTFIX
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other All
: Normal enhancement
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-01-31 12:59 UTC by Julien Isorce
Modified: 2014-02-04 15:42 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
deinterlace: add RGB16 / BGR16 support (3.51 KB, patch)
2014-01-31 12:59 UTC, Julien Isorce
needs-work Details | Review

Description Julien Isorce 2014-01-31 12:59:16 UTC
Created attachment 267726 [details] [review]
deinterlace: add RGB16 / BGR16 support

* actual result: an avi file generated with gst-launch-1.0 videotestsrc num-buffers=200 ! "video/x-raw, format=RGB16" ! avimux ! filesink location=rgb16.avi just fails to play with playbin (with default flags)

* see https://bugzilla.gnome.org/show_bug.cgi?id=723357 and https://bugzilla.gnome.org/show_bug.cgi?id=723358

* with the attached patch the rgb16.avi can be played with playbin but I guess deinterlace enter in passthrough mode.

Because if I do:

gst-launch-1.0 videotestsrc !  interlace ! videoconvert ! deinterlace ! "video/x-raw, format=RGB16" ! videoconvert ! ximagesink

there are some artefacts around the ball curve 

I guess this is because I did not create dedicated deinterlace_frame_rgb16 functions :)
Comment 1 Sebastian Dröge (slomo) 2014-02-04 12:46:39 UTC
Comment on attachment 267726 [details] [review]
deinterlace: add RGB16 / BGR16 support

Yes, you need to add special RGB16/BGR16 processing functions.

Also RGB16... really?! :)
Comment 2 Julien Isorce 2014-02-04 13:37:11 UTC
(In reply to comment #1)
> (From update of attachment 267726 [details] [review])
> Yes, you need to add special RGB16/BGR16 processing functions.
> 
> Also RGB16... really?! :)

And even implementing them it would be very slow on RPI :)

Would it be possible that deinterlace just always work in passthrough when it does not know how to deinterlace ? Because it fails right now if input is RGB16 (interleaved or even if progressive):

gst-launch-1.0 videotestsrc ! "video/x-raw, format=RGB16, interlace-mode=progressive"  !  deinterlace mode = 2 ! fakesink


ERROR:gstdeinterlace.c:426:gst_deinterlace_set_method: assertion failed: (method_type != G_TYPE_INVALID)
Aborted
Comment 3 Sebastian Dröge (slomo) 2014-02-04 15:42:50 UTC
There could be such an auto mode, yes. Mateo is working on that currently, there's another bug for that.