Skip to content
This repository has been archived by the owner on Dec 11, 2022. It is now read-only.

[new-feature] copying resample internals from GeoData #133

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

gottacatchenall
Copy link
Member

@gottacatchenall gottacatchenall commented Oct 26, 2021

What the pull request does

Copies resample/warp internals from GeoData

Type of change

Please indicate the relevant option(s)

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ❇️ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📖 This change requires a documentation update

Checklist

  • The changes are documented
    • The docstrings of the different functions describe the arguments, purpose, and behavior
    • There are examples in the documentation website
  • The changes are tested
  • The changes do not modify the behavior of the previously existing functions
    • If they do, please explain why and how in the introduction paragraph
  • For new contributors - my name and information are added to .zenodo.json
  • The Project.toml field version has been updated
    • Change the last number for a v0.0.x series release, a bug fix, or a performance improvement
    • Change the middle number for additional features that do not break the current test suite (unless you fix a bug in the test suite)
    • Change the first number for changes that break the current test suite

@tpoisot
Copy link
Member

tpoisot commented Oct 26, 2021

Can we do this with the license they have? It might need to be its own file with the proper header, or else we'd need a clean room redesign.

@gottacatchenall
Copy link
Member Author

its MIT, we can acknowledge at the top of file

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants