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

cython nnvm include path error #12123

Closed
petronny opened this issue Aug 10, 2018 · 3 comments
Closed

cython nnvm include path error #12123

petronny opened this issue Aug 10, 2018 · 3 comments

Comments

@petronny
Copy link
Contributor

https://github.com/apache/incubator-mxnet/blob/af15853a5989e8e40a7300669c8d0c42f0be79ad/python/setup.py#L92

This line should be

include_dirs=["../include/", "../3rdparty/tvm/nnvm/include"],

#11130
@marcoabreu
Copy link
Contributor

@tqchen

@tqchen
Copy link
Member

tqchen commented Aug 10, 2018

Thanks for the catch, can you send a PR to fix this?

@petronny
Copy link
Contributor Author

petronny commented Aug 11, 2018 •
edited
Loading

Sorry, I'm afraid not because I have a pool network connection to apache JIRA...
But I have created a PR on github, will it work?

@petronny petronny mentioned this issue Aug 11, 2018
Merged
6 tasks
marcoabreu pushed a commit that referenced this issue Aug 11, 2018
@petronny @marcoabreu
fix cython nnvm include path (#12133)
9fd45b8 
#12123
XinYao1994 pushed a commit to XinYao1994/incubator-mxnet that referenced this issue Aug 29, 2018
@petronny @marcoabreu
fix cython nnvm include path (apache#12133)
8110fee 
apache#12123
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@tqchen @petronny @marcoabreu