Skip to content

dev/ufs-weather-model: Use CPP instead of Switches #515

Merged
aliabdolali merged 1 commit into
NOAA-EMC:dev/ufs-weather-modelfrom
JessicaMeixner-NOAA:ufs/cppPR
Nov 3, 2021
Merged

dev/ufs-weather-model: Use CPP instead of Switches #515
aliabdolali merged 1 commit into
NOAA-EMC:dev/ufs-weather-modelfrom
JessicaMeixner-NOAA:ufs/cppPR

Conversation

@JessicaMeixner-NOAA
Copy link
Copy Markdown
Collaborator

Use CPP ifdefs instead of switches for pre-processing and convert all code from ftn to F90 files. (ftn->src)
The tools for converting ftn->src can be found in the model/tools directory.

Pull Request Summary

This PR is to make the dev/ufs-weather-model branch up to date with develop

Description

Adds switches->CPP commit

Issue(s) addressed

Commit Message

  • Use existing commit message if possible//no merge commit

Testing

  • How were these changes tested? Tested in ufs-weather-model commit

Use CPP ifdefs instead of switches for pre-processing and convert all code from ftn to F90 files. (ftn->src) 
The tools for converting ftn->src can be found in the model/tools directory.
@aliabdolali aliabdolali merged commit 1c2a28d into NOAA-EMC:dev/ufs-weather-model Nov 3, 2021
@JessicaMeixner-NOAA JessicaMeixner-NOAA deleted the ufs/cppPR branch November 13, 2021 12:35
kestonsmith-noaa pushed a commit to kestonsmith-noaa/WW3 that referenced this pull request Jun 17, 2025
* Updates to RRTMGP in FV3/CCPP. New RT for regional configuration using RRTMGP
* Updated submodule to my forked repo.
* Changed default namelist setting for RRTMGP.
* Turned on feature to use GP LW flux-adjustment.
* Removed mistake in previous commit
* Updates to RRTMGP RTs.
* Updated RT nml.
* Updated GP regional RTs
* Reverted deprecated code in FV3/ccpp-physics.
* Removed GP c192L127 RT
* Commented out RRTMGP c192l127 RT in rt.conf.
* GP_2thread RT working.
* Updated FV3/physics
* Revert "GP_2thread RT working."
Co-authored-by: Dom Heinzeller <climbfuji@ymail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants