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

navdata_packet.header now filled, also using cmd_vel.angular.x/y != 0 for hover #38

Merged
merged 2 commits into from
Nov 23, 2012

Conversation

mikehamer
Copy link
Contributor

As per comments in #31 and #34, the requested changes have been made.

Now:

  • The .header of all navdata packets is correctly filled.
  • The messages are defined as class variables rather than as local, thus saving initialization time
  • The parameters control_disable_hover and control_always_send have been removed, in favour of using cmd_vel.angular.x != 0 and cmd_vel.angular.y != 0 to disable hover mode.

Removed ros params command_disable_hover and command_always_send, in
favour of disabling hover manually by setting cmd_vel.angular.x=1 or
cmd_vel.angular.y=1
Header.stamp and header.frame_id are now correctly filled. Messages are
also now member variables rather than local to save initialization time.
mani-monaj added a commit that referenced this pull request Nov 23, 2012
navdata_packet.header now filled, also using cmd_vel.angular.x/y != 0 for hover
@mani-monaj mani-monaj merged commit 837c4bc into AutonomyLab:dev-unstable Nov 23, 2012
mani-monaj added a commit that referenced this pull request Nov 23, 2012
`hover` variable to make the logic correct.
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

Successfully merging this pull request may close these issues.

None yet

2 participants