- Notifications
You must be signed in to change notification settings - Fork94
Minor cleanups: interceptors and OpenAI Agents SDK#211
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
| asyncdef_perform_searches(self,search_plan:WebSearchPlan)->list[str]: | ||
| withcustom_span("Search the web"): | ||
| num_completed=0 |
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.
Do we want to do code cleanup which makes it different from the openai sample?
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.
No strong opinion personally. I didn't know this code was copied.
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.
Reverted. Doesn't seem much point if it's copied code.
569abcb to9a0f59fCompareThis reverts commite7e3e12.
No description provided.