Skip to content

[Domain] expand route guidance placement and activation#251

Merged
learncold merged 2 commits into
mainfrom
codex/route-guidance-room-priority
May 18, 2026
Merged

[Domain] expand route guidance placement and activation#251
learncold merged 2 commits into
mainfrom
codex/route-guidance-room-priority

Conversation

@95x8x9
Copy link
Copy Markdown
Collaborator

@95x8x9 95x8x9 commented May 16, 2026

Summary

  • allow route guidance to be installed inside room space as well as on doors/exits, with clearance checks against walls and non-walkable space
  • persist and render installed guidance positions, and show multiple active guidances consistently in authoring/runtime UI
  • prioritize agents near installed guidance anchors for faster re-evaluation instead of relying only on the global round-robin budget
  • add regression coverage for room-installed guidance, multiple simultaneous guidances, and near-anchor prioritization

Related Issue

Area

  • Engine
  • Domain
  • Application
  • Docs
  • Build
  • Analysis
  • Chore

Architecture Check

  • I kept the dependency direction application -> domain -> engine.
  • I did not add Qt UI code to src/domain.
  • I did not add domain or application dependencies to src/engine.
  • I used src/ as the include root.

Verification

  • cmake --preset windows-debug
  • cmake --build --preset build-debug
  • ctest --preset test-debug
  • Not run (reason below)

Risks / Follow-up

  • default guidance compliance/visibility parameters are still conservative, so scenario tuning may still be needed for stronger in-sim response
  • if room-installed guidance should influence a wider area than the current near-anchor visibility rule, we should follow up by revisiting the activation radius and zone constraints

@95x8x9 95x8x9 requested a review from learncold as a code owner May 16, 2026 10:14
@learncold learncold merged commit 6dbbbbb into main May 18, 2026
2 checks passed
@learncold learncold deleted the codex/route-guidance-room-priority branch May 18, 2026 09:39
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.

Task-Expand route guidance placement and activation behavior

2 participants