-
Notifications
You must be signed in to change notification settings - Fork 30.2k
Deprecate transformers.agents #36415
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
Conversation
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.
As mentioned on Slack (private link):
Fine for me to use these deprecation decorators from huggingface_hub even though they are private. Using private methods makes it more difficult to avoid breaking changes in huggingface_hub. For deprecation decorators it's perfectly fine as they are stable + only meant for a short period of time. That's more of an exception, let's not make a habit of it! 😄
@@ -720,6 +722,10 @@ class ReactAgent(Agent): | |||
The action will be parsed from the LLM output: it consists in calls to tools from the toolbox, with arguments chosen by the LLM engine. | |||
""" | |||
|
|||
@_deprecate_method( | |||
version="4.51.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.
Only 1 release version in advance seems little to me but no strong opinion 🤷♂️
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
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.
Thank you @aymeric-roucher!
@aymeric-roucher This seems like a great idea! One snag though is that Thoughts? |
What does this PR do?
Deprecates
transformers.agents
in favor of smolagents for extended functionality, and a similar API.