Skip to content

SP630 audio video out of sync4 android12#634

Merged
bryanwussow merged 2 commits intosillsdev:developfrom
DaleHensley:SP630-AudioVideoOutOfSync4Android12
Apr 4, 2022
Merged

SP630 audio video out of sync4 android12#634
bryanwussow merged 2 commits intosillsdev:developfrom
DaleHensley:SP630-AudioVideoOutOfSync4Android12

Conversation

@DaleHensley
Copy link
Copy Markdown
Contributor

Per issue #630, update software to proper keep audio and slides in sync.

During Android 12 testing, the encoding of the audio to the slides was out of sync, ie,
the slide video finished earlier than the audio track. This produced a video with the
audio being played over the wrong slide.

The origin code assumed the encoder for a codec mime type (eg, "video/avc") would always
properly encode the video. As it turns out, there can be more than one encoder for
a mime type, so, picking the first encoder in the list may not produce the desired
video. The code was updated to look for a codec encoder with a specific canonical name that
has always worked for us in the past.

The following test procedure was run:

  1. Load a clean version of "002 Lost Coin.bloom" into the device.
  2. Execute Story Producer app and "002 Lost Coin" should appear in the "Story Templates" list.
  3. Select "002 Lost Coin" and the "002 Lost Coin" story is loaded.
  4. Go to the "Accuracy Check" phase and approve all the slides.
  5. Go to the "Finalize" phase and enter a field in "Local Credits" and use the word "Coin" for "Text in Filename". Then select the following option: "Include Pictures". Once this data has been entered, press the "Create Video" button.
  6. A status bar appears showing the progress of the creation of the video.
  7. When the creation of the video is complete, go to the share phase and view the video

Results:

Audio matched the video through the story.

Other combinations of videos were made using background music and text. Videos created by these various combinations of video creation options correctly synced the audio to the video.

The testing was performed on the following configurations:

Pixel 5 Hardware, API 31
Samsung Hardware, API 29
Samsung Hardware, API 28

Copy link
Copy Markdown
Collaborator

@bryanwussow bryanwussow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@bryanwussow bryanwussow merged commit b5a6c13 into sillsdev:develop Apr 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants