NOAA-GFDL / pace

Re-write of FV3GFS weather/climate model in Python
Apache License 2.0
12 stars 11 forks source link

Update Example Build Script for c5 #12

Closed oelbert closed 1 year ago

oelbert commented 1 year ago

Purpose

This adds a build.sh file for Gaea's c5 partition and renames the previous build_gaea.sh to clarify that it's for the c4 partition

Code changes:

FlorianDeconinck commented 1 year ago

LGTM

oelbert commented 1 year ago

I'm concerned that any compiled packages requiring MPI may not be compiled correctly without setting MPICC to the HPE/Cray cc wrapper. For sanity sake may also want to set CC to the same cc wrapper to ensure proper library handling.

I just used a fresh install to run the parallel tests and baroclinic instability case with both the numpy and gt:cpu_kfirst backends and they all pass, so I think we're actually good to go on this.

bensonr commented 1 year ago

@oelbert - thanks for checking, I'll review and merge.