-
Notifications
You must be signed in to change notification settings - Fork 45
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
DOCS-2783: Update detect-people.md #3223
Conversation
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.
looks good, thank you
@@ -97,9 +97,7 @@ Create the service. | |||
|
|||
In the resulting ML model service configuration pane, ensure that **Deploy model on machine** is selected for the **Deployment** field. | |||
|
|||
Click on **Select models** and select the `ml-models-scuttle:people` model from the registry (the **people** model by **ml-models-scuttle**) to deploy an object detection TFLite model that has been trained to be able to detect a person. | |||
|
|||
Click the **Save** button in the top right corner of the page to save your configuration and deploy an object detection TFLite model that has been trained to be able to detect a person. |
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.
I do think we should still tell people to save it though. here I meant to repeat to drill it in ("tell show tell")
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.
why though? they will be saving immediately again right after
5323cdc
to
647173f
Compare
You can view a rendered version of the docs from this PR at https://docs-test.viam.dev/3223 |
No description provided.