Skip to content
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

Add Deformable ConvNets #27

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

souravsingh
Copy link

The codebase is taken from https://github.com/yuyu2172/deformable-conv

@aonotas aonotas self-requested a review March 3, 2019 09:06
@aonotas
Copy link
Contributor

aonotas commented Mar 4, 2019

I will review this PR later!
Thank you for this PR.
Please wait a little.

@aonotas
Copy link
Contributor

aonotas commented Mar 8, 2019

Could you add README.md in ./deformable_convnets/README.md?

I think all file path can be in ./deformable_convnets/. In my opinion, mnist directory is not needed.

Please write requirements library in README.

$ pip install chainercv

@aonotas
Copy link
Contributor

aonotas commented Mar 8, 2019

I am trying to run this PR code,
Is it fine for you?

$ python deformable_convnets/mnist/train_mnist.py --gpu 0
GPU: 0
# Minibatch-size: 100
# epoch: 20
deformable: 1

epoch       main/loss   validation/main/loss  main/accuracy  validation/main/accuracy  elapsed_time
1           2.30162     2.30107               0.112067       0.1135                    95.9675
2           2.30129     2.30102               0.112367       0.1135                    116.356
3           2.30125     2.30106               0.112367       0.1135                    135.722
4           2.30127     2.30101               0.112367       0.1135                    154.666
5           2.30129     2.30095               0.112367       0.1135                    173.193
6           2.30129     2.30103               0.112367       0.1135                    192.252
7           2.30128     2.3011                0.112367       0.1135                    211.127
8           2.30128     2.30107               0.112367       0.1135                    229.825
9           2.30126     2.30098               0.112367       0.1135                    250.089
10          2.30127     2.30106               0.112367       0.1135                    269.097

If you could, please add example result of this code in README.md?

@souravsingh
Copy link
Author

@aonotas The result looks good. Should we also add a model for Pascal VOC object detection, since it is present in the repo?

@aonotas
Copy link
Contributor

aonotas commented Mar 9, 2019

If you could add a model for Pascal VOC object detection and its results, please add it!
Thank you for your great work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants