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 timestamp to rpointer files, but make it backwards compatible with files without the timestamp #484

Closed
ekluzek opened this issue Oct 1, 2024 · 3 comments · Fixed by #486
Assignees
Labels
cesm-coupling For cesm coupling enhancement Improving usability, performance or other types of enhancements high priority Need immediate attention and fix

Comments

@ekluzek
Copy link
Collaborator

ekluzek commented Oct 1, 2024

CESM is going to a system wide change where the restart rpointer text files will have the simulation timestamp in the filename. However, the model will still be backwards compatible to use the older filenames without it. But, new rpointer files created by the model will have the timestamp in the filename.

PR's for CTSM and MOSART for this work are here:

ESCOMP/CTSM#2757
ESCOMP/MOSART#105

@ekluzek ekluzek added enhancement Improving usability, performance or other types of enhancements cesm-coupling For cesm coupling high priority Need immediate attention and fix labels Oct 1, 2024
@ekluzek
Copy link
Collaborator Author

ekluzek commented Oct 2, 2024

@nmizukami and I talked about this and went over what needs to change in the code.

In MOSART and CTSM the restart file was being read to initialize time -- so there were changes around setting the clock based on the clock from the driver rather than the restart file. MizuRoute is already doing that, so that part doesn't need to happen.

What does need to change is the hardcoded rnptfil in route/build/src/public_vars.f90. As well as io_rpointfil.f90 which handles the rpointer file for read and write. Although it could also be left alone and rpntfil updated before the rpointer file is handled. For the backwards compatibility part both the filename with the timestamp on it need to be allowed as well as the generic version.

@nmizukami thinks he'll be able to do this in the next few weeks to a month so that it can come into cesm3_0_beta04. Since, it's not critical for beta04 that should be fine. So we'll have @nmizukami work on this, unless he can't get to it and someone else can get to it beforehand.

@ekluzek
Copy link
Collaborator Author

ekluzek commented Oct 2, 2024

This will also need a tag for it, so it can go into cesm3_0_beta04.

@ekluzek
Copy link
Collaborator Author

ekluzek commented Oct 2, 2024

@jedwards4b notes in an offline email that it isn't absolutely required in beta04, but will be for follow on updates coming into CESM post beta04.

To be clear - the rpointer changes are backward compatible in the sense that current functionality will continue to work for all components. That is - we don't need to change mizuroute or rtm for beta04.
New functionality will require the change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cesm-coupling For cesm coupling enhancement Improving usability, performance or other types of enhancements high priority Need immediate attention and fix
Development

Successfully merging a pull request may close this issue.

2 participants