support non-actuator Dynamixel sensor devices (incl. HX Sensor Tip)#111
support non-actuator Dynamixel sensor devices (incl. HX Sensor Tip)#111jack0682 wants to merge 1 commit into
Conversation
Signed-off-by: JaehongOh <jaehongoh1554@gmail.com>
There was a problem hiding this comment.
Code Review
This pull request introduces a dedicated sensor-only read path to decouple tactile sensor data acquisition from the main actuator control loop, including support for the hx_sensor_tip model and configurable read pacing. It also improves error handling by throwing exceptions instead of exiting and attempts to resolve memory leaks in write operations. However, the review identified critical bugs: the new RAII-managed write buffers result in dangling pointers as the SDK does not copy the data, and the sensor read logic fails to apply unit scaling. Furthermore, the batching logic in ReadSensorOnly incorrectly assumes readTxRx can handle multiple device IDs, and the mandatory routing of GPIO sensors to the new path constitutes a breaking change that should be made optional.
요약
Dynamixel 프로토콜을 사용하는 센서 전용 장치(HX Sensor Tip 등)를
액추에이터와 분리된 read 경로로 처리할 수 있도록 일반화합니다. ROBOTIS HX Sensor
Tip 모델 정의도 함께 포함합니다.
동기
ROBOTIS Dynamixel 생태계는 X-series 모터뿐 아니라 동일 프로토콜을 사용하는
다양한 센서 제품군(예: HX Sensor Tip)도 포함합니다. 그러나 현재
dynamixel_hardware_interface는 모든 장치를 동일한 SyncRead / FastSyncRead그룹으로 묶기 때문에 다음과 같은 한계가 있습니다:
같은 버스 위에서 모터와 센서가 공존하는 구성은 ROBOTIS 표준 사용 패턴이므로,
이를 ros2_control 위에서 안정적으로 지원하기 위한 변경입니다.
주요 변경 사항
1. 센서 전용 read 경로 추가 (
ReadSensorOnly)2.
tactile_read_divider파라미터<hardware>블록 파라미터로 URDF에서 설정 가능3. HX Sensor Tip 모델 정의 추가
param/dxl_model/hx_sensor_tip.model추가4. 기존 동작 보존
호환성
<hardware>파라미터로 opt-in테스트
관련 제품