Skip to content

Rust bindings: eckit::geo grids and gridSpec - #326

Merged
pmaciel merged 14 commits into
developfrom
eckit-geo-rust
Oct 14, 2026
Merged

pmaciel merged 14 commits into
developfrom
eckit-geo-rust

Conversation

@Choochmeque

@Choochmeque Choochmeque commented Aug 18, 2026 •

Copy link
Copy Markdown
Contributor

Description

Adds the eckit-geo feature to the Rust bindings.

Contributor Declaration

By opening this pull request, I affirm the following:

  • All authors agree to the Contributor License Agreement.
  • The code follows the project's coding standards.
  • I have performed self-review and added comments where needed.
  • I have added or updated tests to verify that my changes are effective and functional.
  • I have run all existing tests and confirmed they pass.

🌦️ >> Documentation << 🌦️
https://sites.ecmwf.int/docs/dev-section/eckit/pull-requests/PR-326

@Choochmeque Choochmeque changed the title Geo Grid support Rust bindings: eckit::geo grids and gridSpec Sep 24, 2026
@Choochmeque
Choochmeque marked this pull request as ready for review September 24, 2026 10:32
@codecov-commenter

codecov-commenter commented Sep 24, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 70.58%. Comparing base (a888463) to head (6d27374).

Files with missing lines Patch % Lines
src/eckit/spec/Spec.h 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #326      +/-   ##
===========================================
- Coverage    70.60%   70.58%   -0.03%     
===========================================
  Files         1207     1207              
  Lines        66076    66076              
  Branches      5065     5065              
===========================================
- Hits         46652    46637      -15     
- Misses       19424    19439      +15     

☔ View full report in Codecov by Harness.
📢 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.

@pmaciel pmaciel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Excellent!! 👍

@pmaciel
pmaciel merged commit ce25fa6 into develop Oct 14, 2026
162 of 182 checks passed
@pmaciel
pmaciel deleted the eckit-geo-rust branch October 14, 2026 10:45
@pmaciel pmaciel mentioned this pull request Oct 30, 2026
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