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

refactor(autogpt, forge): Remove autogpts directory #7163

Conversation

kcze
Copy link
Contributor

@kcze kcze commented May 22, 2024

Background

Changes 🏗️

  • Moved autogpt and forge to project root
  • Removed autogpts directory
  • Moved and renamed submodule autogpts/autogpt/tests/vcr_cassettes to autogpt/tests/vcr_cassettes
  • When using CLI agents will be created in agents directory (instead of autogpts)
  • Renamed relevant docs, code and config references from autogpts/[forge|autogpt] to [forge|autogpt] and from *../../* to *../*
  • Updated CODEOWNERS, GitHub Actions and Docker *.yml configs
  • Updated symbolic links in docs

PR Quality Scorecard ✨

  • Have you used the PR description template?   +2 pts
  • Is your pull request atomic, focusing on a single change?   +5 pts
  • Have you linked the GitHub issue(s) that this PR addresses?   +5 pts
  • Have you documented your changes clearly and comprehensively?   +5 pts
  • Have you changed or added a feature?   -4 pts
    • Have you added/updated corresponding documentation?   +4 pts
    • Have you added/updated corresponding integration tests?   +5 pts
  • Have you changed the behavior of AutoGPT?   -5 pts
    • Have you also run agbenchmark to verify that these changes do not regress performance?   +10 pts

@github-actions github-actions bot added Classic AutoGPT Agent size/m documentation Improvements or additions to documentation size/l and removed size/m labels May 22, 2024
Copy link

netlify bot commented May 22, 2024

Deploy Preview for auto-gpt-docs failed.

Name Link
🔨 Latest commit f31eea4
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/664dc2fa1bc687000856d309

@kcze kcze marked this pull request as ready for review May 22, 2024 10:04
@kcze kcze requested review from a team as code owners May 22, 2024 10:04
@kcze kcze requested review from Bentlybro, aarushik93 and Pwuts and removed request for a team, Bentlybro and aarushik93 May 22, 2024 10:04
Copy link

netlify bot commented May 22, 2024

Deploy Preview for auto-gpt-docs canceled.

Name Link
🔨 Latest commit bdf76c8
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/664ddec255d6ab00083bb2df

autogpt/autogpt/app/agent_protocol_server.py Outdated Show resolved Hide resolved
benchmark = ["agbenchmark @ file:///home/reinier/code/agpt/Auto-GPT/benchmark"]
benchmark = ["agbenchmark @ file:///Users/czerwinski/Projects/AutoGPT/benchmark"]
Copy link
Member

Choose a reason for hiding this comment

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

I didn't realize the absolute path was saved in the poetry.lock :')

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We may want to switch to remote dependency

autogpts/autogpt/challenges_already_beaten.json Outdated Show resolved Hide resolved
autogpts/autogpt/data/.keep Outdated Show resolved Hide resolved
forge/forge/agent/agent.py Outdated Show resolved Hide resolved
@kcze kcze merged commit 4c32572 into Significant-Gravitas:master May 22, 2024
9 of 19 checks passed
@kcze kcze deleted the kpczerwinski/open-908-remove-autogpts-dir branch May 22, 2024 12:10
@Pwuts Pwuts mentioned this pull request Jun 5, 2024
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Classic AutoGPT Agent documentation Improvements or additions to documentation size/l
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants