Skip to content

2026-09-18 NanoVirDx domain representation implementation review

Reviewed the minimal NanoVirDx adapter contract for Histo 0.0.127 process_scan.

Work Done

  • Inspected NanoVirDx dataclasses, inference base class, README, packaging, and the in-progress Histo domain-representation adapter.
  • Verified the Histo 0.0.127 process_scan signature and its five fixed OME-Zarr outputs.
  • Confirmed that the 0.0.127 wheel omits the repository-level Hydra config, so a packaged flattened fine28/local config is needed for a default NanoVirDx adapter.
  • Recommended lazy Histo imports for macOS importability, five-artifact batch validation, a README correction, and small adapter/dataclass contract tests.
  • No monorepo files were edited.

Lessons Learned: Pitfalls

  • The local Histo worktree is two commits beyond the 0.0.127 tag and has unrelated working changes. Inspecting the tag explicitly was necessary to avoid reporting a newer API.
  • Listing NanoVirDx recursively without pruning .pixi produced excessive generated-file output. Future inspection should prune generated directories first.

Lessons Learned: Improvements

  • Document in the Histo package/release notes whether runtime config is included in wheels. Consumers otherwise must discover the missing config only after installation.
  • NanoVirDx should have a small test target that can validate lazy optional-platform adapters without installing their Linux-only dependencies.
Navigation

Type to search…

↑↓ navigate↵ selectEsc close