Skip to content

OMI: Re-define state vector only if OMI is used - #71

Merged
jjokella merged 6 commits into
tsmp-pdaf-patched-omifrom
tsmp-pdaf-patched-omi-use-omi-model
Apr 27, 2026
Merged

OMI: Re-define state vector only if OMI is used#71
jjokella merged 6 commits into
tsmp-pdaf-patched-omifrom
tsmp-pdaf-patched-omi-use-omi-model

Conversation

@jjokella

@jjokella jjokella commented Apr 7, 2026

Copy link
Copy Markdown
Collaborator

Introduce use_omi_model (set to value of use_omi) for making the use_omi command line input information available in the model interface routines from interface/model.

use_omi_model is used to decide whether the DA statevector definition routine needs to be called at each update step according to the observation type (OMI) or not (non-OMI).

@jjokella
jjokella merged commit f71ec03 into tsmp-pdaf-patched-omi Apr 27, 2026
@jjokella
jjokella deleted the tsmp-pdaf-patched-omi-use-omi-model branch April 27, 2026 10:15
jjokella added a commit that referenced this pull request Apr 27, 2026
PDAF-OMI development by Yorck Ewerdwalbesloh and Anne Springer from Uni Bonn.

### User
Command line switch
- `use_omi`

### Developer
- additional files in the `interface/framework/Makefile`: Modules for each observation type and callback routines.
- separating backward compatible code from OMI code using the `use_omi` switch
- contains GRACE-DA functionality in more modular way than branch `tsmp-pdaf-patched-grace-da`:
  - #62 

### Contains PRs

* Author additions and bugfix in mod_read_obs (#35)

* GRACE-DA and OMI documentation (#69)

* OMI: EnKF input option for invoking DA increment output (#70)

* OMI: Re-define state vector only if OMI is used (#71)

* OMI: Observation input `dr`: Array of Length 2 in observation file (#74)

---------

Co-authored-by: Yorck Ewerdwalbesloh <s7yoewer@uni-bonn.de>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.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.

1 participant