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

[FR] Add tweaks to 'fast-forward during silence' function #3082

Open
3 tasks
CrimsonFork opened this issue Feb 12, 2020 · 6 comments
Open
3 tasks

[FR] Add tweaks to 'fast-forward during silence' function #3082

CrimsonFork opened this issue Feb 12, 2020 · 6 comments
Labels
feature request Issue is related to a feature in the app player Issues related to any player (main, popup and background)

Comments

@CrimsonFork
Copy link

This feature is amazing already, but I wish for a few options:

  • Set the speed to ramp up and down instead of instantly jumping between different speeds (linear only should be plenty)

  • Control the ff-speed

  • Control the volume threshold (preferably with visual volume feedback)

@Stypox Stypox added feature request Issue is related to a feature in the app player Issues related to any player (main, popup and background) labels Feb 12, 2020
@wb9688
Copy link
Contributor

wb9688 commented Aug 15, 2020

ExoPlayer skipping silence could only be enabled or disabled AFAIK: https://exoplayer.dev/doc/reference/com/google/android/exoplayer2/PlaybackParameters.html#skipSilence

@wb9688 wb9688 closed this as completed Aug 15, 2020
@wb9688
Copy link
Contributor

wb9688 commented Aug 15, 2020

Hmm… not sure if we could do something custom by modifying https://github.com/google/ExoPlayer/blob/release-v2/library/core/src/main/java/com/google/android/exoplayer2/audio/SilenceSkippingAudioProcessor.java and passing that as another AudioProcessor. @Redirion: Do you happen to know more?

@Redirion
Copy link
Member

Its a final class though. But I will take a look. Sounds like a reasonable feature especially for background playback which I personally use more often than video playback.

@wb9688
Copy link
Contributor

wb9688 commented Aug 15, 2020

@Redirion: Copy-paste under a different name is a thing though, see also e.g. how we have https://github.com/TeamNewPipe/NewPipe/blob/dev/app/src/main/java/androidx/fragment/app/FragmentStatePagerAdapterMenuWorkaround.java. Make sure to keep the license header from ExoPlayer in that case though.

@wb9688 wb9688 reopened this Aug 15, 2020
@CrimsonFork
Copy link
Author

CrimsonFork commented Aug 15, 2020

My biggest issue right now is that, while skipping, it doesn't play the audio, which it did in the early implementations. Now, that wouldn't be an issue if it only skipped absolute silence, but simply low volume is considered silence as well, making any speech that is not loud and clear ("dry") hard to understand or even inellegible in some cases.

@cheintz
Copy link

cheintz commented Jun 13, 2023

I was just looking for duplicate issues before creating a new issue for this feature request.

The current version is very aggressive in skipping and results in "drug commercial disclaimer-like" speech patterns. Perhaps good parameterization of silences to skip (or fast-forward through) is minimum silence time and silence speedup factor. So, for example, with a 0.5s minimum silence time and 5x speedup factor, silences of 0.5, 1.0, 1.5, 2 would last for 0.5, 0.6, 0.7, 0.8 seconds, respectively., Maybe also a max time, where silences that would end up longer than a threshold are just trimmed to that threshold.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Issue is related to a feature in the app player Issues related to any player (main, popup and background)
Projects
None yet
Development

No branches or pull requests

5 participants