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.

Sign up for GitHub

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

Jump to bottom

Change plugin topic namespaces #233

Closed
7 tasks done
vooon opened this issue Mar 1, 2015 · 2 comments
Closed
7 tasks done

Change plugin topic namespaces #233

vooon opened this issue Mar 1, 2015 · 2 comments
Labels
API change plugin
Milestone
Version 0.11

Comments

@vooon
Copy link
Member

vooon commented Mar 1, 2015

I think that almost each plugin must be in their own namespace.
I don't like current mix in ~/setpoint/, global_location and raw gps in ~/fix.

Core plugins:

  • gps
  • local_position
  • global_position
  • setpoint_accel
  • setpoint_alttitude
  • setpoint_position
  • setpoint_velocity
@vooon vooon added this to the Version 0.11 milestone Mar 1, 2015
@vooon vooon changed the title Change plugin namespaces Change plugin topic namespaces Mar 1, 2015
@tonybaltovski
Copy link
Contributor

+1

vooon added a commit that referenced this issue Mar 5, 2015
@vooon
plugin api #241: remove global private node handle.
03da33a 
Now all plugins should define their local node handle (see dummy.cpp).
Also partially does #233 (unmerge setpoint topic namespace).
@vooon
Copy link
Member Author

vooon commented Mar 5, 2015

Done.

@vooon vooon closed this as completed Mar 5, 2015
@vooon vooon mentioned this issue Mar 9, 2015
Closed
@dreamkeep dreamkeep mentioned this issue Jul 2, 2015
Closed
nmichael pushed a commit to rislab/mavros that referenced this issue Mar 19, 2016
@vooon
Fix mavlink#233: lost braket in mavlink_param_union_double_t.
bdf0ff9
nmichael pushed a commit to rislab/mavros that referenced this issue Mar 19, 2016
@LorenzMeier
Merge pull request mavlink#234 from vooon/rel4_param_fix
5443285 
Fix mavlink#233: lost braket in mavlink_param_union_double_t.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
API change plugin
Projects
None yet
Milestone
Version 0.11
Development

No branches or pull requests
2 participants
@vooon @tonybaltovski