-
-
Notifications
You must be signed in to change notification settings - Fork 246
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
Retry behavior configuration #80
Labels
good first issue
Good for newcomers
Comments
This and #75 are related (in that the fix for both would be the same) A good solution here would be to allow for extra kwargs to be passed |
mbcrawfo
pushed a commit
to mbcrawfo/atomic-agents
that referenced
this issue
Feb 10, 2025
mbcrawfo
pushed a commit
to mbcrawfo/atomic-agents
that referenced
this issue
Feb 10, 2025
mbcrawfo
pushed a commit
to mbcrawfo/atomic-agents
that referenced
this issue
Feb 10, 2025
mbcrawfo
pushed a commit
to mbcrawfo/atomic-agents
that referenced
this issue
Feb 10, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The
BaseAgent
class currently doesn't support retry functionality through themax_retries
parameter that instructor provides. In theget_response
andrun_async
methods, API calls to instructor don't pass any retry configuration:Adding
max_retries
to theBaseAgentConfig
would be a simple yet valuable improvement.Also, maybe other instructor parameters should be added such as
context
orvalidation_context
.The text was updated successfully, but these errors were encountered: