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

Transcode audio to keep multichannel support #1466

Closed
wants to merge 2 commits into from

Conversation

cewert
Copy link
Member

@cewert cewert commented Nov 2, 2023

Transcode multichannel audio files that downmix to stereo to keep multichannel support i.e. 6+ channel AAC files can be played by the device but downmix to stereo

Everything works with one big exception - video files that are transcoded to preserve multichannel audio play as the should but fast forward and rewind no longer work as well as the length of the video showing 0.

This code was originally from #1366. Making a PR for it so the rest of #1366 can be reviewed and merged.

@jellyfin-bot
Copy link
Contributor

This pull request has merge conflicts. Please resolve the conflicts so the PR can be reviewed. Thanks!

@jellyfin-bot jellyfin-bot added the merge-conflict This PR has a merge conflict label Nov 4, 2023
@jellyfin-bot
Copy link
Contributor

This pull request has merge conflicts. Please resolve the conflicts so the PR can be reviewed. Thanks!

@jellyfin-bot
Copy link
Contributor

This pull request has merge conflicts. Please resolve the conflicts so the PR can be reviewed. Thanks!

@cewert cewert added new-feature A new feature that currently doesn't exist. and removed merge conflict labels Nov 5, 2023
@cewert cewert changed the base branch from unstable to master December 14, 2023 15:07
@jellyfin-bot jellyfin-bot removed the merge-conflict This PR has a merge conflict label Dec 18, 2023
@cewert cewert added the help-wanted Extra attention is needed label Jan 13, 2024
@jellyfin-bot jellyfin-bot added the merge-conflict This PR has a merge conflict label Aug 3, 2024
@jellyfin-bot
Copy link
Contributor

This pull request has merge conflicts. Please resolve the conflicts so the PR can be reviewed. Thanks!

@cewert cewert removed the help-wanted Extra attention is needed label Oct 20, 2024
@cewert cewert closed this Oct 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge-conflict This PR has a merge conflict new-feature A new feature that currently doesn't exist.
Projects
Development

Successfully merging this pull request may close these issues.

2 participants