Fixed up accuracy layer. #1942
Fixed up accuracy layer. #1942
Conversation
src/caffe/proto/caffe.proto
Outdated
There was a problem hiding this comment.
This is added because it allows one to define databases that store several Datums to one keyword.
There was a problem hiding this comment.
Leave that for another PR, where it is used.
There was a problem hiding this comment.
Seconding @sguada. Note also that there is a BlobProtoVector that could be
used for packing multiple pieces of data too.
On Sun, Feb 22, 2015 at 4:01 PM, Sergio Guadarrama <notifications@github.com
wrote:
In src/caffe/proto/caffe.proto
#1942 (comment):@@ -30,6 +30,10 @@ message Datum {
optional bool encoded = 7 [default = false];
}+message DatumVector {
Leave that for another PR, where it is used.
—
Reply to this email directly or view it on GitHub
https://github.com/BVLC/caffe/pull/1942/files#r25138930.
c249d14 to
415e668
Compare
|
The travis test timed out, it AFAIK it builds and tests fine. |
415e668 to
8916348
Compare
|
@sguada @shelhamer I concede DatumVector is out. |
Fixed up accuracy layer.
|
Nice PR. I'm currently using it and it works as advertised. |
|
Thanks for the accuracy layer extension. This needs a rebase after the generalization to N-D blobs in #1970. |
24ed8c8 to
6e20ca1
Compare
|
@shelhamer @jeffdonahue @philkr Thanks. I don't have a GPU implementation planned ATM, but I will push one as soon as I have one. |
6e20ca1 to
1440731
Compare
Merge branch 'master' of https://github.com/BVLC/caffe into fix_accuracy_layer
This PR extends the accuracy layer to behave more like the SoftmaxWithLoss layer. It allows bottom datum to have a width and height, as well as the ignore_label option. Original functionality of the layer remains unchanged.