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

rock-gazebo should ensure that the spherical coordinates heading is rotated by 90 degrees #4

Open
doudou opened this issue Feb 9, 2018 · 2 comments

Comments

@doudou
Copy link
Contributor

doudou commented Feb 9, 2018

In order for Gazebo to use NWU for e.g. GPSes, one need to have a <heading_deg>90</heading_deg> tag in the world. This is more or less a requirement if one wants to use georeferenced coordinates in a Gazebo environment linked to Rock.

So far, the system will misbehave silently if this is not done, I see two possibilities to improve the situation:

  • make rock-gazebo, which already preprocesses the world file, auto-add the tag if it is not present. It would leave an existing heading_deg tag alone so that one can force another heading deg.
  • make rock-gazebo fail if heading_deg is not set to the expected heading.
@doudou
Copy link
Contributor Author

doudou commented Feb 9, 2018

@GuilhermePSC @alexfneves this is why the GPS output is not NWU. You need to add <heading_deg>90</heading_deg> to the spherical coordinates block to fix it. I won't do it myself as it needs to be synchronized with the removal of the workaround you implemented.

@alexfneves
Copy link

It worked! We added the <heading_deg>90</heading_deg> on the sdf file and removed the task we were using to convert the gps data into xyz in gazebo.

I don't know why, the <world_frame_orientation>NWU</world_frame_orientation> didn't seem to work on gazebo, while <heading_deg>90</heading_deg> did the trick. The sdf documentation actually tell us to use -90 deg with default ENU on the orientation, but the 90 deg that seemed right.

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

No branches or pull requests

2 participants