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

Easy run with bat file (with requirements check and install if needed) #870

Merged
merged 2 commits into from
Apr 15, 2023
Merged

Easy run with bat file (with requirements check and install if needed) #870

merged 2 commits into from
Apr 15, 2023

Conversation

DenTheProgrammer
Copy link
Contributor

@DenTheProgrammer DenTheProgrammer commented Apr 11, 2023 •
edited
Loading

Background

With that change users now can clone/pull the repo, optionally update .env file and just run the script

Changes

Added run.bat, run_continuous.bat and check_requirements.py

Documentation

check_requirements.py checks for missing packages and does packages installation only if it's needed

Test Plan

Deleted some packages, run the bat file, works every time

PR Quality Checklist

  • My pull request is atomic and focuses on a single change.
  • I have thouroughly tested my changes with multiple packages configurations
  • 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

@DenTheProgrammer DenTheProgrammer changed the title Easy run bat files with requirements check Easy run with bat file (with requirements check and install if needed) Apr 11, 2023
SuperYeti
SuperYeti previously approved these changes Apr 12, 2023
View reviewed changes
nponeccop
nponeccop previously approved these changes Apr 13, 2023
View reviewed changes
@nponeccop nponeccop mentioned this pull request Apr 14, 2023
Closed
1 task
richbeales
richbeales reviewed Apr 15, 2023
View reviewed changes
run_continious.bat Outdated
@@ -0,0 +1,3 @@
@echo off
Copy link
Contributor

Choose a reason for hiding this comment

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

please spell this file correctly.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@richbeales richbeales merged commit 3e6a3c4 into Significant-Gravitas:master Apr 15, 2023
@itsmarble
Copy link
Contributor

Thank you for the contribution! This looks like a useful addition to the project. I have a question, but please bear with me as I'm not very experienced with this topic. I was wondering if these changes have been tested with a virtual environment setup, such as using pyenv. Do the check_requirements.py script and the provided .bat files work as expected when the packages are installed within a virtual environment? I apologize if this is a basic question, but I would appreciate any insights you can provide. Thanks!

@DenTheProgrammer
Copy link
Contributor Author

Thank you for the contribution! This looks like a useful addition to the project. I have a question, but please bear with me as I'm not very experienced with this topic. I was wondering if these changes have been tested with a virtual environment setup, such as using pyenv. Do the check_requirements.py script and the provided .bat files work as expected when the packages are installed within a virtual environment? I apologize if this is a basic question, but I would appreciate any insights you can provide. Thanks!

Teested it now, seems to work fine with pyenv

@0xf333 0xf333 mentioned this pull request Apr 16, 2023
Closed
5 tasks
SquareandCompass pushed a commit to SquareandCompass/Auto-GPT that referenced this pull request Oct 21, 2023
@Yard1
Do not persist entire AutoMLState in Searcher (Significant-Gravitas#870)
5f67c0a 
* Do not persist entire AutoMLState in Searcher

Signed-off-by: Antoni Baum <[email protected]>

* Fix tests

Signed-off-by: Antoni Baum <[email protected]>

Signed-off-by: Antoni Baum <[email protected]>
sindlinger pushed a commit to Orgsindlinger/Auto-GPT-WebUI that referenced this pull request Sep 25, 2024
@richbeales
Merge pull request Significant-Gravitas#870 from DenTheProgrammer/master
d9aebaf 
Easy run with bat file (with requirements check and install if needed)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nponeccop nponeccop nponeccop approved these changes

@richbeales richbeales richbeales approved these changes

@SuperYeti SuperYeti SuperYeti left review comments

@Slowly-Grokking Slowly-Grokking Slowly-Grokking left review comments

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
@DenTheProgrammer @itsmarble @SuperYeti @nponeccop @richbeales @Slowly-Grokking