[FFmpeg-devel] [PATCH] mpegvideo_enc: add option to disable intra mbs in p frames

[Michael Niedermayer]

On Sat, Jun 09, 2018 at 05:09:13PM +0200, Ramiro Polla wrote:
> Hi Michael,
> 
> On Thu, May 10, 2018 at 11:01 PM, Michael Niedermayer
> <michael at niedermayer.cc> wrote:
> > On Wed, May 09, 2018 at 08:44:25PM +0200, Ramiro Polla wrote:
> >> This option prevents the mpv encoders from using intra macroblocks in
> >> predictive frames.
> >>
> >> It is useful for glitch artists to generate input material. This option
> >> allows them to split and merge two video files while maintaining fluid
> >> motion from the second video without having intra macroblocks restoring
> >> chunks of the first video.
> >
> > maybe a continuous variable like snows intra_penalty could achieve this
> > too but give more flexibility in doing it also just partially if wanted
> 
> I like this idea better. I wanted a simple way to be able to entirely
> disable intra macroblocks, but "-intra_penalty max" could cause an
> overflow, so I set the max value to INT_MAX/2.
> 
> New patch attached.

LGTM

a fate test may also make sense

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Dictatorship naturally arises out of democracy, and the most aggravated
form of tyranny and slavery out of the most extreme liberty. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180610/5f512865/attachment.sig>