Skip to content

Conversation

gmao-qliu
Copy link
Contributor

@gmao-qliu gmao-qliu commented Aug 11, 2025

We need to read S2Sv3 output for offline forecast experiments. Since S2Sv3 doesn't output 3-hourly downward shortwave and longwave radiation, we could rename the land forcing variables in post-processing. Ideally, we would extend get_GEOS() to read both S2Sv3 Forecast and Analysis outputs.

Related PRs:
GEOSldas PR#831

Successfully 0-diff tested for standard Intel builds after

@gmao-qliu
Copy link
Contributor Author

@gmao-rreichle I updated the code. Please begin reviewing the changes.

MET_TAG for S2Sv3 would be:
GEOSS2S3FCST__ens?__YYYYMMDD for forecast forcing
and
GEOSS2S3AODAS for analysis forcing. AODAS forcing data will be stored in daily files under the usual GEOS directory structures (/diag/Y../M../)

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.

@gmao-qliu : I think I have something that should be close to working as desired. Pls note the comment below. We also still need to add documentation of the new reader in the GEOSldas repo: doc/README.MetForcing_and_BCS.md, see GEOS-ESM/GEOSldas#831

PS: The revised implementation requires that the 'time_increment' (sp?) attribute is available in the nc4 file or otherwise finds its way into LDAS_GetVar().

@gmao-qliu
Copy link
Contributor Author

PS: The revised implementation requires that the 'time_increment' (sp?) attribute is available in the nc4 file or otherwise finds its way into LDAS_GetVar().

I tested this revision with "time_increment" attribute. It does work.

@gmao-rreichle gmao-rreichle marked this pull request as ready for review August 26, 2025 15:14
@gmao-rreichle gmao-rreichle requested a review from a team as a code owner August 26, 2025 15:14
@gmao-rreichle gmao-rreichle merged commit 3222d8c into develop Aug 26, 2025
11 checks passed
@gmao-rreichle gmao-rreichle deleted the feature/qliu/metforce_S2Sv3 branch August 26, 2025 15:16
@gmao-rreichle gmao-rreichle changed the title Add functionality to read S2Sv3 output Add reader for surface meteorological forcing from S2S-3 Aug 26, 2025
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