-
Notifications
You must be signed in to change notification settings - Fork 43.7k
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
[Re-arch]: List Files Ability #4791
Conversation
* Update planner´s base.py * improving based on comments * added_selffeedback * improving_docstring_and_interface * improved_data_classes_and_feedback_fn * docstring
* initial logging interface * imports * metadata
* Initial base.py for message broker * type_enum * fixing_imports * classes_added
* re-arch: Initial BudgetManager impl * separate base abc from concrete example * init needs to accept kwargs
* First version of implementation for command base class * Update PR comments --------- Co-authored-by: Daryl Rodrigo <daryl@orkestro.com>
…avitas/Auto-GPT into re-arch/hello-world
…Auto-GPT into re-arch/hello-world
…Auto-GPT into re-arch/hello-world
* Added in a click entry point and status report * renamed app_sketch to runner * Fixing imports adding run commands for the client and server * Make status display a little more formal and flexible --------- Co-authored-by: James Collins <collijk@uw.edu>
* Revise message broker interface * Add emitter * This is the last change for a bit, I think * Incorporate feedback
* feat: plugin base * fix: formatting --------- Co-authored-by: James Collins <collijk@uw.edu>
…o agent-state-encapsulation
…avitas/Auto-GPT into re-arch/hello-world
@collijk One thing I noticed is |
@collijk Do we want to restrict to workspace only? |
related in #4770 |
@@ -84,7 +84,7 @@ def __call__(self, filename: str) -> AbilityResult: | |||
|
|||
class WriteFile(Ability): | |||
default_configuration = AbilityConfiguration( | |||
packages_required=["unstructured"], | |||
packages_required=["os"], |
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.
We don't need builtin packages here.
|
||
class ListFiles(Ability): | ||
default_configuration = AbilityConfiguration( | ||
packages_required=["os"], |
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.
Same
Feel free to add. I haven't dug into knowledge summarization yet so haven't had to use it. |
The workspace should be managing that (and it's user configurable) |
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request. |
Branch deleted for security reasons |
Background
Changes
Documentation
Test Plan
PR Quality Checklist