Skip to content

Add time-varying greenhouse gases to RRTMG shortwave options#1598

Merged
davegill merged 4 commits intowrf-model:release-v4.3.2from
weiwangncar:clwr4sw
Dec 14, 2021
Merged

Add time-varying greenhouse gases to RRTMG shortwave options#1598
davegill merged 4 commits intowrf-model:release-v4.3.2from
weiwangncar:clwr4sw

Conversation

@weiwangncar
Copy link
Collaborator

@weiwangncar weiwangncar commented Dec 13, 2021

TYPE: enhancement

KEYWORDS: time-varying greenhouse gases, RRTMG-sw

SOURCE: internal

DESCRIPTION OF CHANGES:
Problem:
Time-varying greenhouse gases were added to the WRF code in 2013. It was added to CAM longwave and shortwave options. But it was only added to RRTMG longwave scheme.

Solution:

  1. After consulting with the original contributors of the code, the time-varying gases are added to the RRTMG-sw and RRTMG-swf code in this PR.
  2. Also included in this PR is to remove dependency of clwrf_support code for RRTMG-lwk (KIM version) - that code does not use the capability.

In a single 15 km domain test in July 2018, there are small differences between temperature and height at 24 h forecast time as shown below in the domain averaged profile plot:
diff2

LIST OF MODIFIED FILES:
M main/depend.common
M phys/module_ra_rrtmg_sw.F
M phys/module_ra_rrtmg_swf.F

TESTS CONDUCTED:

  1. Time varying gases are added to RRTMG-swf and RRTMG-sw
  2. The Jenfins tests all passing.

RELEASE NOTE: Added time-varying greenhouse gases to RRTMG shortwave options: RRTMG and RRTMG-fast.

@weiwangncar
Copy link
Collaborator Author

Test Type              | Expected  | Received |  Failed
= = = = = = = = = = = = = = = = = = = = = = = =  = = = =
Number of Tests        : 23           24
Number of Builds       : 60           59
Number of Simulations  : 159           157        0
Number of Comparisons  : 96           95        0

Failed Simulations are: 
None
Which comparisons are not bit-for-bit: 
None

@davegill davegill self-requested a review December 14, 2021 00:07
davegill
davegill previously approved these changes Dec 14, 2021
@davegill
Copy link
Contributor

@weiwangncar
Wei,
I am OK with this PR. I approved it.

For a bigger picture, what does this PR do for the v4.4 release topic of time dependent gases?

@weiwangncar
Copy link
Collaborator Author

weiwangncar commented Dec 14, 2021

@davegill Thanks. Currently using time-varying greenhouse gases (not just co2) is a compile option (add -DCLWRFGHG). This PR doesn't change that. This PR only adds the gases to the shortwave code. Ming is working on making this a runtime option for v4.4.
There may be some advantage to add this PR to 4.3.2. Since 4.3 code has been around for a while, users could choose to use this instead of the new 4.4 code. If they do, they could have this fix to use.

@dudhia
Copy link
Collaborator

dudhia commented Dec 14, 2021

Is the plot height or geopotential height? Make sure before merging.

@dudhia
Copy link
Collaborator

dudhia commented Dec 14, 2021

I deleted the comment. lwk is not lwf.

@davegill
Copy link
Contributor

@weiwangncar @dudhia

There may be some advantage to add this PR to 4.3.2. Since 4.3 code has been around for a while, users could choose to use this instead of the new 4.4 code. If they do, they could have this fix to use.

Umm, I just sent out a big note saying no more v4.3.2 mods, and our emails were pointing in the direction of this being on the develop branch. Can this stay on the develop branch? For power users, they can always check out the develop branch for this mod.

dudhia
dudhia previously approved these changes Dec 14, 2021
Copy link
Collaborator

@dudhia dudhia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it is OK for 4.3.2 or develop. Others can decide.

@davegill davegill changed the base branch from develop to release-v4.3.2 December 14, 2021 20:05
@davegill davegill dismissed stale reviews from dudhia and themself December 14, 2021 20:05

The base branch was changed.

@davegill davegill merged commit 8a928d8 into wrf-model:release-v4.3.2 Dec 14, 2021
@davegill
Copy link
Contributor

@weiwangncar @dudhia @kkeene44
Wei,
This compile-time RRTMG sw and swf option for GHG is now in the release-v4.3.2 branch. Kelly waited until we were finished changing the base, so users will now have this option available.

@weiwangncar
Copy link
Collaborator Author

weiwangncar commented Dec 14, 2021 via email

@weiwangncar
Copy link
Collaborator Author

@chenming You may want to include this in your testing and code change for making CLWRFGHG a runtime option.

@davegill
Copy link
Contributor

@weiwangncar @smileMchen @kkeene44

You may want to include this in your testing and code change for making CLWRFGHG a runtime option.

Folks,
GHG mods went to the v4.3.2 branch. Once Kelly also pushes those mods to the develop branch, THEN Ming can use the corrected LW and SW mods for CAM, RRTMG, and RRTMG(f) natively on the develop branch as a place to start. I agree with Wei to a) use those mods, and b) wait for those mods.

vlakshmanan-scala pushed a commit to scala-computing/WRF that referenced this pull request Apr 4, 2024
…el#1598)

TYPE: enhancement

KEYWORDS: time-varying greenhouse gases, RRTMG-sw

SOURCE: internal

DESCRIPTION OF CHANGES:
Problem:
Time-varying greenhouse gases were added to the WRF code in 2013. It was added to CAM longwave and shortwave options. But it was only added to RRTMG longwave scheme. 

Solution:
1. After consulting with the original contributors of the code, the time-varying gases are added to the RRTMG-sw and RRTMG-swf code in this PR. 
2. Also included in this PR is to remove dependency of clwrf_support code for RRTMG-lwk (KIM version) - that code does not use the capability. 

In a single 15 km domain test in July 2018, there are small differences between temperature and height at 24 h forecast time as shown below in the domain averaged profile plot:
![diff2](https://user-images.githubusercontent.com/12705680/145914692-c968b3d9-f807-4932-b86d-39abf0855fe8.png)


LIST OF MODIFIED FILES: 
M       main/depend.common
M       phys/module_ra_rrtmg_sw.F
M       phys/module_ra_rrtmg_swf.F

TESTS CONDUCTED: 
1. Time varying gases are added to RRTMG-swf and RRTMG-sw
2. The Jenkins tests all passing.

RELEASE NOTE: Added time-varying greenhouse gases to RRTMG shortwave options: RRTMG and RRTMG-fast.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants