Skip to content

[Domain] Extend two-floor demo and fix route replanning#163

Closed
95x8x9 wants to merge 1 commit into
codex/exit-reroutefrom
codex/2f-demo-sim-fix
Closed

[Domain] Extend two-floor demo and fix route replanning#163
95x8x9 wants to merge 1 commit into
codex/exit-reroutefrom
codex/2f-demo-sim-fix

Conversation

@95x8x9
Copy link
Copy Markdown
Collaborator

@95x8x9 95x8x9 commented May 2, 2026

Summary

  • add and refine the built-in two-floor demo workflow, including recent-project wiring and editable demo layout data
  • persist additional workspace/scenario state needed by the current authoring flow
  • fix route segment replanning so waypoint metadata vectors stay aligned and no longer trigger the MSVC cannot seek vector iterator after end assertion

Related Issue

Area

  • Domain
  • Application
  • Engine
  • 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 --build --preset build-no-app-debug
  • ctest --preset test-no-app-debug
  • cmake --build --preset build-debug --clean-first
  • cmake --preset windows-debug
  • ctest --preset test-debug
  • Not run (reason below)

Reason:

  • I validated the no-app test path and the Qt app build locally, but did not run the full test-debug preset.

Risks / Follow-up

  • This PR is based on codex/exit-reroute and is intended to merge into that branch first.
  • The diff intentionally includes the currently paired application/domain workspace changes because the verified app build depends on them moving together.

@learncold learncold marked this pull request as ready for review May 2, 2026 15:17
@learncold learncold self-requested a review as a code owner May 2, 2026 15:17
@learncold learncold closed this May 5, 2026
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