Re: [FFmpeg-devel] [PATCH] Ensure scaled video is divisible by n

2019-07-08 Thread Lars Kiesow
Hi Michael, > commit message should begin with a prefix like > avfilter/vf_scale: ... Thanks for the hint. Will use that. > what does this feature have to do with force_original_aspect_ratio ? > i think it should not be under this if() This really only makes sense when using force_original_asp

Re: [FFmpeg-devel] [PATCH] Ensure scaled video is divisible by n

2019-07-05 Thread Michael Niedermayer
On Wed, Jul 03, 2019 at 06:16:51PM +0200, Lars Kiesow wrote: > This patch adds a new option to the scale filter which ensures that the > output resolution is divisible by the given integer similar to using -n > in the `w` and `h` options. But this works even if the > `force_original_aspect_ratio` i

[FFmpeg-devel] [PATCH] Ensure scaled video is divisible by n

2019-07-03 Thread Lars Kiesow
This patch adds a new option to the scale filter which ensures that the output resolution is divisible by the given integer similar to using -n in the `w` and `h` options. But this works even if the `force_original_aspect_ratio` is used. The use case for this is to set a fixed target resolution us

Re: [FFmpeg-devel] [PATCH] Ensure scaled video is divisible by n

2019-07-03 Thread Lars Kiesow
On Wed, 3 Jul 2019 18:04:26 +0200 Paul B Mahol wrote: > On 7/3/19, Lars Kiesow wrote: > > Hi Paul, > > > >> > { "force_original_aspect_ratio", "decrease or increase w/h > >> > if necessary to keep the original AR", > >> > OFFSET(force_original_aspect_ratio), AV_OPT_TYPE_INT, { .i64 = > >>

Re: [FFmpeg-devel] [PATCH] Ensure scaled video is divisible by n

2019-07-03 Thread Paul B Mahol
On 7/3/19, Lars Kiesow wrote: > Hi Paul, > >> > { "force_original_aspect_ratio", "decrease or increase w/h if >> > necessary to keep the original AR", >> > OFFSET(force_original_aspect_ratio), AV_OPT_TYPE_INT, { .i64 = 0}, >> > 0, 2, FLAGS, "force_oar" }, >> > +{ "force_divisible_by", "en

Re: [FFmpeg-devel] [PATCH] Ensure scaled video is divisible by n

2019-07-03 Thread Lars Kiesow
Hi Paul, > > { "force_original_aspect_ratio", "decrease or increase w/h if > > necessary to keep the original AR", > > OFFSET(force_original_aspect_ratio), AV_OPT_TYPE_INT, { .i64 = 0}, > > 0, 2, FLAGS, "force_oar" }, > > +{ "force_divisible_by", "enforce that the output resolution is > >

Re: [FFmpeg-devel] [PATCH] Ensure scaled video is divisible by n

2019-07-03 Thread Paul B Mahol
On 7/3/19, Lars Kiesow wrote: > This patch adds a new option to the scale filter which ensures that the > output resolution is divisible by the given integer similar to using -n > in the `w` and `h` options. But this works even if the > `force_original_aspect_ratio` is used. > > The use case for t

[FFmpeg-devel] [PATCH] Ensure scaled video is divisible by n

2019-07-03 Thread Lars Kiesow
This patch adds a new option to the scale filter which ensures that the output resolution is divisible by the given integer similar to using -n in the `w` and `h` options. But this works even if the `force_original_aspect_ratio` is used. The use case for this is to set a fixed target resolution us

Re: [FFmpeg-devel] [PATCH] Ensure scaled video is divisible by n

2019-07-03 Thread Paul B Mahol
On 7/3/19, Lars Kiesow wrote: > This patch adds a new option to the scale filter which ensures that the > output resolution is divisible by the given integer similar to using -n > in the `w` and `h` options. But this works even if the > `force_original_aspect_ratio` is used. > > The use case for t

[FFmpeg-devel] [PATCH] Ensure scaled video is divisible by n

2019-07-03 Thread Lars Kiesow
This patch adds a new option to the scale filter which ensures that the output resolution is divisible by the given integer similar to using -n in the `w` and `h` options. But this works even if the `force_original_aspect_ratio` is used. The use case for this is to set a fixed target resolution us