Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
* [FFmpeg-devel] [PATCH] doc/filters: expand documentation on libvmaf filter
@ 2023-12-08 18:47 Nil Fons Miret via ffmpeg-devel
  2023-12-10  0:28 ` Stefano Sabatini
  0 siblings, 1 reply; 2+ messages in thread
From: Nil Fons Miret via ffmpeg-devel @ 2023-12-08 18:47 UTC (permalink / raw)
  To: ffmpeg-devel, Kyle Swanson; +Cc: Nil Fons Miret

[-- Attachment #1: Type: text/plain, Size: 10 bytes --]

Attached.

[-- Attachment #2: 0001-doc-filters-expand-documentation-on-libvmaf-filter.patch --]
[-- Type: application/octet-stream, Size: 2625 bytes --]

[-- Attachment #3: Type: text/plain, Size: 251 bytes --]

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [FFmpeg-devel] [PATCH] doc/filters: expand documentation on libvmaf filter
  2023-12-08 18:47 [FFmpeg-devel] [PATCH] doc/filters: expand documentation on libvmaf filter Nil Fons Miret via ffmpeg-devel
@ 2023-12-10  0:28 ` Stefano Sabatini
  0 siblings, 0 replies; 2+ messages in thread
From: Stefano Sabatini @ 2023-12-10  0:28 UTC (permalink / raw)
  To: FFmpeg development discussions and patches; +Cc: Nil Fons Miret, Kyle Swanson

On date Friday 2023-12-08 18:47:28 +0000, ffmpeg-devel Mailing List wrote:
> Attached.

> From e2b98c2dec1248f5000a9291bd6f1177de2a4d0f Mon Sep 17 00:00:00 2001
> From: nilfm <nilf@netflix.com>
> Date: Fri, 8 Dec 2023 14:28:12 +0000
> Subject: [PATCH] doc/filters: expand documentation on libvmaf filter
> 
> Signed-off-by: nilfm <nilf@netflix.com>
> ---
>  doc/filters.texi | 26 ++++++++++++++++++++------
>  1 file changed, 20 insertions(+), 6 deletions(-)
> 
> diff --git a/doc/filters.texi b/doc/filters.texi
> index 6d00ba2c3f..232bd6405b 100644
> --- a/doc/filters.texi
> +++ b/doc/filters.texi
> @@ -16929,19 +16929,33 @@ The first input is the distorted video, and the second input is the reference vi
>  
>  The obtained VMAF score is printed through the logging system.
>  
> -It requires Netflix's vmaf library (libvmaf) as a pre-requisite.
> +It requires Netflix's VMAF library (libvmaf) as a pre-requisite.
>  After installing the library it can be enabled using:
>  @code{./configure --enable-libvmaf}.
>  
> -The filter has following options:
> +The filter has the following options:
>  
>  @table @option
>  @item model
> -A `|` delimited list of vmaf models. Each model can be configured with a number of parameters.

> +A `|` delimited list of VMAF models. Each model can be configured with the following parameters:

This is still ambiguous since it is not defining how the parameters
must be specified. IIUC it should be:

Each model can be configured with a sequence of key=value parameters,
separated by @code{:}. Note that the @code{:} characters is special
and needs to be escaped. The following parameters are supported:
[...]

While at it, it might be good to extend the options with a "models"
alias to clarify plurality.

> +@table @option
> +@item version
> +Built-in model version. Example: @code{vmaf_v0.6.1}.
> +@item name
> +Name that will be associated with this model in the output logs.
> +@item path
> +Path to a model in the local filesystem.
> +@item enable_transform
> +If true, a model-defined transform will be applied to the final score. Default value: @code{false}.
> +@item disable_clip
> +If true, model-defined score clipping will be disabled. Default value: @code{false}.
> +@end table
>  Default value: @code{"version=vmaf_v0.6.1"}
>  
>  @item feature
> -A `|` delimited list of features. Each feature can be configured with a number of parameters.
> +A `|` delimited list of features. Each feature can be configured with a number
> +of parameters. The parameter @code{name} should be set to the name of the
> +feature. Other key-value parameter pairs will be passed in to the feature extractor.

ditto

>  
>  @item log_path
>  Set the file path to be used to store log files.
> @@ -16982,9 +16996,9 @@ ffmpeg -i distorted.mpg -i reference.mpg -lavfi libvmaf='model=version=vmaf_v0.6
>  @end example
>  
>  @item
> -Example with multiple additional features:
> +Example with multiple additional features and feature options:
>  @example
> -ffmpeg -i distorted.mpg -i reference.mpg -lavfi libvmaf='feature=name=psnr|name=ciede' -f null -
> +ffmpeg -i distorted.mpg -i reference.mpg -lavfi libvmaf='feature=name=psnr\\:enable_mse=true|name=ciede' -f null -
>  @end example

looks good otherwise
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-12-10  0:29 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-12-08 18:47 [FFmpeg-devel] [PATCH] doc/filters: expand documentation on libvmaf filter Nil Fons Miret via ffmpeg-devel
2023-12-10  0:28 ` Stefano Sabatini

Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git