On Thu, Dec 09, 2021 at 10:41:17PM +0530, Gyan Doshi wrote:
>
>
> On 2021-12-09 06:49 am, lance.lmw...@gmail.com wrote:
> > From: Limin Wang <lance.lmw...@gmail.com>
> >
> > Signed-off-by: Limin Wang <lance.lmw...@gmail.com>
> > ---
> > libavfilter/af_astats.c | 6 +++---
> > 1 file changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/libavfilter/af_astats.c b/libavfilter/af_astats.c
> > index d0810b5..776a8aa 100644
> > --- a/libavfilter/af_astats.c
> > +++ b/libavfilter/af_astats.c
> > @@ -114,8 +114,8 @@ typedef struct AudioStatsContext {
> > static const AVOption astats_options[] = {
> > { "length", "set the window length", OFFSET(time_constant),
> > AV_OPT_TYPE_DOUBLE, {.dbl=.05}, 0, 10, FLAGS },
> > { "metadata", "inject metadata in the filtergraph", OFFSET(metadata),
> > AV_OPT_TYPE_BOOL, {.i64=0}, 0, 1, FLAGS },
> > - { "reset", "recalculate stats after this many frames",
> > OFFSET(reset_count), AV_OPT_TYPE_INT, {.i64=0}, 0, INT_MAX, FLAGS },
> > - { "measure_perchannel", "only measure_perchannel these per-channel
> > statistics", OFFSET(measure_perchannel), AV_OPT_TYPE_FLAGS,
> > {.i64=MEASURE_ALL}, 0, UINT_MAX, FLAGS, "measure" },
> > + { "reset", "Set number of frame after which stats are going to be
> > recalculated", OFFSET(reset_count), AV_OPT_TYPE_INT, {.i64=0}, 0, INT_MAX,
> > FLAGS },
>
> "Set the number of frames over which cumulative stats are calculated before
> being reset"
>
> > + { "measure_perchannel", "Select the entries which need to be measured
> > per channel", OFFSET(measure_perchannel), AV_OPT_TYPE_FLAGS,
> > {.i64=MEASURE_ALL}, 0, UINT_MAX, FLAGS, "measure" },
>
> "Select the parameters which are measured per channel"
>
> > { "none" , "", 0, AV_OPT_TYPE_CONST,
> > {.i64=MEASURE_NONE }, 0, 0, FLAGS, "measure" },
> > { "all" , "", 0, AV_OPT_TYPE_CONST,
> > {.i64=MEASURE_ALL }, 0, 0, FLAGS, "measure" },
> > { "DC_offset" , "", 0, AV_OPT_TYPE_CONST,
> > {.i64=MEASURE_DC_OFFSET }, 0, 0, FLAGS, "measure" },
> > @@ -143,7 +143,7 @@ static const AVOption astats_options[] = {
> > { "Number_of_NaNs" , "", 0, AV_OPT_TYPE_CONST,
> > {.i64=MEASURE_NUMBER_OF_NANS }, 0, 0, FLAGS, "measure" },
> > { "Number_of_Infs" , "", 0, AV_OPT_TYPE_CONST,
> > {.i64=MEASURE_NUMBER_OF_INFS }, 0, 0, FLAGS, "measure" },
> > { "Number_of_denormals" , "", 0, AV_OPT_TYPE_CONST,
> > {.i64=MEASURE_NUMBER_OF_DENORMALS }, 0, 0, FLAGS, "measure" },
> > - { "measure_overall", "only measure_perchannel these overall
> > statistics", OFFSET(measure_overall), AV_OPT_TYPE_FLAGS,
> > {.i64=MEASURE_ALL}, 0, UINT_MAX, FLAGS, "measure" },
> > + { "measure_overall", "Select the entries which need to be measured
> > overall", OFFSET(measure_overall), AV_OPT_TYPE_FLAGS, {.i64=MEASURE_ALL},
> > 0, UINT_MAX, FLAGS, "measure" },
>
> "Select the parameters which are measured overall"
Thanks for the comments, I'm fine with it. But I saw the descriptions are not
clear so copy them from doc, I'm sure whether we need update the doc also?
>
> Regards,
> Gyan
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
--
Thanks,
Limin Wang
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
