Skip to content

Conversation

@oiffrig
Copy link
Contributor

@oiffrig oiffrig commented Mar 18, 2025

  • Modifies Message.copy to support any Message subclass
  • Adds BUFRMessage class with pack, unpack and from_samples methods
  • Adds a kind argument to the reader constructors (defaults to GRIB for backwards compatibility). Only GRIB and BUFR are supported.

@FussyDuck
Copy link

FussyDuck commented Mar 18, 2025

CLA assistant check
All committers have signed the CLA.

@codecov-commenter
Copy link

codecov-commenter commented Mar 18, 2025

Codecov Report

Attention: Patch coverage is 0% with 29 lines in your changes missing coverage. Please review.

Project coverage is 32.79%. Comparing base (e92cbe7) to head (efa1d70).
Report is 1 commits behind head on develop.

Files with missing lines Patch % Lines
eccodes/highlevel/reader.py 0.00% 17 Missing ⚠️
eccodes/highlevel/message.py 0.00% 11 Missing ⚠️
eccodes/highlevel/__init__.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #124      +/-   ##
===========================================
- Coverage    33.00%   32.79%   -0.21%     
===========================================
  Files           16       16              
  Lines         2606     2622      +16     
  Branches       186      187       +1     
===========================================
  Hits           860      860              
- Misses        1731     1747      +16     
  Partials        15       15              

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

@shahramn shahramn self-assigned this Mar 18, 2025
@shahramn shahramn added the approved-for-ci Approved for CI label Mar 18, 2025
@shahramn shahramn merged commit 4566704 into develop Mar 18, 2025
232 checks passed
@shahramn shahramn deleted the feature/highlevel-bufr branch March 18, 2025 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved-for-ci Approved for CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants