Closed azure-sdk closed 10 months ago
API Spec PR link: https://github.com/Azure/azure-rest-api-specs-pr/pull/15240
Meeting updated by Kailash Joshi
Service Name: Cognitive Services Computer Vision API Review Created By: Kailash Joshi Review Date: 10/24/2023 01:00 PM PT Onboarding Record: PR: Hero Scenarios Link: Not Provided Core Concepts Doc Link: Not Provided
Description: API review for ImageAnalysis API for GA API Spec PR link: Azure/azure-rest-api-specs-pr#15240
Detailed meeting information and documents provided can be accessed here For more information that will help prepare you for this review, the requirements, and office hours, visit the documentation here
Notes from API Review Meeting 10/24/23
Previous feedback:
No more operations in the path -- good
Long running operation ? No, it is synchronous
Body content types missing because the OpenAPI uses x-ms-paths
Why are some fields sent in the parameters and others in the body?
Should we use "confidence", "confidenceScore", "score"?
Bounding box is in pixels ... good. We should make sure it is pixels everywhere.
"TagsResult.values" seems a little verbose ... could it be "Tags.values" ?
Why are we using kebab case rather than camelCase for query parameter names?
All the features and results but Read are nouns, and that one is a verb.
This looks good for GA. Please consider the above and any comments on the PR, then tag us here when you are ready to merge and we'll add the "signedOff" label.
New API Review meeting has been requested.
Service Name: Cognitive Services Computer Vision API Review Created By: Kailash Joshi Review Date: 10/24/2023 01:00 PM PT Onboarding Record: PR: Hero Scenarios Link: Not Provided Core Concepts Doc Link: Not Provided
Description: API review for ImageAnalysis API for GA
Detailed meeting information and documents provided can be accessed here For more information that will help prepare you for this review, the requirements, and office hours, visit the documentation here