Add flag to include the cause in the exception str #137
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mpokrass
approved these changes
Nov 11, 2022
…IConnectionError Many libraries will only show the error string of the raised error, not displaying its cause. Prior to this path, this would mean that the many different kinds of connection errors get translated into a single opaque "Error communicating with OpenAI".
borisdayma
pushed a commit
to borisdayma/openai-python
that referenced
this pull request
Dec 7, 2022
* overload output type depending on stream literal (openai#142) * Bump to v22 * [numpy] change version (openai#143) * [numpy] change version * update comments * no version for numpy * Fix timeouts (openai#137) * Fix timeouts * Rename to request_timeout and add to readme * Dev/hallacy/request timeout takes tuples (openai#144) * Add tuple typing for request_timeout * imports * [api_requestor] Log request_id with response (openai#145) * Only import wandb as needed (openai#146) Co-authored-by: Felipe Petroski Such <[email protected]> Co-authored-by: Henrique Oliveira Pinto <[email protected]> Co-authored-by: Rachel Lim <[email protected]>
cgayapr
pushed a commit
to cgayapr/openai-python
that referenced
this pull request
Dec 14, 2024
* overload output type depending on stream literal (openai#142) * Bump to v22 * [numpy] change version (openai#143) * [numpy] change version * update comments * no version for numpy * Fix timeouts (openai#137) * Fix timeouts * Rename to request_timeout and add to readme * Dev/hallacy/request timeout takes tuples (openai#144) * Add tuple typing for request_timeout * imports * [api_requestor] Log request_id with response (openai#145) * Only import wandb as needed (openai#146) Co-authored-by: Felipe Petroski Such <[email protected]> Co-authored-by: Henrique Oliveira Pinto <[email protected]> Co-authored-by: Rachel Lim <[email protected]>
cgayapr
pushed a commit
to cgayapr/openai-python
that referenced
this pull request
Dec 14, 2024
…IConnectionError (openai#137) Many libraries will only show the error string of the raised error, not displaying its cause. Prior to this path, this would mean that the many different kinds of connection errors get translated into a single opaque "Error communicating with OpenAI".
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.