-
Notifications
You must be signed in to change notification settings - Fork 272
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
Which caffe version supports ceil_mode in Pooling? #1
Comments
@quietsmile pls see: https://github.com/BVLC/caffe/pull/3057/files |
Oh, this is interesting. I mean, is it compatible with cudnn_pooling implementation? Or, just to make the output a little bit larger, even though the elements in the last row/col may still be zeros? Thanks. |
As i know, cudnn does not support ceil mode, but you'd better check cudnn manual by yourself. |
can anyone sent me your caffe? I don't know which version of caffe should be used. I have tried to use a order version without working.But the new caffe didn't have vision_layers.hpp . I don't know what to do.please help me. |
New version doesnot have vision_layers.hpp. In new version the file pooling_layer.hpp is to be modified. |
@Sarah635 I have the same problem ,how to modify on the new version? |
https://github.com/BVLC/caffe/pull/3057/files Follow this link. Add the + lines and delete the - lines. Instead of file vision_layers.hpp,, modify the file pooling_layers.hpp. Modify these 3 files i.e. pooling_layers.hpp, pooling_layers.cpp and caffe.proto according to link above and you are done. |
@Sarah635 Thank you very much, done! |
Need I change the code of pooling layer.cu? Thank U. |
With the latest version of Caffe, you can also get around this by replacing "ceil_mode: false" with "round_mode: FLOOR". No editing of Caffe source code necessary. Refer BVLC/caffe#6282 . |
@keerthanss
|
@keerthanss I tried both |
Nice work!
However, I found that ceil_mode is not supported in the master version of caffe.
Is there an open-sourced caffe version that supports ceil_mode in pooing? Thanks.
The text was updated successfully, but these errors were encountered: