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 a stream for ZBOT for GSWP3 forcing data #162

Open
ekluzek opened this issue May 18, 2022 · 3 comments
Open

Add a stream for ZBOT for GSWP3 forcing data #162

ekluzek opened this issue May 18, 2022 · 3 comments
Labels
CESM Only enhancement New feature or request

Comments

@ekluzek
Copy link
Collaborator

ekluzek commented May 18, 2022

Rather than having ZBOT hardcoded in datm we want to have it read from a file. The file will only have one latitude/longitude and time-sample. Hence, most of this will just be adding this to the streams mechanism for GSWP3.

Note, that the current code doesn't read in ZBOT, like it should, it only overrides it if it isn't read (which it always isn't) and then datm_datamode_clmncep_mod.F90 has this line...

       !--- bottom layer height ---
       if (.not. associated(strm_z)) Sa_z(n) = 30.0_r8

It needs the following to be added before that...

    call shr_strdata_get_stream_pointer( sdat, 'Sa_z'        , strm_z  , rc)
    if (ChkErr(rc,__LINE__,u_FILE_u)) return

And then a fixed stream file added to the stream_definition.xml file for CLMGSWP3v1.Z, to handle Sa_z. It'll be a little simpler than the other blocks since it won't need to handle differing years.

@ekluzek ekluzek added enhancement New feature or request CESM Only labels May 18, 2022
@ekluzek
Copy link
Collaborator Author

ekluzek commented May 18, 2022

This corresponds to the CTSM issue: ESCOMP/CTSM#1760

@wwieder
Copy link

wwieder commented May 19, 2022

Just a note that this bug influences NEON or other tower simulations, where zbot is not being read in off input data. Just pinging other's who may run single point tower sites with nuopc. (it's not an issue with mct runs) @negin513 @danicalombardozzi @adrifoster @swensosc @olyson

@swensosc
Copy link

swensosc commented May 19, 2022 via email

@samsrabin samsrabin moved this to Ready to start (or start again) in CTSM: Upcoming tags Aug 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CESM Only enhancement New feature or request
Projects
Status: Ready to start (or start again)
Development

No branches or pull requests

3 participants