search

JuliaGeo / Geodesy.jl

Work with points defined in various coordinate systems.
MIT License
111 stars 24 forks source link

More idiomatic conversion #4

Closed garborg closed 8 years ago

garborg commented 9 years ago

ENU(Bounds{LLA}...) and friends aren't idiomatic because their return type is not ENU. Other conversions should probably be defined as conversions rather than constructors (assuming constructions that require a reference point are a fit for convert) so both forms work.

I tried to change this already, but it seemed more of a hassle than it's worth to support both 0.3 and 0.4 right now. Perhaps best to wait until 0.4 settles a bit (idioms will be better formed then, too).

garborg commented 9 years ago

See tedsteiner/OpenStreetMap.jl#61 for discussion on adding convenience methods here in Geodesy.jl for converting point types of containers like Dict{Int,ENU} -- for now left as a TBD until language design settles a bit.

andyferris commented 8 years ago

One good solution would have been to define map on Bounds to map the points in the Bounds container according to some function...

andyferris commented 8 years ago

I think this issue isn't relevant anymore? Closing