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

Modify default max tokens for whisper #23

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

echo-lalia
Copy link

This PR is related to issue #22.

This change makes the program work for me. However, this program seemingly did not used to get this error, and I don't know what has changed that could've started causing this issue, and so I'm not confident whether this is a proper solution, or just a quick workaround.

This adjustment was made based on this error printed when running this script: The length of `decoder_input_ids`, including special start tokens, prompt tokens, and previous tokens, is 2, and `max_new_tokens` is 512. Thus, the combined length of `decoder_input_ids` and `max_new_tokens` is: 514. This exceeds the `max_target_positions` of the Whisper model: 448

This adjustment was made based on this error printed when running this script: `The length of `decoder_input_ids`, including special start tokens, prompt tokens, and previous tokens, is 2,  and `max_new_tokens` is 512. Thus, the combined length of `decoder_input_ids` and `max_new_tokens` is: 514. This exceeds the `max_target_positions` of the Whisper model: 448`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant