Skip to content

Get Simulation to work with multiple robots#949

Merged
jaagut merged 11 commits into
mainfrom
feature/seperate_multi_sim_config
Jul 1, 2026
Merged

Get Simulation to work with multiple robots#949
jaagut merged 11 commits into
mainfrom
feature/seperate_multi_sim_config

Conversation

@MegaIng

@MegaIng MegaIng commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Summary

Contains various fixes to make the simulation run with multiple robots, behavior & teamcomm. Works on my laptop

Proposed changes

  • Make sure that each robot reads a different game_settings config for the parameter blackboard so that they don't share a bot_id (and therefore IP address/ports) and role. Indexed by the ROS_DOMAIN_ID. Currently 4 configs are there - more could be added if desired. This required a move from an xml to a python launchfile.
  • Fix camera topics
  • Make the auto setting for team com ip automatically select the loopback IP if we are running in simulation
  • Work around an issue with the builtin timers in simulation by manually implementing timer logic

Related issues

Replaces #924

Checklist

  • Run pixi run build
  • Write documentation
  • Test on your machine
  • Test on the robot
  • Create issues for future work
  • Triage this PR and label it

@github-project-automation github-project-automation Bot moved this to 🆕 New in Software Jun 30, 2026
@MegaIng MegaIng requested review from Flova and texhnolyze June 30, 2026 14:48
@MegaIng MegaIng moved this from 🆕 New to 👀 In review in Software Jun 30, 2026
@MegaIng MegaIng added the enhancement New feature or request label Jun 30, 2026
@MegaIng MegaIng requested a review from Flova July 1, 2026 01:15
@Flova

Flova commented Jul 1, 2026

Copy link
Copy Markdown
Member

The config LGTM. But I am not sure how exactly the python launch handles the overrides, so feel free to give it a quick test.

@MegaIng MegaIng requested a review from jaagut July 1, 2026 03:54
@MegaIng MegaIng force-pushed the feature/seperate_multi_sim_config branch 2 times, most recently from 8bee054 to c00981d Compare July 1, 2026 05:00
@MegaIng MegaIng force-pushed the feature/seperate_multi_sim_config branch from c00981d to 8a2f62e Compare July 1, 2026 05:04
@jaagut jaagut merged commit 464f0b0 into main Jul 1, 2026
3 checks passed
@jaagut jaagut deleted the feature/seperate_multi_sim_config branch July 1, 2026 05:07
@github-project-automation github-project-automation Bot moved this from 👀 In review to ✅ Done in Software Jul 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants