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

Stormcast Customization #799

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

Conversation

jleinonen
Copy link
Collaborator

Modulus Pull Request

Description

Modifies the StormCast code to be trainable with custom datasets, as well as adding a few additional features.

Overview of changes:

  • Combine .to commands in trainer (device and dtype in one call)
  • Save regression and diffusion checkpoints in different directories in rundir to avoid diffusion training trying to load regression checkpoint and vice versa
  • List included HRRR channels in dataset config instead of excluded ones
  • Standardize HRRR dataloader __getitem__ outputs to "background" and "state"
  • Remove retrieval of unused ERA5 time step from HRRR dataloader
  • Refactor creation of model inputs and outputs into build_network_condition_and_target, which is used for training, validation and inference
  • Support for invariant_array being None, indicating no invariants
  • Rename variables in inference.py, move plotting and IO to functions
  • Add support for multiple rounds of gradient accumulation to enable large batch sizes
  • Add support for training with AMP using float16 or bfloat16
  • Align default training parameters with StormCast paper

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.
  • The CHANGELOG.md is up to date with these changes.
  • An issue is linked to this pull request.

Dependencies

@jleinonen jleinonen self-assigned this Feb 26, 2025
@jleinonen
Copy link
Collaborator Author

/blossom-ci

@jleinonen
Copy link
Collaborator Author

/blossom-ci

Copy link
Collaborator

@pzharrington pzharrington left a comment

Choose a reason for hiding this comment

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

LGTM!

@jleinonen
Copy link
Collaborator Author

/blossom-ci

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