Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Commit to file for reproducibility #408

Open
constantinpape opened this issue Mar 5, 2024 · 3 comments
Open

Commit to file for reproducibility #408

constantinpape opened this issue Mar 5, 2024 · 3 comments
Assignees

Comments

@constantinpape
Copy link
Contributor

Add an option to save all the prompts for interactive segmentation.

This would save the prompts per segmented object in an extra directory to enable reproducing the segmentation.
The easiest way to implement this would be adding this to the commit button, and then save the prompts per object to a folder.

See initial discussion in #406

@constantinpape constantinpape self-assigned this Mar 15, 2024
@constantinpape
Copy link
Contributor Author

After thinking about this: the best idea is just to mirror ALL relevant layers (prompts and committed objects) in a zarr file, along the idea in #434 (comment).
I will implement this now.
cc @anwai98

@constantinpape
Copy link
Contributor Author

The core logic is implemented in #451. What's still missing:

  • Document the file format.
  • Save high level information in the zarr file, cf. Add micro-sam version signature to files #445
  • Also save relevant options from widgets, e.g. save the settings of the segment_object or auto segmentation widgets.
  • Reload prompts and reproduce segmentation from them (this is more work and I will likely only have time after NM submission)

@constantinpape constantinpape changed the title Saving all prompts for interactive segmentation Commit to file for reproducibility Mar 15, 2024
@constantinpape
Copy link
Contributor Author

The important metadata (model_type, model hash, micro_sam version) are now saved in the file since #507.
Still missing:

  • Documentation
  • Saving options for the widgets
  • Tests (the logic is quite complex now, so we should write tests for it)
  • Recreating the segmentation from the prompts

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

No branches or pull requests

1 participant