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

Create a virtual env setup guide for development of Ozone in the CONTRIBUTING file #54

Closed
Milind220 opened this issue Mar 10, 2022 · 4 comments · Fixed by #77
Closed
Assignees
Labels
beginner Slightly more complex than 'good first issue', but not difficult either documentation Improvements or additions to documentation

Comments

@Milind220
Copy link
Collaborator

Using this exact environment is not compulsory, but does help to avoid issues with development of the package's features.

  1. Instruct contributors to setup a virtual env with either conda environments or python virtual environments
  2. As of now, Python 3.9 should be used ideally (it's what I've used for the majority of the package, and python 3.10 is not yet supported by conda environments)
  3. the dependencies should be installed from requirements.txt with pip install -r requirements.txt
@lahdjirayhan
Copy link
Contributor

Hold my beer, I got this

(I'm testing out the github workflows)

@Milind220
Copy link
Collaborator Author

Awesome! Works great :)

@Milind220 Milind220 linked a pull request Mar 20, 2022 that will close this issue
@Milind220
Copy link
Collaborator Author

@lahdjirayhan Thanks for your contributions! This was a particularly important issue for us to fix!

@lahdjirayhan
Copy link
Contributor

Thanks for merging my work as well 😃 @Milind220

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beginner Slightly more complex than 'good first issue', but not difficult either documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants