Open amdasilva opened 2 years ago
Please make sure this feature includes the new ACG and StateSpecs and corresponding changes in HISTORY.rc (see issue #93). That is, fix that first. It is mOK to combine the PR for these 2 issues.
Although ExtData.yaml is implemented it is not the default and still needs testing.
@amdasilva @vbuchard Assuming we have a full geosgcm release from @sdrabenh that now includes this new gocart release, we should all you need to do is set
USE_EXTDATA2G: .true.
in the CAP.rc to use it. We have modified gcm_setup/gcm_run.j to handle the new ExtData. In my testing it has been zero-diff. The only exception would be if you cross a "magic date", as this is handled internally in ExtData2G rather than via the ugly scripting.
Please, please test this, the sooner this becomes the default the better from a code maintenance/development perspective.
Background
@bena-nasa has refactored ExtData to use a more versatile YAML resource file. This is a mod that goes beyond GOCART (see below for a list of *ExtFiles.rc that need conversion).
Here are the general steps:
feature/#132/ExtData/yaml
feature/#132/ExtData/yaml/v1.0.0
List of ExtData.rc needing conversion to YAML