-
Notifications
You must be signed in to change notification settings - Fork 21
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
fix: add local built-in agent support #22
Conversation
## What Changed 1. **Local Tool Loading**: Added logic to check and load tools from the local directory first. 2. **Fixed NoneType Error**: Resolved the issue where the version was `None`, causing a `'NoneType' object has no attribute 'replace'` error. 3. **Improved Error Messages**: Enhanced error messages for better clarity and debugging. ## How It Works Now - **Local First**: The system first checks the local tools folder. - **Use Local Tool**: If a tool is found locally, it uses that version. - **Remote Fallback**: If not found locally, it downloads the tool. - **Version Handling**: Retrieves version from local `config.json` to ensure correct versioning. ## Testing - Tested with `math_agent` using local `wikipedia` tool. - Tested with `story_teller` agent. - Confirmed no NoneType errors occur.
## What Changed 1. **Local Tool Loading**: Added logic to check and load tools from the local directory first. 2. **Fixed NoneType Error**: Resolved the issue where the version was `None`, causing a `'NoneType' object has no attribute 'replace'` error. 3. **Improved Error Messages**: Enhanced error messages for better clarity and debugging. ## How It Works Now - **Local First**: The system first checks the local tools folder. - **Use Local Tool**: If a tool is found locally, it uses that version. - **Remote Fallback**: If not found locally, it downloads the tool. - **Version Handling**: Retrieves version from local `config.json` to ensure correct versioning.
## What Changed 1. **Local Tool Loading**: Added logic to check and load tools from the local directory first. 2. **Fixed NoneType Error**: Resolved the issue where the version was `None`, causing a `'NoneType' object has no attribute 'replace'` error. 3. **Improved Error Messages**: Enhanced error messages for better clarity and debugging. ## How It Works Now - **Local First**: The system first checks the local tools folder. - **Use Local Tool**: If a tool is found locally, it uses that version. - **Remote Fallback**: If not found locally, it downloads the tool. - **Version Handling**: Retrieves version from local `config.json` to ensure correct versioning.
## What Changed 1. **Local Tool Loading**: Added logic to check and load tools from the local directory first. 2. **Fixed NoneType Error**: Resolved the issue where the version was `None`, causing a `'NoneType' object has no attribute 'replace'` error. 3. **Improved Error Messages**: Enhanced error messages for better clarity and debugging. ## How It Works Now - **Local First**: The system first checks the local tools folder. - **Use Local Tool**: If a tool is found locally, it uses that version. - **Remote Fallback**: If not found locally, it downloads the tool. - **Version Handling**: Retrieves version from local `config.json` to ensure correct versioning.
- Add debug logs to track agent loading - Fix config handling in agent manager - Update seeact_demo_agent config format The code now: - Shows better error messages - Loads agents more reliably - Keeps full config when packaging"
## What's Changed - Added `--local_agent` flag to support loading agents from local filesystem - Added path validation for local agent loading - Simplified agent execution logic
This PR refactors the configuration management for Cerebrum by moving the URLs for the Agent and Tool Managers into a centralized configuration file (default.yaml). This change allows for easier management and customization of these URLs across different environments. Changes: Added agent_hub_url and tool_hub_url to default.yaml. 2. Updated AutoAgent and AutoTool classes to use URLs from the configuration file.
This reverts commit 1324ed1.
This reverts commit 53fd503.
## What's Changed - Added proper error handling for invalid API keys - Fixed response parsing for LiteLLM completion calls - Added HTTP status codes for different error types: - 402 for API key issues - 500 for other errors
## What Changed 1. Added `_get_builtin_agent_path` method to find local built-in agents 2. Added dependency check for local built-in agents 3. Fixed agent loading flow for local built-in examples
The example still shows the loading of agents from the hub? As it shows the running of agents by calling demo_author/demo_agent. |
Is there an example of using agents from the local path? |
Thanks Kai! The issue earlier was that some modules (e.g., arxiv) were not automatically installed. The updated implementation now checks and installs missing modules automatically during runtime. I tested this by uninstalling arxiv and re-running the agent, and it worked without errors. Let me know if you’d like a local path example as well |
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.
Got, merged.
What Changed
_get_builtin_agent_path
method to find local built-in agents