Skip to content

Zarrv3 quick czireader and other fixes#97

Closed
cgoina wants to merge 54 commits into
zarrv3from
zarrv3-quick-czireader
Closed

Zarrv3 quick czireader and other fixes#97
cgoina wants to merge 54 commits into
zarrv3from
zarrv3-quick-czireader

Conversation

@cgoina

@cgoina cgoina commented May 13, 2026

Copy link
Copy Markdown
Collaborator

This PR includes the following:

  • Scale fixes that are currently in master
  • Support for quick-czireader - I found this much faster when it comes to reading metadata because it goes directly to the fileposition instead of scanning the entire file. I had to add the quick-czireader as a dependency otherwise it didn't pick it
  • Neuronglancer could not open zarr v3 unless it used OME v0.5 so when writing Zarr v3 I use OME v0.5
  • Some bug fixes that I encounter while trying to stitch zarr v3 with sharding
  • A pom profile that can dump all external dependencies - I found this useful for copying the proper dependencies to my Fiji instance

cgoina and others added 30 commits October 28, 2025 14:53
fix SmartSPIM to skip tiles with skip=0 in metadata.json
When detectSubsets() merged two sets because both views of a pair were
in different sets, it would call mergeSets() but never add the pair that
triggered the merge. This caused pairs to be missing from subsets.

The fix adds the current pair to the merged set (which is always at the
end of pairSets after mergeSets() completes).

Also enhanced documentation:
- Added detailed Javadoc note explaining mergeSets() behavior
- Added inline comments explaining each section of mergeSets()
- Added comment at fix location explaining the index logic

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
fixing small history select bug when selecting multiple setups
Signed-off-by: Stephan Preibisch <stephan.preibisch@gmx.de>
Fix the scale and translation transformations in OME-ZARR metadata
Signed-off-by: Stephan Preibisch <stephan.preibisch@gmx.de>
import java.util.HashMap;

import ch.epfl.biop.formats.in.ZeissQuickStartCZIReader;
import loci.formats.FormatTools;

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The whole reason for this file was a bug the exists in ZeissQuickStartCZIReader in getting bitsperpixel value; I filed an issue there but I don't know when will be addressed

@cgoina cgoina closed this Jun 17, 2026
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.

4 participants