Skip to content

Adding MaskFormer #1118

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

Closed
vedantdalimkar opened this issue Apr 8, 2025 · 2 comments
Closed

Adding MaskFormer #1118

vedantdalimkar opened this issue Apr 8, 2025 · 2 comments

Comments

@vedantdalimkar
Copy link
Contributor

MaskFormer is the precursor to Mask2Former and since there are plans to add Mask2Former (#1044), I thought it would be natural to have a MaskFormer implementation first so that it's components can facilitate a more quicker development of Mask2Former. I would be happy to contribute to this issue if it's fine.

@qubvel
Copy link
Collaborator

qubvel commented Apr 8, 2025

Hey @vedantdalimkarm thanks for opening the issue! To minimize the maintained code, I think we should add V2 rather than V1, cause it's a newer version

@vedantdalimkar
Copy link
Contributor Author

Hey @vedantdalimkarm thanks for opening the issue! To minimize the maintained code, I think we should add V2 rather than V1, cause it's a newer version

Sure, that makes sense. Will start working on Mask2Former implementation directly then.

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