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

global_position.cpp plugin #87

Closed
TSC21 opened this issue Aug 7, 2014 · 7 comments · Fixed by #118
Closed

global_position.cpp plugin #87

TSC21 opened this issue Aug 7, 2014 · 7 comments · Fixed by #118

Comments

@TSC21
Copy link
Member

TSC21 commented Aug 7, 2014

Creating this plugin has a reason: gps.cpp gives us GPS_RAW_INT handle, which is the global position, as returned by the Global Positioning System, which means, a RAW sensor value.

For getting the filtered global position, we need to read the GLOBAL_POSITION_INT, which is fused with IMU sensor data onboard the FCU. This can be useful to add some features regarding global<->local conversions.

@TSC21
Copy link
Member Author

TSC21 commented Aug 7, 2014

What do you think of this? It's feasable?

@vooon
Copy link
Member

vooon commented Aug 7, 2014

Yeah, but need find standard messages for this.

@TSC21
Copy link
Member Author

TSC21 commented Aug 7, 2014

Probably sensor_msgs/NavSatFix.msg for lat/lon/alt, sensor_msgs/MagneticField.msg for compass and geometry_msgs/Vector3Stamped.msg to GPS velocities.

Still in doubt relative to the relative altitude (maybe a custom stamped message with a float64 rel_alt field needs to be created - mavros/global_position/rel_alt). But probably it's not the best way. Maybe it can be thrown away in this case, since we have the barometer (or sonar) measures.

To convert to local coordinates, LOCAL_POSITION_NED_SYSTEM_GLOBAL_OFFSET should be considered. A origin should be take in account in this case, I think. Don't know if it's active but there's SET_GPS_GLOBAL_ORIGIN to do that job.

@LorenzMeier I'm calling you also to this topic so you can please clarify and give your opinion in what should be done in this case if we want to do a conversion from GPS to local coordinates. Thanks!

@TSC21
Copy link
Member Author

TSC21 commented Aug 8, 2014

Found what can be used to convert into a pose (GPS to UTM): http://wiki.ros.org/gps_common?distro=hydro

@TSC21
Copy link
Member Author

TSC21 commented Aug 14, 2014

I already started to develop this! I will let you know when I am to finish it out. Probably going to need some help in one or two particular cases.

@vooon
Copy link
Member

vooon commented Aug 15, 2014

Reopen: plugin not complete.

@vooon vooon reopened this Aug 15, 2014
@TSC21
Copy link
Member Author

TSC21 commented Aug 15, 2014

I think as is, it can be closed ;) Next thing to do should be add the possibility to handle GLOBAL_POSITION_INT_COV.

@vooon vooon added this to the Version 0.8.0 milestone Sep 11, 2014
@vooon vooon closed this as completed Sep 11, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants