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 file existence check in yolo_console_dll.cpp #8673

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

Conversation

RRRChangeche
Copy link

I'm trying to use C++ API to integrate darknet with my C++ project on windows. I followed the tutorial on this topic in README.md.

During the test toward my input image files (with #define OPENCV), I found that whenever I gave the wrong file path, the console crashed because the given file path did not exist. To be more convenient for input image files testing, I implemented the file existence check section in yolo_console_dll.cpp.

@cenit
Copy link
Collaborator

cenit commented Nov 4, 2022

the fix is not portable on windows, and so not acceptable. I am sorry.
You could try to rework it with the filesystem library from c++17, maybe

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