Skip to content

[Application] Add scenario-scoped alternative recommendations#255

Merged
learncold merged 1 commit into
mainfrom
codex/alternative-recommendations
May 18, 2026
Merged

[Application] Add scenario-scoped alternative recommendations#255
learncold merged 1 commit into
mainfrom
codex/alternative-recommendations

Conversation

@muzygosu
Copy link
Copy Markdown
Collaborator

@muzygosu muzygosu commented May 17, 2026

Summary

  • 대안추천 data contract와 병목, 출구 사용 균형, 압력 완화, 양방향 흐름 분리, 단계적 배출 추천 생성을 추가했습니다.
  • batch 결과 화면에서 시나리오별로 어떤 대안이 추천되는지 볼 수 있도록 scenario-scoped recommendation UI를 추가했습니다.
  • Compare 패널의 중복 추천 목록을 제거하고, 현재 앱 워크플로에서 지원하지 않는 일방통행 추천은 생성하지 않도록 막았습니다.
  • route guidance는 통로나 문이 아니라 출구 기준으로 설치되도록 정리했습니다.
  • occupant source 편집창에 시작 시각과 실제 배출 구간을 표시하도록 개선했습니다.

Related Issue

#245는 이번 PR에서 추천 규칙 엔진의 핵심 흐름은 구현했지만 Closes가 아니라 Refs로 남겼습니다. 특히 일방통행 전환 추천은 현재 앱에서 실제 시나리오 조작으로 구현할 수 없고, 사용자가 오해할 가능성이 있어서 이번 PR에서 의도적으로 생성하지 않도록 막았습니다. 통로폭 확장이나 출구 추가 개방처럼 구조 자체를 바꾸는 추천도 현재 scenario workflow에서 안전하게 생성하지 않습니다.

Area

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

Architecture Check

  • 의존 방향 application -> domain -> engine을 유지했습니다.
  • src/domain에 Qt UI 코드를 추가하지 않았습니다.
  • src/enginedomain 또는 application 의존성을 추가하지 않았습니다.
  • src/를 include root로 사용하는 기존 구조를 유지했습니다.

Verification

  • cmake --preset windows-debug
  • cmake --build --preset build-debug
  • ctest --preset test-debug
  • cmake --build --preset build-debug --target safecrowd_app
  • cmake --build --preset build-no-app-debug
  • ctest --preset test-no-app-debug
  • Not run (reason below)

Risks / Follow-up

  • 전체 test-debug preset은 실행하지 않았습니다. 이번 검증은 앱 타겟 빌드와 현재 CI 경로에 가까운 no-app 테스트 preset에 집중했습니다.
  • 새 추천과 수동 route guidance 배치는 출구 기준으로 생성됩니다. 다만 이전에 저장된 시나리오에 통로/connection 기준 guidance가 있으면 persistence 경로를 통해 계속 로드될 수 있습니다.
  • #245는 위 Related Issue 설명처럼 일방통행 전환 추천을 의도적으로 막아 두었기 때문에 follow-up으로 남습니다.

@muzygosu muzygosu marked this pull request as ready for review May 17, 2026 10:41
@muzygosu muzygosu requested a review from learncold as a code owner May 17, 2026 10:41
@learncold learncold merged commit af2b259 into main May 18, 2026
7 checks passed
@learncold learncold deleted the codex/alternative-recommendations branch May 18, 2026 10:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants