Skip to content

Added support for python 3 and NCCL#5527

Merged
cypof merged 1 commit intoBVLC:masterfrom
willyd:nccl-py3
Apr 13, 2017
Merged

Added support for python 3 and NCCL#5527
cypof merged 1 commit intoBVLC:masterfrom
willyd:nccl-py3

Conversation

@willyd
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor Author

willyd commented Apr 13, 2017

Thanks

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

lvzhaoyang commented Apr 20, 2017

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
Copy Markdown
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