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.

Sign up for GitHub

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

Jump to bottom

Fix error when using Whisper Speech Recognition on Windows #644

Closed
wants to merge 1 commit into from
Closed

Fix error when using Whisper Speech Recognition on Windows #644

wants to merge 1 commit into from

Conversation

acenturyandabit
Copy link

Due to Windows' limitations on temporary files, we use a custom temporary file provider to ensure Whisper TTS can work on windows.

See https://stackoverflow.com/questions/23212435/permission-denied-to-write-to-my-temporary-file

@ftnext ftnext added whisper Features related to Whisper wontfix labels Jan 9, 2023
@ftnext
Copy link
Collaborator

ftnext commented Jan 9, 2023

Thanks for this great pull request.

Due to Windows' limitations on temporary files

Yeah, #632 shares the same issue and a workaround.

Actually, I updated the implementation to not use tempfile in #647, so this pull request is closed.

@ftnext ftnext closed this Jan 9, 2023
@acenturyandabit acenturyandabit deleted the err_fix branch January 10, 2023 19:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
whisper Features related to Whisper wontfix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@acenturyandabit @ftnext