-
Notifications
You must be signed in to change notification settings - Fork 6.8k
Conversation
@mxnet-label-bot add [pr-awaiting-review] |
@arcadiaphy Thanks for your contribution! |
@sxjscience @KellenSunderland Gentle ping... |
@mxnet-label-bot add [Operator] |
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. Thank you for the improvement!
@sxjscience @KellenSunderland |
I think it should be good. One concern is it may be backward incompatible due to the |
Would be nice to apply to ROIAlign similarly since GluonCV have transitioned to use ROIAlign in recent networks. |
This is great, is the performance increase only due to type changes as in the description? |
@larroy I think the main reason is that the indices of the maximum numbers are saved in this PR. |
LGTM. Many thanks for the speedup / refactor. |
Description
As title. The major changes in implementation:
Benchmarking script
Result
Before:
After:
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.
Changes
Comments