Skip to content

feat(vocab): W23A evidence-gated physical bases for grammar gap closure#13

Merged
Simon-McIntosh merged 1 commit intomainfrom
w23a-vocab-grammar-gaps
Apr 26, 2026
Merged

feat(vocab): W23A evidence-gated physical bases for grammar gap closure#13
Simon-McIntosh merged 1 commit intomainfrom
w23a-vocab-grammar-gaps

Conversation

@Simon-McIntosh
Copy link
Copy Markdown
Owner

W23A Vocabulary Extension — Grammar Gap Closure

Context

W22B review score analysis identified three vocabulary gap classes dragging mean scores from ~0.660 to below the 0.733 target. This PR addresses Class 2 (physics compound nouns) with 7 new physical_base tokens.

Evidence & Rationale

Token DD Source Evidence (N≥3) W22B Impact
bootstrap_current_density core_profiles/profiles_1d/j_bootstrap N=5 scored 0.450
rotation_frequency core_profiles/profiles_1d/ion/rotation_frequency_tor N=8+ scored 0.563
mach_number langmuir_probes/reciprocating/plunge/mach_number_parallel N=3 edge plasma
resistivity wall/*/resistivity, cryostat/*/resistivity N=6 material property
heat_viscosity_current_density edge_profiles/ggd/j_heat_viscosity N=4 gyroviscous j
parallel_viscosity_current_density edge_profiles/ggd/j_parallel_viscosity N=4 parallel viscous j
perpendicular_viscosity_current_density edge_profiles/ggd/j_perpendicular_viscosity N=4 perp viscous j

All tokens meet the N>=3 evidence gate.

Test Results

1107 passed, 68 xfailed, 53 warnings.

…losure

Add 7 physical_base tokens to close vocabulary gaps identified in W22B
review score analysis:

Class 2 — Physics compound nouns:
  - bootstrap_current_density: j_bootstrap (core_profiles, N=5 IDSs)
  - rotation_frequency: rotation_frequency_tor (core_profiles, N=8+)
  - mach_number: mach_number_parallel (langmuir_probes, N=3)
  - resistivity: wall/*/resistivity (wall, cryostat, N=6)

Class 2 — Viscosity current density compounds:
  - heat_viscosity_current_density: j_heat_viscosity (edge/plasma_profiles, N=4)
  - parallel_viscosity_current_density: j_parallel_viscosity (N=4)
  - perpendicular_viscosity_current_density: j_perpendicular_viscosity (N=4)

All tokens meet the N>=3 evidence gate. Sonic rotation frequency is
not added as a separate base because subject=sonic + base=rotation_frequency
correctly composes to sonic_rotation_frequency via existing grammar.
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 26, 2026

⚠️ JUnit XML file not found

The CLI was unable to find any JUnit XML files to upload.
For more help, visit our troubleshooting guide.

@Simon-McIntosh Simon-McIntosh merged commit e444ca8 into main Apr 26, 2026
2 of 4 checks passed
Simon-McIntosh added a commit to Simon-McIntosh/imas-codex that referenced this pull request Apr 26, 2026
…ions

Merge of Simon-McIntosh/IMAS-Standard-Names#13 (w23a-vocab-grammar-gaps).
Adds 7 evidence-gated physical_base tokens:
- bootstrap_current_density, heat_viscosity_current_density, mach_number
- parallel_viscosity_current_density, perpendicular_viscosity_current_density
- resistivity, rotation_frequency

ISN version: 0.7.0rc28.dev2+ge444ca8a9
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.

1 participant