[FFmpeg-devel] [PATCH] avfilter: add tremolo filter
Paul B Mahol
onemda at gmail.com
Sun Sep 20 09:46:42 CEST 2015
Dana 20. 9. 2015. 06:28 osoba "Kyle Swanson" <k at ylo.ph> napisala je:
>
> Signed-off-by: Kyle Swanson <k at ylo.ph>
> ---
> doc/filters.texi | 19 ++++++
> libavfilter/Makefile | 1 +
> libavfilter/af_tremolo.c | 173
+++++++++++++++++++++++++++++++++++++++++++++++
> libavfilter/allfilters.c | 1 +
> libavfilter/version.h | 2 +-
> 5 files changed, 195 insertions(+), 1 deletion(-)
> create mode 100644 libavfilter/af_tremolo.c
>
> diff --git a/doc/filters.texi b/doc/filters.texi
> index 88cb3ce..5bbbaf0 100644
> --- a/doc/filters.texi
> +++ b/doc/filters.texi
> @@ -2568,6 +2568,25 @@ slope
> Determine how steep is the filter's shelf transition.
> @end table
>
> + at section tremolo
> +
> +Sinusoidal amplitude modulation.
> +
> +The filter accepts the following options:
> +
> + at table @option
> + at item f
> +Modulation frequency in Hertz. Modulation frequencies in the subharmonic
range
> +(20 Hz or lower) will result in a tremolo effect.
> +This filter may also be used as a ring modulator by specifying
> +a modulation frequency higher than 20 Hz.
> +Range is 0.1 - 20000.0. Default value is 5.0 Hz.
> +
> + at item d
> +Depth of modulation as a percentage. Range is 0.0 - 1.0.
> +Default value is 0.5.
> + at end table
> +
> @section volume
>
> Adjust the input audio volume.
> diff --git a/libavfilter/Makefile b/libavfilter/Makefile
> index 05effd6..45fca3b 100644
> --- a/libavfilter/Makefile
> +++ b/libavfilter/Makefile
> @@ -82,6 +82,7 @@ OBJS-$(CONFIG_SILENCEDETECT_FILTER) +=
af_silencedetect.o
> OBJS-$(CONFIG_SILENCEREMOVE_FILTER) += af_silenceremove.o
> OBJS-$(CONFIG_STEREOTOOLS_FILTER) += af_stereotools.o
> OBJS-$(CONFIG_STEREOWIDEN_FILTER) += af_stereowiden.o
> +OBJS-$(CONFIG_TREMOLO_FILTER) += af_tremolo.o
generate_wave_table.o
nit: Should be after treble.
> OBJS-$(CONFIG_TREBLE_FILTER) += af_biquads.o
> OBJS-$(CONFIG_VOLUME_FILTER) += af_volume.o
> OBJS-$(CONFIG_VOLUMEDETECT_FILTER) += af_volumedetect.o
> diff --git a/libavfilter/af_tremolo.c b/libavfilter/af_tremolo.c
> new file mode 100644
> index 0000000..7896127
> --- /dev/null
> +++ b/libavfilter/af_tremolo.c
> @@ -0,0 +1,173 @@
> +/*
> + * Tremolo
> + * Copyright (c) 2015 Kyle Swanson <k at ylo.ph>. Some rights reserved.
What this means? Better remove: Some rights reserved.
> + *
> + * This file is part of FFmpeg.
> + *
> + * FFmpeg is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU Lesser General Public
> + * License as published by the Free Software Foundation; either
> + * version 2.1 of the License, or (at your option) any later version.
> + *
> + * FFmpeg is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> + * Lesser General Public License for more details.
> + *
> + * You should have received a copy of the GNU Lesser General Public
> + * License along with FFmpeg; if not, write to the Free Software
> + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
02110-1301 USA
> + */
> +
> +/**
> + * @file
> + * Tremolo
> + */
> +
> +#include "libavutil/opt.h"
> +#include "avfilter.h"
> +#include "internal.h"
> +#include "audio.h"
> +#include "generate_wave_table.h"
> +
> +typedef struct TremoloContext {
> + const AVClass *class;
> + double freq;
> + double depth;
> + double *wave_table;
> + int wave_table_index;
> + int sample_rate;
> +} TremoloContext;
> +
> +#define OFFSET(x) offsetof(TremoloContext, x)
> +#define FLAGS AV_OPT_FLAG_AUDIO_PARAM|AV_OPT_FLAG_FILTERING_PARAM
> +
> +static const AVOption tremolo_options[] = {
> + { "f", "set frequency in hertz", OFFSET(freq),
AV_OPT_TYPE_DOUBLE, {.dbl = 5.0}, 0.1, 20000.0, FLAGS },
> + { "d", "set depth as percentage", OFFSET(depth),
AV_OPT_TYPE_DOUBLE, {.dbl = 0.5}, 0.0, 1.0, FLAGS },
> + { NULL }
> +};
> +
> +AVFILTER_DEFINE_CLASS(tremolo);
> +
> +static float trem_env(AVFilterContext *ctx)
> +{
> + TremoloContext *s = ctx->priv;
> + float env = s->wave_table[s->wave_table_index];
> + s->wave_table_index++;
> + if (s->wave_table_index >= s->sample_rate / s->freq)
> + s->wave_table_index = 0;
> + return 1.0 - (s->depth * env);
> +}
Shouldnt this use doubles instead of floats?
> +
> +static int filter_frame(AVFilterLink *inlink, AVFrame *in)
> +{
> + AVFilterContext *ctx = inlink->dst;
> + AVFilterLink *outlink = ctx->outputs[0];
> + AVFrame *out;
> +
> + if (av_frame_is_writable(in)) {
> + out = in;
> + } else {
> + out = ff_get_audio_buffer(inlink, in->nb_samples);
> + if (!out) {
> + av_frame_free(&in);
> + return AVERROR(ENOMEM);
> + }
> + av_frame_copy_props(out, in);
> + }
> +
> + int channels = inlink->channels;
> + int nb_samples = in->nb_samples;
> + double *dst = (double *)out->data[0];
> + int n, c;
> +
> + for (n = 0; n < nb_samples; n++) {
> + float env = trem_env(ctx);
> + for (c = 0; c < channels; c++) {
> + dst[c] *= env;
Out can be zero if frame is not writable. So use: dst - src * env.
> + }
> + dst += channels;
> + }
> +
> + if (in != out)
> + av_frame_free(&in);
> +
> + return ff_filter_frame(outlink, out);
> +}
> +
> +static int query_formats(AVFilterContext *ctx)
> +{
> + AVFilterFormats *formats;
> + AVFilterChannelLayouts *layouts;
> + static const enum AVSampleFormat sample_fmts[] = {
> + AV_SAMPLE_FMT_DBL,
> + AV_SAMPLE_FMT_NONE
> + };
> + int ret;
> +
> + layouts = ff_all_channel_counts();
> + if (!layouts)
> + return AVERROR(ENOMEM);
> + ret = ff_set_common_channel_layouts(ctx, layouts);
> + if (ret < 0)
> + return ret;
> +
> + formats = ff_make_format_list(sample_fmts);
> + if (!formats)
> + return AVERROR(ENOMEM);
> + ret = ff_set_common_formats(ctx, formats);
> + if (ret < 0)
> + return ret;
> +
> + formats = ff_all_samplerates();
> + if (!formats)
> + return AVERROR(ENOMEM);
> + return ff_set_common_samplerates(ctx, formats);
> +}
> +
> +static av_cold void uninit(AVFilterContext *ctx)
> +{
> + TremoloContext *s = ctx->priv;
> + av_free(s->wave_table);
> +}
> +
> +static int config_input(AVFilterLink *inlink)
> +{
> + AVFilterContext *ctx = inlink->dst;
> + TremoloContext *s = ctx->priv;
> + s->sample_rate = inlink->sample_rate;
> + s->wave_table = av_malloc_array(sizeof(double), s->sample_rate /
s->freq);
> + ff_generate_wave_table(WAVE_SIN, AV_SAMPLE_FMT_DBL, s->wave_table,
s->sample_rate / s->freq, 0.0, 1.0, 0.0);
> + s->wave_table_index = 0;
> + return 0;
> +}
> +
> +static const AVFilterPad avfilter_af_tremolo_inputs[] = {
> + {
> + .name = "default",
> + .type = AVMEDIA_TYPE_AUDIO,
> + .config_props = config_input,
> + .filter_frame = filter_frame,
> + },
> + { NULL }
> +};
> +
> +static const AVFilterPad avfilter_af_tremolo_outputs[] = {
> + {
> + .name = "default",
> + .type = AVMEDIA_TYPE_AUDIO,
> + },
> + { NULL }
> +};
> +
> +AVFilter ff_af_tremolo = {
> + .name = "tremolo",
> + .description = NULL_IF_CONFIG_SMALL("Audio Tremolo."),
> + .priv_size = sizeof(TremoloContext),
> + .priv_class = &tremolo_class,
> + .uninit = uninit,
> + .query_formats = query_formats,
> + .inputs = avfilter_af_tremolo_inputs,
> + .outputs = avfilter_af_tremolo_outputs,
> +};
> diff --git a/libavfilter/allfilters.c b/libavfilter/allfilters.c
> index cab4564..59ba5f5 100644
> --- a/libavfilter/allfilters.c
> +++ b/libavfilter/allfilters.c
> @@ -105,6 +105,7 @@ void avfilter_register_all(void)
> REGISTER_FILTER(STEREOTOOLS, stereotools, af);
> REGISTER_FILTER(STEREOWIDEN, stereowiden, af);
> REGISTER_FILTER(TREBLE, treble, af);
> + REGISTER_FILTER(TREMOLO, tremolo, af);
> REGISTER_FILTER(VOLUME, volume, af);
> REGISTER_FILTER(VOLUMEDETECT, volumedetect, af);
>
> diff --git a/libavfilter/version.h b/libavfilter/version.h
> index e918184..9d44fd0 100644
> --- a/libavfilter/version.h
> +++ b/libavfilter/version.h
> @@ -30,7 +30,7 @@
> #include "libavutil/version.h"
>
> #define LIBAVFILTER_VERSION_MAJOR 6
> -#define LIBAVFILTER_VERSION_MINOR 5
> +#define LIBAVFILTER_VERSION_MINOR 6
> #define LIBAVFILTER_VERSION_MICRO 100
>
> #define LIBAVFILTER_VERSION_INT
AV_VERSION_INT(LIBAVFILTER_VERSION_MAJOR, \
> --
> 1.8.4
>
Will do some minor changes, and apply after are questions I raised resolved.
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
More information about the ffmpeg-devel
mailing list