-
Notifications
You must be signed in to change notification settings - Fork 24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
lat,lon argument order #23
Comments
Hmm yes this is one we've puzzled over quite a lot. People tend to talk about lat-lon while quite a few (certainly not all) geodesy libraries use lon-lat. Is the user more likely to be someone who has done Geodesy in other languages before? Or someone starting out or needing something relatively simple? (The point of the keyword constructor is you can write Thoughts, @c42f? |
Hmm... this leaves me with a thought. You might pass a pointer or data structure to a library that uses lon-lat order, but you probably won't pass a pointer to a human that thinks in lat-lon!! :) Perhaps between the field names, the keyword constructor, and the pretty REPL output a human wouldn't get confused, so there is a decent argument for switching the ordering for ABI compatibility... |
Interesting point Andy, though what would we do we do about the non-keyword constructor? It's still going to be a point of confusion unless we get rid of it. Regarding usage elsewhere, geodetic coordinates are
OGC document on this: Basically, there's significant precedent for having it both ways; nobody can agree and it's all a bit terrible! |
See also: http://www.macwright.org/lonlat/. I'd argue for lonlat. |
Right, the "just use a right handed system because math" and "serialization formats basically all do it this way" are powerful arguments in favour of lon,lat,height. IMO the most powerful counter argument is that "lat long" is the colloquial standard. Some (possibly bad) options to clarify the API:
|
ISO 6709 is an interesting read |
The README has argument order mixed up (
LLA(lat=x, lon=y, alt=z)
andLatLon(lat=x, lon=y)
). Islon, lat
a more common argument order in the external libraries we'll interface with and users will be familiar with?The text was updated successfully, but these errors were encountered: