Repository navigation
Clarify Dehalococcoides recipient processes and conditional cofactor support (batch 30) - #1387
realmarcin wants to merge 1 commit into
Conversation
realmarcin
left a comment
There was a problem hiding this comment.
Source-based self-adversarial review at exact head 5988d5a; not independent approval.
Addressed #1382, #1383 and #1384 after reading the actual diffs against primary sources at the documented access scope. All ten nodes and seven arrows remain. Recipient capabilities are source-only, not reverse/duplicate nutrient transfers. Sulfide toxicity under donor excess is separated from competition and other community experiments. All corrinoid-triculture arrows expose external DMB; donor cofactor synthesis and recipient remodeling are not falsely made exclusive. Native factor-III insufficiency is not active inhibition.
348 passed in 189.71s (0:03:09); six new tests; schema, strict, terms, reference, independent snippet and history checks passed. Zero structural errors, all 31 prior warnings unchanged. Exact participant registry 173 to 174, measured connectivity unchanged, no numeric bounds or algorithms altered. Deterministic rendering, exact descriptions across 456 pages and actual post-commit docs-current passed.
Three guarded writes/histories; 457 other records and 920 caches unchanged. Source-only and partial-full-text limits are explicit, as are uninspected figures/supplements. No new article bodies/PDFs or paid research. Metadata #1385/#1386 remains open, not certified by graph curation.
Decisions, validation and self-adversarial output are committed. Protected CI/main merge and full-corpus completion are not claimed; preserve the dependency stack until main merges.
Network integrity findingsWarnings only — a member with no interaction yet, or a participant matched by ontology id rather than by name, or one on a community-level interaction that resolves to no member. Reported, but does not fail the build. The full report is attached to the workflow run as an artifact. |
a4ca06d to
a6e0370
Compare
5988d5a to
3b4f54d
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head 3b4f54dd705d288c37e52ca173f372a8cc8572dc
after protected main merge of #1312. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 4b39c126a27eb25a16ef82af9440641f55832cd0.
The batch-owned binary patch is byte-identical
(SHA-256 4ea591fb96bd6c6a83000c118387a9cef73a4af48e73b12faaf37f71118d7d47); only ancestry changed.
Scientific records, caches, source-review output, histories, regression tests, and generated pages
are unchanged. Historical validation results and recorded source/access limitations therefore
remain applicable, but required CI must pass at this new head. No new source review or local test
rerun is claimed by this ancestry-only update.
The protected main queue remains mandatory. Keep dependents on their reviewed parent until that
parent is confirmed merged; move dependents before deleting merged branches.
3b4f54d to
6f7947a
Compare
a6e0370 to
61b3033
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head 6f7947a8bdf7f2d7491b551f88a63911eb00e0cd
after protected main merge of #1315. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 4b39c126a27eb25a16ef82af9440641f55832cd0.
The batch-owned binary patch is byte-identical
(SHA-256 4ea591fb96bd6c6a83000c118387a9cef73a4af48e73b12faaf37f71118d7d47); only ancestry changed.
Scientific records, caches, source-review output, histories, regression tests, and generated pages
are unchanged. Historical validation results and recorded source/access limitations therefore
remain applicable, but required CI must pass at this new head. No new source review or local test
rerun is claimed by this ancestry-only update.
The protected main queue remains mandatory. Keep dependents on their reviewed parent until that
parent is confirmed merged; move dependents before deleting merged branches.
6f7947a to
d11c413
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head d11c413201d5f4389312aa2f0470414f209605ea
after protected main merge of #1318. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 4b39c126a27eb25a16ef82af9440641f55832cd0.
The batch-owned binary patch is byte-identical
(SHA-256 4ea591fb96bd6c6a83000c118387a9cef73a4af48e73b12faaf37f71118d7d47); only ancestry changed.
Scientific records, caches, source-review output, histories, regression tests, and generated pages
are unchanged. Historical validation results and recorded source/access limitations therefore
remain applicable, but required CI must pass at this new head. No new source review or local test
rerun is claimed by this ancestry-only update.
The protected main queue remains mandatory. Keep dependents on their reviewed parent until that
parent is confirmed merged; move dependents before deleting merged branches.
1f1d468 to
6cde012
Compare
d11c413 to
1d413bf
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head 1d413bf389704b17dfc67af616c167a53cad8f3f
after protected main merge of #1323. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 4b39c126a27eb25a16ef82af9440641f55832cd0.
The batch-owned binary patch is byte-identical
(SHA-256 4ea591fb96bd6c6a83000c118387a9cef73a4af48e73b12faaf37f71118d7d47); only ancestry changed.
Scientific records, caches, source-review output, histories, regression tests, and generated pages
are unchanged. Historical validation results and recorded source/access limitations therefore
remain applicable, but required CI must pass at this new head. No new source review or local test
rerun is claimed by this ancestry-only update.
The protected main queue remains mandatory. Keep dependents on their reviewed parent until that
parent is confirmed merged; move dependents before deleting merged branches.
6cde012 to
655dd72
Compare
1d413bf to
0248e72
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head 0248e7251ab60e600718774ccbb00f05a9ba003d
after protected main merge of #1328. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 4b39c126a27eb25a16ef82af9440641f55832cd0.
The batch-owned binary patch is byte-identical
(SHA-256 4ea591fb96bd6c6a83000c118387a9cef73a4af48e73b12faaf37f71118d7d47); only ancestry changed.
Scientific records, caches, source-review output, histories, regression tests, and generated pages
are unchanged. Historical validation results and recorded source/access limitations therefore
remain applicable, but required CI must pass at this new head. No new source review or local test
rerun is claimed by this ancestry-only update.
The protected main queue remains mandatory. Keep dependents on their reviewed parent until that
parent is confirmed merged; move dependents before deleting merged branches.
0248e72 to
6006515
Compare
655dd72 to
2be0337
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head 6006515a3a03000098bdbd9c4c912fc9377dd3ed
after protected main merge of #1334. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 4b39c126a27eb25a16ef82af9440641f55832cd0.
The batch-owned binary patch is byte-identical
(SHA-256 4ea591fb96bd6c6a83000c118387a9cef73a4af48e73b12faaf37f71118d7d47); only ancestry changed.
Scientific records, caches, source-review output, histories, regression tests, and generated pages
are unchanged. Historical validation results and recorded source/access limitations therefore
remain applicable, but required CI must pass at this new head. No new source review or local test
rerun is claimed by this ancestry-only update.
The protected main queue remains mandatory. Keep dependents on their reviewed parent until that
parent is confirmed merged; move dependents before deleting merged branches.
6006515 to
9d2eceb
Compare
2be0337 to
7798bc8
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head 9d2eceb63c68ef9506b6ce8c8479a348eaa9facf
after protected main merge of #1339. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 4b39c126a27eb25a16ef82af9440641f55832cd0.
The batch-owned binary patch is byte-identical
(SHA-256 4ea591fb96bd6c6a83000c118387a9cef73a4af48e73b12faaf37f71118d7d47); only ancestry changed.
Scientific records, caches, source-review output, histories, regression tests, and generated pages
are unchanged. Historical validation results and recorded source/access limitations therefore
remain applicable, but required CI must pass at this new head. No new source review or local test
rerun is claimed by this ancestry-only update.
The protected main queue remains mandatory. Keep dependents on their reviewed parent until that
parent is confirmed merged; move dependents before deleting merged branches.
7798bc8 to
4ff3fa1
Compare
9d2eceb to
3680053
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head 3680053b918391ca1939c3f4066a278bea439cc6
after protected main merge of #1346. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 4b39c126a27eb25a16ef82af9440641f55832cd0.
The batch-owned binary patch is byte-identical
(SHA-256 4ea591fb96bd6c6a83000c118387a9cef73a4af48e73b12faaf37f71118d7d47); only ancestry changed.
Scientific records, caches, source-review output, histories, regression tests, and generated pages
are unchanged. Historical validation results and recorded source/access limitations therefore
remain applicable, but required CI must pass at this new head. No new source review or local test
rerun is claimed by this ancestry-only update.
The protected main queue remains mandatory. Keep dependents on their reviewed parent until that
parent is confirmed merged; move dependents before deleting merged branches.
4ff3fa1 to
7a2c786
Compare
3680053 to
20868a1
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head 20868a1cf45a111ecc94b5b9e6817ceb4bbe7f65
after protected main merge of #1350. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 4b39c126a27eb25a16ef82af9440641f55832cd0.
The batch-owned binary patch is byte-identical
(SHA-256 4ea591fb96bd6c6a83000c118387a9cef73a4af48e73b12faaf37f71118d7d47); only ancestry changed.
Scientific records, caches, source-review output, histories, regression tests, and generated pages
are unchanged. Historical validation results and recorded source/access limitations therefore
remain applicable, but required CI must pass at this new head. No new source review or local test
rerun is claimed by this ancestry-only update.
The protected main queue remains mandatory. Keep dependents on their reviewed parent until that
parent is confirmed merged; move dependents before deleting merged branches.
20868a1 to
c34b76c
Compare
7a2c786 to
91af21e
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head c34b76ca31173f09b929082b52cf85b9381d8325
after protected main merge of #1355. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 4b39c126a27eb25a16ef82af9440641f55832cd0.
The batch-owned binary patch is byte-identical
(SHA-256 4ea591fb96bd6c6a83000c118387a9cef73a4af48e73b12faaf37f71118d7d47); only ancestry changed.
Scientific records, caches, source-review output, histories, regression tests, and generated pages
are unchanged. Historical validation results and recorded source/access limitations therefore
remain applicable, but required CI must pass at this new head. No new source review or local test
rerun is claimed by this ancestry-only update.
The protected main queue remains mandatory. Keep dependents on their reviewed parent until that
parent is confirmed merged; move dependents before deleting merged branches.
c34b76c to
a66f315
Compare
91af21e to
902c727
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial integration review at exact head a66f315ccdacde92979962e77342df1414f896a2. Not independent approval.
Integrated already-merged schema prerequisite #1692 (67603e192525899fae75420e2c19e8212b16afc9). Compared with the prior reviewed head c34b76ca31173f09b929082b52cf85b9381d8325, the only changes are the canonical schema pin, shared schema and generated datamodel from that main commit. Every other tracked file is identical; the batch-owned binary patch remains byte-identical (SHA-256 4ea591fb96bd6c6a83000c118387a9cef73a4af48e73b12faaf37f71118d7d47). All 60 published batch patches were checked this way. No scientific assertion, reference cache, history, decision ledger or generated page was edited by this integration.
Adversarial checks: no inherited schema-file edits lost; generated CrossCorpusLink remains standalone and does not add a causal claim; normal strict validation passed all 460 records; required CrossCorpusLink fields enforced. Fresh broad regression runs on both ends of the stack: 302 passed in 172.77s (0:02:52); 756 passed in 255.55s (0:04:15). Docs-current passed clean at batches25/84, and vendored-sync passed. The 58 intermediate combinations were not independently retested. Existing primary-source review limitations and unresolved research issues remain applicable; historic validation receipts are not rewritten as new scientific reviews.
An additional generated-datamodel loading check FAILED and is retained in the integration receipt. Differential testing found exactly the same failures on all 460 paths before and after this integration:417 unhashable TaxonDescriptor and43 unhashable CultureCollectionEnum exceptions, with linkml-runtime1.11.1. Existing tests explicitly use plain YAML instead of this loader. Pre-existing defect #1703 remains OPEN; no datamodel-loader success is claimed. Normal strict schema validation is a separate passing path, not a relabeling of that failure.
Required CI must pass on this exact head and the protected merge-queue candidate. No manual workflow dispatch, artificial status, administrative merge or protection bypass. Dependents stay draft until their parent is confirmed merged; move dependents before branch deletion.
a66f315 to
be022f6
Compare
902c727 to
a84f68f
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head be022f67e3b3bcca237d2561f067cb0d4c1c52bd
after protected main merge of #1361. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 8e621c68a314607021c1c344efe588197b89085f.
The batch-owned binary patch is byte-identical
(SHA-256 4ea591fb96bd6c6a83000c118387a9cef73a4af48e73b12faaf37f71118d7d47); only ancestry changed.
Scientific records, caches, source-review output, histories, regression tests, and generated pages
are unchanged. Historical validation results and recorded source/access limitations therefore
remain applicable, but required CI must pass at this new head. No new source review or local test
rerun is claimed by this ancestry-only update.
The protected main queue remains mandatory. Keep dependents on their reviewed parent until that
parent is confirmed merged; move dependents before deleting merged branches.
be022f6 to
4e0b687
Compare
a84f68f to
46ca547
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial governance integration review at exact head 4e0b6870cc9942383dd90b71c138adb1cedfed58. This is a COMMENT, not independent approval.
Integrated already-merged main PR #1730 (fe1752572a688051e8105e5b89d3099fe2202c63). Compared with prior reviewed head be022f67e3b3bcca237d2561f067cb0d4c1c52bd, exactly two files changed: scripts/.vendored_canon_ref and tests/test_id_label_empty_adapter.py. Each resulting tree differs from its old tree by the exact binary patch from main, and each batch-owned binary patch remains byte-identical (SHA-256 4ea591fb96bd6c6a83000c118387a9cef73a4af48e73b12faaf37f71118d7d47). These invariants passed on all 74 published branches. Original heads are retained in local archive refs/archive/communitymech-pre-governance-20261007. No scientific record, source cache, history, decision ledger, schema, runtime code or generated page changed in this integration.
Adversarial inspection: the adapter regression still checks the empty sentinel and case-insensitive caching, now with an isolated fake oaklib module and a one-call selector assertion. No validation threshold or runtime behavior was relaxed. Fresh regressions at both stack ends: 337 passed in 214.19s (0:03:34); 917 passed in 402.58s (0:06:42). Docs-current and pinned vendored-sync passed at batches26/99. The 72 intermediate combinations were checked by tree/patch hashes, not independently retested. Historical source-review limitations and unresolved issues, including loader #1703 and cache #1091, remain applicable. This is not new paper review, independent approval, or a claim that those outstanding issues are fixed.
Digest-bound integration receipt/logs: /private/tmp/communitymech-governance-integration-20261007/receipt.json. Prior output/review records in reports/causal_graph_review/ remain unchanged. The prior lifecycle receipt is archived and will be updated only after all exact-head comments are verified.
Required CI and independent review must pass for the exact head and protected merge-queue candidate. No CI rerun, manual dispatch, fabricated status, admin merge or protection bypass. Dependents remain draft; move them before deleting a merged parent branch.
46ca547 to
bb47fac
Compare
4e0b687 to
85860d0
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Exact-head self-adversarial restack review at 85860d0d6b56b133ce2b2b9032f21af5e0953e0b.
COMMENT, not independent approval.
The batch-owned binary patch remains byte-identical:
SHA-256 4ea591fb96bd6c6a83000c118387a9cef73a4af48e73b12faaf37f71118d7d47.
Tree f7d3dd6a38f1634527861d8aeee853c3656f5696 matches the independently computed clean merge.
Compared with historical tree 112cf985a4db6e00dc5fe18fd166738aa541f043, only scripts/.vendored_canon_ref
changes, incorporating main PR #1834's pin 849f336e025510316a5f235eb0af8547b8bd50cc.
All curated records, source caches, review outputs, histories, regression tests, and rendered
pages remain unchanged. No new primary-source review or local scientific-test rerun is claimed.
This PR is now part of native stack #1973, rooted at #1363 on main. No protected merge
has occurred. Fresh eligible PR/merge-group CI remains mandatory. No rule bypass, fabricated
status, merge into a topic parent, or premature branch deletion.
85860d0 to
df0fe62
Compare
|
Superseded by the protected cumulative merge #1975 at Patch-inclusion receipt. Existing issue-closing links were transferred to #1975; unresolved research and non-graph gaps are not closed by this action. This PR is being closed as superseded, not represented as individually merged. Archival provenance tags retain source commits; cleanup removes branch refs only and preserves worktree files. |
Scope
Three complete record-level graph reviews: Dhc195/DvH lactate coculture, Dhc195/DvH/PfR7 triculture and BAV1/Fusaro cofactor coculture. Corpus: 135 reviewed / 321 pending / 4 needs_research across all 460 records in both roots.
Closes #1382. Closes #1383. Closes #1384. Separate non-graph follow-ups #1385 and #1386 remain open.
Changes And Adversarial Review
Preserve all ten nodes and seven arrows. Assign recipient dechlorination and remodeling to the recipient instead of duplicate or reversed nutrient transfers. Separate sulfide-associated inhibition under donor excess from hydrogen competition and from donor-limited butyrate-triculture experiments. The roughly threefold rate comparison is not generalized to cell yield.
Make external DMB a coinput throughout the corrinoid triculture. Preserve positive hydrogen and corrinoid transfer, donor cobalamin production as a possible contribution, and recipient remodeling without an exclusive-route claim. Classify measured transcript evidence as IN_VITRO, not merely computational; induction is not proof of gene necessity.
For BAV1/Fusaro preserve positive DMB-associated support and negative native-cobamide compatibility. A missing adequate cofactor is not active factor-III toxicity, and BAV1 activity is not reverse cross-feeding. Distinct strain and donor results are not pooled.
Verification
348 passed in 189.71s (0:03:09); six new focused tests. Schema, strict, term, reference, independent snippet and history checks passed. Raw reference diagnostics and actual snippet counts are recorded separately. Exact targets, one rename and discussion anchors checked.
Zero structural errors; all 31 prior network warnings unchanged. Measured connectivity survey unchanged. Exact participant registry 173 to 174 users, only the corrinoid triculture added; no numeric bounds or algorithms changed.
Three guarded edits, llm-assisted events, append-only histories and regenerated pages. All non-graph raw blocks, 457 other records and 920 reference caches are byte-preserved. Repeat rendering is deterministic and exact descriptions are checked on all 456 pages. Actual post-commit docs-current passed with a clean tree. No screenshot QA is claimed.
Committed decision, validation and adversarial-review records are under reports/causal_graph_review, dated 20261006, batch30. This is source-based self-adversarial review, not independent approval. Source access is explicit per paper: 2014/2017 main text read, 2012 abstract-bounded, 2013 abstract plus partial indexed primary excerpts. No figure images or supplements inspected; no new article body/PDF or Edison job.
Stacking
Draft dependency on #1381. Do not merge into the topic parent. Require exact-head protected main checks/queue after parents merge; move dependents before branch deletion. This batch does not complete the full-corpus goal.