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

Add audio night mode (as preference) to level out/compress audio volume. #1852

Merged

Conversation

robertduplock
Copy link

Add a togglable feature, "Audio night mode", which compresses a video's audio stream and tweaks the EQ, to solve the issue of "dialogue too quiet, explosions too loud".

Changes
Added extra option "Audio night mode" under "Settings > Playback > Audio"

Copy link
Member

@nielsvanvelzen nielsvanvelzen left a comment

Choose a reason for hiding this comment

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

I'm not sure about the naming. Wouldn't something like "Audio leveling" be more descriptive?

@robertduplock
Copy link
Author

Definitely happy to take feedback on the naming. "Night mode" does seem to be a fairly common name for this feature though (Sony, Sonos and Denon use this name on their TVs / soundbars).

@nielsvanvelzen nielsvanvelzen added the next-release Pull request related to a future release which is not being worked on yet label Jul 18, 2022

// Volume normalisation (audio night mode).
if (nightModeEnabled) {
this.enableAudioNightMode(mExoPlayer.getAudioSessionId());
Copy link
Member

Choose a reason for hiding this comment

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

Will this be an exoplayer only feature or will "night mode" also work with libVLC?

Copy link
Author

Choose a reason for hiding this comment

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

TBH I'm not sure at this point. I will test it today and find out.

Copy link
Member

Choose a reason for hiding this comment

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

It will be ExoPlayer only

@nielsvanvelzen
Copy link
Member

Definitely happy to take feedback on the naming. "Night mode" does seem to be a fairly common name for this feature though (Sony, Sonos and Denon use this name on their TVs / soundbars).

If this name is commonly used in other applications it makes sense to use it here too.

@nielsvanvelzen nielsvanvelzen added this to the v0.15.0 milestone Jul 29, 2022
@nielsvanvelzen nielsvanvelzen removed the next-release Pull request related to a future release which is not being worked on yet label Jul 29, 2022
@nielsvanvelzen nielsvanvelzen added the enhancement New feature or request label Aug 11, 2022
@nielsvanvelzen
Copy link
Member

Hey @robertduplock there are still some comments open in this PR before we can merge.

@nielsvanvelzen nielsvanvelzen added playback Issue related to media playback release-highlight Pull request might be suitable for mentioning in the release blog post labels Sep 3, 2022
@nielsvanvelzen
Copy link
Member

I've rebased and squashed your branch and fixed the remaining feedback. Unfortunately the commit author got lost when I removed the changes to the en-gb translations.

@nielsvanvelzen nielsvanvelzen enabled auto-merge (rebase) September 26, 2022 15:33
@nielsvanvelzen nielsvanvelzen merged commit 01dcc65 into jellyfin:master Sep 26, 2022
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 release-highlight Pull request might be suitable for mentioning in the release blog post
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants