Skip to content

Drop the C2/C3 contract labels - #47

Merged
jdkent merged 1 commit into
mainfrom
drop-contract-codes
Oct 21, 2026
Merged

jdkent merged 1 commit into
mainfrom
drop-contract-codes

Conversation

@jdkent

@jdkent jdkent commented Oct 21, 2026

Copy link
Copy Markdown
Member

Drops the contract labels C2 and C3 from the paper-parse schema. The artifacts are now called by name: the parsed paper (ParsedPaper) and the coordinate parse (CoordinateParse).

  • README: the "Contract" column is removed from the paper-parse table; Analysis.source_table_analysis now names a CoordinateParse key.
  • Descriptions in neuroimaging-paper-parse.yaml, paper.yaml, coordinates.yaml and parse_enums.yaml reworded, meaning kept.
  • models/paper_parse.py regenerated. Only two enum docstrings change; no field, enum value or type changes, and the JSON Schemas are unchanged.

python tools/generate_models.py --check passes. Suite: 237 passed, 1 skipped.

Cross-repo references (read-only, not changed here):

  • ns-pond-ingestion-workflow: none (only matches are inside HTML/gz test fixtures).
  • pondie: docs/extraction-workflow-experiments.md lines 197, 198, 204, 208, 212, 271, 387, 391, 597, 615 use C2/C3/C4 as experiment-arm names (critic, targeted re-ask), not contract labels.
  • neurostore: store/review-frontend/src/inspector/Inspector.test.tsx:184 ends a test title with "(C2)"; likely a roadmap code, not this contract.

🤖 Generated with Claude Code

Description text only; models regenerated.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@jdkent
jdkent marked this pull request as ready for review October 21, 2026 01:48
@jdkent jdkent self-assigned this Oct 21, 2026
@jdkent
jdkent merged commit 116b3b5 into main Oct 21, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants