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

Feature/add driver checkpoint #391

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Conversation

elynnwu
Copy link
Collaborator

@elynnwu elynnwu commented Nov 22, 2022

Purpose

This PR adds driver-in/out checkpointer. Currently also supports non orchestrated backends and only tested using numpy.

Code changes:

  • checkpoint: added checkpoint selector to expose checkpoint type to driver
  • driver: added checkpointer_config to driver config, defaults to NullCheckpointer
  • grid and initialization: for serialbox type, add option to read the netcdf file
  • fv_dynamics: add checkpoint driver-in
  • update_atmos_state: add checkpoint driver-out
  • test_checkpoints: add test_driver driver-in/out checkpoint

Infrastructure changes:

  • driver checkpoint test added
  • added all folder for c12_6ranks_baroclinic_dycore_microphysics which combines all savepoint netcdf to one folder

Checklist

Before submitting this PR, please make sure:

  • You have followed the coding standards guidelines established at Code Review Checklist.
  • Docstrings and type hints are added to new and updated routines, as appropriate
  • All relevant documentation has been updated or added (e.g. README, CONTRIBUTING docs)
  • For each public change and fix in pace-util, HISTORY has been updated
  • Unit tests are added or updated for non-stencil code changes

@FlorianDeconinck FlorianDeconinck added the blocked Blocked by another PR / feature label Dec 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Blocked by another PR / feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants