Improve support for gridSpec in encoder - #257
Open
tweska wants to merge 14 commits into
Open
Conversation
…epping for SH with gridSpec
tweska
force-pushed
the
feature/gridSpec-in-grib2mars
branch
from
September 1, 2026 16:02
5fe98c5 to
019556c
Compare
tweska
force-pushed
the
feature/gridSpec-in-grib2mars
branch
from
September 1, 2026 16:15
ca182a0 to
bfa66fe
Compare
tweska
marked this pull request as ready for review
September 2, 2026 06:24
tweska
force-pushed
the
feature/gridSpec-in-grib2mars
branch
from
September 3, 2026 07:21
bfa66fe to
e0b1d3d
Compare
tweska
force-pushed
the
feature/gridSpec-in-grib2mars
branch
from
September 3, 2026 07:37
e0b1d3d to
778852a
Compare
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## 50r2-dev #257 +/- ##
===========================================
Coverage ? 44.47%
===========================================
Files ? 465
Lines ? 21064
Branches ? 1521
===========================================
Hits ? 9369
Misses ? 11695
Partials ? 0 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
All features in support of mode when
skipSection3is enabled + fixes for a few issuesgridgridSpecfrom the sample to set MARSgridin grib2marstruncationis no longer set!ECCODES_ECKIT_GEO=1in the environment ifskipSection3is enabled in any of grib2mars, mars2mars or mars2gribskipSection3ingrib-to-mtg2toolCI fails because it relies on the
truncationmethod being available on theGridclass ineckit::geo. This is not in develop yet, it seems.The following packages have branches that work with this branch/PR in metkit:
feature/eckit-geofix/metkit-encoder-testFix metkit unit tests mir#55fix/mars2mars-api-callRemove misc overwrite logic multio#295This PR contains the fix from #259
📋 Metkit Documentation 📋
https://sites.ecmwf.int/docs/metkit/pull-requests/PR-257