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

NNVM memory optimization example does not work #6272

Closed
MarvinTeichmann opened this issue May 16, 2017 · 10 comments
Closed

NNVM memory optimization example does not work #6272

MarvinTeichmann opened this issue May 16, 2017 · 10 comments

Comments

@MarvinTeichmann
Copy link

MarvinTeichmann commented May 16, 2017

Steps to reproduce

Run the memcost example provided here.

  • make with_inplace, make with_sharing and make with_both all three use the same amount of Memory Total 1784 MB allocated.

  • make no_optimization runs indefinitely

Environment info

MXNet versions tested:

  • 0.9.5-1
  • build from source / master

Operating System: Linux

@MarvinTeichmann MarvinTeichmann changed the title NNVM GPU memory optimization does not work NNVM memory optimization example does not work May 16, 2017
@MarvinTeichmann
Copy link
Author

@tqchen

@eric-haibin-lin
Copy link
Member

I think this example is out of date, currently MXNet always enable inplace_update, and the macro for shared_memory is updated as NNVM_EXEC_MATCH_RANGE

https://github.com/dmlc/mxnet/blob/986b736b816018b96e9d1e2c358bb7665b80944d/src/executor/graph_executor.cc#L403

https://github.com/dmlc/nnvm/blob/b279286304ac954098d94a2695bca599e832effb/src/pass/plan_memory.cc#L277

@mli we should either remove outdated examples, or keep it up to date by testing it regularly

@yajiedesign
Copy link
Contributor

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!

@MarvinTeichmann
Copy link
Author

The issue still persists. Closing the issue on github does not fix the bug.

@yajiedesign yajiedesign reopened this Sep 30, 2017
@yajiedesign
Copy link
Contributor

@MarvinTeichmann ok it reopen,

@szha
Copy link
Member

szha commented Dec 30, 2017

@apache/mxnet-committers: This issue has been inactive for the past 90 days. It has no label and needs triage.

For general "how-to" questions, our user forum (and Chinese version) is a good place to get help.

@samskalicky
Copy link
Contributor

@MarvinTeichmann is this still an issue? If so, can you update your reproducibility instructions to 1.3 and we'll try and debug if the issue still persists. Thanks!

@szha
Copy link
Member

szha commented Oct 5, 2018

@samskalicky the example hasn't been updated so the instruction remains the same.

@kalyc
Copy link
Contributor

kalyc commented Nov 15, 2018

@MarvinTeichmann your issue has been fixed by this #13235 (comment) can you close it?

@MarvinTeichmann
Copy link
Author

Thanks.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants