Skip to content

[Application] Seek replay to exact T90/T95#181

Merged
learncold merged 3 commits into
mainfrom
codex/t90-t95-keyframe-seek
May 5, 2026
Merged

[Application] Seek replay to exact T90/T95#181
learncold merged 3 commits into
mainfrom
codex/t90-t95-keyframe-seek

Conversation

@95x8x9
Copy link
Copy Markdown
Collaborator

@95x8x9 95x8x9 commented May 4, 2026

Summary

  • t90, t95 시점을 그래프에서 클릭해 리플레이를 해당 시점으로 이동(상태는 pause)
  • T90/T95 키프레임을 evacuation 전환 시점에서 캡처해 시간/상태 오차 제거(기존 bottleneck detectionFrame 저장 방식과 동일)

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
  • cmake --build --preset build-no-app-debug --target safecrowd_tests
  • ctest --preset test-no-app-debug

Risks / Follow-up

  • None

@95x8x9 95x8x9 requested a review from learncold as a code owner May 4, 2026 16:14
Remove specific directories from .gitignore
Copy link
Copy Markdown
Owner

@learncold learncold left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

T90/T95 클릭 replay 기능의 방향은 맞지만, 현재 키프레임 저장 시점 때문에 '정확한 T90/T95' 상태를 보장하지 못합니다. 아래 코멘트의 frame capture 위치를 수정한 뒤 다시 확인하는 것이 좋겠습니다.

Comment thread src/domain/ScenarioSimulationSystems.cpp Outdated
@95x8x9
Copy link
Copy Markdown
Collaborator Author

95x8x9 commented May 5, 2026

리뷰 코멘트 반영했습니다.\n\n- T90/T95 키프레임 캡처를 RenderSync(ScenarioFrameSyncSystem)에서 제거하고, evac 완료 전환 감지 시점(ScenarioSimulationMotionSystem)에서 바로 frame을 캡처하도록 변경했습니다.

  • elapsedSeconds만 덮어쓰는 방식은 제거했고, bottleneck/hotspot의 detectionFrame 저장 방식과 동일하게 '해당 시점의 상태+시간'을 같이 저장합니다.
  • 변경 파일: src/domain/ScenarioSimulationMotionSystem.cpp, src/domain/ScenarioSimulationSystems.cpp, src/domain/ScenarioSimulationSystems.h

Copy link
Copy Markdown
Owner

@learncold learncold left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수정 반영 확인했습니다. T90/T95 키프레임을 evacuation 완료 전환 감지 시점에서 캡처하도록 변경되어, 기존에 지적한 시간/상태 불일치 문제가 해결됐습니다.

확인:

  • cmake --preset windows-debug-no-app
  • cmake --build --preset build-no-app-debug --target safecrowd_tests
  • ctest --preset test-no-app-debug

@learncold learncold merged commit 622dace into main May 5, 2026
4 checks passed
@learncold learncold deleted the codex/t90-t95-keyframe-seek branch May 5, 2026 15:22
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.

[Domain] T90/T95 시점 키프레임 저장으로 리플레이 seek 기능 추가

2 participants