Skip to content

[Application] show hazard exposure results#266

Open
SilverSupplier wants to merge 4 commits into
mainfrom
codex/hazard-exposure-results
Open

[Application] show hazard exposure results#266
SilverSupplier wants to merge 4 commits into
mainfrom
codex/hazard-exposure-results

Conversation

@SilverSupplier
Copy link
Copy Markdown
Collaborator

@SilverSupplier SilverSupplier commented May 23, 2026

Summary

  • Adds hazard exposure result summaries for fire and smoke in single-scenario result cards, graph tabs, and navigation panels.
  • Adds batch comparison exposure tables and selected-scenario exposure details.
  • Persists hazard exposure result artifacts and covers project save/load round-trip behavior.

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 (not rerun; existing configured build tree reused)
  • cmake --build --preset build-debug
  • ctest --preset test-debug
  • cmake --build --preset build-no-app-debug
  • ctest --preset test-no-app-debug
  • Not run (reason below)

Risks / Follow-up

  • UI rendering was checked through the local app build path, but there is no automated Qt screenshot test for the new Exposure tabs yet.
  • Pressure result persistence was split into PR [Application] persist pressure result artifacts #268 to keep this PR focused on hazard exposure display and persistence.
  • Integration note: this PR overlaps with open PR [Domain] 운영 갈등 지표 및 결과 오버레이 구현 #267 in result navigation, workspace state, and result widget code paths. This PR intentionally does not resolve the cross-PR integration conflict. The PR that is merged later should rebase and reconcile src/application/ProjectWorkspaceState.h, src/application/ScenarioResultNavigation.cpp, and src/application/ScenarioResultWidget.cpp. Cross-PR UI integration should be handled by the later PR owner.

@SilverSupplier SilverSupplier force-pushed the codex/hazard-exposure-results branch from 49cbff7 to d21c8e9 Compare May 23, 2026 20:44
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.

1 participant