androidx / media

Jetpack Media3 support libraries for media use cases, including ExoPlayer, an extensible media player for Android
https://developer.android.com/media/media3
Apache License 2.0
1.73k stars 415 forks source link

Values obtained via Transformer.getProgress() are incorrect for sections of a Composition that contain MotionPhotos #1164

Closed jadoblrb closed 6 months ago

jadoblrb commented 8 months ago

Version

Media3 1.3.0

More version details

No response

Devices that reproduce the issue

Pixel7a

Devices that do not reproduce the issue

No response

Reproducible in the demo app?

Not tested

Reproduction steps

1) Create a Composition that contains an asset that represents a MotionPhoto .jpg 2) Export this Composition using a Transformer whose progress is obtained via Transformer.getProgress()

For the segment of the composition that contains the MotionPhoto, the transformer's progress seems to race through the MotionPhoto and then stall at the end of the MotionPhoto, while actual procesing for the MotionPhoto continues on 'silently'. If this progress value is used to update the status of a progressBar, then this progressBar appears to freeze while the MotionPhoto is being processed.

Expected result

the progress reported while the Transformer processes a MotionPhoto should be smooth, similar to how progress is reported for the sections of a Composition that contain a movie file

Actual result

The Progress that is reported by Transformer.getProgress() is not fluid while a MotionPhoto is being processed

Media

Media will be sent via email

Bug Report

tof-tof commented 8 months ago

In this file, is the aim to transform/process the "video" element of the motion photo or the image element. If it's the image element, then this issue is the same issue as described in #1181.

jadoblrb commented 8 months ago

The aim is to transform/process the image element of the MotionPhoto. Agreed that it is similar to 1181.

tof-tof commented 8 months ago

@droid-girl please advise what to do with this bug in light of this

droid-girl commented 6 months ago

Duplicate for https://github.com/androidx/media/issues/1181.