Repository navigation
Review cellobiose biofilm, yeast-cellulose and X514 causal graphs (batch 21) - #1339
Conversation
realmarcin
left a comment
There was a problem hiding this comment.
Source-based self-adversarial review at exact head b4fd7d01144bf9815c3c5b5ce8c1c7b628a0a018, not independent approval.
Addressed #1335, #1336 and #1337. Retained all ten nodes and seven arrows, with five exact renames and no new graph structure. Preserved glucose/necromass controls, respiratory protection and conditional mutualism, measured products and viable populations, and the positive 39E supplementation control. Distinguished assay regimes, polymer versus cellobiose conversion, genomic qPCR versus viability, biofilm biomass versus unmeasured products, and proposed flux/B12 contributions versus selective causal demonstrations.
All 30 local gates pass: 262 passed in 220.78s (0:03:40); 83 MATCH snippets, zero other classes; 3 passed in 2.34s; 55 unrelated raw blocks and four caches byte-preserved. Two identical full renders with exact node/edge-description coverage; actual post-commit just check-docs-current passed and left the tree clean. Only the two intentional participant-registry users were added; exact-set and connectivity/GTDB census bounds remain unchanged.
The decision/adversarial/validation records under reports/causal_graph_review/ bind every node/arrow and record/cache hashes. Three primary main bodies with Methods and tables/captions were read; the 2011 mechanism paper was abstract-only. No visual image/supplement review or copyrighted full-body addition is claimed.
Non-graph #1338 and existing #746 remain open. Stacked coverage is 112/460 reviewed, not corpus completion. Required CI and protected main merge queue remain mandatory; do not merge into the topic parent. The separate batch13 CI fix will be incorporated through the protected parent restack.
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. |
cf322d4 to
258aa4f
Compare
b4fd7d0 to
189de5c
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Source-based self-adversarial review refreshed at exact head 189de5cb28b628390a7cda118c5ffa8911a4434b after protected merge of #1302. Not independent approval.
All 1 batch-owned commit(s) were retained. The batch's diff against its parent is byte-identical to its prior reviewed diff (SHA-256 ac269d653c71522b6dc9f53fdaf952a846cf4f10a49a43a9511ef3dbd9485a40). The only whole-tree change is the exact four-file #1329 history-base CI fix, already reviewed and merged through #1302. No additional scientific, grounding, history, generated-page or test-bound edits were made. New tree: 733239bcfd1eaf6e0e7b7bcf60f6a130891a9d96.
Committed decision/adversarial/validation records remain the scientific review evidence; their base SHAs are historical provenance. Required CI must pass for this new head. Do not merge into a topic parent or bypass the protected main queue.
258aa4f to
33e821b
Compare
189de5c to
86df6a9
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Source-based self-adversarial review refreshed at exact head 86df6a9a7a4ff8782d865522ea6f4b283099f4ef after integrating merged website PR #1303 (a937575f88ab11ff74683a8902fe13fd80c96165). Not independent approval.
The website merge conflicted only in the parent generated browser page; it was regenerated from the merged renderer and preserved KB. Each dependent owned commit was then rebased, generated pages refreshed, and actual post-commit just check-docs-current run successfully in every worktree. All branches were pushed atomically with exact leases.
Every non-document blob is verified identical to its previously reviewed version except the nine exact upstream website files. This includes unchanged scientific records, taxa, caches, histories, review ledgers and batch-owned regression code. Non-document manifest SHA-256: b89789441e2a789869eff70ef00bcf29c8bc48825b31937ad82c70e7b6111c96. New whole-tree SHA: 07a40b9dc0dba3503aceb6c85dc17f143d251062. Whole-tree equality to the previous head is NOT claimed because upstream rendering changed.
The parent integration-focused tests passed. The final combined stack suite, including all batch regressions, site/link tests, corpus/network checks, reference-validator canaries and publication-rights tests, reports: 302 passed in 223.43s (0:03:43). All twelve worktrees passed actual docs-current. Batch-specific scientific/access limitations remain as recorded; batch25 still explicitly retains three primary-verified Results quotes absent from its abstract-only cache, not a passed reference gate.
The decision, adversarial-review and validation files remain historical source-review provenance. Required CI must pass at this new head and the protected main queue remains mandatory. Do not merge a child into a topic parent; delete merged branches only after their dependents move.
33e821b to
6c5a7cc
Compare
86df6a9 to
1053b8a
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head 1053b8ae600ff75dfa87e26bb1a2895cf087828f
after protected main merge of #1307. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 07a40b9dc0dba3503aceb6c85dc17f143d251062.
The batch-owned binary patch is byte-identical
(SHA-256 e47240c7236dcba54864354407fa1e41366d618cace9faba3b13b6d4d610d0a3); 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.
1053b8a to
cb1ebef
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head cb1ebefa6fd7bc6c6c2e237d3f604f0531e9af7e
after protected main merge of #1312. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 07a40b9dc0dba3503aceb6c85dc17f143d251062.
The batch-owned binary patch is byte-identical
(SHA-256 e47240c7236dcba54864354407fa1e41366d618cace9faba3b13b6d4d610d0a3); 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.
2341d1d to
c7f2b2f
Compare
cb1ebef to
6519f8f
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head 6519f8f209e1b196212864bf33c644b3ffc4227c
after protected main merge of #1315. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 07a40b9dc0dba3503aceb6c85dc17f143d251062.
The batch-owned binary patch is byte-identical
(SHA-256 e47240c7236dcba54864354407fa1e41366d618cace9faba3b13b6d4d610d0a3); 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.
6519f8f to
b586c46
Compare
c7f2b2f to
a43c4cf
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head b586c468ddc183548c6f7dcb49236674168024fd
after protected main merge of #1318. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 07a40b9dc0dba3503aceb6c85dc17f143d251062.
The batch-owned binary patch is byte-identical
(SHA-256 e47240c7236dcba54864354407fa1e41366d618cace9faba3b13b6d4d610d0a3); 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.
b586c46 to
b7eef67
Compare
a43c4cf to
89fc203
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head b7eef67787f2eec618b976b21f318e488e97486d
after protected main merge of #1323. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 07a40b9dc0dba3503aceb6c85dc17f143d251062.
The batch-owned binary patch is byte-identical
(SHA-256 e47240c7236dcba54864354407fa1e41366d618cace9faba3b13b6d4d610d0a3); 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.
b7eef67 to
b3ca1ff
Compare
89fc203 to
a5bdf07
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head b3ca1ff77e66e0e7cd9661a7795ebf3be154a53c
after protected main merge of #1328. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 07a40b9dc0dba3503aceb6c85dc17f143d251062.
The batch-owned binary patch is byte-identical
(SHA-256 e47240c7236dcba54864354407fa1e41366d618cace9faba3b13b6d4d610d0a3); 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.
Summary
Closes #1335. Closes #1336. Closes #1337. Non-graph #1338 and existing #746 remain open.
Validation
Review Limits
Source-based self-adversarial review, not independent approval. Three primary main bodies, Methods and tables/captions were read. The 2011 X514 mechanism paper was available only as a complete abstract. No visual image or supplementary-file inspection. No copyrighted full body added; four caches unchanged. Unrelated metadata is preserved, not certified.
Stacked on batch20 PR #1334. Do not merge into the topic parent or bypass checks. After protected parent merges, restack the owned commit, retarget to main, refresh exact-head review and use the protected queue. Delete parent branches only after dependents move. The pending batch13 CI-fix commit is not yet in this branch; incorporate it through the protected parent restack.