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

Make Tool single-input #3684

Closed
wants to merge 2 commits into from
Closed

Make Tool single-input #3684

wants to merge 2 commits into from

Commits on Apr 27, 2023

  1. Add validation on agent instantiation for multi-input tools (#3681)

    Tradeoffs here:
    - No lint-time checking for compatibility
    - Differs from JS package
    - The signature inference, etc. in the base tool isn't simple
    - The `args_schema` is optional
    
    Pros:
    - Forwards compatibility retained
    - Doesn't break backwards compatibility
    - User doesn't have to think about which class to subclass (single base
    tool or dynamic `Tool` interface regardless of input)
    -  No need to change the load_tools, etc. interfaces
    
    Co-authored-by: Hasan Patel <mangafield@gmail.com>
    vowelparrot and TheSeer507 committed Apr 27, 2023
    Configuration menu
    Copy the full SHA
    027638a View commit details
    Browse the repository at this point in the history

Commits on Apr 28, 2023

  1. Configuration menu
    Copy the full SHA
    30b4251 View commit details
    Browse the repository at this point in the history