Re: [FFmpeg-devel] [PATCH 2/2] libvpxenc: Report encoded VP9 level

2016-11-28 Thread James Zern 
On Mon, Nov 28, 2016 at 10:32 AM, Alex Converse  wrote:
> On Tue, Nov 22, 2016 at 3:10 PM, James Zern  wrote:
>> On Tue, Nov 22, 2016 at 12:08 PM, James Zern  wrote:
>>> On Tue, Nov 22, 2016 at 12:04 PM, James Zern  wrote:
 On Fri, Nov 18, 2016 at 2:01 PM, Alex Converse  
 wrote:
> Report the actual level of the encoded output if a level is
> targeted or the level is passively tracked with a target of 0.
> ---
>  libavcodec/libvpxenc.c | 31 +++
>  1 file changed, 31 insertions(+)
>

 lgtm.
 I don't know if there's a better way to report this at the stream
 level (AV_PKT_DATA_STRINGS_METADATA?), there doesn't seem to be
 anything specific right now. This info can be translated quickly if
 someone wants to make this more structured or has any opinion now.

>>>
>>> I forgot that there was some discussion around adding this to the
>>> codec extradata in webm. That could be a followup if there's
>>> documentation on the format for that.
>>
>> http://wiki.webmproject.org/vp9-codecprivate
>
> Are there any tools that read or write this data. libavformat and
> libvpx/webmenc.cc don't seem to implement it. I'd rather not be the
> guinea pig for this.

libwebm has support in the muxer sample and webm_info for reporting:
https://chromium.googlesource.com/webm/libwebm/+/master/mkvmuxer_sample.cc#471
https://chromium.googlesource.com/webm/libwebm/+/master/webm_info.cc#381
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 2/2] libvpxenc: Report encoded VP9 level

2016-11-28 Thread Alex Converse 
On Tue, Nov 22, 2016 at 3:10 PM, James Zern  wrote:
> On Tue, Nov 22, 2016 at 12:08 PM, James Zern  wrote:
>> On Tue, Nov 22, 2016 at 12:04 PM, James Zern  wrote:
>>> On Fri, Nov 18, 2016 at 2:01 PM, Alex Converse  
>>> wrote:
 Report the actual level of the encoded output if a level is
 targeted or the level is passively tracked with a target of 0.
 ---
  libavcodec/libvpxenc.c | 31 +++
  1 file changed, 31 insertions(+)

>>>
>>> lgtm.
>>> I don't know if there's a better way to report this at the stream
>>> level (AV_PKT_DATA_STRINGS_METADATA?), there doesn't seem to be
>>> anything specific right now. This info can be translated quickly if
>>> someone wants to make this more structured or has any opinion now.
>>>
>>
>> I forgot that there was some discussion around adding this to the
>> codec extradata in webm. That could be a followup if there's
>> documentation on the format for that.
>
> http://wiki.webmproject.org/vp9-codecprivate

Are there any tools that read or write this data. libavformat and
libvpx/webmenc.cc don't seem to implement it. I'd rather not be the
guinea pig for this.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 2/2] libvpxenc: Report encoded VP9 level

2016-11-22 Thread James Zern 
On Tue, Nov 22, 2016 at 12:08 PM, James Zern  wrote:
> On Tue, Nov 22, 2016 at 12:04 PM, James Zern  wrote:
>> On Fri, Nov 18, 2016 at 2:01 PM, Alex Converse  
>> wrote:
>>> Report the actual level of the encoded output if a level is
>>> targeted or the level is passively tracked with a target of 0.
>>> ---
>>>  libavcodec/libvpxenc.c | 31 +++
>>>  1 file changed, 31 insertions(+)
>>>
>>
>> lgtm.
>> I don't know if there's a better way to report this at the stream
>> level (AV_PKT_DATA_STRINGS_METADATA?), there doesn't seem to be
>> anything specific right now. This info can be translated quickly if
>> someone wants to make this more structured or has any opinion now.
>>
>
> I forgot that there was some discussion around adding this to the
> codec extradata in webm. That could be a followup if there's
> documentation on the format for that.

http://wiki.webmproject.org/vp9-codecprivate
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 2/2] libvpxenc: Report encoded VP9 level

2016-11-22 Thread James Zern 
On Tue, Nov 22, 2016 at 12:04 PM, James Zern  wrote:
> On Fri, Nov 18, 2016 at 2:01 PM, Alex Converse  
> wrote:
>> Report the actual level of the encoded output if a level is
>> targeted or the level is passively tracked with a target of 0.
>> ---
>>  libavcodec/libvpxenc.c | 31 +++
>>  1 file changed, 31 insertions(+)
>>
>
> lgtm.
> I don't know if there's a better way to report this at the stream
> level (AV_PKT_DATA_STRINGS_METADATA?), there doesn't seem to be
> anything specific right now. This info can be translated quickly if
> someone wants to make this more structured or has any opinion now.
>

I forgot that there was some discussion around adding this to the
codec extradata in webm. That could be a followup if there's
documentation on the format for that.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 2/2] libvpxenc: Report encoded VP9 level

2016-11-22 Thread James Zern 
On Fri, Nov 18, 2016 at 2:01 PM, Alex Converse  wrote:
> Report the actual level of the encoded output if a level is
> targeted or the level is passively tracked with a target of 0.
> ---
>  libavcodec/libvpxenc.c | 31 +++
>  1 file changed, 31 insertions(+)
>

lgtm.
I don't know if there's a better way to report this at the stream
level (AV_PKT_DATA_STRINGS_METADATA?), there doesn't seem to be
anything specific right now. This info can be translated quickly if
someone wants to make this more structured or has any opinion now.

> +
>  static av_cold int vpx_free(AVCodecContext *avctx)
>  {
>  VPxContext *ctx = avctx->priv_data;
> +int level_out = 0;

could relocate this to avoid unused warnings against older libs.

> +
> +#if VPX_ENCODER_ABI_VERSION >= 12
> +if (ctx->level >= 0 && !(avctx->flags & AV_CODEC_FLAG_PASS1) &&
> +!codecctl_intp(avctx, VP9E_GET_LEVEL, _out))
> +  av_log(avctx, AV_LOG_INFO, "Encoded level %.1f\n", level_out * 0.1);

indent is off.

> +#endif
>
>  vpx_codec_destroy(>encoder);
>  if (ctx->is_alpha)
> --
> 2.8.0.rc3.226.g39d4020
>
> ___
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel