-
Notifications
You must be signed in to change notification settings - Fork 36
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
Comments
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
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:
AnyLoc/scripts/SAM_many.sh
Line 13 in a9fda68
AnyLoc/scripts/SAM_many.sh
Line 14 in a9fda68
AnyLoc/scripts/SAM_many.sh
Line 17 in a9fda68
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.
The text was updated successfully, but these errors were encountered: