Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MP4 output from Elastic Transcoder needs to be updated #1021

Open
collinpreston opened this issue Dec 15, 2022 · 1 comment
Open

MP4 output from Elastic Transcoder needs to be updated #1021

collinpreston opened this issue Dec 15, 2022 · 1 comment
Assignees

Comments

@collinpreston
Copy link
Contributor

collinpreston commented Dec 15, 2022

Steps to Reproduce

  1. Create or use an existing collection in OVS that is associated with a course on edX.
  2. Upload a video to the collection.
  3. Attempt to download the video either from s3 (https://s3.console.aws.amazon.com/s3/buckets/odl-video-service-transcoded-rc?prefix=transcoded/&region=us-east-1) or by adding the video ID to a course component and specifying "user can download" from the advanced video settings (instructions for this are explained here: Mp4 video download #1011).

Expected Behavior

  1. The downloaded mp4 video should have the following details:
  • 320 kbps
  • 48 kHz audio

Actual Behavior

View the mp4 video's codec settings are slightly different.

Additional information

  1. The Elastic Encoder preset for mp4 files is defined here: https://github.com/mitodl/odl-video-service/blob/f89d0de9ec3893d2502c4069e4409bdd4cde3357/config/et_mp4_preset.json
@collinpreston collinpreston changed the title MP4 videosettings do not match MP4 output from Elastic Transcoder do not match the preset Dec 15, 2022
@collinpreston collinpreston changed the title MP4 output from Elastic Transcoder do not match the preset MP4 output from Elastic Transcoder needs to be updated Dec 15, 2022
@pdpinch
Copy link
Member

pdpinch commented Dec 16, 2022

I believe @collinpreston already fixed this in production, but it needs to be tested.

@pdpinch pdpinch self-assigned this Dec 16, 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

No branches or pull requests

2 participants