Skip to content

fix: NPC (TrafficSimulation) DO NOT stop on TrafficLight#417

Merged
mackierx111 merged 2 commits into
mainfrom
fix/traffic_lanelet
Aug 22, 2025
Merged

fix: NPC (TrafficSimulation) DO NOT stop on TrafficLight#417
mackierx111 merged 2 commits into
mainfrom
fix/traffic_lanelet

Conversation

@rokamu623

Copy link
Copy Markdown
Contributor

Checklist before requesting a review

  • Follow coding standards.
  • Follow AWSIM architecture.
  • No unnecessary comments or Debug.Log() remain.
  • The latest main branch has been merged into the work branch.
  • The expected behavior is achieved after implementation. Furthermore, existing functions are not affected.

Problem & Fix

Behavior test

Problem

Screencast.from.2025.08.22.13.44.04.webm

Solve

Screencast.from.2025.08.22.16.19.19.webm

Signed-off-by: rokamu623 <ryo.okamura.3@tier4.jp>
Signed-off-by: rokamu623 <ryo.okamura.3@tier4.jp>
@rokamu623 rokamu623 self-assigned this Aug 22, 2025
@rokamu623 rokamu623 added the bug Something isn't working label Aug 22, 2025
@rokamu623 rokamu623 requested a review from mackierx111 as a code owner August 22, 2025 07:24
@rokamu623 rokamu623 changed the title fix/traffic lanelet fix: NPC (TrafficSimulation) DO NOT stop on TrafficLight Aug 22, 2025

@mackierx111 mackierx111 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will merge this pull request.
However, I found another issue, so I will share it next week.

@mackierx111 mackierx111 merged commit 7b5d632 into main Aug 22, 2025
@mackierx111 mackierx111 deleted the fix/traffic_lanelet branch September 9, 2025 00:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants