Skip to content
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

Latitude/Longitude inversion #6

Closed
EmilyBourne opened this issue Oct 12, 2017 · 1 comment
Closed

Latitude/Longitude inversion #6

EmilyBourne opened this issue Oct 12, 2017 · 1 comment

Comments

@EmilyBourne
Copy link

The geo-co-ordinates provided by SUMo are in (Longitude,Latitude) format, however GeoPosition as defined in artery/utility/Geometry.h is in format (latitude,longitude). This means that when the GeoPosition is created as in the function VehicleController::getGeoPosition() (artery/traci/VehicleController.cc: line 38) the longitude is saved as latitude and latitude as longitude

riebl added a commit that referenced this issue Oct 13, 2017
fixes issue #6 reported by EmilyBourne, thanks!

Change-Id: I4b2e7a57d2b44da535d649dcf9f2454ed1bc42e0
@riebl
Copy link
Owner

riebl commented Oct 13, 2017

Oops, this is a really unfortunate mistake probably introduced during some code refactoring. I have changed the relevant code fragment to refer to GeoPosition's attributes explicitly by name so this bug is unlikely to be reintroduced. Fixed by 82afb97.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants