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

Add CYGNSS soil moisture observation reader #76

Merged
merged 25 commits into from
Jan 31, 2025

Conversation

amfox37
Copy link
Contributor

@amfox37 amfox37 commented Jan 17, 2025

This PR aims to add ability to read CYGNSS L3 soil moisture daily and subdaily soil moisture v3.2 observations from daily files available from: https://podaac.jpl.nasa.gov/dataset/CYGNSS_L3_SOIL_MOISTURE_V3.2
Subdaily observations represent 6 hour long time slices and are assimilated at 3z, 9z, 15z, 21z. Daily observations are assimilated at 12z. NB It is not anticipated both subdaily and daily observations will be jointly assimilated, although the code alllows this.
There is no capability to assimilate two subdaily time slices simultaneously, so assimilation windows longer than 6hrs will error out.
Individual observation "sigma" values are currently read, but not used. errstd from nml is used.
A fixed mask is read and applied with obs removed when any flags are set.
This mask is on the same 36km grid as the 36km resolution observations. If the observations are not on this grid it will error out.
For now, employs scale_obs_sfmc_zscore(), which uses scaling parameters on lat/lon grid in nc4 file (as in ASCAT_MET* species).

@amfox37 amfox37 added enhancement New feature or request 0-diff labels Jan 17, 2025
Copy link
Collaborator

@gmao-rreichle gmao-rreichle left a comment

Choose a reason for hiding this comment

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

@amfox37, I added a few comments and questions. I might have misunderstood bits and pieces. I currently don't have access to Discover or GMAO Linux servers, so working entirely with the GitHub.com interface, which is somewhat limiting.

GEOSlandassim_GridComp/clsm_ensupd_read_obs.F90 Outdated Show resolved Hide resolved
GEOSlandassim_GridComp/clsm_ensupd_read_obs.F90 Outdated Show resolved Hide resolved
GEOSlandassim_GridComp/clsm_ensupd_read_obs.F90 Outdated Show resolved Hide resolved
GEOSlandassim_GridComp/clsm_ensupd_read_obs.F90 Outdated Show resolved Hide resolved
GEOSlandassim_GridComp/clsm_ensupd_read_obs.F90 Outdated Show resolved Hide resolved
GEOSlandassim_GridComp/clsm_ensupd_read_obs.F90 Outdated Show resolved Hide resolved
GEOSlandassim_GridComp/clsm_ensupd_read_obs.F90 Outdated Show resolved Hide resolved
GEOSlandassim_GridComp/clsm_ensupd_read_obs.F90 Outdated Show resolved Hide resolved
@gmao-rreichle
Copy link
Collaborator

After e274718

  • CYGNSS reader functionality successfully tested by @amfox37
  • Passed LDAS nightly 0-diff tests (@gmao-rreichle)

@gmao-rreichle gmao-rreichle marked this pull request as ready for review January 31, 2025 17:01
@gmao-rreichle gmao-rreichle requested a review from a team as a code owner January 31, 2025 17:01
@gmao-rreichle gmao-rreichle merged commit f48d06e into develop Jan 31, 2025
11 checks passed
@gmao-rreichle gmao-rreichle deleted the feature/amfox/cygnss_obs_reader branch January 31, 2025 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0-diff enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants