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

Add MODIS 5km to 500m and 250m interpolation #16

Merged
merged 1 commit into from
Feb 13, 2020

Conversation

djhoese
Copy link
Member

@djhoese djhoese commented Jun 3, 2019

This was an attempt to see how easy it would be to add MODIS 5km->500m and 5km->250m interpolation. It is an abomination and should not be merged without some real thought.

See http://www.icare.univ-lille1.fr/tutorials/MODIS_geolocation for how the real interpolations are supposed to work.

This PR cheats to make the output the right size by repeating a lot of pixels on the edges. The other option would be to interpolate twice (5km to 1km, 1km to Xm) but that would add a ton of operations to the dask graph for no reason.

I'm leaving this here as reference but it should probably not be merged, ever.

  • Closes #xxxx
  • Tests added
  • Tests passed
  • Passes git diff origin/master **/*py | flake8 --diff
  • Fully documented

@djhoese djhoese self-assigned this Jun 3, 2019
@coveralls
Copy link

Coverage Status

Coverage remained the same at ?% when pulling cf031c3 on djhoese:feature-5km-to-500-250m into 7c5cc8a on pytroll:master.

@mraspaud mraspaud merged commit 2c27197 into pytroll:master Feb 13, 2020
@djhoese djhoese deleted the feature-5km-to-500-250m branch September 6, 2021 23:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants