Repository navigation
Correct assay boundaries, strain direction and modeled mechanisms (batch 34) - #1416
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 fcc5445; not independent approval.
Addressed #1410/#1411/#1412 after complete recorded primary-text and actual diff review. Nine nodes retained, four arrows qualified and two cross-assay arrows removed. Distinguish drought field genomic capacity, maize watered-control responses and single-isolate iron amendment. Correct CC17B-1-to-DSM1-25280 proposed exclusion, retaining shared-ID strain distinctions, CFU limits and staged temperature/oxygen conditions. ENIGMA measured physiology remains separate from modeled NO exchange and high-condition nitrite exposure; no obligate dependence, categorical competition or N2O-toxicity claim.
376 passed in 175.08s (0:02:55); eight new tests. Focused schema/strict/terms/snippet/history checks, deterministic renders, exact descriptions across 456 pages and actual post-commit docs-current passed. Reference-validator performed zero checks despite exit 0: no-op is not evidence validation. Zero structural errors, 31 unchanged warnings. Participant registry gains two records; ENIGMA scope shift accounts for measured census change, with no algorithm/numeric bound changes. All non-graph raw blocks, 457 other records and 920 caches preserved.
#647/#1413/#1414/#1415 remain open. No new causal structure or provider job/charge. Cached main-body text reviewed; no visual figures, supplements, raw data or browser screenshots claimed. Committed decision, validation and adversarial output records disclose limitations. Protected main merge and full-corpus completion are not asserted.
|
CI scheduling note: 73 cancellation requests were accepted for current heads of task-owned dependent draft PRs; 7 of the 80 identified runs had already completed. Those drafts are not ready to merge and their ancestry changes as parents land. No cancelled check is counted as successful validation. The seven normal required checks on merge-ready #1315 at Local audited plan/receipt: |
441f630 to
3507e2c
Compare
fcc5445 to
320ccc5
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head 320ccc53dbb11013b5ce6b545fc78e2c015fbd14
after protected main merge of #1315. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree b1133b3c622afdf3a22e1514fe2ff2b10948d9dc.
The batch-owned binary patch is byte-identical
(SHA-256 6488e9c21e74f30855b7167ca69d84821d2cf46ba4d0d3d6294ae4d833011c18); 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.
320ccc5 to
a4d2690
Compare
3507e2c to
7441e72
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head a4d269002b4ad68a71bb062252d4d38fe01e6ebf
after protected main merge of #1318. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree b1133b3c622afdf3a22e1514fe2ff2b10948d9dc.
The batch-owned binary patch is byte-identical
(SHA-256 6488e9c21e74f30855b7167ca69d84821d2cf46ba4d0d3d6294ae4d833011c18); 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.
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. |
|
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. |
7441e72 to
4d900a7
Compare
a4d2690 to
4413e26
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head 4413e2662d9e737d0578017913c704a9085e32f1
after protected main merge of #1323. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree b1133b3c622afdf3a22e1514fe2ff2b10948d9dc.
The batch-owned binary patch is byte-identical
(SHA-256 6488e9c21e74f30855b7167ca69d84821d2cf46ba4d0d3d6294ae4d833011c18); 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.
4d900a7 to
d572bc7
Compare
4413e26 to
7a775c8
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head 7a775c81789d54275d11d46c9f46e4d822a5e03a
after protected main merge of #1328. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree b1133b3c622afdf3a22e1514fe2ff2b10948d9dc.
The batch-owned binary patch is byte-identical
(SHA-256 6488e9c21e74f30855b7167ca69d84821d2cf46ba4d0d3d6294ae4d833011c18); 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.
d572bc7 to
94c2336
Compare
7a775c8 to
1a05dfe
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head 1a05dfe6a682b416397d2bbc4ef8d27cdf168acb
after protected main merge of #1334. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree b1133b3c622afdf3a22e1514fe2ff2b10948d9dc.
The batch-owned binary patch is byte-identical
(SHA-256 6488e9c21e74f30855b7167ca69d84821d2cf46ba4d0d3d6294ae4d833011c18); 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.
1a05dfe to
d49713b
Compare
94c2336 to
d3db0fe
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head d49713bd482730168aa95a6e8f0ab9723a91a3db
after protected main merge of #1339. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree b1133b3c622afdf3a22e1514fe2ff2b10948d9dc.
The batch-owned binary patch is byte-identical
(SHA-256 6488e9c21e74f30855b7167ca69d84821d2cf46ba4d0d3d6294ae4d833011c18); 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.
d49713b to
f8afe3f
Compare
d3db0fe to
f6543e9
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head f8afe3fa4f0d21cff683ba51910b498e2dfdcb60
after protected main merge of #1346. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree b1133b3c622afdf3a22e1514fe2ff2b10948d9dc.
The batch-owned binary patch is byte-identical
(SHA-256 6488e9c21e74f30855b7167ca69d84821d2cf46ba4d0d3d6294ae4d833011c18); 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.
f8afe3f to
cd6f44c
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head cd6f44c3f117db2e1a01fc237c28b45fbd34c4fa
after protected main merge of #1350. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree b1133b3c622afdf3a22e1514fe2ff2b10948d9dc.
The batch-owned binary patch is byte-identical
(SHA-256 6488e9c21e74f30855b7167ca69d84821d2cf46ba4d0d3d6294ae4d833011c18); 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.
cd6f44c to
af73094
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head af7309465652b6150e7239613bf1c2e2eee9e4f4
after protected main merge of #1355. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree b1133b3c622afdf3a22e1514fe2ff2b10948d9dc.
The batch-owned binary patch is byte-identical
(SHA-256 6488e9c21e74f30855b7167ca69d84821d2cf46ba4d0d3d6294ae4d833011c18); 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.
195a0a3 to
e44c5c0
Compare
af73094 to
6b2d3ac
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial integration review at exact head 6b2d3ac97c21972fc9a3fe25b016714057bc4bf8. Not independent approval.
Integrated already-merged schema prerequisite #1692 (67603e192525899fae75420e2c19e8212b16afc9). Compared with the prior reviewed head af7309465652b6150e7239613bf1c2e2eee9e4f4, 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 6488e9c21e74f30855b7167ca69d84821d2cf46ba4d0d3d6294ae4d833011c18). 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.
e44c5c0 to
bd27199
Compare
6b2d3ac to
abf2a11
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial review refreshed at exact head abf2a113461638161dc5a826c1b742d45ef06a87
after protected main merge of #1361. This is not independent approval.
The entire tree remains identical to the preceding reviewed tree 7d91d383dc06c5701e62a2f688c4e7abf87e088b.
The batch-owned binary patch is byte-identical
(SHA-256 6488e9c21e74f30855b7167ca69d84821d2cf46ba4d0d3d6294ae4d833011c18); 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.
abf2a11 to
6fc6a28
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Self-adversarial governance integration review at exact head 6fc6a2830d8a2a9aa68f667d96ede15fe4f067bf. This is a COMMENT, not independent approval.
Integrated already-merged main PR #1730 (fe1752572a688051e8105e5b89d3099fe2202c63). Compared with prior reviewed head abf2a113461638161dc5a826c1b742d45ef06a87, 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 6488e9c21e74f30855b7167ca69d84821d2cf46ba4d0d3d6294ae4d833011c18). 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.
2b8f2b0 to
3c9c9ad
Compare
6fc6a28 to
17d5884
Compare
realmarcin
left a comment
There was a problem hiding this comment.
Exact-head self-adversarial restack review at 17d5884900312be64475b81b952c75aa5e3c7435.
COMMENT, not independent approval.
The batch-owned binary patch remains byte-identical:
SHA-256 6488e9c21e74f30855b7167ca69d84821d2cf46ba4d0d3d6294ae4d833011c18.
Tree 0eb69c383ec4be43714eea3420d845f18831ae09 matches the independently computed clean merge.
Compared with historical tree 674aff3457be13c1a2a2a47da66bbafd1139e5b4, 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.
17d5884 to
2805b96
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 completed existing-graph reviews: drought-iron rhizosphere, dual Bacillus/Pseudomonas LA fermentation, and ENIGMA denitrification. Corpus: 144 reviewed / 309 pending / 7 needs_research, across 460 records in both roots.
Closes #1410. Closes #1411. Closes #1412. Non-graph #1413/#1414/#1415 and existing cultivation-boundary #647 remain open.
Changes And Review
Nine nodes retained, four arrows qualified and two unsupported cross-assay arrows removed. No new causal structure or Edison job/charge. Field genomic associations, maize TOM1 controls and single-isolate sorghum amendments remain distinct.
Correct the proposed Bacillus exclusion direction to CC17B-1 toward DSM1-25280, preserving strain identity despite a shared species ID. CFU decline is not demonstrated extinction; competition, product inhibition and culturability alternatives remain explicit. No direct PAD-vanillin catalysis or donor-neutrality claim. Preserve aerobic 30 C ZL pretreatment and the subsequent anaerobic 50 C Bacillus relay.
ENIGMA joint growth and measured chemistry are distinct from modeled reverse NO transfer and high-condition nitrite exposure. Community scope resolves complementary exchange without a false single direction, obligate dependence or categorical competition. Negative causal link refers to nitrite, not N2O toxicity.
Verification
376 passed in 175.08s (0:02:55); eight new tests. Schema, strict, terms, independent snippets and history checks passed. Three reference-validator commands exit 0 but perform zero checks: a no-op, not evidence validation. Text matching is not scientific support.
Zero structural errors and 31 exactly unchanged baseline warnings. Participant registry 178 to 180 adds only Dual Bacillus and ENIGMA. ENIGMA changes mixed to community-level-only, accounting for two additional sole-rule taxa; no taxon, algorithm or numeric test-bound change.
Three guarded writes, append-only histories, llm-assisted events and regenerated pages. All non-graph raw blocks, 457 other records and 920 caches preserved. Repeated renders deterministic; exact descriptions checked across 456 pages. Actual post-commit docs-current passed with a clean tree.
Committed decision ledger, validation output and adversarial-review records: reports/causal_graph_review, dated 20261006, batch34. Primary cached main bodies, captions and tables read; no visual figures, supplements or raw data reviewed. Stale abstract_only cache headers disclosed, not changed. Source-based self-adversarial review, not independent approval.
Dependency
Draft depends on #1409; do not merge into the topic parent. Require protected exact-head checks and main merge queue after parents merge. Move dependents before branch deletion. Full-corpus completion is not claimed.