Skip to content

Conversation

bouweandela
Copy link
Member

@bouweandela bouweandela commented May 23, 2024

Description

Add xESMF regriding scheme for the regrid preprocessor.

Example usage in preprocessor in recipe:

regrid:
  target_grid: 1x1
  scheme:
    reference: esmvalcore.preprocessor.regrid_schemes:xESMF
    method: bilinear
    periodic: true

Link to documentation: https://esmvaltool--2433.org.readthedocs.build/projects/ESMValCore/en/2433/api/esmvalcore.regridding_schemes.html#esmvalcore.preprocessor.regrid_schemes.xESMF


Before you get started

Checklist

It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.


To help with the number pull requests:

Copy link

codecov bot commented May 23, 2024

Codecov Report

❌ Patch coverage is 33.33333% with 26 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.34%. Comparing base (2fc8761) to head (b639462).
⚠️ Report is 273 commits behind head on main.

Files with missing lines Patch % Lines
esmvalcore/preprocessor/_regrid_xesmf.py 31.57% 26 Missing ⚠️

❌ Your patch status has failed because the patch coverage (33.33%) is below the target coverage (100.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2433      +/-   ##
==========================================
- Coverage   94.51%   94.34%   -0.17%     
==========================================
  Files         246      247       +1     
  Lines       14023    14062      +39     
==========================================
+ Hits        13254    13267      +13     
- Misses        769      795      +26     

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@bouweandela bouweandela added preprocessor Related to the preprocessor dask related to improvements using Dask labels May 23, 2024
@flicj191
Copy link
Contributor

message for myself @flicj191 as I want to try to use this regridder. let me know if you would like me to do anything

Copy link
Contributor

In order to maintain a backlog of relevant pull requests, we automatically label them as stale after 180 days of inactivity.

If this pull request is still important to you, please comment below to remove the stale label. Otherwise, this pull request will be automatically closed in 60 days. If this pull request only suffers from a lack of reviewers, please tag the @ESMValGroup/technical-lead-development-team so they can help you find a suitable reviewer.

@github-actions github-actions bot added the Stale label Jun 27, 2025
Copy link
Contributor

This stale pull request has been automatically closed due to a lack of community activity.

If you still care about this pull request, please re-open it. If you do not have sufficient permissions for this, tag the @ESMValGroup/technical-lead-development-team in a comment below including a short message that this pull request is still relevant to you. They will re-open it on your behalf.

@github-actions github-actions bot closed this Aug 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dask related to improvements using Dask preprocessor Related to the preprocessor Stale
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

2 participants