Commit e3a432a
fix: Handle QuantStudio Stop Time and non-ASCII delta columns (#1203)
## Summary
- Accept "Experiment Run Stop Time" as alias for "Experiment Run End
Time" in header metadata parsing (some QuantStudio exports use this
alternate field name)
- Explicitly read `ΔΔCт - Fsigma` and `ΔΔCт + Fsigma` columns with valid
Python identifiers before `get_unread()`, preventing `TypeError: Field
names must be valid identifiers` when these flow through
`make_dataclass()`
## Test plan
- [x] Added `appbio_quantstudio_example10.xlsx` test file exercising
both fixes (384-well, Comparative Ct with "Stop Time" and ΔΔCт columns)
- [x] All 28 QuantStudio parser tests pass
- [x] All 312 vendor discovery tests pass
- [x] Lint clean
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>1 parent 1e99faf commit e3a432a
3 files changed
Lines changed: 2775 additions & 1 deletion
File tree
- src/allotropy/parsers/appbio_quantstudio
- tests/parsers/appbio_quantstudio/testdata
Lines changed: 5 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
82 | 82 | | |
83 | 83 | | |
84 | 84 | | |
85 | | - | |
| 85 | + | |
| 86 | + | |
| 87 | + | |
86 | 88 | | |
87 | 89 | | |
88 | 90 | | |
| |||
474 | 476 | | |
475 | 477 | | |
476 | 478 | | |
| 479 | + | |
| 480 | + | |
477 | 481 | | |
478 | 482 | | |
479 | 483 | | |
| |||
0 commit comments