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

detect.py changes for windows #85

Closed
apiszcz opened this issue Feb 1, 2019 · 4 comments
Closed

detect.py changes for windows #85

apiszcz opened this issue Feb 1, 2019 · 4 comments

Comments

@apiszcz
Copy link

apiszcz commented Feb 1, 2019

Initial changes for windows

$ diff detect.py_orig detect.py
2a3,4
> import shutil
> import os
25a28,29
>     if os.path.exists(output): shutil.rmtree(output)
>     os.makedirs(output)
27,28d30
<     os.system('rm -rf ' + output)
<     os.makedirs(output, exist_ok=True)
110c112
<             results_img_path = os.path.join(output, path.split('/')[-1])
---
>             results_img_path = os.path.join(output, path.split(os.sep)[-1])

@glenn-jocher
Copy link
Member

Thanks for the feedback! I've updated detect.py now to be fully windows compatible. I modified this line to use the python pathlib library based on #90 (comment). Let me know if you find any other windows incompatibilities!

save_path = str(Path(output) / Path(path).name)

@sanazss
Copy link

sanazss commented Jul 19, 2019

this is the image after i ran detect.py on gray scale

image

@sanazss
Copy link

sanazss commented Jul 19, 2019

would you kindly let me know what cause this problem? The bottom of the image just doesnt make sense

@glenn-jocher
Copy link
Member

glenn-jocher commented Jul 19, 2019

Hello, thank you for your interest in our work! This is an automated response. Please note that most technical problems are due to:

  • Your changes to the default repository. If your issue is not reproducible in a fresh git clone version of this repository we can not debug it. Before going further run this code and ensure your issue persists:
sudo rm -rf yolov3  # remove exising repo
git clone https://github.com/ultralytics/yolov3 && cd yolov3 # git clone latest
python3 detect.py  # verify detection
python3 train.py  # verify training (a few batches only)
# CODE TO REPRODUCE YOUR ISSUE HERE
  • Your custom data. If your issue is not reproducible with COCO data we can not debug it. Visit our Custom Training Tutorial for exact details on how to format your custom data. Examine train_batch0.jpg and test_batch0.jpg for a sanity check of training and testing data.
  • Your environment. If your issue is not reproducible in a GCP Quickstart Guide VM we can not debug it. Ensure you meet the requirements specified in the README: Unix, MacOS, or Windows with Python >= 3.7, Pytorch >= 1.1, etc. You can also use our Google Colab Notebook to test your code in working environment.

If none of these apply to you, we suggest you close this issue and raise a new one using the Bug Report template, providing screenshots and minimum viable code to reproduce your issue. Thank you!

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

3 participants