fix: Support OME-TIFF page-indexed reading for live viewing#36

Merged

hongquanli merged 19 commits into

mainfrom

fix/ome-tiff-page-indexed-reading

Mar 25, 2026

Commits on Mar 24, 2026

fix: support page-indexed TIFF reading for OME-TIFF live viewing

maragall
and
claude
committed

Commits on Mar 25, 2026

fix: Add error handling and improve logging for page-indexed TIFF paths

hongquanli
and
claude
committed
refactor: Use page_idx parameter instead of #suffix for OME-TIFF pages

hongquanli
and
claude
committed
fix: Tighten types, fix tests, and use `is None` check

hongquanli
and
claude
committed
perf: Cache TiffFile handles for multi-page OME-TIFF reads

hongquanli
and
claude
committed
fix: Only cache TiffFile handles for multi-page files, narrow lock scope

hongquanli
and
claude
committed
fix: Remove GIL-dependent double-checked locking, add multi-page test

hongquanli
and
claude
committed
fix: Evict stale handles on IndexError, always cache, cleanup on end

hongquanli
and
claude
committed
fix: Close evicted handle, hold lock through page read, add test

hongquanli
and
claude
committed
fix: Only cache TiffFile handles for multi-page files (page_idx > 0)

hongquanli
and
claude
committed
fix: Add LRU eviction to TiffFile handle cache (max 128)

hongquanli
and
claude
committed
fix: Use per-file locks, remove page_idx==0 special case

hongquanli
and
claude
committed
test: Add edge case tests for multi-page TIFF loading

hongquanli
and
claude
committed
fix: Acquire per-file lock before closing handles, validate page_idx

hongquanli
and
claude
committed
fix: Use pytest.raises instead of self.assertRaises in plain class

hongquanli
and
claude
committed
fix: Don't close handle on IndexError eviction (use-after-close race)

hongquanli
and
claude
committed
Merge remote-tracking branch 'origin/main' into fix/ome-tiff-page-indexed-reading
hongquanli
committed
test: Add LRU cache stress test and concurrent register+load test

hongquanli
and
claude
committed
fix: Fail loudly if test runs against wrong ndviewer_light install

hongquanli
and
claude
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

fix: Support OME-TIFF page-indexed reading for live viewing#36

fix: Support OME-TIFF page-indexed reading for live viewing#36
hongquanli merged 19 commits into
mainfrom
fix/ome-tiff-page-indexed-reading

Commits on Mar 24, 2026

fix: support page-indexed TIFF reading for OME-TIFF live viewing

Commits on Mar 25, 2026

fix: Add error handling and improve logging for page-indexed TIFF paths

refactor: Use page_idx parameter instead of #suffix for OME-TIFF pages

fix: Tighten types, fix tests, and use `is None` check

perf: Cache TiffFile handles for multi-page OME-TIFF reads

fix: Only cache TiffFile handles for multi-page files, narrow lock scope

fix: Remove GIL-dependent double-checked locking, add multi-page test

fix: Evict stale handles on IndexError, always cache, cleanup on end

fix: Close evicted handle, hold lock through page read, add test

fix: Only cache TiffFile handles for multi-page files (page_idx > 0)

fix: Add LRU eviction to TiffFile handle cache (max 128)

fix: Use per-file locks, remove page_idx==0 special case

test: Add edge case tests for multi-page TIFF loading

fix: Acquire per-file lock before closing handles, validate page_idx

fix: Use pytest.raises instead of self.assertRaises in plain class

fix: Don't close handle on IndexError eviction (use-after-close race)

Merge remote-tracking branch 'origin/main' into fix/ome-tiff-page-indexed-reading

test: Add LRU cache stress test and concurrent register+load test

fix: Fail loudly if test runs against wrong ndviewer_light install