Closed gyglim closed 7 years ago
I made some progress regarding this. The padding here is incorrect: https://github.com/axon-research/c3d-keras/blob/master/c3d_model.py#L49
The original has no padding: https://github.com/facebook/C3D/blob/master/C3D-v1.0/src/caffe/layers/pool3d_layer.cpp#L46 or, rather it has right only padding, through this: https://github.com/facebook/C3D/blob/master/C3D-v1.0/src/caffe/layers/pool3d_layer.cpp#L92
by changing above line to
model.add(ZeroPadding3D(padding=((0, 0), (0, 1), (0, 1)), name='zeropad5'))
I now get probablities that are closer to the reference implementation:
wiffle ball: 28.18%
knife throwing: 13.11%
croquet: 10.82%
disc golf: 7.88%
rounders: 4.97%
kickball: 4.46%
bocce: 3.11%
boomerang: 2.02%
dodgeball: 1.78%
pitch and putt: 1.40%
Also, I used the mean from here, which helped getting more accurate results: https://github.com/Lasagne/Recipes/blob/master/examples/Video%20features%20with%20C3D.ipynb
@gyglim thanks for your effort. i was puzzled myself. right-padding instead of symmetric padding did make it close to reference results.
@gyglim : made this change. thanks for your contribution!
First, thanks a lot for providing this tensorflow implementation. I successfully run it and got the same probabilities as given in the script. However I compared on the snippet used in https://github.com/Lasagne/Recipes/blob/master/examples/Video%20features%20with%20C3D.ipynb
and get quite different probabilities compared to the caffe reference implementation. This code:
Caffe:
did you ever verify the output against the caffe implementation? Any idea what the reason for this might be?