[FFmpeg-devel] [PATCH v1 2/2] fftools/ffmpeg: add exif orientation support per frame's metadata

[Jun Li]

On Mon, May 27, 2019 at 7:56 AM Nicolas George <george at nsup.org> wrote:

> Jun Li (12019-05-25):
> > I believe it has been enabled by default, the 'autorotate' value is true
> by
> > default.
>
> Yes, but it is currently enabled for constant rotation. Variable
> rotation is more tricky, so the question of enabling it by default must
> be asked again.
>
> > Do you mean the orientation case 1, 2, 3, 4 have the same resolution and
> 5,
> > 6, 7, 8 are rotated, so the reinit is only necessary when switch between
> > these two cases?
>
> I do not know the numeric values by rote, but I think that is what I
> mean.
>
> > I think it is doable. That transpose filter can do partially dynamic
> > processing based on frame's orientation, but the filter args still need
> to
> > set resolution info.
>
> You can have the filter examine the metadata directly, as an option.
>
> > Addressed in the new version.
>
> Did you send it? If so I missed it.
>
>
Hi Nicolas,
I updated the version as follows, it addresses your feedback and leveraged
current flip functionality, as well new option to auto-flip or
auto-transpose.
https://patchwork.ffmpeg.org/patch/13311/
https://patchwork.ffmpeg.org/patch/13312/

Thanks for your review.

Best Regards,
Jun



> Regards,
>
> --
>   Nicolas George
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".