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

Bugfix: final snapshot iter number is off by one #2072

Merged
merged 1 commit into from
May 14, 2015

Conversation

jeffdonahue
Copy link
Contributor

Since #1228 the final snapshot has saved as *_iter_N.{solverstate,caffemodel} where N = max_iter + 1. This fixes that. @longjon

weiliu89 added a commit to weiliu89/caffe that referenced this pull request Apr 14, 2015
@longjon
Copy link
Contributor

longjon commented May 14, 2015

Looks good, this logic is cleaner and more correct. Thanks @jeffdonahue!

@longjon longjon closed this May 14, 2015
@longjon longjon reopened this May 14, 2015
longjon added a commit that referenced this pull request May 14, 2015
Bugfix: final snapshot iter number is off by one
@longjon longjon merged commit 07f56a0 into BVLC:master May 14, 2015
@jeffdonahue jeffdonahue deleted the final-snapshot-off-by-one branch May 15, 2015 01:25
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