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

Aberta Para iniciantes
#97 1 comentário 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
2/5
Tempo estimado
1-3 horas
Facilidade para iniciantes
68/100
Tipo de issue
Bug
Clareza
Razoavelmente clara
Status de atividade
Ativa
Stack de tecnologia
c
Domínio
distributed-systems, embedded-iot

Direção de pesquisa

Comece pelo executável ping_pong no pacote micro_ros_demos_rclc e acompanhe como os IDs dos dispositivos e o estado de rand() são inicializados. Reproduza o problema usando os comandos fornecidos de build com colcon e de contêineres e, em seguida, verifique se os dois contêineres trocam mensagens Ping e Pong com IDs distintos.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

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

Linguagem predominante
C
Estrelas
101
Forks
35
Merge médio
4d 16h
PRs com merge (30d)
3

Guia de contribuição

Abrir o guia de contribuição

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de micro-ROS/micro-ROS-demos

Todas as issues de micro-ROS/micro-ROS-demos

Issues semelhantes

Mais issues de C

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.