-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Adding Custom Vision object detection sample #952
base: main
Are you sure you want to change the base?
Conversation
@luisquintanilla, I think this is ready for review. I could have missed some stuff, so just let me know if anything needs to be updated. |
@luisquintanilla This should now be ready for a review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jwood803 Looks good. Added a few comments to the README
samples/csharp/end-to-end-apps/StopSignDetection_ONNX/README.md
Outdated
Show resolved
Hide resolved
samples/csharp/end-to-end-apps/StopSignDetection_ONNX/README.md
Outdated
Show resolved
Hide resolved
Additional comments:
|
@luisquintanilla Updated, but let me know if any more changes are needed. |
Thanks @jwood803. For some reason it's still not saving to the top-level directory. |
@jwood803 check out my branch for changes that work in VS. Make sure that |
Adding weather recognition sample to go along with the Custom Vision ONNX tutorial