-
Notifications
You must be signed in to change notification settings - Fork 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
Add MobileNetV3 architecture for Detection #3253
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
datumbox
force-pushed
the
mobilenetv3_detection
branch
from
January 14, 2021 16:40
24f1e90
to
30299cc
Compare
13 tasks
datumbox
force-pushed
the
mobilenetv3_detection
branch
2 times, most recently
from
January 15, 2021 09:29
0455f35
to
30299cc
Compare
datumbox
force-pushed
the
mobilenetv3_detection
branch
from
January 17, 2021 22:51
4609c36
to
11408d6
Compare
datumbox
changed the title
[WIP] Add MobileNetV3 architecture for Detection
Add MobileNetV3 architecture for Detection
Jan 18, 2021
fmassa
approved these changes
Jan 18, 2021
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.
Looks great to me, thanks a lot Vasilis!
facebook-github-bot
pushed a commit
that referenced
this pull request
Jan 21, 2021
Summary: * Minor refactoring of a private method to make it reusuable. * Adding a FasterRCNN + MobileNetV3 with & w/o FPN models. * Reducing Resolution to 320-640 and anchor sizes to 16-256. * Increase anchor sizes. * Adding rpn score threshold param on the train script. * Adding trainable_backbone_layers param on the train script. * Adding rpn_score_thresh param directly in fasterrcnn_mobilenet_v3_large_fpn. * Remove fasterrcnn_mobilenet_v3_large prototype and update expected file. * Update documentation and adding weights. * Use buildin Identity. * Fix spelling. Reviewed By: datumbox Differential Revision: D25954553 fbshipit-source-id: 41009b5c6e855cd0b2639902165c57402387d0be
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #1999
Trained using the code committed at 11408d6.
The current temporary pre-trained model was trained:
Submitted batch job 34945231
Accuracy metrics:
Validated with:
Submitted batch job 34643680
Speed benchmark:
0.14 sec per image on CPU