Skip to content

use string comparison to implementation_identifier, not by addresses.#3073

Open
fujitatomoya wants to merge 1 commit intorollingfrom
pull/3068/followup
Open

use string comparison to implementation_identifier, not by addresses.#3073
fujitatomoya wants to merge 1 commit intorollingfrom
pull/3068/followup

Conversation

@fujitatomoya
Copy link
Collaborator

Description

follow up of #3068, especially for #3068 (comment)

Fixes # (issue)

Is this user-facing behavior change?

No

Did you use Generative AI?

Yes, Claude Sonnet 4.5 to help to find the root cause.

Additional Information

Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
@fujitatomoya
Copy link
Collaborator Author

@mini-1235 can you try this patch?

@mini-1235
Copy link
Contributor

No this does not fix the issue :(

What I can confirm at this point is that running:

ros2 launch nav2_bringup tb4_simulation_launch.py use_intra_process_comms:=True

often results in segfault

If I disable intraprocess communication (default off):

ros2 launch nav2_bringup tb4_simulation_launch.py

the issue does not reproduce

Unfortunately I don't have time this week to create a smaller reproducible example, as it is Chinese New Year here in Indonesia

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments