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? |
goodsong81
left a comment
There was a problem hiding this comment.
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
goodsong81
left a comment
There was a problem hiding this comment.
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.