Repository navigation
Bound EcoFAB and E. coli causal graphs to tested mechanisms (batch 36) - #1430
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 a691e1d; not independent approval.
Addressed #1424/#1425/#1426 after primary-text and complete diff review.
Nine nodes retained; one out-of-system node and two unsupported arrows removed. No added causal structure or Edison job/charge.
EcoFAB retains three whole-treatment outcome arrows. Separate sterile-plant nitrogen experiments do not belong in the SynCom17 graph. OAS925 isolate traits are intrinsic, with a HYPOTHESIZED contribution to relative dominance. Remove dominance-to-metabolome mediation and categorical facilitation, competition and cross-feeding; bulk profiles do not resolve donor/recipient flux.
HMO reciprocal exchange remains supported for K12 and an infant B. bifidum isolate under nutrient limitation. Preserve mutant/viability controls and chromatographic fucose/galactose evidence; spent-medium growth required pH readjustment to 6.3. Return-to-cysteine maintenance is PARTIAL. Remove the part/whole mutualism-summary arrow; do not claim host abundance regulation or a resolved cysteine release route.
GL10/XL12 retains a single node and zero arrows. Substrate specialization overlaps, not exclusive sugar use or direct interstrain exchange. Both strain labels remain distinct despite their shared taxon ID. The abstract and Results xylose endpoints remain explicitly unreconciled; the existing cross-feeding gap and cache-separation safeguards remain intact.
392 passed in 179.44s (0:02:59); eight new regression tests. Focused schema, strict, term, independent snippet and history checks passed. Three reference-validator commands exit 0 but perform zero checks: a no-op, not evidence validation. Textual matches are not scientific support.
Zero structural errors; all 33 baseline network warnings preserved exactly, none added. The 183-user participant registry and numeric bounds are unchanged; OAS925 intrinsic representation changes mixed scope 122 to 123, community-only 265 to 264, and sole broad credit 1222 to 1221.
Three guarded writes and append-only histories; non-graph raw blocks, 457 other records and all 920 cache files unchanged. Repeat renders deterministic with exact descriptions checked across 456 pages. Actual post-commit docs-current passed with clean tree.
Committed decisions, actual validation output (including baseline) and self-adversarial review are under reports/causal_graph_review, dated 20261006, batch36. Four cached primary main-body texts reviewed at recorded scope; no visual figures, separate supplements, raw-data or browser review claimed. This is source-based self-adversarial review, not independent approval.
#1427/#1428/#1429 remain OPEN. Protected main merge and full-corpus completion are not asserted.
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. |
54ce25c to
caa8c96
Compare
a691e1d to
8f192ba
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head 8f192ba2038bd1dd5986b448ffb33892a8526995
after protected main merge of #1315. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 2d768412771a01f162425275107046bb75a2e95c.
The batch-owned binary patch is byte-identical
(SHA-256 609c412592075a60f9e861d17e6a611ec2f9982a1f197a9b77f691f485507f6c); 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.
caa8c96 to
f45b2af
Compare
8f192ba to
4e90cf3
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head 4e90cf3cae68b86b0f7cc489b2600efc9fac0141
after protected main merge of #1318. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 2d768412771a01f162425275107046bb75a2e95c.
The batch-owned binary patch is byte-identical
(SHA-256 609c412592075a60f9e861d17e6a611ec2f9982a1f197a9b77f691f485507f6c); 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.
|
Reference-counter correction (#1469) The earlier statement that linkml-reference-validator 0.1.7's The source-backed correction and historical candidate inventory are committed in #1470 at The new Fucoidan batch separately retains 14 baseline reference-CLI failures and 15 baseline snippet-audit mismatches for supplement-sidecar excerpts. Those are not failures attributed to this older PR. This correction is self-review, not independent approval or merge authorization beyond the existing protected workflow. |
f45b2af to
e6e6da2
Compare
4e90cf3 to
17fc4f2
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head 17fc4f2c903617109ea1baa3fc3117935e73d03a
after protected main merge of #1323. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 2d768412771a01f162425275107046bb75a2e95c.
The batch-owned binary patch is byte-identical
(SHA-256 609c412592075a60f9e861d17e6a611ec2f9982a1f197a9b77f691f485507f6c); 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.
17fc4f2 to
f03e17a
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head f03e17a7567581a84a6c4cd7ddf4cb8c8836d8c5
after protected main merge of #1328. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 2d768412771a01f162425275107046bb75a2e95c.
The batch-owned binary patch is byte-identical
(SHA-256 609c412592075a60f9e861d17e6a611ec2f9982a1f197a9b77f691f485507f6c); 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.
4565d99 to
d00d603
Compare
f03e17a to
52b818a
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head 52b818adda430dd9cbdeca17a9dea5b9b885ed75
after protected main merge of #1334. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 2d768412771a01f162425275107046bb75a2e95c.
The batch-owned binary patch is byte-identical
(SHA-256 609c412592075a60f9e861d17e6a611ec2f9982a1f197a9b77f691f485507f6c); 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.
d00d603 to
c8d24c1
Compare
52b818a to
1498a92
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head 1498a920404f7bb38e51c81279f128b9f22033dc
after protected main merge of #1339. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 2d768412771a01f162425275107046bb75a2e95c.
The batch-owned binary patch is byte-identical
(SHA-256 609c412592075a60f9e861d17e6a611ec2f9982a1f197a9b77f691f485507f6c); 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.
c8d24c1 to
9c0c1fa
Compare
1498a92 to
6c0df81
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head 6c0df81cf49d915bbd33a786242827b932d69947
after protected main merge of #1346. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 2d768412771a01f162425275107046bb75a2e95c.
The batch-owned binary patch is byte-identical
(SHA-256 609c412592075a60f9e861d17e6a611ec2f9982a1f197a9b77f691f485507f6c); 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.
9c0c1fa to
b1d90b0
Compare
6c0df81 to
15a100b
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head 15a100bb871aad87cba00d15edad00937056d953
after protected main merge of #1350. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 2d768412771a01f162425275107046bb75a2e95c.
The batch-owned binary patch is byte-identical
(SHA-256 609c412592075a60f9e861d17e6a611ec2f9982a1f197a9b77f691f485507f6c); 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.
b1d90b0 to
e3bcef7
Compare
15a100b to
3e7b727
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head 3e7b72739c5f7b6dfd899ed0763c07ad1d2d5021
after protected main merge of #1355. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 2d768412771a01f162425275107046bb75a2e95c.
The batch-owned binary patch is byte-identical
(SHA-256 609c412592075a60f9e861d17e6a611ec2f9982a1f197a9b77f691f485507f6c); 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.
e3bcef7 to
fa49a29
Compare
3e7b727 to
ad974f8
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial integration review at exact head ad974f8d26062262e0d0507a8630e0a0273a4ec5. Not independent approval.
Integrated already-merged schema prerequisite #1692 (67603e192525899fae75420e2c19e8212b16afc9). Compared with the prior reviewed head 3e7b72739c5f7b6dfd899ed0763c07ad1d2d5021, 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 609c412592075a60f9e861d17e6a611ec2f9982a1f197a9b77f691f485507f6c). 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.
fa49a29 to
ec9500f
Compare
ad974f8 to
71b5746
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head 71b574603f932d5a895fd2ad712f556ec2c9b0a6
after protected main merge of #1361. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 66f13a20b26dc2dcdb4055faf913ff8d30d48b2f.
The batch-owned binary patch is byte-identical
(SHA-256 609c412592075a60f9e861d17e6a611ec2f9982a1f197a9b77f691f485507f6c); 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.
ec9500f to
dfe7d50
Compare
71b5746 to
7f60e2c
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial governance integration review at exact head 7f60e2c60e5b8fafe37947954945a662b61ef752. This is a COMMENT, not independent approval.
Integrated already-merged main PR #1730 (fe1752572a688051e8105e5b89d3099fe2202c63). Compared with prior reviewed head 71b574603f932d5a895fd2ad712f556ec2c9b0a6, 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 609c412592075a60f9e861d17e6a611ec2f9982a1f197a9b77f691f485507f6c). 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.
7f60e2c to
a83b413
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Exact-head self-adversarial restack review at a83b413d7eeed6c173432a0d1b432c9a8b4f016b.
COMMENT, not independent approval.
The batch-owned binary patch remains byte-identical:
SHA-256 609c412592075a60f9e861d17e6a611ec2f9982a1f197a9b77f691f485507f6c.
Tree e630787c44b50898a895dccb8f5207f3eee58577 matches the independently computed clean merge.
Compared with historical tree 372dd1721f952da997fe03286f9ea3e4bd03c3b8, 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.
a83b413 to
5287486
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 existing-graph reviews: EcoFAB SynCom17, E. coli-B. bifidum HMO mutualism, and GL10/XL12 mixed-sugar fermentation. Corpus: 150 reviewed / 303 pending / 7 needs_research across 460 records in both roots.
Closes #1424. Closes #1425. Closes #1426. Non-graph #1427/#1428/#1429 remain open.
Changes
Nine nodes retained; one out-of-system node and two unsupported arrows removed. No added causal structure or Edison job/charge.
EcoFAB retains three whole-treatment outcome arrows. Separate sterile-plant nitrogen experiments do not belong in the SynCom17 graph. OAS925 isolate traits are intrinsic, with a HYPOTHESIZED contribution to relative dominance. Remove dominance-to-metabolome mediation and categorical facilitation, competition and cross-feeding; bulk profiles do not resolve donor/recipient flux.
HMO reciprocal exchange remains supported for K12 and an infant B. bifidum isolate under nutrient limitation. Preserve mutant/viability controls and chromatographic fucose/galactose evidence; spent-medium growth required pH readjustment to 6.3. Return-to-cysteine maintenance is PARTIAL. Remove the part/whole mutualism-summary arrow; do not claim host abundance regulation or a resolved cysteine release route.
GL10/XL12 retains a single node and zero arrows. Substrate specialization overlaps, not exclusive sugar use or direct interstrain exchange. Both strain labels remain distinct despite their shared taxon ID. The abstract and Results xylose endpoints remain explicitly unreconciled; the existing cross-feeding gap and cache-separation safeguards remain intact.
Verification And Output Records
392 passed in 179.44s (0:02:59); eight new regression tests. Focused schema, strict, term, independent snippet and history checks passed. Three reference-validator commands exit 0 but perform zero checks: a no-op, not evidence validation. Textual matches are not scientific support.
Zero structural errors; all 33 baseline network warnings preserved exactly, none added. The 183-user participant registry and numeric bounds are unchanged; OAS925 intrinsic representation changes mixed scope 122 to 123, community-only 265 to 264, and sole broad credit 1222 to 1221.
Three guarded writes and append-only histories; non-graph raw blocks, 457 other records and all 920 cache files unchanged. Repeat renders deterministic with exact descriptions checked across 456 pages. Actual post-commit docs-current passed with clean tree.
Committed decisions, actual validation output (including baseline) and self-adversarial review are under reports/causal_graph_review, dated 20261006, batch36. Four cached primary main-body texts reviewed at recorded scope; no visual figures, separate supplements, raw-data or browser review claimed. This is source-based self-adversarial review, not independent approval.
Dependency
Draft depends on #1423; do not merge into the topic parent. Require eligible exact-head checks and the protected main merge queue after parents merge. Move dependents before branch deletion. Full-corpus completion is not claimed.