Skip to content

Adding accumulated GHI and fixing registry units for solar diagnostics package#1610

Merged
davegill merged 3 commits intowrf-model:developfrom
twjuliano:solar_diag_update_121721
Dec 24, 2021
Merged

Adding accumulated GHI and fixing registry units for solar diagnostics package#1610
davegill merged 3 commits intowrf-model:developfrom
twjuliano:solar_diag_update_121721

Conversation

@twjuliano
Copy link
Contributor

@twjuliano twjuliano commented Dec 20, 2021

TYPE: enhancement, bug fix

KEYWORDS: WRF-Solar, diagnostics

SOURCE: Timothy Juliano and Pedro Jimenez (NCAR/RAL)

DESCRIPTION OF CHANGES:
We have added a new 2D variable (ghi_accum) and time series variable (ts_ghi_accum) for the WRF-Solar
diagnostics package. This variable uses SWDNB as computed by the radiation parameterization to accumulate
downwelling GHI at the surface. The new variable is set to zero at the beginning of the simulation for either
restart=.true. or restart=.false..

As a separate but very minor change, we have clarified units in the registry for cloud base and cloud height
WRF-Solar variables.

LIST OF MODIFIED FILES:
M Registry/Registry.EM_COMMON
M Registry/registry.solar_fields
M dyn_em/start_em.F
M phys/module_diag_solar.F
M phys/module_diagnostics_driver.F
M phys/module_physics_init.F
M share/wrf_timeseries.F
M run/README.tslist

TESTS CONDUCTED:

  1. Conducted test on Cheyenne to ensure that GHI is summed over time and set to zero properly at initialization for
    both 2D and time series variables.
  2. All jenkins test are passing.

RELEASE NOTE: A new variable that accumulates GHI over time is added to the WRF-Solar diagnostics package. Cloud base and cloud height units (AGL) are clarified in the registry.

@twjuliano twjuliano requested review from a team as code owners December 20, 2021 17:25
@davegill
Copy link
Contributor

@weiwangncar @twjuliano
Wei,
There are "optional" and "if present" pieces in this code that are not required, since we have removed the NMM core. I am thinking that we just accept PRs with this issue, and then we go into phy_init and clean them all up ourselves. If you are OK with that plan, I am ready to approve this PR.

@weiwangncar
Copy link
Collaborator

@davegill @twjuliano I'm ok with it.

@davegill davegill self-requested a review December 23, 2021 02:24
davegill
davegill previously approved these changes Dec 23, 2021
@davegill davegill requested a review from weiwangncar December 23, 2021 02:24
@davegill davegill self-requested a review December 24, 2021 18:51
@davegill davegill merged commit 3bb5e91 into wrf-model:develop Dec 24, 2021
vlakshmanan-scala pushed a commit to scala-computing/WRF that referenced this pull request Apr 4, 2024
…s package (wrf-model#1610)

TYPE: enhancement, bug fix

KEYWORDS: WRF-Solar, diagnostics

SOURCE: Timothy Juliano and Pedro Jimenez (NCAR/RAL)

DESCRIPTION OF CHANGES:
We have added a new 2D variable (`ghi_accum`) and time series variable (`ts_ghi_accum`) for the WRF-Solar 
diagnostics package. This variable uses `SWDNB` as computed by the radiation parameterization to accumulate 
downwelling GHI at the surface. The new variable is set to zero at the beginning of the simulation for either 
`restart=.true.` or `restart=.false.`.

As a separate but very minor change, we have clarified units in the registry for cloud base and cloud height 
WRF-Solar variables.

LIST OF MODIFIED FILES:
M Registry/Registry.EM_COMMON
M Registry/registry.solar_fields
M dyn_em/start_em.F
M phys/module_diag_solar.F
M phys/module_diagnostics_driver.F
M phys/module_physics_init.F
M share/wrf_timeseries.F
M run/README.tslist

TESTS CONDUCTED: 
1. Conducted test on Cheyenne to ensure that GHI is summed over time and set to zero properly at initialization for 
both 2D and time series variables.
2. All jenkins test are passing.

RELEASE NOTE: A new variable that accumulates GHI over time is added to the WRF-Solar diagnostics package. Cloud base and cloud height units (AGL) are clarified in the registry.
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.

3 participants