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

Catch generic exception from transformers in huggingface #690

Merged

Conversation

jmartin-tech
Copy link
Collaborator

@jmartin-tech jmartin-tech commented May 21, 2024

Explicitly return None for all results and log when exception raised for empty prompt

Testing update reflect expected results.

Explicitly return `None` for all results and long when exception raised for empty prompt

Signed-off-by: Jeffrey Martin <[email protected]>
@jmartin-tech
Copy link
Collaborator Author

See: #689 (comment)

@jmartin-tech jmartin-tech requested a review from leondz May 21, 2024 21:34
@jmartin-tech jmartin-tech merged commit 60675d2 into NVIDIA:main May 22, 2024
4 checks passed
@jmartin-tech jmartin-tech deleted the fix/hf-transformer-api-change branch May 22, 2024 13:34
@github-actions github-actions bot locked and limited conversation to collaborators May 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant