Skip to content

Modify the kinematics plugin to match with new interface diagram and msg#36

Merged
greg-burgess merged 4 commits intomasterfrom
match_new_interface_msg
Mar 24, 2021
Merged

Modify the kinematics plugin to match with new interface diagram and msg#36
greg-burgess merged 4 commits intomasterfrom
match_new_interface_msg

Conversation

@woensug-choi
Copy link
Copy Markdown
Collaborator

Baseground

Changelog

The plugin is modified to match with the new interface diagram solely to deal with glider motions and its actual values (from gazebo values, not that of estimated values obtained from sensors) according to the new UwGliderStatus msg

  • Change the plugin name and command/status topic name
    • Plugin name
      • Before : direct_kinematics_ros_plugin / After : kinematics_ros_plugin
    • Command/status ROS topic name
      • Before : /glider_hybrid_whoi/direct_kinematics/UwGliderCommand, /glider_hybrid_whoi/direct_kinematics/UwGliderStatus
      • After : /glider_hybrid_whoi/kinematics/UwGliderCommand, /glider_hybrid_whoi/kinematics/UwGliderStatus
  • Remove sensor data manipulation from the kinematics plugin
  • Add GDAL transform to calculate Lat/Lon from XYZ

Note

@greg-burgess greg-burgess merged commit eb0fa15 into master Mar 24, 2021
@woensug-choi
Copy link
Copy Markdown
Collaborator Author

@daewok Please note that the topic name for glider command and status is changed

  • Command/status ROS topic name
    • Before : /glider_hybrid_whoi/direct_kinematics/UwGliderCommand, /glider_hybrid_whoi/direct_kinematics/UwGliderStatus
    • After : /glider_hybrid_whoi/kinematics/UwGliderCommand, /glider_hybrid_whoi/kinematics/UwGliderStatus

@woensug-choi woensug-choi deleted the match_new_interface_msg branch July 30, 2021 04:58
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.

2 participants