-
Notifications
You must be signed in to change notification settings - Fork 444
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
Enable deit_tiny template by default #2282
Conversation
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## develop #2282 +/- ##
=======================================
Coverage 0.00% 0.00%
=======================================
Files 498 498
Lines 37574 37577 +3
=======================================
- Misses 37574 37577 +3
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
@wonjuleee , is it okay to merge? or do you want to merge this PR after checking the effect of the Geti side? |
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.
Let's hold this PR until the Geti implements static model template mapping. It's OK for testing purpose.
Sorry for interrupting, we need to check documentation coverage if we promote DeiT template as default. FYI, the performance of multi-label and h-label classification is not validated for DeiT template. Thanks! |
I am currently working on manual testing on Geti. |
85c5994
to
06c4d85
Compare
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.
Checked that DeiT is registered as custom model on Geti.
Summary
How to test
Checklist
License
Feel free to contact the maintainers if that's a concern.