Skip to content

Conversation

sguada
Copy link
Contributor

@sguada sguada commented Jan 24, 2014

Rearranged the code to do the snapshot after computing the loss and the test accuracy.

@Yangqing
Copy link
Member

Is there a specific use case for this change...? I feel indifferent both ways, but it seems to make more sense to snapshot to disk as soon as the network update step.

@sguada
Copy link
Contributor Author

sguada commented Jan 24, 2014

The motivation is to know the loss and accuracy of the network before
trying to save it to disk. Since I was getting errors sometime in while
saving, I couldn't know that. But it would fine otherwise.

Sergio

2014/1/23 Yangqing Jia [email protected]

Is there a specific use case for this change...? I feel indifferent both
ways, but it seems to make more sense to snapshot to disk as soon as the
network update step.


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

Yangqing added a commit that referenced this pull request Jan 24, 2014
Do snapshot after computing loss and test accuracy
@Yangqing Yangqing merged commit f17cfa0 into BVLC:master Jan 24, 2014
mitmul pushed a commit to mitmul/caffe that referenced this pull request Sep 30, 2014
Do snapshot after computing loss and test accuracy
lukeyeager pushed a commit to lukeyeager/caffe that referenced this pull request Nov 3, 2015
Cherry-pick batch normalization PR.  Still pending cuDNNv4 version
happynear pushed a commit to happynear/caffe that referenced this pull request May 12, 2016
…03_21

Merge bvlc/windows@{2016-03-21} into master
Cysu pushed a commit to Cysu/caffe that referenced this pull request May 20, 2016
DVEfremov pushed a commit to DVEfremov/caffe that referenced this pull request Feb 2, 2017
- in fact caffe uses DType as float but looks like double type support not correctly implemented:
  there are kerenel compilation errors on startup atleast for Mali GPU even if
DOUBLE_SUPPORT_AVAILABLE not defined, so looks like usages of DOUBLE_SUPPORT_AVAILABLE not for all required parts of code,
DVEfremov pushed a commit to DVEfremov/caffe that referenced this pull request Feb 2, 2017
DVEfremov pushed a commit to DVEfremov/caffe that referenced this pull request Feb 2, 2017
DVEfremov pushed a commit to DVEfremov/caffe that referenced this pull request Feb 2, 2017
DVEfremov pushed a commit to DVEfremov/caffe that referenced this pull request Feb 3, 2017
- add ability to disable DOUBLE SUPPORT with compilation flag
  DISABLE_DOUBLE_SUPPORT
  (hardcoded DOUBLE_SUPPORT_AVAILABLE atleast doesn't work for Mali GPU at caffe time runtime)
DVEfremov pushed a commit to DVEfremov/caffe that referenced this pull request Feb 6, 2017
DVEfremov pushed a commit to DVEfremov/caffe that referenced this pull request Feb 7, 2017
- add ability to disable DOUBLE SUPPORT with compilation to cl_kernels.sh and header.cl (fix review comment)
- add generated cl_kernels.cpp (as mentioned before for MacOS and Windows users where code generation does not work at this moment)
naibaf7 added a commit that referenced this pull request Feb 7, 2017
OpenCl kernel compilation errors for android #51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants