NOAA-GFDL / SHiELD_build

Other
7 stars 21 forks source link

Updating RTS tests: c5 and Yaml #34

Closed laurenchilutti closed 3 weeks ago

laurenchilutti commented 10 months ago

Description

The GAEA_RTS tests #SBATCH headers have been modified to use c5 by default. I have deleted the CI runscript for Gaea C4 as well.

FMS 2024.01 will have an option to use yaml format for data_table and field_table. To use yaml versions of these tables: compile FMS with -Duse_yaml and specify via namelist options to use yaml for these tables (field_manager_nml :: use_field_table_yaml = .T. and data_override_nml :: use_data_table_yaml = .T.)

Building FMS with -Duse_yaml was already implemented in SHiELD_build. To do this you simply execute the Build/COMPILE script with the argument "yaml".

The CI and GAEA_RTS tests have been modified in this PR to work with and without yaml tables. There is a variable use_yaml set by default to false. When false, the original field_table and data_table are used. If set to true, the data_table.yaml and field_table.yaml are used and the appropriate namelist flags are set to true.

Fixes # (issue)

How Has This Been Tested?

I have tested this with FMS 2023.03 and FMS2024.01-alpha2 (our current testing tag for the coming FMS release) I have run in debug and repro, I have compiled with GNU and Intel.

Checklist:

Please check all whether they apply or not

laurenchilutti commented 10 months ago

@uramirez8707 @abrooks1085 @bensonr could you please review this PR. It has been tested by Andrew and myself and is working as expected. I go on leave in 1 week (or less), and would like to get this in.

laurenchilutti commented 3 weeks ago

Closing this and replacing with a new PR coming with the SHiELD release of GFDL_atmos_cubed_sphere and SHiELD_physics