-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[Feature] Support ViT-Adapter #1209
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## dev-1.x #1209 +/- ##
============================================
+ Coverage 0.02% 87.11% +87.08%
============================================
Files 121 170 +49
Lines 8217 14356 +6139
Branches 1368 2267 +899
============================================
+ Hits 2 12506 +12504
+ Misses 8215 1449 -6766
- Partials 0 401 +401
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
There are several points of concern, including ・Duplication of ViTAdapter and BEiTAdapter Please review. |
Get it, we will review it soon. We will try to merge it into the new version. |
is the |
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.
LGTM.
Motivation
paper: https://arxiv.org/abs/2205.08534
code: https://github.com/czczup/ViT-Adapter
issue: open-mmlab/mmdetection#9044
Related PR
open-mmlab/mmdetection#9354
open-mmlab/mmcv#2451
open-mmlab/mmcv#2452
Remaining task
Checklist
Before PR:
After PR: