update cli scenarios #21106
update cli scenarios #21106
Conversation
There was a problem hiding this comment.
@briacht thanks for updating this.
I added a comment. Let me know what you think.
| - recommendation | ||
| - Future: other machine learning tasks such as image-classification, ranking, anomaly-detection, clustering | ||
| - image classification | ||
| - Future: other machine learning tasks such as ranking, anomaly detection, clustering |
There was a problem hiding this comment.
I wonder if we even need this future bullet point. It's not in the product now, so would remove it and just list what the product supports. Sounds more like a promise and might create confusion.
There was a problem hiding this comment.
Yes, docs shouldn't mention future plans according to the contributor's guide. @briacht Could you update and then we'll merge?
| - recommendation | ||
| - Future: other machine learning tasks such as image-classification, ranking, anomaly-detection, clustering | ||
| - image classification | ||
| - Future: other machine learning tasks such as ranking, anomaly detection, clustering |
There was a problem hiding this comment.
Yes, docs shouldn't mention future plans according to the contributor's guide. @briacht Could you update and then we'll merge?
|
@briacht Ping |
|
I had to create a new PR: #23283 Also I got rid of the "Future" statement. |
|
Closing PR since there's a new one. |
6 participants
Summary
add image classification to cli scenarios