CICE-Consortium / CICE

Development repository for the CICE sea-ice model
Other
53 stars 128 forks source link

Standalone gadi (#2) #947

Closed anton-seaice closed 2 months ago

anton-seaice commented 2 months ago

This is the setup for doing standalone builds on the ACCESS computers. This affects build set-up only, not CICE code.

For detailed information about submitting Pull Requests (PRs) to the CICE-Consortium, please refer to: https://github.com/CICE-Consortium/About-Us/wiki/Resource-Index#information-for-developers

PR checklist

Add standalone build config for gadi supercomputer

anton-seaice commented 2 months ago

I can't see if/how I made this fail in the CI tests:

Run cd $HOME/cice
  cd $HOME/cice
  ./cice.setup -m conda -e macos --test unittest --pes 2x1 -s helloworld --testid hw01
  cd *helloworld*hw01
  ./cice.build
  ./cice.run
  shell: /bin/csh -e {0}

./cice.setup:
./cice.setup: ERROR in tsfile, this should never happen
Error: Process completed with exit code 255.
apcraig commented 2 months ago

I think github actions changed again and it's broken. UGH!!!

See also https://github.com/apcraig/CICE/actions/runs/8912669968/job/24476572429 which is just running the current CICE main (but off my fork). Will try to do some debugging.

apcraig commented 2 months ago

Please merge/rebase from the current main head. GHActions should be working again.

apcraig commented 2 months ago

Github Actions is working again. But this PR now shows the changes associated with the merge. I think we prefer not to see those. Did you rebase? Any chance you could fix that? Another option is to create a new branch off Consortium main, merge this branches changes onto the new branch, and create a new PR. Let me know if that makes sense. Maybe you have a more clever way to handle this.

anton-seaice commented 2 months ago

Thanks Tony - yes operator error. Ill fix

anton-seaice commented 2 months ago

It looks like we might have an intermittent bug in the CI:

https://github.com/ACCESS-NRI/CICE/actions/runs/8916589708 passed and https://github.com/CICE-Consortium/CICE/actions/runs/8916589961/job/24488267484 failed on the same commit !

Anyway, I made the change suggested and it passed this time (but I can't see how its related). Don't merge yet, ill have a go tomorrow at setting a more meaningful value for _ICE_MACHINEENVINFO

anton-seaice commented 2 months ago

This should be ready to merge @apcraig :)