line 9~20 in train.py may not work about yolov3 HOT 5 CLOSED

Resolved with PR #45.

from yolov3.

Oh, thats an interesting comment @violet17. Yes at first this did not work, which is why I added line 23 before importing test.py, to prevent the first namespace from impacting the second.

I did not think about your solution, but yes this could work also. Which one is better from a 'best-practices' standpoint do you think?

from yolov3.

Thanks for your reply. Yes, the namespace in the test.py is used in your code.

from yolov3.

I was thinking about doing what you said, but I realied if we moved the parser arguments inside test.py into the if __name__ == '__main__': statement, we still need to tell it to use latest.pt instead of the default yolov3.pt. I do this now by redefining the the test.py -weights_path in line 205, but if the namespace were not defined yet I wouldn't be able to do this.

from yolov3.

So sorry about that. And thank you for the reply. I was confused about the scope of the namespace before. Now I realize that the first namespace could be used in train.py, and would be used in test.py. And the scope of the second namespace is test.py.

from yolov3.