Skip to content

Create successor pre-registration for item 3 (#59) - #63

Merged
InauguralPhysicist merged 2 commits into
mainfrom
cloud-59
Oct 28, 2026
Merged

InauguralPhysicist merged 2 commits into
mainfrom
cloud-59

Conversation

@InauguralPhysicist

Copy link
Copy Markdown
Member

Closes #59

Built by a Codex Cloud task: https://chatgpt.com/codex/tasks/task_e_6ae2567843c8832da67a4f01afdb1d1f. Its red/green and gate evidence is in the task log.

@InauguralPhysicist

Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 28, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-28T20:59:30.126405Z 891f451 Manual request
🔒 Security Review ✅ Completed 2026-10-28T20:59:43.434734Z 891f451 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

Under the successor rule all 35 frozen-rule settled reverse runs decay to the
origin, so that direction has denominator 0 and gives no positive evidence.
The controls cover three of four directions; the record now says so instead
of leaving it implicit in the oracle list.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01KC99CmwatKssQYggkBCgwF

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 891f451268

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread prereg/59/check.py
Comment on lines +29 to +30
digest = hashlib.sha256(new.read_bytes()).hexdigest()
print(f"PASS: successor table sha256={digest}")

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Verify the successor table against its registered hash

If oracle_settled.csv is regenerated with different denominators or accidentally modified, this acceptance check still prints PASS and exits successfully: it computes the digest but never compares it with oracle_settled.family_b_table_sha256 in manifest.json. Since this table freezes the successor experiment's coverage denominators, accepting a changed table silently changes the retention gate. Load the registered digest and assert equality before reporting success.

Useful? React with 👍 / 👎.

@InauguralPhysicist
InauguralPhysicist merged commit 731a937 into main Oct 28, 2026
1 check passed
@InauguralPhysicist
InauguralPhysicist deleted the cloud-59 branch October 28, 2026 21:03
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.

#55 item 3: the oracle-settled rule counts origin-decayed runs as settled, so neither its coverage floor nor its positive controls can pass

1 participant