Skip to content

Comments

Fix crash after failed audio download in process_conversation_inputs#217

Open
kirklandsign wants to merge 1 commit intohuggingface:mainfrom
kirklandsign:fix-audio-download
Open

Fix crash after failed audio download in process_conversation_inputs#217
kirklandsign wants to merge 1 commit intohuggingface:mainfrom
kirklandsign:fix-audio-download

Conversation

@kirklandsign
Copy link

The except block printed an error but let execution fall through to torchaudio.load(buf) where buf was undefined, causing an UnboundLocalError. Re-raise as RuntimeError with the URL and cause. Also add timeout=30 to requests.get() to prevent indefinite hangs.

The except block printed an error but let execution fall through to
torchaudio.load(buf) where buf was undefined, causing an
UnboundLocalError. Re-raise as RuntimeError with the URL and cause.
Also add timeout=30 to requests.get() to prevent indefinite hangs.
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