Skip to content

MARS2GRIB: Make MARS number optional in numberOfForecastsInEnsemble resolver#156

Open
tweska wants to merge 1 commit intodevelopfrom
fix/em-es-mars-number
Open

MARS2GRIB: Make MARS number optional in numberOfForecastsInEnsemble resolver#156
tweska wants to merge 1 commit intodevelopfrom
fix/em-es-mars-number

Conversation

@tweska
Copy link
Member

@tweska tweska commented Feb 6, 2026

MARS key 'number' is not available in ensemble means, thus we cannot always do the validation of perturbation number in this resolver.

@codecov-commenter
Copy link

codecov-commenter commented Feb 6, 2026

Codecov Report

❌ Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 54.18%. Comparing base (b13ae4d) to head (692c3c7).

Files with missing lines Patch % Lines
...b/backend/deductions/numberOfForecastsInEnsemble.h 0.00% 4 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #156   +/-   ##
========================================
  Coverage    54.18%   54.18%           
========================================
  Files          300      300           
  Lines        11564    11564           
  Branches      1025     1025           
========================================
  Hits          6266     6266           
  Misses        5298     5298           

☔ 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.

MARS key 'number' is not available in ensemble means
@tweska tweska force-pushed the fix/em-es-mars-number branch from c8d3a03 to 692c3c7 Compare February 6, 2026 16:34
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