[FFmpeg-devel] [PATCH] Implement NewTek NDI support
Steinar H. Gunderson
steinar+ffmpeg at gunderson.no
Thu Aug 3 10:41:55 EEST 2017
On Thu, Aug 03, 2017 at 08:21:09AM +0300, Maksym Veremeyenko wrote:
>> You should reject bottom field first formats, as NDI explicitly
>> disallows them.
> even if it disallowed, why to drop? may be warning message would be enough?
FWIW, the documentation may say that formats are always TFF, but on the wire,
it seems the bottom field actually goes first.
(I guess this is a bug that isn't noticed because there's actually no way
from the API to get at the individual fields)
> +NDI is very picky about the formats it supports. Pixel format is always
Or bgra (internally, uyvya4224, I suppose).
> + recv_create_desc.allow_video_fields = ctx->allow_video_fields;
I'm not convinced you ever want to set this to false, as what's happening if
you do is simply that the SDK applies a vertical blur to incoming interlaced
frames. FFmpeg has much better deinterlacers if that's what you want.
/* Steinar */
More information about the ffmpeg-devel