Skip to content
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.

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

Python interface to blobs and blob data through boost::python #11

Merged
merged 2 commits into from
Jan 20, 2014

Conversation

longjon
Copy link
Contributor

@longjon longjon commented Dec 7, 2013

This is the Python blob interface as it stands. If you'd rather not touch blob.hpp (which I think is reasonable), I think that can be accomplished by switching from inheritance to composition for CaffeBlob, and writing a few more wrappers.

@shelhamer
Copy link
Member

I'm interested in merging this but your PR mentions changes to blob.hpp yet the commits in this branch don't touch it. Are there still changes to commit?

@longjon
Copy link
Contributor Author

longjon commented Jan 15, 2014

I just today updated this to confine the changes to pycaffe.cpp and to expose diff as well as data and params as well as blobs. (I assumed that github would link this PR to a particular commit, but I guess it links to a branch name.) So, yes, there are still changes to commit, and feel free to pull them if you see fit, as I think previous reservations have been addressed.

@shelhamer
Copy link
Member

I checked this out and everything compiles but the python wrapper segfaults instantly. Are there any lingering commits you need to push to your branch?

@longjon
Copy link
Contributor Author

longjon commented Jan 15, 2014

Hm, it works for me on ICSI (although I did have to fiddle my path to avoid a SIGABRT caused by a protobuf version mismatch). Where are you building? You get a segfault on import of pycaffe?

@shelhamer
Copy link
Member

Ok, it could just be my error. Import is fine, but Caffe calls segfault.
I'll check my dims-assume I just screwed up for now.

Le mercredi 15 janvier 2014, longjon notifications@github.com a écrit :

Hm, it works for me on ICSI (although I did have to fiddle my path to
avoid a SIGABRT caused by a protobuf version mismatch). Where are you
building? You get a segfault on import of pycaffe?


Reply to this email directly or view it on GitHubhttps://github.com//pull/11#issuecomment-32426392
.

shelhamer added a commit that referenced this pull request Jan 20, 2014
Python interface to blobs and blob data through boost::python
@shelhamer shelhamer merged commit f8039bc into BVLC:master Jan 20, 2014
@shelhamer
Copy link
Member

Thanks for the wrapper enhancement! (Problems were from miscellaneous build issues on my end–the code was fine.)

@jermainewang jermainewang mentioned this pull request Jan 20, 2014
mitmul pushed a commit to mitmul/caffe that referenced this pull request Sep 30, 2014
Python interface to blobs and blob data through boost::python
andpol5 pushed a commit to andpol5/caffe that referenced this pull request Aug 24, 2016
@ih4cku ih4cku mentioned this pull request Sep 12, 2016
mindcont pushed a commit to mindcont/caffe that referenced this pull request Apr 25, 2017
mbassov pushed a commit to mbassov/caffe that referenced this pull request Nov 10, 2017
DEV-28879 Lifted Structured Embedding Code from rksltnl
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants