cogeotiff / rio-tiler-mosaic

A rio-tiler plugin for creating tiles from multiple observations.
MIT License
32 stars 6 forks source link

refactor pixel selection methods to allow custom classes #3

Closed vincentsarago closed 5 years ago

vincentsarago commented 5 years ago

closes #1 #2

⚠️ This is a major update and will break the previous release

This PR adds an abstract base class for pixel_selection method, allowing custom classes. The base class is relatively simple with 3 methods/classes

This work is mostly inspired by https://github.com/mapbox/watchbot-progress-py

I'd love to get feedback on

Note: This PR removes the last methods because it's was making things more complicated and the logic should (IMO) be moved to the top level app.

vincentsarago commented 5 years ago

here is a gist showing the new usage and how to create simple custom methods https://gist.github.com/vincentsarago/5c71f4a91115277dd73f92f7577faad2