8000 Added support for python 3 and NCCL by willyd · Pull Request #5527 · BVLC/caffe · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Added support for python 3 and NCCL #5527

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Apr 13, 2017
Merged

Added support for python 3 and NCCL #5527

merged 1 commit into from
Apr 13, 2017

Conversation

willyd
Copy link
Contributor
@willyd willyd commented Apr 13, 2017

@cypof I reworked the NCCL and python3 compatibility of #5400. Can you try this and let me know.

@cypof
Copy link
Member
cypof commented Apr 13, 2017

OK, everything seems to work, thanks

@cypof cypof merged commit a54b943 into BVLC:master Apr 13, 2017
@willyd
Copy link
Contributor Author
willyd commented Apr 13, 2017

Thanks

This was referenced Apr 14, 2017
Open
Merged
@lvzhaoyang
Copy link

Hi,

I have compiling issue which is

/home/zlv30/framework/caffe/python/caffe/_caffe.cpp: In function ‘boost::python::api::object caffe::NCCL_New_Uid()’:
/home/zlv30/framework/caffe/python/caffe/_caffe.cpp:374:10: error: could not convert ‘uid’ from ‘std::__cxx11::string {aka std::__cxx11::basic_string<char>}’ to ‘boost::python::api::object’
   return uid;

which I think it is caused by this pull request. Can you take a look at it?

I am using python 2 on ubuntu 16.04.

@willyd
Copy link
Contributor Author
willyd commented Apr 20, 2017

This was supposed to be fixed in eeebdab

@lvzhaoyang Can you confirm that you are using the latest master?

@dncliss dncliss mentioned this pull request Nov 6, 2018
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@willyd @cypof @lvzhaoyang
0