[FFmpeg-devel] 4:2:0 DV25 file wrongly identified as 4:1:1

[Dave Rice]

On Sep 21, 2010, at 7:39 AM, Niobos wrote:

> On 2010-09-21 12:04, Michael Niedermayer wrote:
>> On Fri, Sep 03, 2010 at 02:06:13PM +0200, Niobos wrote:
>>> The reason why I'm raising this issue here (as well as with the tool
>>> that produced the file), is that a simple fix (attached) will:
>>> * make ffmpeg work for our production system
>>> * conform to the original comment (which is now lost) by checking the
>>> APT flag instead of the AP1 flag. 
>>> 
>>> Obviously, this change may break DV decoding if the current behaviour is
>>> desired. I'm no expert on the DV format, comments and insights are welcome.
>> 
>>> dvdata.c |    2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>> 24b14fe94c5cfd32b6d52ee8660360dc63f67594  dv25-wrongly-identified-as-411.patch
>> 
>> as noone wants to maintain dv ...
>> patch should be ok if tested & passes fate 
> 
> Patch is tested on two sample 4:2:0 files. I don't have a 4:1:1 clip to
> test it against.

If it helps, here's a some 4:1:1 DV from the Internet Archive:
http://www.archive.org/download/TripTheLightPromo2/DanceTest2.mov ('dvc ')
http://www.archive.org/download/ThroughtTheFirstBoca/ThroughFirstBoca.avi ('dvsd', DVCPRO)
http://www.archive.org/download/Timothy_Leary_Archives_237_.dv/237_.dv (DVCPRO)
Dave Rice
avpreserve.com

> make fate passes on my machine (output attached). Most notably, these
> test succeed: vsynth{1,2}-dv{,50}, lavf-dv_fmt, seek-dv{411,50,}_dv,
> seek-lavf_dv
> 
> Greetings,
> Niobos
> <fate-output.txt>_______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at mplayerhq.hu
> https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-devel