gz_ros2_control icon indicating copy to clipboard operation
gz_ros2_control copied to clipboard

Add IMU as a separate plugin of ros2_control hardware interface

Open ahcorde opened this issue 4 years ago • 7 comments

Add IMU as a separate plugin of ros2_control hardware interface

ahcorde avatar Nov 30 '21 10:11 ahcorde

Hi @ahcorde I'd like to take up this task. Can you guide me as to how to proceed and what do we want to implement?

akshatpandya avatar Dec 06 '21 04:12 akshatpandya

Hi @akshatpandya, are you still interested into this? I can guide you if you want.

This probably means that IMU is added as implementation of "SensorInterface"

This would also lead to more specific plugins for different sensors, e.g., IMU and FTS.

destogl avatar Dec 23 '21 16:12 destogl

Hi @destogl, yes I'm interested in taking up this issue.

akshatpandya avatar Dec 27 '21 00:12 akshatpandya

@akshatpandya, sorry I missed your comment. Here the list of tasks that I think has to be done for this functionality. (@ahcorde feel free to extend/correct as you think).

  • all files should be added into ign_ros2_control package
  • [ ] Create header file ignition_sensor_interface.hpp with IgnitionSensorInterface class inheriting hardware_interface::SensorInterface (similar to ignition_system_interface.hpp file
  • [ ] Create ign_imu_sensor.hpp file similar to the ign_system.hppfile
  • [ ] Implement ign_imu_sensor.cpp file (For the interface names try using standard names defined in IMU semantic_component from ros2_control package)

Please let me know if you have any questions.

destogl avatar Jan 18 '22 17:01 destogl

@destogl thanks for the explanation. I'll take up this task and keep you updated.

akshatpandya avatar Jan 24 '22 22:01 akshatpandya

Has anyone made any progress on this? Currently Nav2 tutorials found here only shows how to add an IMU for simulation in Gazebo Classic. Is something similar currently available for Ignition? @destogl

jclinton830 avatar Nov 13 '23 04:11 jclinton830