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

Question about SAM #27

Open
Graysonggg opened this issue Mar 30, 2024 · 1 comment
Open

Question about SAM #27

Graysonggg opened this issue Mar 30, 2024 · 1 comment

Comments

@Graysonggg
Copy link

Amazing job!

I noticed that in addition to dino, you also tried other fundamental models such as sam and clip, and sam's model selection is reflected in the following two lines in the document:

model_types=("vit_b")

out_layer_numbers=(12)

num_clusters=(128)

I'm curious about that what is the consideration behind these choices? Why isn't the cluster center set to 32 like dinov2? And have you considered to use larger sam ​like sam-vit-L?

Looking forward to your reply.

@JayKarhade
Copy link
Contributor

Hi @Graysonggg we had done preliminary evaluations on SAM with those parameters, however we did not see better performance than DINOv2 and hence didn't probe further.

The hyperparameters that have been used are just an artifact of those evaluations. We did benchmark against bigger ViT models too, but didn't see much difference.

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

2 participants