Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Extend regridder saving/loading to all regridders #357

Merged
merged 11 commits into from
May 30, 2024

Conversation

stephenworsley
Copy link
Contributor

Closes #318

Copy link

codecov bot commented May 28, 2024

Codecov Report

Attention: Patch coverage is 99.37500% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 98.92%. Comparing base (cf86c1d) to head (d812343).

Current head d812343 differs from pull request most recent head f077809

Please upload reports for the commit f077809 to get more accurate results.

Files Patch % Lines
esmf_regrid/experimental/io.py 98.05% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #357      +/-   ##
==========================================
+ Coverage   98.85%   98.92%   +0.07%     
==========================================
  Files          37       37              
  Lines        3929     4196     +267     
==========================================
+ Hits         3884     4151     +267     
  Misses         45       45              

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

* main:
  Move benchmark code from nox to bm_runner (SciTools#361)
  Bump scitools/workflows from 2024.01.0 to 2024.04.3 (SciTools#356)
  Bump peter-evans/create-pull-request from 5.0.2 to 6.0.5 (SciTools#355)
  [pre-commit.ci] pre-commit autoupdate (SciTools#336)
  Updated environment lockfiles (SciTools#344)
@stephenworsley stephenworsley marked this pull request as ready for review May 29, 2024 15:03
Copy link
Member

@pp-mo pp-mo left a comment

Choose a reason for hiding this comment

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

I'm not really competent to comment on all the mechanics here.
But I have raised some 'style' points, including docstrings that seem to need changing.

Also, I think it probably needs a changelog entry ?

esmf_regrid/experimental/io.py Show resolved Hide resolved
esmf_regrid/experimental/io.py Show resolved Hide resolved
esmf_regrid/experimental/io.py Show resolved Hide resolved
esmf_regrid/experimental/io.py Show resolved Hide resolved
@pp-mo
Copy link
Member

pp-mo commented May 30, 2024

OK I think this is all good now

@pp-mo pp-mo merged commit d4f4c2d into SciTools:main May 30, 2024
15 checks passed
stephenworsley added a commit to stephenworsley/iris-esmf-regrid that referenced this pull request Jul 18, 2024
* main:
  Bump scitools/workflows from 2024.06.0 to 2024.06.5 (SciTools#377)
  Fix typo (SciTools#371)
  Bump scitools/workflows from 2024.04.3 to 2024.06.0 (SciTools#365)
  Update to v0.11.dev0. (SciTools#363)
  Update to v0.10.0 (SciTools#362)
  Extend regridder saving/loading to all regridders (SciTools#357)
  Ensure dtype is preserved after regridding (SciTools#239)
  Move benchmark code from nox to bm_runner (SciTools#361)

# Conflicts:
#	esmf_regrid/schemes.py
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.

Support saving/loading of ESMFAreaweightedRegridder, ESMFBilinearRegridder and ESMFNearestRegridder.
2 participants