Skip to content

mars2grib: Support ocean fields#181

Open
tweska wants to merge 3 commits intodevelopfrom
feature/mars2grib-ocean
Open

mars2grib: Support ocean fields#181
tweska wants to merge 3 commits intodevelopfrom
feature/mars2grib-ocean

Conversation

@tweska
Copy link
Member

@tweska tweska commented Mar 12, 2026

This PR adds support for ORCA grids in the representation concept and adds level mappings required to encode ocean fields.

This PR is meant to be used with these changes in MultIO: ecmwf/multio#247

@tweska tweska changed the title mars2grib: support ocean fields mars2grib: Support ocean fields Mar 12, 2026
@codecov-commenter
Copy link

codecov-commenter commented Mar 12, 2026

Codecov Report

❌ Patch coverage is 1.85185% with 53 lines in your changes missing coverage. Please review.
✅ Project coverage is 62.34%. Comparing base (838d1bc) to head (f7e0b9b).

Files with missing lines Patch % Lines
...it/mars2grib/backend/concepts/level/levelMatcher.h 0.00% 29 Missing ⚠️
...d/concepts/representation/representationEncoding.h 6.66% 14 Missing ⚠️
...etkit/mars2grib/backend/concepts/level/levelEnum.h 0.00% 9 Missing ⚠️
...nd/concepts/representation/representationMatcher.h 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #181      +/-   ##
===========================================
- Coverage    62.61%   62.34%   -0.27%     
===========================================
  Files          303      303              
  Lines        11554    11603      +49     
  Branches      1029     1037       +8     
===========================================
  Hits          7234     7234              
- Misses        4320     4369      +49     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dsarmany dsarmany force-pushed the feature/mars2grib-ocean branch from d4d7d69 to f7e0b9b Compare March 15, 2026 11:27
DEF(LevelType::OceanSurfaceToBottom, "oceanSurfaceToBottom");
DEF(LevelType::LakeBottom, "lakeBottom");
DEF(LevelType::MixingLayer, "mixingLayer");
DEF(LevelType::OceanModel, "oceanModel");
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we will need to introduce some additional logic for the levtype=o3d fields (oceanModel and oceanModelLayer) to set the vertical information.

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.

3 participants