-
Notifications
You must be signed in to change notification settings - Fork 44.4k
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
feat(rnd): AutoGPT Agent wrapper for Agent Server #7365
Conversation
✅ Deploy Preview for auto-gpt-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
846984a
to
53a07d6
Compare
include_os_info = ( | ||
self.code_executor.config.execute_local_commands | ||
if hasattr(self, "code_executor") | ||
else False | ||
) |
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.
Without this it throws errors when code_executor
is removed.
@@ -33,6 +33,7 @@ gTTS = "^2.3.1" | |||
jinja2 = "^3.1.2" | |||
jsonschema = "*" | |||
litellm = "^1.17.9" | |||
numpy = ">=1.26.0,<2.0.0" |
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.
Incompatible dependencies when using ^2
class BlockAgentSettings(AgentSettings): | ||
disabled_components: list[str] = Field(default_factory=list) |
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.
If we move this to autogpt
AgentSettings
then I think we can remove BlockAgent
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #7365 +/- ##
==========================================
- Coverage 54.21% 54.20% -0.01%
==========================================
Files 122 122
Lines 6875 6876 +1
Branches 881 881
==========================================
Hits 3727 3727
- Misses 3015 3016 +1
Partials 133 133
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@@ -1,5 +1,6 @@ | |||
from autogpt_server.blocks import sample, reddit, text, ai, wikipedia, discord | |||
from autogpt_server.data.block import Block | |||
from autogpt_server.data.agent_block import AutoGPTAgentBlock |
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.
from autogpt_server.data import agent_block
and add agent_block
to all ?
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.
I moved agent_block.py
to blocks/agent.py
and added agent
module to __all__
class Input(BlockSchema): | ||
task: str | ||
input: str | ||
disabled_components: list[str] = Field(default_factory=list) |
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.
Is it easier to exclude than to include ?
If I want to execute the agent with only two tools, how many entries should I add here ?
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.
For now it's be better to have enabled_componets
, I updated it. However, SystemComponent
is force-enabled because agent may not work well without it.
input: str | ||
disabled_components: list[str] = Field(default_factory=list) | ||
disabled_commands: list[str] = Field(default_factory=list) | ||
fast_mode: bool = False |
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.
Can we also pass the api_key here?
Same thing as:
https://github.com/Significant-Gravitas/AutoGPT/pull/7373/files#diff-8bc0ec9590a3208160360ef6789128eae490a405fa67349418d414dd0f2c4261R19
it will autoload it from env with the key OPENAI_API_KEY
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.
done
Failure "fixed" by actions/toolkit#1723 and caused by actions/upload-artifact#485 |
Fix macos build
If we have a choice @kcze then I much prefer better results slower (better LLM) than worse results quickly (fast LLM) by default. |
There's a switch on the block now :) I just made it fast for the test, it's smart by default for the block. |
Changes 🏗️
Add AutoGPT Agent wrapper block for the Agent Server that allows running a temporary agent with full AutoGPT capabilities. You can limit components and commands it has access to, specify
task
andinput
data.autogpt
andforge
dependency to theautogpt_server
AutoGPTAgentBlock
that initializes and runs a single agent loop on executionBlockAgent
that inherits fromautogpt
Agent
and is a thin extension on the agent that allows to disable componentsOutputComponent
that addsoutput
command for the agentPR Quality Scorecard ✨
+2 pts
+5 pts
+5 pts
+5 pts
-4 pts
+4 pts
+5 pts
-5 pts
agbenchmark
to verify that these changes do not regress performance?+10 pts