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

Added in a click entry point and status report #4005

Merged
merged 4 commits into from
May 9, 2023

Conversation

Swiftyos
Copy link
Contributor

@Swiftyos Swiftyos commented May 8, 2023

Added in a status command so we can quickly get an idea of where the re-arch is up to.
These commands are to be deleted when the re-arch is completed.

Usage:

./run.sh  status   
All packages are installed.
Getting v2 agent status...
|-----------------|-----------------|
| Name            | Status          |
|-----------------|-----------------|
| Agent           | INTERFACE_DONE  |
| Budget          | BASIC_DONE      |
| Command         | IN_PROGRESS     |
| Configuration   | INTERFACE_DONE  |
| LLM             | INTERFACE_DONE  |
| Logging         | INTERFACE_DONE  |
| Memory          | TODO            |
| Messaging       | BASIC_DONE      |
| Planning        | INTERFACE_DONE  |
| Plugin          | TODO            |
| Workspace       | INTERFACE_DONE  |
|-----------------|-----------------|

To view handover notes by the last person to work on a module:

./run.sh  status -d
All packages are installed.
Getting v2 agent status...

Here are some handover notes from the last contributor to work on the system. 
These are not necessarily up to date, but should give you a good idea of where to jump in.

Agent:
         Interface has been created. Work is needed on the agent factory.

Budget:
         Interface has been completed and a basic implementation has been created.

Command:
         More work is needed, basic ideas are in place.

Configuration:
         Interface has been created. Basic example needs to be created.

LLM:
         Interface has been created. Next up is creating a basic implementation.

Logging:
         Interface has been created. Basic example needs to be created.

Memory:
         The memory subsystem has not been started yet.

Messaging:
         Interface has been completed and a basic implementation has been created.

Planning:
         Interface has been created. Basic example needs to be created.

Plugin:
         The plugin system has not been started yet.

Workspace:
         Interface has been created. Basic example needs to be created.

@vercel
Copy link

vercel bot commented May 8, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
docs ⬜️ Ignored (Inspect) May 9, 2023 0:10am

@Swiftyos Swiftyos requested a review from collijk May 8, 2023 10:19
@github-actions
Copy link
Contributor

github-actions bot commented May 8, 2023

This PR exceeds the recommended size of 200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size

@github-actions
Copy link
Contributor

github-actions bot commented May 8, 2023

This PR exceeds the recommended size of 200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size

@Swiftyos Swiftyos self-assigned this May 8, 2023
from autogpt.core.planning import Planner
from autogpt.core.plugin import Plugin, PluginManager
from autogpt.core.workspace import Workspace
import click
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Move this out of here and into something like autogpt.cli.v2. core will operate like a library interface, so this init will be non-trivial as we get to implementation. Alternatively we could have the cli and the app sketch live in autogpt.re_arch_app or similar. Let me know what you think as I'll have to do some motion in other branches.

from autogpt.core.planning.base import ModelPrompt
from autogpt.core.plugin.base import PluginManager
from autogpt.core.workspace.base import Workspace
from autogpt.core.configuration.base import Configuration
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't drop the typechecking blocks yet. There will be cyclic dependencies in the method signatures, but I'm still ironing out the details. I don't want to have to come monkey with imports all the time. Easier to just resolve this stuff with interfaces have stabilized

@github-actions
Copy link
Contributor

github-actions bot commented May 8, 2023

This PR exceeds the recommended size of 200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size

@collijk collijk merged commit b55d3ff into re-arch/hello-world May 9, 2023
@collijk collijk deleted the re-arch/hw-status branch May 9, 2023 00:10
@github-actions
Copy link
Contributor

github-actions bot commented May 9, 2023

This PR exceeds the recommended size of 200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants