-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Update MixVisionTransformer #975
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@brianhou0208 Thanks for one more feature! I really appreciate your contributions! See the comments below
c0399c5
to
af6a1b1
Compare
Hey, @brianhou0208! Can you please rebase/merge the main? |
af6a1b1
to
a039526
Compare
Waiting for the tests and merging! Thanks for the update and iterations! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merging!
Hi, @qubvel
This PR introduces support for different output strides in the Mit Encoder. The original Mit Encoder required extra parameters (H, W) to be passed along with the features during forward propagation, which is not compatible with the current self.get_stage format.
This update enables PAN, DeepLabv3, and DeepLabv3+ to support the Mit encoder.
Update
Test Code
output