ros-controls/ros2_controllers

diff_drive_controller code tidy-up + many tests are missing from ROS1

Aperta

#304 aperta il 25 feb 2022

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (0 assegnatari)C++ (520 fork)auto 404
good second issuehelp wanted

Metriche repository

Star
 (810 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Review the code and feature list of diff_drive_controller from ROS1 and write up a comparison with the current version in ROS2, don't forget about testing either. Please make sure to highlight messy or inefficient parts in the current ROS2 implementation, there should be a few of those still...

https://github.com/ros-controls/ros_controllers/tree/noetic-devel/diff_drive_controller https://wiki.ros.org/diff_drive_controller

The ticket then should be extended with either a list of TODOs or a list of additional tickets to address the missing parts.

Guida contributor

Direzione di ricerca
Confronta l'implementazione del diff drive controller di ROS1 (in ros controllers/noetic devel) con la versione ROS2 corrente in questo repository. Identifica test mancanti, codice disordinato e inefficienze. Crea un elenco di TODO o ticket di follow up per colmare le lacune.
Tech stack
ccpp
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++ROS
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60