protomaps / go-pmtiles

Single-file executable tool for working with PMTiles archives
BSD 3-Clause "New" or "Revised" License
350 stars 48 forks source link

Update show.go to show (latitude, longitude), not (long, lat). #129

Closed bramp closed 6 months ago

bramp commented 6 months ago

The general convention per ISO_6709, is that a coordinate is (lat, long), not (long, lat). It took me a minute of staring at pmtiles show output to realise the tool flipped them.

https://en.wikipedia.org/wiki/ISO_6709

bdon commented 6 months ago

https://macwright.com/lonlat/

let's just make it print out long: X lat: Y then

bramp commented 6 months ago

oh gah, even more confusing than I realised.

Updated the PR with your suggestion.

bdon commented 6 months ago

Thanks!

bdon commented 6 months ago

https://github.com/protomaps/go-pmtiles/releases/tag/v1.14.1