Skip to content

[Application] Improve project save/load result persistence#206

Merged
learncold merged 1 commit into
mainfrom
codex/application-save-load-persistence
May 9, 2026
Merged

[Application] Improve project save/load result persistence#206
learncold merged 1 commit into
mainfrom
codex/application-save-load-persistence

Conversation

@95x8x9
Copy link
Copy Markdown
Collaborator

@95x8x9 95x8x9 commented May 8, 2026

Summary

  • 프로젝트 저장 시 결과 화면의 분석 아티팩트(timingSummary 추가 필드, densitySummary, exitUsage, zoneCompletion, placementCompletion)를 함께 저장/복원하도록 확장했습니다.
  • 결과 화면에서 저장한 경우 다시 열 때 결과 화면으로 복원되도록 활성 화면 판별 순서를 수정했습니다.
  • 결과 화면의 마지막 결과 탭(Bottleneck, Hotspot, Zone, Groups)과 authoring 우측 패널 모드를 저장/복원하도록 보강했습니다.

Related Issue

  • None (application-only PR)

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
  • Not run (reason below)

Risks / Follow-up

  • baseline/alternative 결과를 여러 개 저장하고 비교하는 기능은 아직 포함하지 않았습니다.
  • authoring 우측 패널 상태는 저장되지만, run 전용 우측 패널 경험 자체는 별도 개선이 필요할 수 있습니다.

@95x8x9 95x8x9 marked this pull request as ready for review May 8, 2026 15:45
@95x8x9 95x8x9 requested a review from learncold as a code owner May 8, 2026 15:45
@95x8x9
Copy link
Copy Markdown
Collaborator Author

95x8x9 commented May 8, 2026

추가 커밋은 pr 199번과 겹치는 내용이라고 판단해 revert 처리했습니다.

@learncold learncold force-pushed the codex/application-save-load-persistence branch from 89d0eff to 51d5a28 Compare May 9, 2026 16:09
@learncold learncold merged commit ad9327b into main May 9, 2026
2 checks passed
@learncold learncold deleted the codex/application-save-load-persistence branch May 9, 2026 16:10
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