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.

Sign up for GitHub

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

Jump to bottom

Install of nco required for ice post-processing #23

Closed
aidanheerdegen opened this issue Feb 29, 2024 · 4 comments · Fixed by #27
Closed

Install of nco required for ice post-processing #23

aidanheerdegen opened this issue Feb 29, 2024 · 4 comments · Fixed by #27
Labels
add dependency Add another dependency to the conda environment

Comments

@aidanheerdegen
Copy link
Member

aidanheerdegen commented Feb 29, 2024 •
edited
Loading

Ice post-processing (concatenation) of daily ice data files requires installation of cdo nco

ACCESS-NRI/access-om2-configs#16 (comment)
@aidanheerdegen aidanheerdegen added the add dependency Add another dependency to the conda environment label Feb 29, 2024
@aidanheerdegen aidanheerdegen mentioned this issue Mar 12, 2024
Closed
@aidanheerdegen aidanheerdegen changed the title Install of cdo required for ice post-processing Install of eco required for ice post-processing Mar 12, 2024
@aidanheerdegen aidanheerdegen changed the title Install of eco required for ice post-processing Install of nco required for ice post-processing Mar 12, 2024
@anton-seaice
Copy link

anton-seaice commented Mar 22, 2024 •
edited
Loading

Where is micromamba installed (on gadi) ?

@anton-seaice anton-seaice mentioned this issue Mar 22, 2024
Closed
@aidanheerdegen
Copy link
Member Author

Where is micromamba installed (on gadi) ?

In this case I don't think it is. The workflow creates the environment on the GitHub runner and uses conda pack to create a self-contained environment that is unpacked on gadi

https://github.com/ACCESS-NRI/payu-condaenv/blob/main/.github/workflows/deploy.yml#L31-L50

But @CodeGat would know more.

@anton-seaice
Copy link

I am actually not sure we need to make this change? We can just module load nco and it is provided by the system ?

@aidanheerdegen
Copy link
Member Author

I am actually not sure we need to make this change? We can just module load nco and it is provided by the system ?

Yes we could. We have to install other supporting software so to my mind it seems reasonable to include nco as well.

Avoiding module load also makes the script less NCI specific in my mind. Potentially also testable outside an NCI environment .

@aidanheerdegen aidanheerdegen mentioned this issue Apr 2, 2024
Merged
@jo-basevi jo-basevi mentioned this issue Apr 12, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
add dependency Add another dependency to the conda environment
Projects
None yet
Milestone
No milestone
2 participants
@aidanheerdegen @anton-seaice