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

HQ-SAM #25

Open
dlangenk opened this issue Jun 11, 2024 · 3 comments
Open

HQ-SAM #25

dlangenk opened this issue Jun 11, 2024 · 3 comments
Labels

Comments

@dlangenk
Copy link
Member

There is a finetuned variant of SAM available at https://github.com/SysCV/sam-hq which promises better segmentation masks at basically no cost.

To upgrade to sam-hq, we basically only have to replace the sam package that is installed with pip with sam-hq (also pip compatible) and download the new weights. The front end is completely compatible with the new embeddings (already tested).

Although there is no very big perceived improvement, it might be worth it.

@mzur
Copy link
Member

mzur commented Jun 11, 2024

The low resolution of the feature representation is probably still the most limiting factor. If you don't see a big improvement in the cases you tested in #9, I don't see an immediate need to implement this 🤔

@dlangenk
Copy link
Member Author

Yes it is. But if we solved that it might be worth to implement it. Creating the pull request will only take 10-40 min.

@mzur
Copy link
Member

mzur commented Jun 11, 2024

Right. But then I want an example that HQ SAM provides better results than the current implementation 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: No status
Development

No branches or pull requests

2 participants