Skip to content

Handle issues with OpenAI responses gracefully.#426

Open
bobvandevijver wants to merge 1 commit intoathombv:developfrom
bobvandevijver:handle-openai-errors
Open

Handle issues with OpenAI responses gracefully.#426
bobvandevijver wants to merge 1 commit intoathombv:developfrom
bobvandevijver:handle-openai-errors

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