Enable automatic segmentation state caching for SAM2#1295
Conversation
|
Okay I tried it on a Mac, looks pretty good. What do you think? @constantinpape |
constantinpape
left a comment
There was a problem hiding this comment.
I would change / clarify a few things:
- Is the loading now also lazy / on-demand for tiled / 3d data?
- I would try to save the state in zarr, as part of the embedding data structure. Also add a respective attribute to the zarr so we can record the state of this.
- Rename
amg_stateto something more general.
|
I'll answer the high-level questions here:
Yes, that's right -- it's lazy on-demand loading. I updated it a bit to compat with the catched zarr dispatcher now!
Took care of it. Here's a high-level file structure inside the cached zarr:
This one's a bit of a thinker. I am thinking of going with a general alias, eg. |
|
Okay, this is ready for the next round of review @constantinpape |
WIP. Need to check how this works across tools!