-
Notifications
You must be signed in to change notification settings - Fork 11
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 page: Set up Spack for building ACCESS models on Gadi #789
Conversation
|
bf7e614
to
8192ad3
Compare
Version 1.3 of my preferred text and formatting is: Set up Spack for building ACCESS modelsSpack is a build-from-source package manager, specifically designed to simplify the installation of scientific software on supercomputers. To use Spack, please familiarise yourself with the Basic Usage instructions and Environments. We also recommend that you refer to the Spack 101 Tutorial. PrerequisitesThese instructions are tailored specifically for Gadi. To use Spack on Gadi, you must have an NCI account. For instructions on how to set up an account, refer to Set Up your NCI Account. The steps for setting up Spack on Gadi are:Create a directory on the filesystem where Spack will be installed. e.g.
Clone the relevant git repositories:
Link
Test Spack (OPTIONAL)To test Spack we will create an ACCESS-OM2 environment and build the relevant packages. Then, we will uninstall all the packages and remove the environment.
|
Written like this it seems as if, to use Spack for building a model, a user needs to read those pages (basic usage and environments). However, reading those pages is a step that might be helpful to get more comfortable with Spack (and maybe do some more advanced stuff). The minimum steps for building a new version of an ACCESS model (for example a version with the source code of a component changed) should be as simple as running a few commands, and should not require the user to read extra documentation about the usage of Spack.
Creating a directory for the Spack installation is not a prerequisite (in fact is also optional, although reccommended), but rather an actual (optional) step for the setup. General comments:
|
2baa737
to
dc8fb89
Compare
Hi @atteggiani , I think it's important to appreciate that these instructions are for advanced users that plan to modify the source code and/or change dependencies of ACCESS models. They are not for users that just want to run unmodified ACCESS models.
We want the user to read the "basic usage" and "environments" sections, if they are going to use Spack to build ACCESS models.
It is a prerequisite. We want the user to keep all the Spack related files/directories in a new directory. I'll change
If you prefer to remove the enumeration, remove the numbers but please keep the prose.
Unfortunately, it's the quickest ACCESS model to build. You make a good point, perhaps we need to create a Spack environment just for testing purposes. That can be an objective of the second iteration of these instructions. |
dc8fb89
to
032a24c
Compare
Hi @harshula, thank you for your comments.
I would not consider users that plan to modify the source code and change dependencies that "advanced", in the sense that they might be researchers that don't have a lot of experience with ACCESS models and Gadi.
Spack is an amazing tool for simplifying builds and making them more robust (from a reproducibility and scientific point of view). I think we should have a Spack workflow that minimizes the knowledge a user needs to have about Spack to be able to build a specific ACCESS model (and I think it is possible despite current Spack/
In the Hive Docs we list as prerequisites all the steps that would prevent users from doing further steps listed in the guide and, for this reason, need to be taken care of before following the actual guide.
The prose is similar. If you have suggestions bout that please comment on the specific portion and I will address the comments/suggestions.
Ok, for the time being is fine. But I agree it would be best to have a Spack environment just for testing. Thank you! |
Warning message: |
Hi @atteggiani , Thanks for the chat, I've updated the text and formatting to v1.2. |
Sure! Happy to drop "configurations" It was just a draft to get an idea down that @atteggiani will finesse :) |
Thanks @heidinett ! Hi @atteggiani , I've started a Test build on Gadi to reproduce the build failure. |
Hi @atteggiani , The test build on Gadi succeeded. What was the error message you received? |
I propose "This page is tailored to experienced users and collaborators developing ACCESS models." |
Mine failed again. |
The main error seems to be the following:
|
Actual error was:
|
I've updated the text to v1.3. I added |
The build was failing because I had the |
I found out what the error is caused by. It is the I tried purging the environment, then exporting the After I unset the |
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.
Thanks for nice and compact PR @atteggiani ;)! Looks good to me.
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 only reviewed spack.md
. Approved.
…e 'Getting Started' Addressed some of Harshula's comments to the PR + further corrections.
Fixed some typos
92225db
to
f4c570c
Compare
Fixes #785.