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

Lower testing of RTM in CESM3 #44

Closed
ekluzek opened this issue Dec 21, 2023 · 2 comments · Fixed by #43
Closed

Lower testing of RTM in CESM3 #44

ekluzek opened this issue Dec 21, 2023 · 2 comments · Fixed by #43
Assignees

Comments

@ekluzek
Copy link
Contributor

ekluzek commented Dec 21, 2023

The use of RTM is now only for Paleo work in CESM3. As such we should lower the testing for it, and only test what's needed for Paleo work in CESM3. In CESM4 we hope to move to mizuRoute and hence remove RTM in CESM4.

This relates to #42

@ekluzek
Copy link
Contributor Author

ekluzek commented Jan 10, 2024

To facilitate this let's remove the flooding tests which is available in MOSART, and remove izumi as a machine to test on. So testing will just be done on Derecho for test list that is very short.

ekluzek added a commit to mvertens/rtm that referenced this issue Jan 10, 2024
@ekluzek
Copy link
Contributor Author

ekluzek commented Jan 18, 2024

I was wrong in the above statement, flooding is NOT available nor working in MOSART. See

ESCOMP/MOSART#80

As such we should keep the flooding tests in RTM so we ensure passing the fields through the CPL continues to work. We do want a working version in MOSART, but until then we should continue to test in RTM.

Note, also that CTSM runs some flooding tests in aux_clm:

ERP_P64x2_D.f10_f10_mg37.I2000Clm50SpRtmFl.derecho_intel.clm-default
ERS_D_Ld5.f10_f10_mg37.I2000Clm50BgcCropRtm.derecho_intel.rtm-rtmOnFloodOnEffvelOn

So the RTM test list should continue to test those.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant