search

developmentseed / titiler

Build your own Raster dynamic map tile services
https://developmentseed.org/titiler/
MIT License
765 stars 157 forks source link

rename `/crop` -> `/bbox` and `/feature` ? #702

Closed vincentsarago closed 11 months ago

vincentsarago commented 11 months ago

In the tiler factory we create a set of endpoint with /crop prefix to handle both bbox (GET) and feature (POST) image queries.

https://github.com/developmentseed/titiler/blob/108283f7d4366394e5d32831055d2d9982de4e41/src/titiler/core/titiler/core/factory.py#L930-L938

https://github.com/developmentseed/titiler/blob/108283f7d4366394e5d32831055d2d9982de4e41/src/titiler/core/titiler/core/factory.py#L993-L1005

I'm not a big fan of the /crop name and I think it could make sense to change the endpoint prefix to something with more meanings

geospatial-jeff commented 11 months ago

I've always thought crop is confusing so this seems like a good change to me 👍