Skip to content

rig: run_all.sh 의 calibration phase 제거 — 값을 낸 적이 없고, 내던 값은 보정값이 못 되는 축 (#744 후속) - #751

Merged
Khyojae merged 1 commit into
mainfrom
rig/drop-calibration-phase
Sep 15, 2026
Merged

Khyojae merged 1 commit into
mainfrom
rig/drop-calibration-phase

Conversation

@Khyojae

@Khyojae Khyojae commented Sep 14, 2026

Copy link
Copy Markdown
Member

무엇

run_all.sh 의 calibration phase 를 지우고 박스 보정을 calibrate_box(이미 preflight·ai-venv bootstrap 에서 자동) 로 일원화한다. #744 이슈의 두 번째 안, #750 이 «결정은 사용자» 로 열어둔 것 — 2026-09-14 사용자 결정.

왜 고치지 않고 지우나

바뀐 것

파일 변경
loadtest/aws/run_all.sh CALIB_RIG·TIMEOUT_CALIB 변수, phase_calibration, 디스패치 삭제(−62). 옛 레시피의 PHASES 에 calibration 이 남아 있어도 ⏭ 한 줄 + PHASE_LOG 에 SKIP 행으로 받아 라운드가 안 막힌다. calibrate_box 머리 주석의 «어느 라운드에도 없다» 는 낡아서 «08-26 이전엔 없다 · cpu 축이 인용 축» 으로
loadtest/aws/README.md «보정값(從 R11)» 절을 calibrate_box 기준으로 재작성 — 산출물 두 자리(preflight calibration.tsv · ai-venv bootstrap → ai_venv_conditions.txt), 인용은 cpu 축, P6 도커 손 명령 불필요. 설정 표에서 두 변수 제거
loadtest/AWS-RIDE-ALONG.md R11 행 정정 — «phase 로 상설화(09-02)» 는 값을 낸 적이 없었다
docs/decisions/round-to-round-nonreproducibility.md §2 «R6 rig 1워커 판이 정확히 이것이다» 에 정정 블록 — 보정 정의 자체가 틀린 축이었다. #498 ③ 의 답이 여기

확인한 것 / 안 한 것

  • bash -n 통과. 실 EC2 재실행은 안 했다 — 지운 것이라, 다음 라운드에서 preflight 가 calibration.tsv 를 남기는지로 확인
  • 결과 디렉터리(loadtest/results/…)는 안 건드렸다 — 축 B rig 의 레시피에 calibration 이 적혀 있지만 그건 그때의 기록이고, 위 ⏭ 처리로 재사용해도 안 막힌다

머지 순서

🤖 Generated with Claude Code

https://claude.ai/code/session_016Ams2Xf6XxkBp8Qs7bUXuG

…값이 못 되는 축이다 (#744)

09-02 에 «상설화» 했다던 `calibration` phase(R6 rig 의 1워커 판 → scaling_1w.txt)는
저장소 루트 cwd 에서 `app.core` 를 못 찾아 어느 라운드에서도 값을 낸 적이 없다 —
loadtest/results/ 어디에도 scaling_1w.txt 가 없다(#750 이 확인). 從이라 라운드를 안
막았고 ⚠️ 한 줄만 남겨서 아무도 몰랐다.

고치는 대신 지운다. 고쳐도 그 phase 가 내는 값(1스레드 추론 fps)은 calibrate_box 의
infer 축과 같은 종류인데, 축 A(08-26)·축 B(09-14) 실측이 «그 축은 보정값이 못 된다»
고 답했다 — 처리량비를 설명하는 건 파이썬 루프 원속도(cpu 축, 3%p 안)이고 infer 축은
같은 부팅 안에서도 5~12% 흔들린다. 보정값은 이미 preflight 의 calibrate_box 와
ROLE=ai-venv bootstrap 이 calibration.tsv 로 남기고 있다.

  run_all.sh      CALIB_RIG·TIMEOUT_CALIB 변수, phase_calibration 함수, 디스패치 삭제.
                  옛 레시피(축 B rig 등)의 PHASES 에 `calibration` 이 남아 있어도 라운드가
                  안 막히게 ⏭ 한 줄 + PHASE_LOG SKIP 행으로 받는다
  aws/README.md   «보정값(從 R11)» 절을 calibrate_box 기준으로 다시 씀 — 산출물 두 자리,
                  cpu 축이 인용 축, P6 도커 손 명령은 더는 불필요. 설정 표에서 두 변수 제거
  AWS-RIDE-ALONG  R11 행 정정 — «phase 로 상설화(09-02)» 는 값을 낸 적이 없었다
  설계 §2         «R6 rig 1워커 판이 정확히 이것이다» 문단에 정정 블록 — 보정 정의 자체가
                  틀린 축이었다. #498 ③(R6 79.9 fps 를 왜 못 쓰나)의 이유가 여기 있다

bash -n 통과. 실 EC2 는 안 돌렸다 — 지운 것이라 다음 라운드의 PHASE_LOG 에 `calibration
SKIP` 이 찍히는지(옛 레시피 사용 시)와 calibration.tsv 가 preflight 에서 생기는지로 확인.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016Ams2Xf6XxkBp8Qs7bUXuG
@coderabbitai

coderabbitai Bot commented Sep 14, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 25 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: b316ef54-1587-465a-9950-82983e7970f2

📥 Commits

Reviewing files that changed from the base of the PR and between 0014100 and 6465dbd.

📒 Files selected for processing (4)
  • docs/decisions/round-to-round-nonreproducibility.md
  • loadtest/AWS-RIDE-ALONG.md
  • loadtest/aws/README.md
  • loadtest/aws/run_all.sh

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Khyojae

Khyojae commented Sep 14, 2026

Copy link
Copy Markdown
Member Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Sep 14, 2026 •

Copy link
Copy Markdown
⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@Khyojae
Khyojae merged commit e4d5015 into main Sep 15, 2026
3 checks passed
@Khyojae
Khyojae deleted the rig/drop-calibration-phase branch September 15, 2026 06:08
Khyojae added a commit that referenced this pull request Sep 15, 2026
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