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

Handle issues with OpenAI responses gracefully. #426

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

Conversation

bobvandevijver
Copy link
Contributor

Instead of throwing away the output (for which you paid), it is now written in a fail file so you as developer might still be able to use it. It also now gives an valid error when the the processing was stopped without hitting a natural stopping point, instead of a general message the resulting JSON is invalid.

Instead of throwing away the output (for which you paid), it is now
written in a fail file so you as developer might still be able to use
it. It also now gives an valid error when the the processing was stopped
without hitting a natural stopping point, instead of a general message
the resulting JSON is invalid.
@RobinBol RobinBol requested a review from WeeJeWel August 30, 2024 14:15
@OlivierZal
Copy link
Contributor

I love this idea, it's fair to avoid developers paying for a failure... Hope this will be reviewed.
Thanks @bobvandevijver!

@shaarkys
Copy link

I would love that to be integrated. Currently trying to translate 25kB driver.flow.compose.json and it fails but the output, even partial, is gone. @WeeJeWel ?

@WeeJeWel WeeJeWel requested review from jeroenwienk and removed request for WeeJeWel November 26, 2024 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants