On Mon, Mar 14, 2022 at 19:35:36 +0000, Soft Works wrote: > > I've been at the same point some time ago where I wondered why ffmpeg is > not setting this, but then I had found that it is overridden by the call > to ass_set_pixel_aspect(). > > ass_set_pixel_aspect() is setting settings.par and if I'm not mistaken, > an existing par setting leads to the storage size setting to be ignored:
It’s not overridden. Only the explicit PAR is currently preferd over the implicit derivation from storage and frame size. However as I stated in the patch description and the comment: “some tags depend on the exact storage resolution of the video” “it actually goes farther than just the aspect ratio” I.e. there's more info in the storage size than just the PAR. You can also easily test the files I linked to empirically validate that there is in fact a difference. > But perhaps I'm missing something.. > > softworkz _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".