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.

Sign up for GitHub

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

Jump to bottom

Refactor ToolRegistry #3444

Open
jacopo-chevallard opened this issue Oct 31, 2024 — with Linear · 1 comment
Open

Refactor ToolRegistry #3444

jacopo-chevallard opened this issue Oct 31, 2024 — with Linear · 1 comment
Assignees
@jacopo-chevallard

Comments

@jacopo-chevallard Linear
Copy link
Collaborator

Currently, we deal with tools as in core/quivr_core/llm_tools/web_search_tools.py.

We should rather define our own QuivrToolBase class where tools inherit from, e.g.

class TavilyTool(BaseTool)

We can then use a ProxyMapping as a registry to the tools. We could create tools (as classes or objects) and retrieve them.
@linear Linear
Copy link

linear bot commented Oct 31, 2024

CORE-275 Refactor `ToolRegistry`

@jacopo-chevallard jacopo-chevallard self-assigned this Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jacopo-chevallard jacopo-chevallard

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jacopo-chevallard