Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Revert "Static memory allocation for cached_op (#10817)" #11311

Merged
merged 1 commit into from
Jun 15, 2018

Conversation

marcoabreu
Copy link
Contributor

@marcoabreu marcoabreu commented Jun 15, 2018

This reverts commit 2dbd143.

See #11171 for reference. #10817 has not been approved and is causing regressions.

@marcoabreu marcoabreu requested a review from szha as a code owner June 15, 2018 21:02
@zheng-da
Copy link
Contributor

can we move forward instead of backward?

@tqchen tqchen closed this Jun 15, 2018
@tqchen
Copy link
Member

tqchen commented Jun 15, 2018

close due to bugfix introduced in #11310

@marcoabreu marcoabreu reopened this Jun 15, 2018
@tqchen tqchen closed this Jun 15, 2018
@tqchen
Copy link
Member

tqchen commented Jun 15, 2018

#11310

@marcoabreu marcoabreu reopened this Jun 15, 2018
@tqchen tqchen closed this Jun 15, 2018
@marcoabreu marcoabreu reopened this Jun 15, 2018
@marcoabreu marcoabreu merged commit e48a8fd into apache:master Jun 15, 2018
@tqchen
Copy link
Member

tqchen commented Jun 15, 2018

CI has not yet passed, and this is a self-merging...

tqchen added a commit that referenced this pull request Jun 15, 2018
piiswrong added a commit to piiswrong/mxnet that referenced this pull request Jun 15, 2018
@marcoabreu marcoabreu deleted the revert-10817 branch June 15, 2018 22:23
@marcoabreu
Copy link
Contributor Author

I'd like to introduce you to the Apache guidelines: https://www.apache.org/foundation/voting.html

"2. Code modifications"

@tqchen
Copy link
Member

tqchen commented Jun 15, 2018

with great power comes great responsibility. I would never abused that power to any changes just to make a point and bring things backward.

zheng-da pushed a commit to zheng-da/incubator-mxnet that referenced this pull request Jun 28, 2018
XinYao1994 pushed a commit to XinYao1994/incubator-mxnet that referenced this pull request Aug 29, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants