Skip to content

feat: Add Vi-Cell XR single-sample report format support#1222

Merged
nathan-stender merged 1 commit into
mainfrom
feat/vi-cell-xr-report-format
Jun 3, 2026
Merged

feat: Add Vi-Cell XR single-sample report format support#1222
nathan-stender merged 1 commit into
mainfrom
feat/vi-cell-xr-report-format

Conversation

@nathan-stender
Copy link
Copy Markdown
Collaborator

Summary

  • Adds support for the Vi-CELL XR 2.04 single-sample "report" export format (xlsx)
  • This format uses a key-value layout (one sample per file) rather than the tabular multi-sample format already supported
  • Detection is based on row 4 containing "Sample ID" as a label in column A (report format) vs as part of a column header row (tabular format)
  • All measurement data, settings, and metadata are correctly mapped to the same ASM cell-counting schema

Test plan

  • New test file added (Beckman_Vi-Cell-XR_report_format.xlsx) with expected JSON output
  • All 13 Vi-Cell XR parser tests pass (existing + new)
  • Report format detection does not false-positive on any existing tabular test files
  • Discover vendor tests pass (sniff correctly identifies the new format)
  • Lint passes

🤖 Generated with Claude Code

Vi-CELL XR 2.04 can export a single-sample "report" format (xlsx) with
key-value layout instead of the tabular multi-sample format. This adds
ViCellXRReportReader to handle files where row 4 contains "Sample ID" as
a label (key-value) rather than as a column header (tabular).

Detection: checks if row 4, col A == "Sample ID" (report) vs being part
of a column header row (tabular). Falls through to existing tabular reader
if not detected.

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 3, 2026 14:45
@nathan-stender nathan-stender merged commit f01c47d into main Jun 3, 2026
9 checks passed
@nathan-stender nathan-stender deleted the feat/vi-cell-xr-report-format branch June 3, 2026 18:15
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