Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Variance Partition estimates of Stddev #1037

Merged
merged 24 commits into from
Apr 17, 2024
Merged

Conversation

andytangborn
Copy link
Collaborator

This PR creates code and a yaml (adapted from SOCA) that computes time dependent background error standard deviations using neighboring grid points. These are created "on the fly", eliminating a need for staticB stddev files that need to be switched in or out each season.

utils/chem/chem_diagb.h Outdated Show resolved Hide resolved
utils/chem/chem_diagb.cc Outdated Show resolved Hide resolved
utils/chem/chem_diagb.h Outdated Show resolved Hide resolved
utils/chem/chem_diagb.h Outdated Show resolved Hide resolved
utils/chem/chem_diagb.h Outdated Show resolved Hide resolved
utils/chem/chem_diagb.h Outdated Show resolved Hide resolved
utils/chem/chem_diagb.h Outdated Show resolved Hide resolved
utils/chem/chem_diagb.h Outdated Show resolved Hide resolved
utils/chem/chem_diagb.h Outdated Show resolved Hide resolved
utils/chem/chem_diagb.h Outdated Show resolved Hide resolved
parm/aero/berror/aero_diagb.yaml Outdated Show resolved Hide resolved
utils/chem/chem_diagb.h Show resolved Hide resolved
Copy link
Contributor

@CoryMartin-NOAA CoryMartin-NOAA left a comment

Choose a reason for hiding this comment

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

One more thing then I think it's ready to be approved

Copy link
Contributor

@guillaumevernieres guillaumevernieres left a comment

Choose a reason for hiding this comment

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

Seems good to me. See comments.

parm/aero/berror/aero_diagb.yaml.j2 Outdated Show resolved Hide resolved
utils/chem/chem_diagb.h Outdated Show resolved Hide resolved
utils/chem/chem_diagb.h Show resolved Hide resolved
@andytangborn
Copy link
Collaborator Author

Not sure what this means.

andytangborn dismissed guillaumevernieres’s stale review via 837de5a now

@CoryMartin-NOAA
Copy link
Contributor

it means his previous review is no longer applicable because you updated the branch

@andytangborn
Copy link
Collaborator Author

I just noticed these lines in chem_diagb.h. These should be removed I think?

 // Get the std dev to add to sst
  double sstBkgErrMin(0.0);
  fullConfig.get("min sst", sstBkgErrMin);

  // Get the max std dev for ssh
  double sshMax(0.0);
  fullConfig.get("max ssh", sshMax);

@CoryMartin-NOAA
Copy link
Contributor

@andytangborn yes you don't need ssh or sst

Copy link
Contributor

@CoryMartin-NOAA CoryMartin-NOAA left a comment

Choose a reason for hiding this comment

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

Thanks @andytangborn for this, I'll approve and merge, and we can fix any minor issues later when we incorporate it into the workflow. Great work!

@CoryMartin-NOAA CoryMartin-NOAA merged commit d25dbe6 into develop Apr 17, 2024
4 of 5 checks passed
@CoryMartin-NOAA CoryMartin-NOAA deleted the feature/chem_diagb branch April 17, 2024 20:48
danholdaway added a commit that referenced this pull request Apr 23, 2024
* upstream/develop:
  More marine DA bufixes  (#1069)
  Add basin flags to the ocean obs (#1067)
  Revert "Prepare observations for snow DA updates to the ensemble members" (#1066)
  Fix aero_diagb template (#1064)
  Refactoring prepocnobs task (#1049)
  Ensemble diagnostics not triggered properly (#1060)
  New ocean/ice B diag (#1050)
  Remove R2D2 and SOLO from the GitHub Actions workflow CI (#1058)
  Variance Partition estimates of Stddev for aerosols (#1037)
  Removing ctests of an obsolete bufr_converter in SOCA (#1056)
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.

None yet

3 participants