-
Notifications
You must be signed in to change notification settings - Fork 0
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
Initial Development MR #1
Conversation
Add dependencies Update dependencies Add pyaml as dependency Add loguru and rich Update .gitignore Add default configurations Add template files Commit semi-finished `new` command(s) Commit semi-finished `install` command(s) Remove dependency on poetry Classes for 'providers' WIP implementation of install Restructure files Rename methods, use with env .venv by default Add install poetry step Flesh out class Multiple changes - implement logging - code cleanup - skip steps if not needed - improve spack python package parsing - add docstrings - fix spack multiprocessing issues Add global verbosity levels Update to new loggers squash with global loggers Formatting changes Move installer class, rename pkg to pyvarium Rename to pyvarium Rename package directory Rename variables Delete templated tests until I come up with good ones Tidy up code Update dependencies Fix error Restructure installers Add tests Add docstrings, clarify var/func names Add fastroce dep Cast paths to Path Use delegates for kwargs, use same var names Update deps for tests, add more coverage Add test image build workflow Add test image build workflow Implement test workflow using test image Update deps Add runs on Set up CI tests Explicitly use bash shell for run Split up step Prepare for integration tests Update test image Spit unit and integration tests Add compiler find call Fix typos Explicitly install gcc Print off compilers list Don't clear env vars by default Try unified codecov upload Adjust artifact paths Try codecov files with quotes Add poetry test Update lockfile Set environ to none, set executable to bash Add version flag, adjust temp paths md instead of rst readme Update deps, add pytest opts Restructure deps Tidy up Update to work with new installers Split unit and integration tests Add nox file Use tomlkit and poetry pyproject classes Add pyvarium tests Add test image build workflow Update lockfile Rename integration test venv for act compat Make cmd_mp work like cmd Add module generation printer Tidy up, finish modulegen
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
ee1605a
to
0ac6d94
Compare
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Signed-off-by: Robert Rosca <[email protected]>
This comment was marked as outdated.
This comment was marked as outdated.
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #1 +/- ##
=======================================
Coverage ? 93.02%
=======================================
Files ? 13
Lines ? 416
Branches ? 0
=======================================
Hits ? 387
Misses ? 29
Partials ? 0 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
This pull request introduces 4 alerts when merging 30b66b9 into 317fe93 - view on LGTM.com new alerts:
Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog. |
30b66b9
to
cb4599b
Compare
cb4599b
to
9f7a64e
Compare
This pull request introduces 4 alerts when merging 9f7a64e into 317fe93 - view on LGTM.com new alerts:
Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog. |
This pull request introduces 4 alerts when merging f95a178 into 317fe93 - view on LGTM.com new alerts:
Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog. |
This pull request introduces 4 alerts when merging 309a912 into 317fe93 - view on LGTM.com new alerts:
Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog. |
This pull request introduces 4 alerts when merging 4ca8c91 into 317fe93 - view on LGTM.com new alerts:
Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog. |
This pull request introduces 4 alerts when merging e870889 into 317fe93 - view on LGTM.com new alerts:
Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog. |
e870889
to
01c079a
Compare
This pull request introduces 4 alerts when merging 01c079a into 317fe93 - view on LGTM.com new alerts:
Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog. |
This pull request introduces 4 alerts when merging 5a47570 into 317fe93 - view on LGTM.com new alerts:
Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog. |
This pull request introduces 4 alerts when merging fe07ba4 into 317fe93 - view on LGTM.com new alerts:
Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog. |
The readme has a quick start section describing usage. The fastest and easiest way to play around with this is in a docker container, specifically the one used for testing as this has python, pip, and a few other packages pre-compiled and installed inside it, saving a lot of time during the tests.
To get started within the container:
Then you can just
cd /tmp
,pyvarium new env
, and follow the instructions in the readme to see if everything works as expected. To make testing easier numpy is already present, so once you've set up a pyvarium environmentpyvarium add spack py-numpy
should finish pretty quickly, then runpyvarium add pipenv extra-data
(or anything else) to add in packages from pip.