-
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
Locating pre-release binaries is difficult #70
Comments
Comments welcome @CodeGat @dougiesquire |
Linking #66 |
Por qué no los dos? |
I think the environment modules would be the best way to go for most people? I'm also considering the fact that the "This module will be deployed with the following versions..." comment is quite large already. |
The This prerelease is accessible using Or does it require a little more stuff? |
I'm happy with only providing info on the modules. Once Payu can find the executable from the environment module, that will definitely be the preferred option. |
Is the only/correct way to find where everything was installed to look in the |
You can also do |
I like this |
Does "where the binaries shall be on your $PATH" make enough sense? |
I think so, yes |
Dunno what the priority would be, though |
But can you? Isn't this the weird error I encountered with I also like this It is a nice distillation of the massive tsunami of information I put at the top. |
Because conciseness. I thought it would be too wordy to include both but Tommy has managed that nicely. |
Alright PR enjoyers, I have a linked PR that will do just that :) Let me know if you want me to go through it |
@aidanheerdegen I'd just tested it on |
We're putting a lot of useful information in comments on PRs, but not the paths to the actual executables, or how to access the installed model using
spack
ormodule load
:One use case of the pre-release functionality is to trial updates to a model, which requires utilising the pre-release builds in configurations.
There is work being done to make it easier to switch between model versions using
module load
, but it is still worthwhile providing information on the different ways the pre-release install can be accessed. e.g.To access this pre-release:
spack
:The
spack.location
file shows all the paths where packages are installed, e.g.The mom5 binaries are in the
bin
sub-directoryspack
also adds this directory to thePATH
when an environment is activated, so usingwhich
also works:Once the the prerelease module is loaded the binaries should be in
$PATH
:Or they can be accessed directly (soon) with
payu
by having this inconfig.yaml
So the question is, what additional information should we provide in the PR comments? Arguably the last module example is the most concise, straightforward to explain and easiest to implement.
The text was updated successfully, but these errors were encountered: