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

vision_position -> vision_pose_estimate #123

Closed
TSC21 opened this issue Aug 15, 2014 · 0 comments
Closed

vision_position -> vision_pose_estimate #123

TSC21 opened this issue Aug 15, 2014 · 0 comments

Comments

@TSC21
Copy link
Member

TSC21 commented Aug 15, 2014

I propose changing name vision_position to vision_pose_estimate, just to be according with mocap_pose_estimate plugin.
So it will be:
mocap_pose_estimate
vision_pose_estimate

Also propose move this plugin and vision_speed to mavros_extras, since also not anyone is going to use them as plugins.

@vooon vooon closed this as completed in de5905b Aug 16, 2014
vooon added a commit that referenced this issue Aug 18, 2014
* master:
  plugins: Change UAS FCU link name.
  scripts: mavcmd: Add takeoffcur and landcur commands
  Plugins: finish moving plugins
  Closes #122, closes #123; plugins: move mocap & vision plugins to extras, change vision plugins name
  plugins: UAS remove std::atomic<double>
  plugin: global_position: Fill NavSatFix status filed.
  plugins: Add GPS data to UAS
  plugins: Move setpoint_mixin.h
  plugin: mocap: Fix load.
  plugins: global_position: get pose orientation from the one stored in uas
  plugins: global_position: use relative_alt on position.z; mavros_plugins.xml - corrected declaration of mocap_pose_estimate
  plugin - global_position - changed parameter path / orientation source
  launch: APM2 blacklist global_position plugin
  plugin: global_position: Unit unification.
  plugin: global_position: Move heaedr; Style fix.
  added rel_pos and compass_hdg pub; minor corrections
  global_position plugin - initial commit
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