-
Notifications
You must be signed in to change notification settings - Fork 488
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
YoloNAS benchmarking tutorial #1076
Conversation
…nas_guide' into feature/SG-865_benchmarking_yolonas_guide
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.
Thanks!
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.
my comment inline
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.
LGTM, the content is clear but there are some links/format to be fixed.
Also added some comments on the display but these are just notes/ideas.
One thing that we should do is to add this to mkdocs.yaml so that this would be displayed on the documentation website.
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.
LGTM, the content is clear but there are some links/format to be fixed.
Also added some comments on the display but these are just notes/ideas.
One thing that we should do is to add this to mkdocs.yaml so that this would be displayed on the documentation website.
@shaydeci great improvement, aside small typos - LGTM
|
Hehe sure! |
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.
some small typos
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.
LGTM
No description provided.