search

GEOS-ESM / GEOSgcm_App

Repo containing the scripts for running the GEOS Earth System Model GCM
Apache License 2.0
2 stars 1 forks source link

Use nccmp, add readers/writers test to regress #573

Closed mathomp4 closed 8 months ago

mathomp4 commented 8 months ago

Closes #571 Closes #572

This PR moves gcm_regress.j to use nccmp instead of cmp for comparison of restarts (non-MOM).

It also changes NUM_READERS and NUM_WRITERS to 6 for the 1x6 layout test as a test of that.

NOTE: We move to use nccmp as it was found that there seem to be differences in the HDF5 Superblock when written with multiple writers. The data and metadata is all the same, but something changes and then, boom, different files technically such that cmp will not work.

mathomp4 commented 8 months ago

Welp, it seems to work for me. Undrafting.