[Bug 785211] gst-validate: considers as a failure if decoder changes width, height or framerate.

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=785211

--- Comment #5 from Edward Hervey <bilboed at bilboed.com> ---
(In reply to Tim-Philipp Müller from comment #2)
> Are you saying you feel it's wrong that mpegvideoparse is putting display
> width/height into the caps? I don't think so, I think that's exactly how it
> should be.

In the case of mpeg, that display width/height is used for pan&scan (there is a
per-picture display extension specifying the recommended location of that
display window). Do we want to force showing 4:3 videos for 16:9 videos with
pan/scan information ? Note the decoders will *not* crop it to 4:3.

(In reply to Hyunjun Ko from comment #4)
> Created attachment 356249 [details]
> vp9 sample
> 
> Note that this is not only about mpeg2 but vp9.
> I'm attaching sample to reproduce.
> 
> How about lowering the level of message from critical to warning?

  I would very much prefer we keep this issue critical. IIRC it was one of the
reasons validate was started for (3rd party decoder implementations that would
return incoherent/invalid width/height/framerate/par).

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.