Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Bugfix: Illogical "Avoid computing higher temperatures on no_speech" … #1

Merged
merged 1 commit into from
Dec 13, 2024

Conversation

pooyaravari
Copy link
Owner

…(openai#1903)

  • Bugfix: Illogical "Avoid computing higher temperatures on no_speech"

Bugfix for openai#1279

It's "silence" when decoding has failed due to compression_ratio_threshold too, when further down the code it's not "silence" anymore.

"Silence" should be only when decoding has failed due to logprob_threshold.

Like described there:
https://github.com/openai/whisper/blob/8bc8860694949db53c42ba47ddc23786c2e02a8b/whisper/transcribe.py#L421

And in code there:
https://github.com/openai/whisper/blob/8bc8860694949db53c42ba47ddc23786c2e02a8b/whisper/transcribe.py#L243-L251

  • Fix if "logprob_threshold=None"

…1903)

* Bugfix: Illogical "Avoid computing higher temperatures on no_speech"

Bugfix for #1279

It's "silence" when decoding has failed due to `compression_ratio_threshold` too, when further down the code it's not "silence" anymore.

"Silence" should be only when decoding has failed due to `logprob_threshold`.

Like described there:
https://github.com/openai/whisper/blob/8bc8860694949db53c42ba47ddc23786c2e02a8b/whisper/transcribe.py#L421

And in code there:
https://github.com/openai/whisper/blob/8bc8860694949db53c42ba47ddc23786c2e02a8b/whisper/transcribe.py#L243-L251

* Fix if "logprob_threshold=None"

---------

Co-authored-by: Jong Wook Kim <jongwook@openai.com>
@pooyaravari pooyaravari merged commit a48344f into pooyaravari:main Dec 13, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants