docs(oatp): formalize the +BSA PSu,inf deferral (data-blocked) after closing the auto-ECM xfail#104
Conversation
…closing the auto-ECM xfail A 3-agent investigation of the diagnosis section 9 '+BSA / PSu,inf OATP' lever found the re-anchor already shipped (#66: abundance 5.0e5->1.3e5 onto non-holdout pitavastatin) and the lever headline-inert (DE-44). #103 closed the last loose end (test_predict_auto_ecm) via a stack-robust fold-error gate. The remaining +BSA PS reparameterization is now formally deferred: the per-substrate PSu,inf(+BSA)/Kp values are in the paywalled Li/Benet 2020 primary source (not to be fabricated), and it is near-zero-observable (the abundance re-anchor absorbs the PS change; pravastatin is validation-only) on top of headline-inert. Records this in the DE-44 status line and a 2026-07-06 experiment-log entry so it is a closed, documented decision rather than a dangling TODO.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 551b6ad86b
ℹ️ 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".
| - **+BSA PSu,inf formally deferred (DE-44):** the mechanistic upgrade (replace `hepatic_ecm.json` | ||
| seed PS midpoints with Li/Benet 2020 +BSA-derived PSu,inf, then re-derive abundance) is | ||
| **data-blocked** — the per-substrate PSu,inf/Kp values are in the paywalled AAPS J primary source | ||
| ([DOI](https://doi.org/10.1208/s12248-020-00528-y)), not to be fabricated — and near-zero-observable | ||
| (the abundance re-anchor absorbs the PS change; only *relative* statin shifts; pravastatin is | ||
| validation-only) on top of headline-inert. Deferred until the primary-source table is available. |
There was a problem hiding this comment.
Update the roadmap diagnosis when deferring the lever
This new entry formally defers the diagnosis §9 +BSA/OATP lever, but docs/research/diagnosis.md:101 still tells readers it is “the one un-foreclosed accuracy lever” and maps it to the FLUX-1 statin re-anchor. For anyone starting from the diagnosis/README roadmap after this commit, the project still presents the deferred, data-blocked work as active and actionable, so please update that section to match the DE-44 deferral status.
Useful? React with 👍 / 👎.
What
Formalizes the OATP1B1 ECM "+BSA PSu,inf" thread as a closed, documented decision.
Findings (3-agent investigation of the diagnosis §9 lever)
test_oatp_ecm_statins[pravastatin,pitavastatin]already passing.test_predict_auto_ecmhad two exact Cmax pins (0.0294/0.00116) stale under the old 5.0e5 abundance → replaced with the sibling's stack-robust fold-error gate and un-xfailed.hepatic_ecm.json+ abundance re-derivation) is data-blocked — the per-substrate PSu,inf/Kp values are in the paywalled AAPS J primary source (DOI), not to be fabricated — and near-zero-observable (the abundance re-anchor absorbs the PS change; pravastatin is the only holdout ECM drug and stays validation-only) on top of headline-inert.Changes (tracked)
dead-ends.mdDE-44: adds the 2026-07-06 status (re-anchor + xfail shipped; +BSA PS reparameterization deferred as data-blocked).experiment-log.md: 2026-07-06 entry recording the investigation, the test(ecm): close the FLUX-1 auto-ECM xfail with a stack-robust fold-error gate #103 xfail closure, and the deferral.(Local-only
docs/_internal/status.mdand the re-anchor spec were updated in the working tree to match; both are gitignored.)Headline 2.743 untouched. This is the third correctness-not-accuracy finding this cycle (after RBP honest-1.0 and the prospective re-score).