Skip to content
This repository was archived by the owner on Aug 3, 2026. It is now read-only.

validator: reject cleanly when op4 eval fails (no scoring crash) - #64

Merged
bitzic merged 1 commit into
mainfrom
fix/op4-crash
Jun 26, 2026
Merged

validator: reject cleanly when op4 eval fails (no scoring crash)#64
bitzic merged 1 commit into
mainfrom
fix/op4-crash

Conversation

@bitzic

@bitzic bitzic commented Jun 26, 2026

Copy link
Copy Markdown
Contributor
  • op4 returns (False, detail, None) when a checkpoint won't load into RalphBase and the patched-workdir re-eval also fails; judge_submission stored hidden_eval=None and returned a "passing" result
  • score_and_decide then dereferenced None.val_bpb -> AttributeError that took down the whole epoch loop (a DoS surface for any unloadable ckpt)
  • mirror op1-op3: on op4 failure set result.rejected and return
  • score_and_decide: defense-in-depth None-guard before .val_bpb
  • test: op4 failure rejects cleanly with reason op4_hidden_eval

- op4 returns (False, detail, None) when a checkpoint won't load into
  RalphBase and the patched-workdir re-eval also fails; judge_submission
  stored hidden_eval=None and returned a "passing" result
- score_and_decide then dereferenced None.val_bpb -> AttributeError that
  took down the whole epoch loop (a DoS surface for any unloadable ckpt)
- mirror op1-op3: on op4 failure set result.rejected and return
- score_and_decide: defense-in-depth None-guard before .val_bpb
- test: op4 failure rejects cleanly with reason op4_hidden_eval

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@bitzic
bitzic merged commit 77f79f7 into main Jun 26, 2026
4 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant