RobotecAI/ros2-for-unity

Editor crashing after mismatching ROS2 distributions

Open

#37 opened on Mar 4, 2022

View on GitHubContributor guide
 (1 comment) (0 reactions) (0 assignees)C# (82 forks)auto 404
good first issuehelp wanted

Repository metrics

Stars
 (628 stars)
PR merge metrics
 (PR metrics pending)

Description

Describe the bug When compiling Ros2ForUnity asset and launching Unity Editor with different distro sourced, the Editor crashes without any log info.

To Reproduce Steps to reproduce the behavior:

  1. Build R2FU asset for ROS2 foxy version
  2. Run Unity Editor with ROS2 galaxy sourced
  3. Run simulation with example talker included in scene

Expected behavior The Editor doesn't die and logs information that ROS2 distros has been mixed.

Desktop (please complete the following information):

  • OS: Ubuntu20.04
  • ros2 distro foxy/galactic
  • ros2-for-unity version 8bd00f3ce5f2da1e67812cd2d2f4cdbdba584e91
  • ros2cs version cbb0e00c773fd45fca9c5d64d27e20f04be53fff (preload_typesupport branch)
  • ros2 dds middleware w/e
  • ros2 environment setup single pc

Contributor guide

Research direction
Reproduce by building R2FU asset for one ROS2 distro (e.g., foxy) and sourcing a different distro (e.g., galactic) before launching Unity Editor. Capture terminal logs for crash details. Investigate adding a runtime check for distribution mismatch.
Tech stack
csharp
Domain
desktopdevops
Issue type
Bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
GitROS2Unity
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70