Skip to content

Tile and Stitching#384

Open
srivarra wants to merge 4 commits intomainfrom
tile
Open

Tile and Stitching#384
srivarra wants to merge 4 commits intomainfrom
tile

Conversation

@srivarra
Copy link
Collaborator

@srivarra srivarra commented Mar 5, 2026

No description provided.

Signed-off-by: Sricharan Reddy Varra <sricharan.varra@biohub.org>
Signed-off-by: Sricharan Reddy Varra <sricharan.varra@biohub.org>
Copy link
Contributor

@talonchandler talonchandler left a comment

Choose a reason for hiding this comment

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

Suggested next iteration:

  • z tiling support
  • benchmark with zyx tiles at zebrafish scale, with 3x3 through ~9x9 tiles...how long does i/o w/ and w/o blending take?
  • a path to parallelization...it's okay if it's serial on a single job for now

If this is ready by next Wednesday, I think I can trial it on ~9 zebrafish tiles, maybe 512x512x256 per tile.

# --------------------
# See how tiles are laid out with overlap.

slicer = Slicer(data, tile_size={"y": 32, "x": 64}, overlap={"y": 8, "x": 16})
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm unsure about if this Slicer is user facing or who should use it

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

removed explicit mention of it in the examples.

Comment on lines +71 to +72
result = map_tiles(
data,
Copy link
Contributor

Choose a reason for hiding this comment

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

Perhaps a better name? what did Jordao name his?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Jordao called it apply_tiled, I've changed the name to apply_func_tiled.

Comment on lines +74 to +75
tile_size={"y": 32, "x": 64},
overlap={"y": 8, "x": 16},
Copy link
Contributor

Choose a reason for hiding this comment

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

What's the plan for "z" tiling? We'll need this for zebrafish.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

For z tiling not much really changes, I just needed to generalize the blending and the logic for creating composites in the overlap regions to support 3D.

Signed-off-by: Sricharan Reddy Varra <sricharan.varra@biohub.org>
Signed-off-by: Sricharan Reddy Varra <sricharan.varra@biohub.org>
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