Addition of cropping.py #229
Conversation
There was a problem hiding this comment.
As discussed in our meeting with @talonchandler. I think the convenience functions like get_positions andget_channel_indices would be great to have inside iohub.
@ziw-liu what would be a good place to have these (iohub.ngff?)
| chunks = (1, 1, 1, shape[3], shape[4]) | ||
|
|
||
| # position keys | ||
| position_keys = [self.validate_position_key(path) for path, _ in positions] |
There was a problem hiding this comment.
could run everything in a single for loop
Can you explain why is |
|
Feel free to ignore this comment because no one asked for my review. Since the unified API was merged, I think it would be nice for this function to take a |
The catch here is that OME-Zarr UAPI is still pending... |
Sample script to index into either time, channels, z, y, and x into an existing ome-zarr store and create a new ome-zarr store with crops. Code is currently structured as a class with some helper methods but can be re-factored as some utils methods that are used in the script directly (without the class structure).