0001-prores_ks-use-CodecContext-for-color-information-if-.patch
Description: Binary data
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
On 10/4/18, Marc-Antoine ARNAUD wrote:
>
>
Why?
IIRC this patch is not needed.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
In our case we have some files with bad colorspaces (in HD but with bt601
colorspace).
So we use -colorspace, -color_trc, -color_primaries to force the output
colorspace.
We keep compatibility with "old command line", we get source colorspace if
nothing is mentionned.
It work like that for Mpeg2vi
On 10/5/18, Marc-Antoine ARNAUD wrote:
> In our case we have some files with bad colorspaces (in HD but with bt601
> colorspace).
> So we use -colorspace, -color_trc, -color_primaries to force the output
> colorspace.
>
> We keep compatibility with "old command line", we get source colorspace if
>
For me it's the only codec who use picture colorspace as source.
All others uses only the CodecContext.
I don't know the exact reason, but I suppose it can be easiest to manage
output colorspace during merge of video, as a video can have only one
"static" video colorspace.
So for me it made sense
On 10/10/18, Marc-Antoine ARNAUD wrote:
> For me it's the only codec who use picture colorspace as source.
> All others uses only the CodecContext.
> I don't know the exact reason, but I suppose it can be easiest to manage
> output colorspace during merge of video, as a video can have only one
> "
I have updated the patch with our discussion.
It took information only from the codec context.
Marc-Antoine
Le mer. 10 oct. 2018 à 11:36, Paul B Mahol a écrit :
> On 10/10/18, Marc-Antoine ARNAUD wrote:
> > For me it's the only codec who use picture colorspace as source.
> > All others uses on
Le mer. 10 oct. 2018 à 17:13, Marc-Antoine ARNAUD <
arnaud.marcanto...@gmail.com> a écrit :
> I have updated the patch with our discussion.
> It took information only from the codec context.
>
> Marc-Antoine
>
> Hello,
If i correctly understand (which is not sure :-) :
the colorspace for AVCodec
Hi Martin,
For my point of view, the AVFrame contains the colorspace for each frame,
which can be different (maybe not volunteer..).
The colorspace in AVCodecContext is the "pre-computed" colorspace, used to
generate the output file, as header/footer of wrappers can require
colorspace informations
Le lun. 15 oct. 2018 à 11:24, Marc-Antoine ARNAUD <
arnaud.marcanto...@gmail.com> a écrit :
> Hi Martin,
>
> For my point of view, the AVFrame contains the colorspace for each frame,
> which can be different (maybe not volunteer..).
> The colorspace in AVCodecContext is the "pre-computed" colorspa
10 matches
Mail list logo