Opened 20 months ago
Last modified 8 months ago
#9456 new defect
ffprobe coded_width, coded_height, and all dec_ctx->properties fields broken (regression)
Reported by: | Gregory Beauregard | Owned by: | |
---|---|---|---|
Priority: | important | Component: | ffprobe |
Version: | git-master | Keywords: | codecpar regression |
Cc: | Blocked By: | ||
Blocking: | Reproduced by developer: | yes | |
Analyzed by developer: | no |
Description (last modified by )
Summary of the bug: The coded_height and coded_width parameters don't work in ffprobe; they always display the same as the width and height fields.
How to reproduce: Run the following command on any 1080p h264 file. Expected coded_height is 1088, but it will show 1080.
ffprobe -select_streams v -show_entries stream=coded_width,coded_height file.mkv
This behavior is a regression since 04f49645a519233f3638104e0df5215758652fcb
This behavior was also broken in the past and was fixed in https://trac.ffmpeg.org/ticket/6958 before regressing.
ffprobe should not be outputting wrong info
Change History (8)
comment:1 by , 20 months ago
comment:2 by , 20 months ago
Removal or re-implementation of all fields that depended on ist->dec_ctx->properties
being set (like closed_captions
and film-grain and more?) will also need backported to 4.4 if the API has already been removed (precluding the revert of the commit). All ffprobe fields that depended on this being set regressed in the same commit and now show fake wrong results with ffprobe.
If the API is not yet removed the better solution is probably reverting the commit in the 4.4 branch so the deprecation or fix of all these ffprobe fields can be done correctly in master.
comment:3 by , 20 months ago
avstream.codec was removed before 4.4 release: https://github.com/FFmpeg/FFmpeg/commit/3749eede
So we will need to backport a deprecation or re-implementation of these ffprobe fields to 4.4 branch
comment:4 by , 20 months ago
Summary: | ffprobe coded_width and coded_height don't work (regression) → ffprobe coded_width, coded_height, and all dec_ctx->properties fields broken (regression) |
---|
comment:5 by , 19 months ago
Component: | ffprobe → avcodec |
---|---|
Keywords: | codecpar regression added |
Priority: | normal → important |
updated properties to correspond to triage on previous instance of this bug https://trac.ffmpeg.org/ticket/6958 since the bug is the same except for also breaking a few additional things
comment:6 by , 19 months ago
Component: | avcodec → ffprobe |
---|
follow-up: 8 comment:7 by , 8 months ago
Description: | modified (diff) |
---|
Please provide the command line you tested together with the complete, uncut console output and an input sample to make this a valid ticket.
comment:8 by , 8 months ago
Reproduced by developer: | set |
---|
Replying to Carl Eugen Hoyos:
Please provide the command line you tested together with the complete, uncut console output and an input sample to make this a valid ticket.
+reproduced by developer (communication in irc a few months ago)
You can find the sample and commandline output on the trac linked in the description, but if you're uncomfortable using that uploaded sample you can also use probably most of the samples in fate or that have been uploaded if they don't have the right multiple (i.e. any 1080p h264 file)
My personal dog in this is I would like to access this info so I have a way to do calculations needed for bitstream filter (bsf) cropping, but this should be removed rather than displaying the wrong value if the wish is to deprecate it.
A fix or removal would also need backported to 4.4.