-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
feat: add indictor for tool failure to FunctionExecutionResult #5428
Conversation
@microsoft-github-policy-service agree company="Amazon" |
1 similar comment
@microsoft-github-policy-service agree company="Amazon" |
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.
You can run uv sync --all-extras in the 'python' directory to sync your local environment. There is no dependency change so there shouldn't be update to the uv.lock file
I did that. Doesn't help. I manually reverted it, but the
|
Could you try to purge your local virtualenv completely and recreate on again. Also make sure your
I just ran the CI tests and it works. So likely your local environment setup has some issue. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #5428 +/- ##
==========================================
- Coverage 78.32% 78.32% -0.01%
==========================================
Files 165 165
Lines 9800 9803 +3
==========================================
+ Hits 7676 7678 +2
- Misses 2124 2125 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
python/packages/autogen-ext/src/autogen_ext/agents/openai/_openai_assistant_agent.py
Show resolved
Hide resolved
Thanks! For now we don't really have a sample showing this. I believe with Anthropic model client, we can utlize the error field to indicate error, as described there: https://docs.anthropic.com/en/docs/build-with-claude/tool-use#troubleshooting-errors. |
Why are these changes needed?
Some LLMs recieve an explicit signal about tool use failures. This change will allow
Related issue number
Closes #5273
Checks
We probably want to add some additional testing. I need some advice where to add it.