-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Comments
ExoPlayer skipping silence could only be enabled or disabled AFAIK: https://exoplayer.dev/doc/reference/com/google/android/exoplayer2/PlaybackParameters.html#skipSilence |
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 |
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. |
@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. |
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. |
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. |
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)
The text was updated successfully, but these errors were encountered: