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

Incorrect arg parsing for yolo_video? #220

Open
leo-p opened this issue Aug 24, 2018 · 2 comments · May be fixed by #543
Open

Incorrect arg parsing for yolo_video? #220

leo-p opened this issue Aug 24, 2018 · 2 comments · May be fixed by #543

Comments

@leo-p
Copy link

leo-p commented Aug 24, 2018

I was recently trying to perform prediction on images with a pre-trained yolov3 model that I had already converted to keras with convert.py. The issue was that the path options such as --classes or --model where not being taken into account.

After further digging, it appears that the argparse and attribute names don't match. For instance with the model path:

I've found an easy fix by adding a destination tag to the argparse:

  • model: '--model', type=str, dest='model_path',
  • anchors: '--anchors', type=str, dest='anchors_path',
  • classes: '--classes', type=str, dest='classes_path',

Do you know if I missed anything?

@tom-foster-wcc
Copy link

tom-foster-wcc commented Aug 24, 2018

I would suggest a pull for this. I was about to work on this problem myself but just read this so thanks for saving me time!

The other way was to just change the command line argument to '--model_path' etc, but yours matches the documentation which is much nicer 🥇

@yush1ga yush1ga linked a pull request Aug 29, 2019 that will close this issue
@josephofiowa
Copy link

thanks @leo-p !

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 a pull request may close this issue.

3 participants