Skip to content

Change localization parameters#707

Merged
Dooruk merged 2 commits intodevelopfrom
feature/ygyu/change_param_localization
Mar 11, 2026
Merged

Change localization parameters#707
Dooruk merged 2 commits intodevelopfrom
feature/ygyu/change_param_localization

Conversation

@metdyn
Copy link
Contributor

@metdyn metdyn commented Feb 13, 2026

Description

A better estimate for localization parameters in observations/localization directory:

  • lengthscale: 1500.e3
  • max nobs: 10000

Impact

Have an effect on Local Ensemble DA (still being tested)

->  lengthscale: 1500.e3
->  max nobs: 10000
@metdyn metdyn requested a review from rtodling February 13, 2026 22:00
@metdyn
Copy link
Contributor Author

metdyn commented Feb 19, 2026

The resulting increment graphics is not good from this set of parameters.

@metdyn metdyn closed this Feb 19, 2026
@metdyn
Copy link
Contributor Author

metdyn commented Mar 11, 2026

I think we can afford:
obs localizations:
lengthscale: 1500.e3
max nobs: 10000

Because I have thin obs data by 75%.

@metdyn metdyn reopened this Mar 11, 2026
@Dooruk
Copy link
Collaborator

Dooruk commented Mar 11, 2026

@metdyn, would it be possible to create a tier 1 configuration with c12 an tier 2 with c90? You can see an example for marine here:
https://github.com/GEOS-ESM/swell/blob/develop/src/swell/suites/3dvar_marine_cycle/suite_config.py

I'm testing it with actions, then I can merge this.

Also does this close an open issue?

@Dooruk
Copy link
Collaborator

Dooruk commented Mar 11, 2026

Currently, localensembleda takes a very long time with c90 (~40 mins!) we really need to cut that down. Maybe you can create c12 states using JEDI changestate?

@metdyn
Copy link
Contributor Author

metdyn commented Mar 11, 2026

@Dooruk,
I know your concern. The code slowdown is due to qd.local_ensemble_use_linear_observer(False) in develop and C90 resolution is not the leading factor. Using linear observer has a large speed up in code, but the code may crash until we use two steps in GETKF, i.e., the two steps GETKF PR (#692) will cure both run stability and runtime speed issue. (We should neglect the code speed problem for this PR, 40min for C90. We will get to 10 min for C90 from PR 692).

@Dooruk Dooruk merged commit 9f2f4cc into develop Mar 11, 2026
14 checks passed
@Dooruk Dooruk deleted the feature/ygyu/change_param_localization branch March 11, 2026 21:04
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