Skip to content

Comments

Added check before register ptr to python#3866

Closed
gongenhao wants to merge 1 commit intoBVLC:masterfrom
gongenhao:master
Closed

Added check before register ptr to python#3866
gongenhao wants to merge 1 commit intoBVLC:masterfrom
gongenhao:master

Conversation

@gongenhao
Copy link

This PR added check before register ptr to python for Net,Blob,Layers which is able to solve the warning related to:
#3575

and
https://groups.google.com/forum/#!topic/caffe-users/C_air48cISU

On master branch, there are warnings when run pycaffe.
/home/ubuntu/caffe/python/caffe/pycaffe.py:13: RuntimeWarning: to-Python converter for boost::shared_ptrcaffe::Net already registered; second conversion method ignored.
from ._caffe import Net, SGDSolver, NesterovSolver, AdaGradSolver,
/home/ubuntu/caffe/python/caffe/pycaffe.py:13: RuntimeWarning: to-Python converter for boost::shared_ptrcaffe::Blob already registered; second conversion method ignored.
from ._caffe import Net, SGDSolver, NesterovSolver, AdaGradSolver,
/home/ubuntu/caffe/python/caffe/pycaffe.py:13: RuntimeWarning: to-Python converter for boost::shared_ptrcaffe::Solver already registered; second conversion method ignored.
from ._caffe import Net, SGDSolver, NesterovSolver, AdaGradSolver, \

@shelhamer
Copy link
Member

This doesn't seem quite right. With boost 1.60 I have this kind of error on make pytest:

TypeError: No to_python (by-value) converter found for C++ type: boost::shared_ptrcaffe::Net

This is why #3575 was merged, though I agree the current warnings with other versions are a hassle and I would like to resolve the issue.

@ZhengRui
Copy link

My warnings disappeared after using this fix, my boost version 1.58, osx10.10.

@shelhamer
Copy link
Member

shelhamer commented Apr 28, 2016

Closing as fixed by #4069 (which works across boost versions without warnings). Thanks @gongenhao for the potential fix.

@shelhamer shelhamer closed this Apr 28, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants