Skip to content

Handle unitless QMCFC parameters - #87

Merged
galjos merged 1 commit into
mainfrom
fix/qmcfc-unitless-parameters
Jul 23, 2026
Merged

Handle unitless QMCFC parameters#87
galjos merged 1 commit into
mainfrom
fix/qmcfc-unitless-parameters

Conversation

@galjos

@galjos galjos commented Jul 22, 2026

Copy link
Copy Markdown
Member

Summary

  • normalize missing QMCFC units without inventing scientific units
  • omit unavailable units from plot labels and show n/a in explicit TUI unit fields
  • cover single plots, dashboards, terminal charts, and the TUI with a minimal QMCFC fixture

Verification

  • reproduced with the issue attachment and parsed all 11,897 rows after the fix
  • 200 unit and integration tests passed with 100% coverage
  • 9 GUI and TUI end-to-end tests passed
  • 210 total non-benchmark tests passed

Fixes #86

@galjos galjos added the bug Something isn't working label Jul 22, 2026
@galjos
galjos requested a review from ape33 July 22, 2026 11:35
@galjos galjos mentioned this pull request Jul 22, 2026
@galjos
galjos merged commit fc0c5e3 into main Jul 23, 2026
1 check passed
@galjos
galjos deleted the fix/qmcfc-unitless-parameters branch July 23, 2026 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error Reading QMCFC Files

2 participants