[ ] All sub component pull requests have been reviewed by their code managers.
[X] Run the full Intel+GNU RT suite (compared to current baselines) on either Hera/Derecho/Hercules
[X] Commit 'test_changes.list' from previous step
Description:
This work is synonymous with https://github.com/ufs-community/ufs-weather-model/pull/2158. See https://github.com/NOAA-EMC/fv3atm/pull/881 for most of the work, which is primarily cleanup of unused variables and metadata. The original PR into production/RRFS.v1 also had many changes related to always allocating conditionally-allocated variables in fv3atm (at least to zero size) due to NCO's requirement that code pass the -check all debug flag, but those changes are no longer required due to other changes in fv3atm and ccpp-framework. This work has been tested with the -check all flag added to the cmake debug mode as in https://github.com/ufs-community/ufs-weather-model/pull/2158/files#diff-077be989f84571193e53c052e8bfecd6f073df5ce504350826c657d023c10bfb, although those changes ARE NOT included in this PR since it is beyond the scope of my work and I'm unsure whether all components will work with it (or even inherit all compilation flags from ufs-weather-model).
Commit Message:
* UFSWM - Second reconciliation PR from production/RRFS.v1 related to FV3 unused and optional variables
* FV3 - Second reconciliation PR from production/RRFS.v1 related to FV3 unused and optional variables
* ccpp-physics - Second reconciliation PR from production/RRFS.v1 related to FV3 unused and optional variables
Commit Queue Requirements:
[X] Commit 'test_changes.list' from previous step
Description:
This work is synonymous with https://github.com/ufs-community/ufs-weather-model/pull/2158. See https://github.com/NOAA-EMC/fv3atm/pull/881 for most of the work, which is primarily cleanup of unused variables and metadata. The original PR into production/RRFS.v1 also had many changes related to always allocating conditionally-allocated variables in fv3atm (at least to zero size) due to NCO's requirement that code pass the
-check all
debug flag, but those changes are no longer required due to other changes in fv3atm and ccpp-framework. This work has been tested with the-check all
flag added to the cmake debug mode as in https://github.com/ufs-community/ufs-weather-model/pull/2158/files#diff-077be989f84571193e53c052e8bfecd6f073df5ce504350826c657d023c10bfb, although those changes ARE NOT included in this PR since it is beyond the scope of my work and I'm unsure whether all components will work with it (or even inherit all compilation flags from ufs-weather-model).Commit Message:
Priority:
Git Tracking
UFSWM:
Sub component Pull Requests:
UFSWM Blocking Dependencies:
Changes
Regression Test Changes (Please commit test_changes.list):
Input data Changes:
Library Changes/Upgrades:
Testing Log: