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.

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

Python wrapper fixes and re-arrangement #291

Closed
wants to merge 1 commit into from
Closed

Python wrapper fixes and re-arrangement #291

wants to merge 1 commit into from

Conversation

shelhamer
Copy link
Member

N.B. this PR is against master to address users' current problems, and reduce the number of issues raised that are better solved by examples and documentation.

This is a placeholder to let everyone know I'm working on (list in progress):

  • formatting helpers to convert images to Caffe format for deployment, or convert Caffe images back to normal images for viewing
  • batching helper for feeding inputs to the net with padding as needed
  • python exceptions instead of library crashes
  • a more complete wrapper example that shows how image formatting, computing the forward pass,
    and reading off predictions works
  • properly separate responsibilities between the pycaffe wrapper, the wrapper example (wrapper.py) and detection example (detector.py)

@shelhamer shelhamer changed the title Python wrapper fixes and re-arrangement (work in progress) Python wrapper fixes and re-arrangement Apr 3, 2014
@shelhamer shelhamer mentioned this pull request Apr 5, 2014
Merged
@longjon @shelhamer
switch from inheritance to directly overriding methods for caffe.Net
5b940f7 
Also, internally drop the "Caffe" prefix for classes exported by the
internal _caffe module. This change means that SGDSolver will
automatically return a Net with the "nice" Python interface rather than
the internal one, (and avoids the extra layer of method wrapping that
results from switching to composition).

[Cherry-picked by @shelhamer from longjon/pycaffe-solver@a8a0191 to
simplify future merge of master fixes to dev.]
@shelhamer
Copy link
Member Author

Closing in favor of a PR to dev after merge of #286 and its re-arrangement of caffe.Net, after which the python improvements will be cherry-picked to master.

@shelhamer shelhamer closed this Apr 9, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@longjon longjon

Labels
interface
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@shelhamer @longjon