Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
This repository was archived by the owner on Apr 30, 2026. It is now read-only.

Feature/add driver checkpoint#391

Open
elynnwu wants to merge 14 commits intomainai2cm/pace:mainfrom
feature/add-driver-checkpointai2cm/pace:feature/add-driver-checkpointCopy head branch name to clipboard
Open

Feature/add driver checkpoint#391
elynnwu wants to merge 14 commits intomainai2cm/pace:mainfrom
feature/add-driver-checkpointai2cm/pace:feature/add-driver-checkpointCopy head branch name to clipboard

Conversation

@elynnwu
Copy link
Copy Markdown
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 subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

blocked Blocked by another PR / feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.