-
Notifications
You must be signed in to change notification settings - Fork 40
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
Suggestions for micro-sam tool #406
Comments
Updates based on discussions today:
|
Providing a demo notebook for:
Hint: something like this - https://github.com/facebookresearch/segment-anything/blob/main/notebooks/automatic_mask_generator_example.ipynb |
I will close this since most points here are either solved or covered by other issues.
This is being implemented as part of #408.
This is a bit outdated. It's now possible to compute tiled embeddings without using the embedding file. In any case, we should improve the documentation on this, but this is covered by other issues already.
Both changes are now implemented. (Together with a better documentation of the parameters of the CLI scripts)
Also covered by #408. |
I received some feedback and requests:
micro_sam.annotator
and not pass the embedding file (the compute image embeddings in .zarr format), it would be nice to lead the users towards how to precompute embeddings (using a link or redirecting to a brief mention of scripts).Some observations from my side:
micro_sam.precompute_embeddings
- expects-o
attribute to store the image embeddings. It would be nice (and probably consistent) to expose the attribute as-e
(the same expectation as forannotator_2d
, etc., or vice-versa)micro_sam.annotator_2d
- the choice of the model is limited to--model_type
. It would be nice to have-m
attribute exposed as well.The text was updated successfully, but these errors were encountered: