Skip to content

Interfaces for restarts - #12

Merged
peverwhee merged 8 commits into
ESMCI:developmentfrom
peverwhee:interfaces-for-restarts
Oct 5, 2026
Merged

peverwhee merged 8 commits into
ESMCI:developmentfrom
peverwhee:interfaces-for-restarts

Conversation

@peverwhee

Copy link
Copy Markdown
Collaborator

Adds new interfaces to hist_api and hist_buffer to grab the sample count and standard deviation buffer for a given field.

Used for restart files.

@peverwhee
peverwhee requested a review from nusbaume June 23, 2026 21:25
@peverwhee peverwhee self-assigned this Jun 23, 2026

@nusbaume nusbaume left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for adding the new API routines @peverwhee! I have some requests and questions, mostly related to the intents of various subroutine and function arguments, but hopefully nothing that will be too problematic (and if there is something just let me know). Thanks again!

Comment thread src/hist_api.F90 Outdated
Comment thread src/hist_api.F90 Outdated
Comment thread src/hist_api.F90 Outdated
Comment thread src/hist_api.F90 Outdated
Comment thread src/hist_buffer.F90 Outdated
Comment thread src/hist_buffer.F90 Outdated
Comment thread src/hist_buffer.F90 Outdated
@peverwhee
peverwhee requested a review from nusbaume August 24, 2026 17:45

@nusbaume nusbaume left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for responding to my requests @peverwhee! I just had one last (optional) request, but otherwise everything looks great to me now.

Comment thread src/hist_buffer.F90 Outdated
@peverwhee
peverwhee merged commit 228b127 into ESMCI:development Oct 5, 2026
1 check passed
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.

2 participants