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

The demo experiment cannot be run successfully #1

Closed
Yhdian opened this issue Jul 29, 2021 · 3 comments
Closed

The demo experiment cannot be run successfully #1

Yhdian opened this issue Jul 29, 2021 · 3 comments

Comments

@Yhdian
Copy link

Yhdian commented Jul 29, 2021

Hello.

Thanks you for the great work.

I used your checkpoints to test the deepfashon2 images, but I encountered some problems. I tried the following modifications:

  1. The file reppoints_detector_kp_gt.py is missing, so I comment out import RepPointsDetectorKpGT. I'm not sure if this will affect the model accuracy.
  2. In the file https://github.com/ShenhanQian/KGDet/blob/master/configs/kgdet_moment_r50_fpn_1x-deepfashion2.py, the neck type is FPN2, but there is no FPN2 module. I change the FPN2 type into FPN and comment out select_out=[2].

Finally, I can run the demo. However, the output is not correct. Can you give some advises so that I can run the demo experiment successfully? Thanks.

@ShenhanQian
Copy link
Owner

Thanks for your attention, and sorry for the bugs.

  1. reppoints_detector_kp_gt.py is redundant, so I have removed it in the latest commit.
  2. I have added fpn2.py in the latest commit. FPN2 is a variation of FPN with minor modifications. It's up to your choice whether to use it. But if you need to use our checkpoint, please keep the default config.

When trying to replicate your problem, I found another problem in the DeepFashion2API due to version conflicts and fixed them. Therefore, I highly recommend that you pull the recent commits and re-install deepfashion2_api in the root directory.

Also, I include the demo dataset in this repo to ease testing.

@Yhdian
Copy link
Author

Yhdian commented Jul 29, 2021

Thanks very much for your update. I can run the demo successfully now.

@Yhdian Yhdian closed this as completed Jul 29, 2021
@ShenhanQian
Copy link
Owner

You are welcome.

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

No branches or pull requests

2 participants