Skip to content

[Application] default result overlay to peak density#271

Merged
learncold merged 1 commit into
mainfrom
application/default-peak-density-overlay
Jun 4, 2026
Merged

[Application] default result overlay to peak density#271
learncold merged 1 commit into
mainfrom
application/default-peak-density-overlay

Conversation

@SilverSupplier

Copy link
Copy Markdown
Collaborator

Summary

  • Default the results screen map overlay to Peak Density (previously Occupancy) and relabel the option "Peak Density", so the first results view matches the documented behavior.
  • The remaining overlay options (Occupancy, Pressure, Hotspots, Bottlenecks, Cross Flow, None) are unchanged.

Area

  • Engine
  • Domain
  • Application

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)

Built the Qt app in release: cmake --preset windows-release + cmake --build --preset build-release --target safecrowd_app -> succeeds. The change is a default selection and a label only.

Risks / Follow-up

  • UI default only; no change to overlay computation or logic.

@SilverSupplier SilverSupplier requested a review from learncold as a code owner June 2, 2026 19:26
@learncold learncold merged commit 2b85a6a into main Jun 4, 2026
2 checks passed
@learncold learncold deleted the application/default-peak-density-overlay branch June 4, 2026 15:33
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