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 back rpointer timestamps #316

Merged
merged 1 commit into from
Dec 6, 2024

Conversation

jedwards4b
Copy link
Contributor

Description of changes

add rpointers to timestamps.

Specific notes

Contributors other than yourself, if any:

CDEPS Issues Fixed (include github issue #):

Are there dependencies on other component PRs (if so list):

Are changes expected to change answers (bfb, different to roundoff, more substantial):

Any User Interface Changes (namelist or namelist defaults changes):

Testing performed (e.g. aux_cdeps, CESM prealpha, etc):

Hashes used for testing:

@jedwards4b jedwards4b self-assigned this Dec 6, 2024
@jedwards4b jedwards4b merged commit a4e9d13 into ESCOMP:main Dec 6, 2024
1 check passed
@jedwards4b jedwards4b deleted the reinsert_rpointer_tstamp branch December 6, 2024 17:12
@@ -1068,8 +1068,7 @@ subroutine dshr_restart_write(rpfile, case_name, model_name, inst_suffix, ymd, t

call shr_cal_datetod2string(date_str, ymd, tod)
write(rest_file_model ,"(7a)") trim(case_name),'.', trim(model_name),trim(inst_suffix),'.r.', trim(date_str),'.nc'
! temporarily turn off timestamp, remove this code and comment in alpha05c
lrpfile = rpfile(:len_trim(rpfile)-17)

! write restart info to rpointer file
if (my_task == main_task) then
open(newunit=nu, file=trim(lrpfile), form='formatted')

Choose a reason for hiding this comment

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

Hi @jedwards4b

Sorry - me again, confused again.

I think at line 1074, you meant to change lrpfile back to rpfile ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for catching this, I will correct.

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

Successfully merging this pull request may close these issues.

3 participants