[gstreamer-bugs] [Bug 595286] New: dvdec rank MARGINAL, yet it is the only dv decoder that does PAR properly

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=595286
  GStreamer | gst-plugins-good | git

           Summary: dvdec rank MARGINAL, yet it is the only dv decoder
                    that does PAR properly
    Classification: Desktop
           Product: GStreamer
           Version: git
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: Normal
         Component: gst-plugins-good
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: wingo at pobox.com
         QAContact: gstreamer-bugs at lists.sourceforge.net
      GNOME target: ---
     GNOME version: ---


If I decode a dv video with HEAD gstreamer, I get this:

$ gst-launch -v filesrc location=/kipple/lumroom.mov ! decodebin !
video/x-raw-yuv ! fakesink
...
/GstPipeline:pipeline0/GstDecodeBin:decodebin0/ffdec_dvvideo:ffdec_dvvideo0.GstPad:src:
caps = video/x-raw-yuv, width=(int)720, height=(int)480,
framerate=(fraction)2997/100, format=(fourcc)Y41B, interlaced=(boolean)true
...


Yet if I edit the dvdec source code to make the libdv decoder PRIMARY instead
of MARGINAL, I get this:


$ gst-launch -v filesrc location=/kipple/lumroom.mov ! decodebin !
video/x-raw-yuv ! fakesink
...

/GstPipeline:pipeline0/GstDecodeBin:decodebin0/GstDVDec:dvdec0.GstPad:src: caps
= video/x-raw-yuv, format=(fourcc)YUY2, width=(int)720, height=(int)480,
framerate=(fraction)2997/100, pixel-aspect-ratio=(fraction)10/11,
interlaced=(boolean)true


Note the difference? PAR is correctly set in the libdv case, and not in the
ffmpeg case.

I think the right thing is to either fix gst-ffmpeg, or revert this one:

commit 268b93fd1e24f9b2d94b37500234bba0525642e6
Author: Edward Hervey <bilboed at bilboed.com>
Date:   Thu Jun 26 09:37:23 2008 +0000

    ext/dv/gstdv.c: Marking rank of dvdec as GST_RANK_MARGINAL since it's the
slowest

    Original commit message from CVS:
    * ext/dv/gstdv.c: (plugin_init):
    Marking rank of dvdec as GST_RANK_MARGINAL since it's the slowest
    DV decoder available.
    Fixes #532393

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.