From e78802d0d85c2305339fc79fa1be1d0d43a899bf Mon Sep 17 00:00:00 2001 From: simakova Date: Wed, 17 Jul 2024 03:28:56 -0700 Subject: [PATCH] Update recommendation on setting frame rate for images PiperOrigin-RevId: 653167049 --- .../androidx/media3/transformer/EditedMediaItem.java | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/libraries/transformer/src/main/java/androidx/media3/transformer/EditedMediaItem.java b/libraries/transformer/src/main/java/androidx/media3/transformer/EditedMediaItem.java index 4e1d2f1543..b6e32289ff 100644 --- a/libraries/transformer/src/main/java/androidx/media3/transformer/EditedMediaItem.java +++ b/libraries/transformer/src/main/java/androidx/media3/transformer/EditedMediaItem.java @@ -169,9 +169,12 @@ public final class EditedMediaItem { /** * Sets the {@link MediaItem} frame rate in the output video, in frames per second. * - *

This should be set for inputs that don't have an intrinsic frame rate (e.g. images, which - * are recommended to be set for a frame rate of 30). It will be ignored for inputs that do have - * an intrinsic frame rate (e.g. video). + *

This should be set for inputs that don't have an intrinsic frame rate (e.g., images). It + * will be ignored for inputs that do have an intrinsic frame rate (e.g., video). + * + *

For images, the frame rate depends on factors such as desired look, output format + * requirement, and whether the content is static or dynamic (e.g., animation). However, 30 fps + * is suitable for most use cases. * *

No frame rate is set by default. *