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

Allow all supported audio codecs in ExoPlayer transcoding profile #2381

Merged

Conversation

nielsvanvelzen
Copy link
Member

I've only tested this one a single video item that needed transcoding for subs but also transcoded the audio. Not sure why the transcoding profile didn't allow all audio codecs.

Changes

  • Allow all supported audio codecs in ExoPlayer transcoding profile

Issues

@nielsvanvelzen nielsvanvelzen added the enhancement New feature or request label Jan 2, 2023
@nielsvanvelzen nielsvanvelzen modified the milestone: v0.16.0 Jan 2, 2023
@thornbill
Copy link
Member

👀 I wonder if this is the cause of all the reports of unnecessary audio transcoding...

@nielsvanvelzen
Copy link
Member Author

I can confirm it fixes that issue. Previously the dashboard said the transcode reason was because of audio, now it says it's because of the subtitles. So far everything seems to work fine for me. But I'm still not sure if it fits for 0.15.1 or if we should put it back to 0.16.0.

@nielsvanvelzen nielsvanvelzen added the playback Issue related to media playback label Jan 3, 2023
@nielsvanvelzen
Copy link
Member Author

After tinkering about it for a bit I think we should include this in 0.15.z. I don't expect new issues from this change and we can always revert if a bug appears.

@nielsvanvelzen nielsvanvelzen added this to the v0.15.1 milestone Jan 3, 2023
Copy link
Member

@thornbill thornbill left a comment

Choose a reason for hiding this comment

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

Untested but lgtm!

@nielsvanvelzen nielsvanvelzen merged commit 096a436 into jellyfin:master Jan 4, 2023
@nielsvanvelzen nielsvanvelzen deleted the exo-transcode-profile-audio branch January 4, 2023 07:54
@VampiricAlien
Copy link

https://exoplayer.dev/supported-formats.html
Does the app include FFmpeg extension?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request playback Issue related to media playback
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants