Skip to content
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

[Feature Request]: Enable overwrite_tools in register_for_llm for GPTAssistantAgent #2575

Open
jtrugman opened this issue May 2, 2024 · 1 comment
Assignees
Labels
enhancement New feature or request openai-assistant Issues related to assistants from OpenAI

Comments

@jtrugman
Copy link
Contributor

jtrugman commented May 2, 2024

Is your feature request related to a problem? Please describe.

Re: Conversation between Eric and I We should have the ability to set the tools a GPTAssistantAgent can use via the register_for_llm paradigm that is set up. Currently, register_for_llm does not work for GPTAssistantAgent because it does not update the tools for the OpenAI Assistants API. This results in the issue described here in PR-2375 where fetching the tools via llm_config["tools"] returns the proper tools, but seeing the tools the Assistant API has access too on the OpenAI Platform Portal shows the tools not being there.

Describe the solution you'd like

We agreed that we should have the ability to overwrite_tools for register_for_llm. Basically the logic in overwrite_tools that enables you to use the underlying OpenAI APIs to update the tool available to the Assistant API should be applied for then you are register_for_llm on a GPTAssistantAgent. This we have feature parity between GPTAssistantAgent and ConversableAgent.

Additional context

No response

@jtrugman jtrugman added the enhancement New feature or request label May 2, 2024
@jtrugman
Copy link
Contributor Author

jtrugman commented May 2, 2024

@ekzhu can you assign this issue to me? I can work on it

@ekzhu ekzhu changed the title [Feature Request]: Enable overwrite_tools in register_for_llm [Feature Request]: Enable overwrite_tools in register_for_llm for GPTAssistantAgent May 3, 2024
@ekzhu ekzhu added the openai-assistant Issues related to assistants from OpenAI label May 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request openai-assistant Issues related to assistants from OpenAI
Projects
None yet
Development

No branches or pull requests

2 participants