[FFmpeg-devel] [PATCH] Implement NewTek NDI support

Steinar H. Gunderson sgunderson at bigfoot.com
Thu Aug 3 10:15: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
> +uyvy422.

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 */
Homepage: https://www.sesse.net/

More information about the ffmpeg-devel mailing list