Changed temperature default value to None #553
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue :
Sometimes, the generated transcription gets stuck repeating the same words, and there is a warning :
Compression ratio threshold is not met with temperature 0.0
The problem is that the default value for temperature is 0 in speaches. The problem can be solved if we pass a higher temperature but it will apply to all the transcript.
The default behavior in faster-whisper is that if temperature is not specified, it is considered 0 and only when compression ratio threshold or log probability threshold are not met for a certain segment then it will try with temperature 0.2 and so on from the list [0, 0.2, 0.4, 0.6, 0.8, 1.0] for that segment.
Fix :
Change the default value for temperature to the same default list used in faster-whisper to let it handle retries for segments where compression ratio threshold or log probability threshold criterias are not met.