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.

Sign up for GitHub

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

Jump to bottom

Barebones checkpointing #324

Closed
ali-ramadhan opened this issue Aug 1, 2019 · 0 comments · Fixed by #326
Closed

Barebones checkpointing #324

ali-ramadhan opened this issue Aug 1, 2019 · 0 comments · Fixed by #326
Assignees
@ali-ramadhan
Labels
feature 🌟 Something new and shiny testing 🧪 Tests get priority in case of emergency evacuation
Milestone
v1.0

Comments

@ali-ramadhan
Copy link
Member

We currently have no checkpointing anymore. We need a barebones checkpointer that only checkpoints and saves the essentials. Should be more transparent than serializing the Model to disk, which also comes with multiple issues.

Should also be useful for long runs (e.g. seasonal cycle simulations) and more rigorous regression tests that actually test the dynamics.
@ali-ramadhan ali-ramadhan added feature 🌟 Something new and shiny testing 🧪 Tests get priority in case of emergency evacuation labels Aug 1, 2019
@ali-ramadhan ali-ramadhan added this to the v1.0 milestone Aug 1, 2019
@ali-ramadhan ali-ramadhan self-assigned this Aug 1, 2019
@ali-ramadhan ali-ramadhan mentioned this issue Aug 2, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ali-ramadhan ali-ramadhan

Labels
feature 🌟 Something new and shiny testing 🧪 Tests get priority in case of emergency evacuation
Projects
None yet
Milestone
v1.0
Development

Successfully merging a pull request may close this issue.

Checkpointer revival CliMA/Oceananigans.jl
1 participant
@ali-ramadhan