-
Notifications
You must be signed in to change notification settings - Fork 1.6k
web_search_preview tool call is not limited by max_turns option #783
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
Comments
All the |
I have encountered more than 6 sets of the following web_search_call events and those are regarded as a single turn:
Here is full response stream: My intention is that it would be great if |
Gotcha. We can't do this today, because max_turns is a local (SDK) param but the web searches happen in the API. An alternate approach is to set a max_tokens instead, which will force the model to use fewer tokens and hence fewer web searches? |
Thanks @rm-openai , I see that we cannot control the API. I solved this problem by counting |
Makes sense. I'll also look into adding more configurabilty to the API. thanks for the feedback. |
Please read this first
Yes
Describe the bug
max_turns of run_streamed() is ignored for
web_search_preview
toolDebug information
Repro steps
https://gist.github.com/jihun-im-open/b1351fb44320f83d72d50ab9f98a50da
Expected behavior
MaxTurn Error should be raised
The text was updated successfully, but these errors were encountered: