Skip to content

Enhanced snapshotting, access to solver parameters, const correctness and style improvements. #5549

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
Open
wants to merge 24 commits into
base: master
Choose a base branch
from
Open

Enhanced snapshotting, access to solver parameters, const correctness and style improvements. #5549

wants to merge 24 commits into from

Conversation

ensafi
Copy link

@ensafi ensafi commented Apr 19, 2017 •
edited
Loading

  1. Added const getters and fixed constness of existing getters for various parameters that applications may wish to track via the C++ API during training.
  2. Added member variables to make temporary variables persistent and accessible via getters, including learning rate and mean scores (accuracy).

Added const getters and fixed constness of existing getters for vario…
1edd005 
…us parameters that applications may wish to track via the C++ API during training. Added member variables and initialization to make temporary variables persistent and accessible via getters, including learning rate and mean scores (accuracy).
@ensafi ensafi changed the title Added const getters and fixed constness of existing getters for vario… Enhanced snapshotting, access to solver parameters, const correctness and style improvements. May 22, 2017
@ensafi
Copy link
Author

ensafi commented May 22, 2017 •
edited
Loading

  1. Added protocol buffer snapshotting support and a custom snapshot callback to facilitate user-defined behavior without necessarily relying on disk files for saving/restoring snapshots.
  2. Renamed global macros to include CAFFE prefix: CAFFE_STRINGIFY and CAFFE_AS_STRING.

Eskandar Ensafi added 10 commits June 15, 2017 15:10
Merge branch 'master' of https://github.com/BVLC/caffe
aced0f4
SolverState is now optional, to maximize flexibility of custom snapsh…
55c2012 
…ot functions.
Merge branch 'master' of https://github.com/BVLC/caffe
8d3be4b
Merge branch 'master' of https://github.com/BVLC/caffe
5922a92
Merge branch 'master' of https://github.com/BVLC/caffe
03c20c8
Fixed bug related to storing mean score, now properly stored for all …
ce926ec 
…test nets, for all blobs. Learning rate now part of Solver base class, but it is up to implementations to update its value.
Conforming to 2 space convention between code and comment.
e94c5f3
Reverted to local arrays, and storing copies of mean scores as a memb…
006e8b3 
…er of Solver.
Storing all test net mean scores and output blob indices, with API re…
76f57e3 
…vised to allow future addition of training net scores and indices.
Merge branch 'master' of https://github.com/BVLC/caffe
69a3a0e
@Noiredd Noiredd mentioned this pull request Feb 8, 2018
Open
19 tasks
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
enhancement style
Projects
None yet
Milestone
No milestone
Development

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