Skip to content

feat: Add HIAC .xls file support to Beckman PharmSpec parser#1232

Merged
nathan-stender merged 3 commits into
mainfrom
feat/hiac-xls-support
Jun 17, 2026
Merged

feat: Add HIAC .xls file support to Beckman PharmSpec parser#1232
nathan-stender merged 3 commits into
mainfrom
feat/hiac-xls-support

Conversation

@nathan-stender

Copy link
Copy Markdown
Collaborator

Summary

  • Extends the Beckman PharmSpec parser to support HIAC 9703+ .xls files (both binary OLE2 and HTML-masquerading-as-XLS formats)
  • Adds sniff logic to detect HIAC "Run Counter Test" files by reading cell content
  • Handles alternative header field names (Sample Name vs Probe, View Volume (%) vs View Volume)
  • Updates parser-generator skill documentation with correct test data generation procedure

Test plan

  • Both HIAC .xls test files parse correctly (binary and HTML formats)
  • All 17 PharmSpec parser tests pass
  • All 318 vendor discovery tests pass (no sniff conflicts with Vi-Cell XR or other .xls parsers)
  • Linting passes
  • Generated JSON output manually verified against source data (all measurements, metadata, and calculated averages match)

🤖 Generated with Claude Code

The HIAC 9703+ instrument outputs .xls files in two sub-formats (binary
OLE2 and HTML masquerading as .xls) with a "Run Counter Test" layout.
This extends the existing PharmSpec parser to handle both formats since
the data structure (particle size distributions) is identical.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@nathan-stender
nathan-stender requested review from a team and slopez-b as code owners June 17, 2026 15:13
@nathan-stender
nathan-stender requested a review from tamargrey June 17, 2026 15:13
stephenworlow
stephenworlow previously approved these changes Jun 17, 2026
nathan-stender and others added 2 commits June 17, 2026 09:10
- Fix CalculatedDataConfigWithOptional import path in calcdocs/api.py
- Use project read_excel wrapper instead of pd.read_excel in PharmSpec parser/reader
- Fix pd.read_html call to pass decoded string instead of BytesIO

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@nathan-stender
nathan-stender merged commit e37b078 into main Jun 17, 2026
9 checks passed
@nathan-stender
nathan-stender deleted the feat/hiac-xls-support branch June 17, 2026 19:17
nathan-stender added a commit that referenced this pull request Jun 19, 2026
### Added

- Add HIAC .xls file support to Beckman PharmSpec parser (#1232)

### Fixed

- Accept numeric-only Machine ID in Mabtech APEX parser (#1231)
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