Re: [FFmpeg-devel] avcodec/proresaw_enc : improvment (vendor and color properties, 4444Xq)

2018-12-04 Thread Martin Vignali
>
> LGTM
>
> thanks
>
> [...]
> --
> Michael
>

Pushed, thanks.

Martin
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] avcodec/proresaw_enc : improvment (vendor and color properties, 4444Xq)

2018-12-03 Thread Michael Niedermayer
Hi

On Sun, Dec 02, 2018 at 09:44:38PM +0100, Martin Vignali wrote:
>  internal.h |   12 
>  utils.c|   19 +++
>  2 files changed, 31 insertions(+)
> 46843c218e2c53b93a283656dc3e89f93775a286  
> 0001-avcodec-utils-add-ff_int_from_list_or_default-func.patch
> From 09477289e48362c81fb25e4ffbf563de1729270a Mon Sep 17 00:00:00 2001
> From: Martin Vignali 
> Date: Sun, 2 Dec 2018 21:36:24 +0100
> Subject: [PATCH 1/4] avcodec/utils : add ff_int_from_list_or_default func
> 
> to check valid value, or return default_value
> ---
>  libavcodec/internal.h | 12 
>  libavcodec/utils.c| 19 +++
>  2 files changed, 31 insertions(+)

LGTM

thanks

[...]
-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

There will always be a question for which you do not know the correct answer.


signature.asc
Description: PGP signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] avcodec/proresaw_enc : improvment (vendor and color properties, 4444Xq)

2018-12-02 Thread Martin Vignali
Hello,

No strong opinion about the func name, so change to
ff_int_from_list_or_default
Change int array to const int (in func and in prores_aw)

Martin


0003-avcodec-prores_aw-only-set-color-prim-trc-space.patch
Description: Binary data


0001-avcodec-utils-add-ff_int_from_list_or_default-func.patch
Description: Binary data


0004-avcodec-prores_aw-add--xq-support.patch
Description: Binary data


0002-avcodec-prores_aw-add-vendor-option.patch
Description: Binary data
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] avcodec/proresaw_enc : improvment (vendor and color properties, 4444Xq)

2018-12-02 Thread Michael Niedermayer
On Sun, Dec 02, 2018 at 01:52:49PM +0100, Martin Vignali wrote:
> Hello,
> 
> New patch in attach
> 
> 001 : add ff_int_is_in_list func to libavcodec/utils.c
> 002 : add vendor option (unchanged)
> 003 : only write color properties if supported (use new ff_int_is_in_list
> func)
> 004 : Add 444Xq (unchanged)
> 
> Martin

>  proresenc_anatoliy.c |   13 ++---
>  1 file changed, 10 insertions(+), 3 deletions(-)
> 8ba4720a3b18d2ee2f865463d5ad402ba7a04a4b  
> 0003-avcodec-prores_aw-only-set-color-prim-trc-space.patch
> From 219d1a49472712d601dc722a2719abf547321c32 Mon Sep 17 00:00:00 2001
> From: Martin Vignali 
> Date: Sun, 2 Dec 2018 13:40:02 +0100
> Subject: [PATCH 3/4] avcodec/prores_aw : only set color prim, trc, space 
>  values if supported
> 
> set to unspecified if frame have another value
> ---
>  libavcodec/proresenc_anatoliy.c | 13 ++---
>  1 file changed, 10 insertions(+), 3 deletions(-)
> 
> diff --git a/libavcodec/proresenc_anatoliy.c b/libavcodec/proresenc_anatoliy.c
> index 8047f1c242..6263d2b83b 100644
> --- a/libavcodec/proresenc_anatoliy.c
> +++ b/libavcodec/proresenc_anatoliy.c
> @@ -52,6 +52,12 @@ static const int qp_start_table[5] = {  8, 3, 2, 1, 1};
>  static const int qp_end_table[5]   = { 13, 9, 6, 6, 5};
>  static const int bitrate_table[5]  = { 1000, 2100, 3500, 5400, 7000};
>  
> +static int valid_primaries[9]  = { AVCOL_PRI_RESERVED0, AVCOL_PRI_BT709, 
> AVCOL_PRI_UNSPECIFIED, AVCOL_PRI_BT470BG,
> +   AVCOL_PRI_SMPTE170M, AVCOL_PRI_BT2020, 
> AVCOL_PRI_SMPTE431, AVCOL_PRI_SMPTE432,INT_MAX };
> +static int valid_trc[4]= { AVCOL_TRC_RESERVED0, AVCOL_TRC_BT709, 
> AVCOL_TRC_UNSPECIFIED, INT_MAX };
> +static int valid_colorspace[5] = { AVCOL_SPC_BT709, AVCOL_SPC_UNSPECIFIED, 
> AVCOL_SPC_SMPTE170M,
> +   AVCOL_SPC_BT2020_NCL, INT_MAX };

these should be const static

[...]
>  internal.h |   12 
>  utils.c|   19 +++
>  2 files changed, 31 insertions(+)
> 1d57ba15526595e41879dc1b4aa65444941e4d68  
> 0001-avcodec-utils-add-ff_int_is_in_list-func.patch
> From 3489b650f90c62ecf621f4f930239bb639515aca Mon Sep 17 00:00:00 2001
> From: Martin Vignali 
> Date: Sun, 2 Dec 2018 13:28:33 +0100
> Subject: [PATCH 1/4] avcodec/utils : add ff_int_is_in_list func
> 
> to check valid value, or return default_value
> ---
>  libavcodec/internal.h | 12 
>  libavcodec/utils.c| 19 +++
>  2 files changed, 31 insertions(+)
> 
> diff --git a/libavcodec/internal.h b/libavcodec/internal.h
> index 0c2133f092..03f1d29bc2 100644
> --- a/libavcodec/internal.h
> +++ b/libavcodec/internal.h
> @@ -404,6 +404,18 @@ int ff_alloc_a53_sei(const AVFrame *frame, size_t 
> prefix_len,
>   */
>  int64_t ff_guess_coded_bitrate(AVCodecContext *avctx);
>  
> +/**
> + * Check if a value is in the list. If not, return the default value
> + *
> + * @param ctxContext for the log msg
> + * @param val_name   Name of the checked value, for log msg
> + * @param array_valid_values Array of valid int, ended with INT_MAX
> + * @param default_value  Value return if checked value is not in the 
> array
> + * @return   Value or default_value.
> + */
> +int ff_int_is_in_list(void *ctx, const char * val_name, int val,
> +  int * array_valid_values, int default_value);

maybe
ff_int_from_list_or_default()
is a clearer name, what do you think ?

but iam fine with either, just wondering 

thx

[...]


-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I know you won't believe me, but the highest form of Human Excellence is
to question oneself and others. -- Socrates


signature.asc
Description: PGP signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] avcodec/proresaw_enc : improvment (vendor and color properties, 4444Xq)

2018-12-02 Thread Martin Vignali
Hello,

New patch in attach

001 : add ff_int_is_in_list func to libavcodec/utils.c
002 : add vendor option (unchanged)
003 : only write color properties if supported (use new ff_int_is_in_list
func)
004 : Add 444Xq (unchanged)

Martin


0003-avcodec-prores_aw-only-set-color-prim-trc-space.patch
Description: Binary data


0004-avcodec-prores_aw-add--xq-support.patch
Description: Binary data


0002-avcodec-prores_aw-add-vendor-option.patch
Description: Binary data


0001-avcodec-utils-add-ff_int_is_in_list-func.patch
Description: Binary data
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] avcodec/proresaw_enc : improvment (vendor and color properties, 4444Xq)

2018-11-26 Thread Michael Niedermayer
On Mon, Nov 26, 2018 at 04:19:20PM +0100, Martin Vignali wrote:
> > > +
> > > +switch (pict->colorspace) {
> > > +case AVCOL_SPC_BT709:
> > > +case AVCOL_SPC_UNSPECIFIED:
> > > +case AVCOL_SPC_SMPTE170M:
> > > +case AVCOL_SPC_BT2020_NCL:
> > > +colorspace = pict->colorspace;
> > > +break;
> > > +default:
> > > +av_log(avctx, AV_LOG_DEBUG,
> > > +"Frame colorspace %d are not supported in prores frame.
> > Set prores frame value to unspecified\n", pict->colorspace);
> > > +break;
> > > +}
> > > +*buf++ = color_primaries;
> > > +*buf++ = color_trc;
> > > +*buf++ = colorspace;
> >
> > we maybe should write a generic function that takes a list of supported
> > types
> > and the undefined one and then does the check, debug print and all that
> > that could be used in other encoders too.
> > just an idea, that can be done later
> >
> > the patch
> > LGTM
> >
> >
> Don't know for others codecs, but this check can also probably be add when
> setting AvFrame color properties in prores dec.
> 
> As a generic function, do you mean something like this :
> (If yes, where do you think it's better to declare this func)
> (untested, and not sure, what is the right log level for this (debug or
> warning ?))
> 
> int ff_check_value(AVCodecContext *avctx, const char * val_name, int val,

ff_int_is_in_list() or something similar would be more descriptive and specific
this one doesnt suggest though that the return is teh value or default so its
still maybe not the best identifer


> int * array_valid_values, int nb_values, int
> default_value) {

nb_values could be replaced by a -1 or so terminated array, making the
API slightly more robust

AVCodecContext* should be void *

but yes this is what i had in mind
If it has uses in multiple libs then it has to be in a lib that all
depend on. This would dictate in which lib to put it


thx

[...]
-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Dictatorship: All citizens are under surveillance, all their steps and
actions recorded, for the politicians to enforce control.
Democracy: All politicians are under surveillance, all their steps and
actions recorded, for the citizens to enforce control.


signature.asc
Description: PGP signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] avcodec/proresaw_enc : improvment (vendor and color properties, 4444Xq)

2018-11-26 Thread Martin Vignali
> > (alpha 12b encoding is probably
> > easy to add)
>
> Are you sure alpha is 12 bit? As long as I remember, it is 16 bit.
>
>
Alpha is stored in 16b, but like the input pix fmt is 10b or (later) 12b
alpha, the alpha val is shift during encoding to obtain a 16b val.
The right way will be to add yuv444p12_with16bAlpha, but probably not an
easy task to integrate this inside swscale.

Martin
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] avcodec/proresaw_enc : improvment (vendor and color properties, 4444Xq)

2018-11-26 Thread Reto Kromer
> On 26 Nov 2018, at 15:28, Martin Vignali  wrote:
> 
> (alpha 12b encoding is probably
> easy to add)

Are you sure alpha is 12 bit? As long as I remember, it is 16 bit.

Best regards, Reto
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] avcodec/proresaw_enc : improvment (vendor and color properties, 4444Xq)

2018-11-26 Thread Martin Vignali
>
> So I applied these patches and it appears that ffmpeg will encode XQ as
> 10-bit, but it will decode as 12-bit. Is this intentional? I worry that I
> might have applied the patches incorrectly.  And is it possible to add
> 12-bit encoding?
>
>
Hello,

Seems like you applied patch "add 12b decoding in prores dec", and "add
Proresxq in prores_aw".

With these patchs, decoding of prores , and  XQ, are in 12b
But the encoding bitdepth is not modify (stay in 10b). The "Add Xq encoding
patch", only define quantif matrix, and target size.
Will be interesting to add later support to 12b input in prores_aw, but
doesn't know where to start to add 12b DCT (alpha 12b encoding is probably
easy to add)

Martin
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] avcodec/proresaw_enc : improvment (vendor and color properties, 4444Xq)

2018-11-26 Thread Martin Vignali
> > +
> > +switch (pict->colorspace) {
> > +case AVCOL_SPC_BT709:
> > +case AVCOL_SPC_UNSPECIFIED:
> > +case AVCOL_SPC_SMPTE170M:
> > +case AVCOL_SPC_BT2020_NCL:
> > +colorspace = pict->colorspace;
> > +break;
> > +default:
> > +av_log(avctx, AV_LOG_DEBUG,
> > +"Frame colorspace %d are not supported in prores frame.
> Set prores frame value to unspecified\n", pict->colorspace);
> > +break;
> > +}
> > +*buf++ = color_primaries;
> > +*buf++ = color_trc;
> > +*buf++ = colorspace;
>
> we maybe should write a generic function that takes a list of supported
> types
> and the undefined one and then does the check, debug print and all that
> that could be used in other encoders too.
> just an idea, that can be done later
>
> the patch
> LGTM
>
>
Don't know for others codecs, but this check can also probably be add when
setting AvFrame color properties in prores dec.

As a generic function, do you mean something like this :
(If yes, where do you think it's better to declare this func)
(untested, and not sure, what is the right log level for this (debug or
warning ?))

int ff_check_value(AVCodecContext *avctx, const char * val_name, int val,
int * array_valid_values, int nb_values, int
default_value) {
int i;
for (i = 0 ; i < nb_values ; i ++) {
if (val == array_valid_values[i]){
return val;
}
}

/* val is not a valid value */
av_log(avctx, AV_LOG_DEBUG,
   "%s %d are not supported. Set to default value : %d\n",
val_name, val, default_value);
return default_value;
}


Martin
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] avcodec/proresaw_enc : improvment (vendor and color properties, 4444Xq)

2018-11-26 Thread Kieran O Leary
On Mon, Nov 26, 2018 at 8:06 AM Kieran O Leary 
wrote:

>
>
> On Sat, 24 Nov 2018, 22:11 Martin Vignali 
>> Hello,
>>
>> Patch in attach add some improvments to prores aw encoder
>>
>> 012 : Add vendor option (code come from prores_ks encoder)
>> 013 : Only write color properties, if defined in rdd36 (other values are
>> "converted" to unspecified)
>> 014 : Add  XQ encoding support
>>
>
> I hope to test these patches out today with 12-bit RGB DPX/FFV1 as input.
> Is the  XQ encoder intended to be 10 or 12-bit?
>

So I applied these patches and it appears that ffmpeg will encode XQ as
10-bit, but it will decode as 12-bit. Is this intentional? I worry that I
might have applied the patches incorrectly.  And is it possible to add
12-bit encoding?

$ ./ffmpeg -f lavfi -i testsrc -c:v prores -profile:v 5 xq.mov

ffmpeg version N-92541-g598755603c Copyright (c) 2000-2018 the FFmpeg
developers

  built with Apple LLVM version 9.0.0 (clang-900.0.39.2)

  configuration:

  libavutil  56. 24.101 / 56. 24.101

  libavcodec 58. 40.100 / 58. 40.100

  libavformat58. 23.100 / 58. 23.100

  libavdevice58.  6.100 / 58.  6.100

  libavfilter 7. 46.100 /  7. 46.100

  libswscale  5.  4.100 /  5.  4.100

  libswresample   3.  4.100 /  3.  4.100

Input #0, lavfi, from 'testsrc':

  Duration: N/A, start: 0.00, bitrate: N/A

Stream #0:0: Video: rawvideo (RGB[24] / 0x18424752), rgb24, 320x240
[SAR 1:1 DAR 4:3], 25 tbr, 25 tbn, 25 tbc

File 'xq.mov' already exists. Overwrite ? [y/N] y

Stream mapping:

  Stream #0:0 -> #0:0 (rawvideo (native) -> prores (native))

Press [q] to stop, [?] for help

Output #0, mov, to 'xq.mov':

  Metadata:

encoder : Lavf58.23.100

Stream #0:0: Video: prores (XQ) (ap4x / 0x78347061), yuv444p10le,
320x240 [SAR 1:1 DAR 4:3], q=2-31, 200 kb/s, 25 fps, 12800 tbn, 25 tbc

Metadata:

  encoder : Lavc58.40.100 prores

frame=14747 fps=2015 q=-0.0 Lsize=  469689kB time=00:09:49.84
bitrate=6523.3kbits/s speed=80.6x

video:469627kB audio:0kB subtitle:0kB other streams:0kB global headers:0kB
muxing overhead: 0.013138%

Exiting normally, received signal 2.

telecine:ffmpeg administrator$ ./ffmpeg -i xq.mov

ffmpeg version N-92541-g598755603c Copyright (c) 2000-2018 the FFmpeg
developers

  built with Apple LLVM version 9.0.0 (clang-900.0.39.2)

  configuration:

  libavutil  56. 24.101 / 56. 24.101

  libavcodec 58. 40.100 / 58. 40.100

  libavformat58. 23.100 / 58. 23.100

  libavdevice58.  6.100 / 58.  6.100

  libavfilter 7. 46.100 /  7. 46.100

  libswscale  5.  4.100 /  5.  4.100

  libswresample   3.  4.100 /  3.  4.100

Input #0, mov,mp4,m4a,3gp,3g2,mj2, from 'xq.mov':

  Metadata:

major_brand : qt

minor_version   : 512

compatible_brands: qt

encoder : Lavf58.23.100

  Duration: 00:09:49.88, start: 0.00, bitrate: 6522 kb/s

Stream #0:0(eng): Video: prores (XQ) (ap4x / 0x78347061),
yuv444p12le(tv, progressive), 320x240, 6521 kb/s, SAR 1:1 DAR 4:3, 25 fps,
25 tbr, 12800 tbn, 12800 tbc (default)

Metadata:

  handler_name: VideoHandler
  encoder : Lavc58.40.100 prores


Best,

Kieran.

>
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] avcodec/proresaw_enc : improvment (vendor and color properties, 4444Xq)

2018-11-26 Thread Kieran O Leary
On Sat, 24 Nov 2018, 22:11 Martin Vignali  Hello,
>
> Patch in attach add some improvments to prores aw encoder
>
> 012 : Add vendor option (code come from prores_ks encoder)
> 013 : Only write color properties, if defined in rdd36 (other values are
> "converted" to unspecified)
> 014 : Add  XQ encoding support
>

I hope to test these patches out today with 12-bit RGB DPX/FFV1 as input.
Is the  XQ encoder intended to be 10 or 12-bit?

Kieran.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] avcodec/proresaw_enc : improvment (vendor and color properties, 4444Xq)

2018-11-25 Thread Michael Niedermayer
On Sat, Nov 24, 2018 at 11:04:38PM +0100, Martin Vignali wrote:
>  proresenc_anatoliy.c |   49 ++---
>  1 file changed, 46 insertions(+), 3 deletions(-)
> 44fe346a2be4d3d3ce2c903daf9cd599437627cc  
> 0013-avcodec-prores_aw-only-set-color-prim-trc-space.patch
> From 119a246e570ff346490aef88710d8c8b4aae34e7 Mon Sep 17 00:00:00 2001
> From: Martin Vignali 
> Date: Sat, 24 Nov 2018 22:48:08 +0100
> Subject: [PATCH 13/14] avcodec/prores_aw : only set color prim, trc, space 
>  values if supported
> 
> set to unspecified if frame have another value
> ---
>  libavcodec/proresenc_anatoliy.c | 49 
> ++---
>  1 file changed, 46 insertions(+), 3 deletions(-)
> 
> diff --git a/libavcodec/proresenc_anatoliy.c b/libavcodec/proresenc_anatoliy.c
> index 8047f1c242..8098336b3b 100644
> --- a/libavcodec/proresenc_anatoliy.c
> +++ b/libavcodec/proresenc_anatoliy.c
> @@ -656,6 +656,7 @@ static int prores_encode_frame(AVCodecContext *avctx, 
> AVPacket *pkt,
> const AVFrame *pict, int *got_packet)
>  {
>  ProresContext *ctx = avctx->priv_data;
> +int color_primaries = 0, color_trc = 0, colorspace = 0;/* unspecified by 
> default */
>  int header_size = 148;
>  uint8_t *buf;
>  int pic_size, ret;
> @@ -686,9 +687,51 @@ static int prores_encode_frame(AVCodecContext *avctx, 
> AVPacket *pkt,
>  *buf++ = 0x82; // 422, not interlaced
>  }
>  *buf++ = 0; /* reserved */
> -*buf++ = pict->color_primaries;
> -*buf++ = pict->color_trc;
> -*buf++ = pict->colorspace;
> +
> +switch (pict->color_primaries) {
> +case AVCOL_PRI_RESERVED0:
> +case AVCOL_PRI_BT709:
> +case AVCOL_PRI_UNSPECIFIED:
> +case AVCOL_PRI_BT470BG:
> +case AVCOL_PRI_SMPTE170M:
> +case AVCOL_PRI_BT2020:
> +case AVCOL_PRI_SMPTE431:
> +case AVCOL_PRI_SMPTE432:
> +color_primaries = pict->color_primaries;
> +break;
> +default:
> +av_log(avctx, AV_LOG_DEBUG,
> +   "Frame color primaries %d are not supported in prores frame. 
> Set prores frame value to unspecified\n", pict->color_primaries);
> +break;
> +}
> +
> +switch (pict->color_trc) {
> +case AVCOL_TRC_RESERVED0:
> +case AVCOL_TRC_BT709:
> +case AVCOL_TRC_UNSPECIFIED:
> +color_trc = pict->color_trc;
> +break;
> +default:
> +av_log(avctx, AV_LOG_DEBUG,
> +"Frame color_trc %d are not supported in prores frame. Set 
> prores frame value to unspecified\n", pict->color_trc);
> +break;
> +}
> +
> +switch (pict->colorspace) {
> +case AVCOL_SPC_BT709:
> +case AVCOL_SPC_UNSPECIFIED:
> +case AVCOL_SPC_SMPTE170M:
> +case AVCOL_SPC_BT2020_NCL:
> +colorspace = pict->colorspace;
> +break;
> +default:
> +av_log(avctx, AV_LOG_DEBUG,
> +"Frame colorspace %d are not supported in prores frame. Set 
> prores frame value to unspecified\n", pict->colorspace);
> +break;
> +}
> +*buf++ = color_primaries;
> +*buf++ = color_trc;
> +*buf++ = colorspace;

we maybe should write a generic function that takes a list of supported types
and the undefined one and then does the check, debug print and all that
that could be used in other encoders too.
just an idea, that can be done later

the patch
LGTM

thx

[...]

-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Republics decline into democracies and democracies degenerate into
despotisms. -- Aristotle


signature.asc
Description: PGP signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] avcodec/proresaw_enc : improvment (vendor and color properties, 4444Xq)

2018-11-25 Thread Michael Niedermayer
On Sat, Nov 24, 2018 at 11:04:38PM +0100, Martin Vignali wrote:
>  proresenc_anatoliy.c |   36 +++-
>  1 file changed, 35 insertions(+), 1 deletion(-)
> 61e39289b0bfaafd515c5dab14805c1b4cdc0ad7  
> 0012-avcodec-prores_aw-add-vendor-option.patch
> From cf9e3b3bc0c739c347a36b79678d3aeb594aeb96 Mon Sep 17 00:00:00 2001
> From: Martin Vignali 
> Date: Sat, 24 Nov 2018 22:46:59 +0100
> Subject: [PATCH 12/14] avcodec/prores_aw : add vendor option
> 
> code come from prores_ks
> ---
>  libavcodec/proresenc_anatoliy.c | 36 +++-
>  1 file changed, 35 insertions(+), 1 deletion(-)

probably ok

[...]

-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have often repented speaking, but never of holding my tongue.
-- Xenocrates


signature.asc
Description: PGP signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


[FFmpeg-devel] avcodec/proresaw_enc : improvment (vendor and color properties, 4444Xq)

2018-11-24 Thread Martin Vignali
Hello,

Patch in attach add some improvments to prores aw encoder

012 : Add vendor option (code come from prores_ks encoder)
013 : Only write color properties, if defined in rdd36 (other values are
"converted" to unspecified)
014 : Add  XQ encoding support

Martin


0012-avcodec-prores_aw-add-vendor-option.patch
Description: Binary data


0014-avcodec-prores_aw-add--xq-support.patch
Description: Binary data


0013-avcodec-prores_aw-only-set-color-prim-trc-space.patch
Description: Binary data
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel