-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
Implementation of YOLACT forward_dummy to enable get_flops #6079
Conversation
Codecov Report
@@ Coverage Diff @@
## master #6079 +/- ##
==========================================
+ Coverage 61.44% 61.82% +0.38%
==========================================
Files 304 315 +11
Lines 23855 25163 +1308
Branches 3942 4185 +243
==========================================
+ Hits 14657 15558 +901
- Misses 8415 8796 +381
- Partials 783 809 +26
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
The change does not influence |
My local lint passed, is there any updates on the docformatter check? |
My local lint passed too. Really wired. @hhaAndroid @ZwwWayne any ideas? |
lint passed @RangiLyu |
The test failed for something not related to this change |
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
…b#6079) * update forward_dummy * improve code * fix lint * include forward_dummy in unit test * remove hardcode * fix lint * fix bug * change simple_test to forward * update forward_dummy * fix problems * add test comment * fix lint
Motivation
To enable the
get_flops
feature on YOLACT, theforward_dummy
needs to be implementedModification
The
forward_dummy
function is implemented.Closes #5891