-
Notifications
You must be signed in to change notification settings - Fork 40
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
Refactoring all the code to simplify things #10
base: master
Are you sure you want to change the base?
Conversation
GPU single element access is much slower
Trying to mimic exactly Caffe's SGD
…viderROI to use FRCNN instead
…g. Need to check SPP
Cleaned a bit the Tester. Need to add argcheck to functions (but first need to figure out how to pass several parameters without it crashing, maybe using ordered arguments)
Speeds up bounding box processing time
Temporary fix for #18 I should add groups to `nn` soon.
Just a heads up on this branch. |
Hi, What's the status of this PR? Was there a problem that you didn't merge in the main code? Are you getting correct results? |
Hey @erfannoury , |
Hi @fmassa, Yes, I understand. I'm looking for a working implementation of Fast R-CNN in either Torch or PyTorch, but unfortunately there aren't any other working implementations out there except Girshick's implementation in pycaffe. |
As far as I remember, the implementation of Fast R-CNN here was working. |
This PR substitutes #5.