-
Notifications
You must be signed in to change notification settings - Fork 507
Python wrapper #59
base: master
Are you sure you want to change the base?
Python wrapper #59
Conversation
…apper (run_c3d_classification.py)
Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at [email protected]. Thanks! |
Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks! |
I believe the way you read in the mean is wrong (mixed dimension). I had the problem that the mean didn't match in my Pull request to Lasagne Recipes: Lasagne/Recipes#41 It appears the values are read in a different order from the one they are written. The elements in C3D are shaped as (channel,frameNr,y,x), but you read in the mean as (frameNr,channel,y,x). Here is the mean difference between the frame means visualized: Difference between the first channel of image 6 and image 5: i.e. there is a jump in the values, which comes from reading the input wrong. |
@gyglim thanks for the fix. i'll incorporate this in this PR soon. |
Added some fixes / sample script for UCF-101. |
I believe this commit completes all there is to support C3D in python scripts. Please let me know if anyone has a feedback or a question. Thanks. |
@chuckcho Thank you very much for your contribution. I did not have time to fully review this for now (so it may take sometime to merge as soon as I fully try/test it). |
@chuckcho I wonder whether I could directly load the pre-trained Sport1m model in |
@fengwhu.
|
@chuckcho Fine. It works now. Thanks a lot! |
Made pycaffe available for C3D classification with a sample python wrapper (run_c3d_classification.py)