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

GRIB 2 output #44

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

GRIB 2 output #44

wants to merge 4 commits into from

Conversation

clechartre
Copy link

@clechartre clechartre commented May 31, 2024

Purpose

This is the equivalent of DRAFT PR #42 now adapted to this repository (previously tailored to the MeteoSwsiss/neural-lam fork)
Outputting the inference results as GRIB format. Therefore, this needs the inference PR to be merged previously (loads of changes come from this).
This integrates the functionality of transforming the .npy array that is originally produced into a GRIB from a template.

Code changes:

  • ar_model: add to the function on_predict_epoch_end a function generate_time_step to generate time steps markers for the grib files and a function that translates the npy output into a grib file.
  • other modifications to the code include adaptation to tsa, imports of new libraries for this functionality, and references to templates in data_config.yaml

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)

Review

For the review process follow the guidelines at Checklist

@clechartre clechartre changed the title GRIB output GRIB 2 output May 31, 2024
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