From: Stefano Sabatini <stefasab@gmail.com>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Subject: Re: [FFmpeg-devel] [PATCH] doc/encoders/libxvid: fix references for me_quality option
Date: Fri, 6 Oct 2023 09:54:00 +0200
Message-ID: <ZR+9GJxZTE2tetOQ@mariano> (raw)
In-Reply-To: <c0a4a6b8-6bd3-5a6f-1f76-ed70bc968de4@gyani.pro>
On date Wednesday 2023-09-06 10:56:43 +0530, Gyan Doshi wrote:
> On 2023-08-26 04:26 pm, Stefano Sabatini wrote:
> > Drop reference to constants removed in 94eed68ace9f2416af8.
> >
> > In particular, rename me_method to me_quality and add description for
> > supported values.
> >
> > Address trac issue:
> > http://trac.ffmpeg.org/ticket/10003
> > ---
> > doc/encoders.texi | 21 +++++++++------------
> > 1 file changed, 9 insertions(+), 12 deletions(-)
> >
> > diff --git a/doc/encoders.texi b/doc/encoders.texi
> > index 14c69fc7b3..25c40a6486 100644
> > --- a/doc/encoders.texi
> > +++ b/doc/encoders.texi
> > @@ -2981,27 +2981,24 @@ Place global headers in extradata instead of every keyframe.
> > @item trellis
> > -@item me_method
> > -Set motion estimation method. Possible values in decreasing order of
> > +@item me_quality
> > +Set motion estimation quality level. Possible values in decreasing order of
> > speed and increasing order of quality:
> > @table @samp
> > -@item zero
> > +@item 0
> > Use no motion estimation (default).
> > -@item phods
> > -@item x1
> > -@item log
> > +@item 1, 2
> > Enable advanced diamond zonal search for 16x16 blocks and half-pixel
> > -refinement for 16x16 blocks. @samp{x1} and @samp{log} are aliases for
> > -@samp{phods}.
> > +refinement for 16x16 blocks.
> > -@item epzs
> > +@item 3, 4
> > Enable all of the things described above, plus advanced diamond zonal
> > -search for 8x8 blocks, half-pixel refinement for 8x8 blocks, and motion
> > -estimation on chroma planes.
> > +search for 8x8 blocks and half-pixel refinement for 8x8 blocks, also
> > +enable motion estimation on chrom planes for P and B-frames.
> s/chrom/chroma
>
> > -@item full
> > +@item 5, 6
> > Enable all of the things described above, plus extended 16x16 and 8x8
> > blocks search.
> > @end table
>
> Else, LGTM.
Applied, thanks.
_______________________________________________
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".
prev parent reply other threads:[~2023-10-06 7:54 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-26 10:56 Stefano Sabatini
2023-09-05 23:12 ` Stefano Sabatini
2023-09-06 5:26 ` Gyan Doshi
2023-10-06 7:54 ` Stefano Sabatini [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ZR+9GJxZTE2tetOQ@mariano \
--to=stefasab@gmail.com \
--cc=ffmpeg-devel@ffmpeg.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
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