Skip to content

Conversation

nedbat
Copy link
Member

@nedbat nedbat commented Jan 1, 2024

We are required to call use_tool_id, otherwise a ValueError occurs: "ValueError: tool 1 is not in use." And we cannot use an ID that is already in use, or "ValueError: tool 1 is already in use" occurs, as mentioned above.

This paragraph seems simply wrong. Let me know if I have misunderstood.


📚 Documentation preview 📚: https://cpython-previews--113619.org.readthedocs.build/

We are required to call use_tool_id, otherwise a ValueError occurs:
"ValueError: tool 1 is not in use."  And we cannot use an ID that is
already in use, or "ValueError: tool 1 is already in use" occurs, as
mentioned above.

This paragraph seems simply wrong.  Let me know if I have misunderstood.
@nedbat nedbat requested a review from markshannon January 1, 2024 11:51
@bedevere-app bedevere-app bot added docs Documentation in the Doc dir skip news awaiting review labels Jan 1, 2024
@nedbat
Copy link
Member Author

nedbat commented Jan 19, 2024

@markshannon what do you think of this?

@markshannon
Copy link
Member

This looks good, thanks

@markshannon markshannon merged commit 07ef63f into python:main Jan 25, 2024
@nedbat nedbat deleted the nedbat/clarify-use-tool-id branch March 19, 2024 20:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation in the Doc dir skip issue skip news

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants