-
Notifications
You must be signed in to change notification settings - Fork 5
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
Single entry point for all applications and utilities #252
Conversation
I still need to make the corresponding changes to the tier1 and tier2 testing. |
I see a few module changes but is this still using spack 1.4 modules? |
Yes we are still using spack-stack 1.4. The package changes are just for GitHub. Now everything goes through the top level script, which includes an import of everything in one way or another. Previously calling a test meant only needing to import what that test needed. We could have probably avoided that with a less monolithic top level but I figured this makes it easier for folks to look at and understand what's going on in the future. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I quite like the single entry point implementation. Indeed, it makes understanding the application easier. Looking forward to working with these changes when merged.
Co-authored-by: Colin Egerer <[email protected]>
This is great, |
Description
Instead of having many scripts we have a single entry point
swell
.Examples of new use:
swell create ufo_testing
will create the experiment directory. The oldswell_prepare_experiment_config
andswell_create_experiment
wrapped into one command.swell launch /path/to/yaml
instead ofswell_launch_experiment
swell task GetObservations ...
instead ofswell_task
.Note that now you can do
swell --help
and see all features of the entire package. Then you can do things likeswell create --help
to go down into each application to see the help. So the user has only one entry point and can figure out the entire application from there.h/t @aerorahul for this nice suggestion.
Dependencies
Impact
N/A