Skip to content

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

@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?

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

Successfully merging this pull request may close these issues.

3 participants