-
Notifications
You must be signed in to change notification settings - Fork 143
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
Add optional exception attribute to ErrorArtifact #747
Add optional exception attribute to ErrorArtifact #747
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fantastic, thank you for the PR! Could you also please add here?
Sure thing, added. |
Ah could you also add a section in Something like:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like you just need to merge the latest base branch changes into your branch. Then we're good!
CHANGELOG.md
Outdated
@@ -23,6 +23,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 | |||
- `output_task_input` and `output_task_output` fields to `FinishStructureRunEvent`. | |||
- `AmazonS3FileManagerDriver` for managing files on Amazon S3. | |||
- `MediaArtifact` as a base class for `ImageArtifact` and future media Artifacts. | |||
- Added optional `exception` field to `ErrorArtifact`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hyper nit: Added
is not needed here. The other bullet points read with an implicit Added
through the section header.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
As discussed on discord channel back in March (handling content management policy error), adding optional exception attribute to ErrorArtifact.
This will allow to check Exception instance against eg. OpenAI exceptions to handle it properly in your application.