Skip to content

[Domain] Refine fire smoke reaction behavior#229

Merged
learncold merged 2 commits into
mainfrom
codex/issue-222-fire-smoke-reaction
May 14, 2026
Merged

[Domain] Refine fire smoke reaction behavior#229
learncold merged 2 commits into
mainfrom
codex/issue-222-fire-smoke-reaction

Conversation

@SilverSupplier
Copy link
Copy Markdown
Collaborator

Summary

  • Consolidates fire/smoke hazard runtime rules in the domain layer and removes duplicated UI-side hazard logic.
  • Improves hazard-aware route scoring using actual route geometry and separates physical exposure from detection sensitivity.
  • Preserves open-ended hazard schedules in authoring UI and aligns smoke slowdown with Pathfinder visibility-based behavior as a partial Task-smoke / visibility / FED / FDS 연동 파이프라인 확장 #102 follow-up.

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
  • MSBuild safecrowd_app.vcxproj Debug x64
  • build/vs2022/windows-debug-no-app/Debug/safecrowd_tests.exe

Risks / Follow-up

@SilverSupplier SilverSupplier requested a review from learncold as a code owner May 12, 2026 16:43
@SilverSupplier SilverSupplier changed the base branch from main to codex/issue-226-fire-smoke-hazard-foundation May 12, 2026 17:07
Base automatically changed from codex/issue-226-fire-smoke-hazard-foundation to main May 14, 2026 04:35
@learncold learncold force-pushed the codex/issue-222-fire-smoke-reaction branch from 63d02c0 to aeec4ee Compare May 14, 2026 04:39
@learncold learncold merged commit c2149b6 into main May 14, 2026
2 checks passed
@learncold learncold deleted the codex/issue-222-fire-smoke-reaction branch May 14, 2026 04:40
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-Fire/Smoke crowd reaction behavior 구현

2 participants