Skip to content

Improve DA RANDOMCV capability#967

Merged
jamiebresch merged 1 commit intowrf-model:developfrom
jamiebresch:randomcv
Aug 23, 2019
Merged

Improve DA RANDOMCV capability#967
jamiebresch merged 1 commit intowrf-model:developfrom
jamiebresch:randomcv

Conversation

@jamiebresch
Copy link
Contributor

@jamiebresch jamiebresch commented Jul 30, 2019

TYPE: enhancement

KEYWORDS: WRFDA, RANDOMCV

SOURCE: Jamie Bresch (NCAR/MMM), Tom Auligne (previously NCAR, initiated the change in 2010)

DESCRIPTION OF CHANGES:

Simplify and cleanup the analysis_type="RANDOMCV" capability.
(1) Remove unnecessary and redundant code for anal_type_randomcv.
Non-relevant files and log messages are not written out.
(2) Add a new namelist n_randomcv (default=1) to allow the generation of multiple perturbed output in one run.
For example, wrfvar_output_randomcv.e001, wrfvar_output_randomcv.e002, etc.

The changes have no impact on non-RANDOMCV applications.

LIST OF MODIFIED FILES:
M Registry/registry.var
M var/README.namelist
M var/da/da_define_structures/da_initialize_cv.inc
M var/da/da_main/da_solve.inc
M var/da/da_main/da_wrfvar_top.f90
M var/da/da_tools/da_set_randomcv.inc

TESTS CONDUCTED:
With default n_randomcv=1, the new code generates identical wrfvar_output as the origonal code.

RELEASE NOTE: WRFDA analysis_type="RANDOMCV" is improved. To get multiple perturbed output in one WRFDA run, set new namelist n_randomcv to the desired number.

modified:   Registry/registry.var
modified:   var/README.namelist
modified:   var/da/da_define_structures/da_initialize_cv.inc
modified:   var/da/da_main/da_solve.inc
modified:   var/da/da_main/da_wrfvar_top.f90
modified:   var/da/da_tools/da_set_randomcv.inc
@liujake
Copy link
Contributor

liujake commented Aug 23, 2019

@jamiebresch this PR was approved. Can you merge it?

@jamiebresch jamiebresch merged commit f814821 into wrf-model:develop Aug 23, 2019
@jamiebresch jamiebresch deleted the randomcv branch August 23, 2019 19:46
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.

2 participants