-
Notifications
You must be signed in to change notification settings - Fork 6.8k
Static memory allocation for cached_op #10817
Conversation
include/mxnet/op_attr_types.h
Outdated
@@ -145,6 +145,11 @@ class OpStatePtr { | |||
void reset() { | |||
ptr_.reset(); | |||
} | |||
/* \brief checks whether the managed object is managed only by the current | |||
OpStatePtr instance */ | |||
bool unique() { |
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.
where is this used? why expose this detail?
b223a77
to
78d178a
Compare
b6c263b
to
b1bf748
Compare
…pache#10817)" This reverts commit 2dbd143.
…pache#10817)" This reverts commit 2dbd143.
@piiswrong I have mixed results with this PR. Did you benchmark your code? It is DenseNet121 running on multilabel classification task on multigpu. Batches are fixed shapes (N, 3, 224, 224) Here are my results:
No warning or error was thrown. ~speed slightly faster and outside of normal variations. |
…pache#10817)" This reverts commit 2dbd143.
…ed_op (apache#10817)" (apache#11311)" This reverts commit e48a8fd.
…pache#10817)" (apache#11311) This reverts commit 2dbd143.
…pache#10817)" (apache#11311) This reverts commit 2dbd143.
Description
(Brief description on what this PR is about)
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.
Changes
Comments