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.

Sign up for GitHub

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

Jump to bottom

Autogpt namespace fix imports #1380

Merged
merged 4 commits into from
Apr 14, 2023
Merged

Autogpt namespace fix imports #1380

merged 4 commits into from
Apr 14, 2023

Conversation

waynehamadi
Copy link
Contributor

@waynehamadi waynehamadi commented Apr 14, 2023 •
edited
Loading

Background

thanks for your big help @dhensen :
see: #329

Changes

Documentation

Test Plan

PR Quality Checklist

  • My pull request is atomic and focuses on a single change.
  • I have thoroughly tested my changes with multiple different prompts.
  • I have considered potential risks and mitigations for my changes.
  • I have documented my changes clearly and comprehensively.
  • I have not snuck in any "extra" small tweaks changes

dhensen and others added 3 commits April 14, 2023 10:27
@dhensen @waynehamadi
Convert to python module named autogpt.
d64f866 
Also fixed the Dockerfile.
Converting to module makes development easier.
Fixes coverage script in CI and test imports.
@waynehamadi
added more autogpt prefixes in imports
adf7c3a
@waynehamadi
added message to redirect users
8dbc71d
@waynehamadi waynehamadi mentioned this pull request Apr 14, 2023
Closed
@waynehamadi waynehamadi force-pushed the autogpt-namespace-fix-imports branch from a033067 to 9b56ebe Compare April 14, 2023 18:55
@waynehamadi
Copy link
Contributor Author

@richbeales @p-i-

@richbeales richbeales merged commit 5389b2d into Significant-Gravitas:master Apr 14, 2023
@calebanthony calebanthony mentioned this pull request Apr 14, 2023
Closed
1 task
sadmuphin added a commit to sadmuphin/Auto-GPT that referenced this pull request Apr 14, 2023
@sadmuphin
Fixed with merge Significant-Gravitas#1380
47397f1
@dhensen
Copy link
Contributor

dhensen commented Apr 14, 2023

Tears of Joy, great news!!! AutoGPT is module now finally!!!! Thanks all @merwanehamadi @nponeccop @richbeales @Torantulino 🥇🏅🥉🥈🎖️🥇🥇🥇

@nponeccop
Copy link
Contributor

Cross me out I was against lol

@RodjaMe
Copy link

RodjaMe commented Apr 22, 2023

Sorry guys but I'm totally clueless about coding, was trying to install Auto-gpt, my cmd line is giving me this message "python: can't open file 'C:\AI\AutoGPT\Auto-GPT\scripts\main.py': [Errno 2] No such file or directory" which according to what I understood from the above; you have fixed, can you tell a total non-coder (me), how to fix this ? if it's difficult then don't bother , thanks in advance

@dhensen
Copy link
Contributor

dhensen commented Apr 23, 2023

Hi @RodjaMe did you clone AutoGPT a few weeks ago? script/main.py does not exist anymore, please try starting from scratch. Maybe rename Auto-GPT directory to Auto-GPT-old so that you don't lose any essential stuff. Then git clone the current repository, follow the README.me and only delete the Auto-GPT-old when you're sure everything is running fine.

sindlinger pushed a commit to Orgsindlinger/Auto-GPT-WebUI that referenced this pull request Sep 25, 2024
@richbeales
Merge pull request Significant-Gravitas#1380 from merwanehamadi/autog…
e3e76ed 
…pt-namespace-fix-imports

Autogpt namespace fix imports
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@richbeales richbeales richbeales approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@waynehamadi @dhensen @nponeccop @RodjaMe @richbeales @nhra168