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

Standardize tool interface #72

Closed
2 tasks done
ErikBjare opened this issue Feb 4, 2024 · 5 comments
Closed
2 tasks done

Standardize tool interface #72

ErikBjare opened this issue Feb 4, 2024 · 5 comments
Labels
enhancement New feature or request tool

Comments

@ErikBjare
Copy link
Owner

ErikBjare commented Feb 4, 2024

Tools currently need to be registered in gptme/tools/__init__.py.

  • It'd be nice if we didn't have to enumerate them in __init__.py and instead auto-discover them, or something.
  • enable/disable them in config or with a CLI param (done in 48d559b)
@ErikBjare ErikBjare added enhancement New feature or request tool labels Feb 21, 2024
@ErikBjare
Copy link
Owner Author

Most of the work done in #86

@ErikBjare
Copy link
Owner Author

This is pretty clean now.

@ErikBjare
Copy link
Owner Author

ErikBjare commented Oct 22, 2024

Should maybe make a @tool decorator that takes a callable or ToolSpec and registers it/makes it loadable, then explicitly specify which tools to load during init.

Mainly so we can get rid of the stuff in tools/init.py where all the stuff gets imported (although we would still have to import it somehow, maybe dynamically during init).

@jrmi
Copy link
Contributor

jrmi commented Jan 10, 2025

Solved by #336?

@ErikBjare
Copy link
Owner Author

Yes! 🥳

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request tool
Projects
None yet
Development

No branches or pull requests

2 participants