ping_pong rand() is not seeded so device ids are likely the same across nodes

Abierto Apto para principiantes
#97 1 comentario 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
68/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Activo
Stack tecnológico
c
Área
distributed-systems, embedded-iot

Línea de trabajo

Comienza con el ejecutable ping_pong del paquete micro_ros_demos_rclc y sigue cómo se inicializan sus identificadores de dispositivo y el estado de rand(). Reproduce el problema usando los comandos proporcionados de compilación con colcon y de los contenedores, y después verifica que los dos contenedores intercambian mensajes tanto Ping como Pong con identificadores distintos.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Issue template

Steps to reproduce the issue

Build this repo on your target

  cd /ros2_ws
  . /opt/ros/humble/setup.bash
  . install/local_setup.bash
  colcon build --packages-select micro_ros_demos_rclc
  /ros2_ws/install/micro_ros_demos_rclc/lib/micro_ros_demos_rclc/ping_pong
Expected behavior

I expect to see ping/pong messages

Actual behavior

You will see only Ping or only Pong messages

Additional information

Running on Ubuntu 24.04 host, docker compose driving a pair of containers to pubsub.

In my testing, seeding random with a nanosecond clock is sufficient to generate different device ids. I'd file a PR but I'd like to see if this was intended or maybe just an artifact of launching two container simultaneously

Lenguaje dominante
C
Estrellas
101
Forks
35
Merge medio
4 d 16 h
PR fusionados (30 d)
3

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de micro-ROS/micro-ROS-demos

Todos los issues de micro-ROS/micro-ROS-demos

Issues similares

Más issues de C

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.