Skip to content

probe(P-D4): refit the base rate inside each bootstrap draw; the post-hoc gate does not pass - #1725

Merged
AdaWorldAPI merged 1 commit into
mainfrom
lab/pd4-refit-base-rate-20261022
Oct 23, 2026
Merged

AdaWorldAPI merged 1 commit into
mainfrom
lab/pd4-refit-base-rate-20261022

Conversation

@AdaWorldAPI

Copy link
Copy Markdown
Owner

This PR fixes the codex P2 on #1724, which merged before the fix landed.

P-D4 scores abstentions at the base rate π. π is estimated from the same 40 blocks that the bootstrap resamples, but #1724 held it fixed across draws. That left π's own estimation error out of the interval.

With π refitted inside every draw:

Brier gain 95% interval
Before (π fixed) 0.0242 [0.0079, 0.0409]
After (π refitted) 0.0242 [−0.0062, 0.0552]

The interval now includes 0, so the kill condition fires: the post-hoc gate's gain is not established. P-D4 as a whole fails. The pre-registered margin was already vacuous, and the post-hoc arm does not rescue it.

Changes

  • crates/causal-edge/tests/p_d4_decision_gate_probe.rs:
    • adds base_rate_in, which refits π on the drawn blocks;
    • renames the test to post_hoc_gate_gain_is_not_established, which pins the failing result;
    • prints the fixed-π interval for comparison.
  • .claude/board/entries/2026-10-22-probes-pd4-pe2-pd3.md: corrects the status, the P-D4 result and the closeout line. The ablation gains also used fixed π and are marked as overstated.

The within-block selective-accuracy check does not use π and is unchanged: +0.120 [0.058, 0.185] on real labels, with an interval that includes 0 on shuffled labels. P-E2 and P-D3 are also unchanged.

Checks

  • Release tests pass; fmt and clippy are clean on this file.
  • Disable run: restoring fixed π makes the pinning test fail.
  • The entries index and the supersession index are current.

Generated by Claude Code

)

pi is estimated from the same blocks being resampled; holding it fixed left
its error out of the interval. Refitted: 0.0242 [-0.0062, 0.0552], so the
post-hoc gate does not clear the kill condition. The test now pins that, and
the board entry is corrected.

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

coderabbitai Bot commented Oct 22, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 55ad56f5-2285-463b-87f1-e1247c24f2e8
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Warning

Billing warning: we have not been able to collect payment for this subscription for more than 72 hours. Please update the payment method or pay any pending invoices in Billing to avoid service interruption.

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.

@chatgpt-codex-connector

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 🔄 Running since 2026-10-22T22:37:35.480105Z 57fbec4 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.

@AdaWorldAPI
AdaWorldAPI merged commit b342d47 into main Oct 23, 2026
12 checks passed
AdaWorldAPI added a commit that referenced this pull request Oct 23, 2026
…261023

board: catch-up for #1660–#1725 (PR_ARC, INTEGRATION_PLANS, LATEST_STATE)
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