Improve scaling code a bit

georust / geozero

Zero-Copy reading and writing of geospatial data.

Apache License 2.0

322 stars 30 forks source link

Improve scaling code a bit #152

Closed nyurik closed 11 months ago

nyurik commented 11 months ago

On a second though, I think to_mvt_unscaled is more accurate. raw here would be confusing.
There is no need to store bounds and do additional math ops on it. Can precompute a few things earlier on.
Default can be auto-derived

P.S. @pka I think its better to push PRs to individual's forks, and only keep project-maintained versions in the primary repo. This way there would be no experimental branches in the main repo, reducing confusion.

nyurik commented 11 months ago

P.S. Once you +1 this (or merge it into your branch), i can +1/merge yours into main