[ai-vision-face] should this package be under sdk/vision directory?

[jeremymeng]

The package is currently at sdk/face/ai-vision-face-rest directory. face doesn't feel like an azure service.

There's already sdk/vision/ai-vision-image-analysis-rest there.

[chungshengfu]

According to this documentation: What are Azure AI services?, face, vision, speech, document intelligence are parallel services, so current structure looks reasonable to me.
Besides, since they logically belong to Azure AI Services (formerly Azure Cognitive Services), probably the structure in azure-rest-api-spec is more like what you want? https://github.com/Azure/azure-rest-api-specs/tree/main/specification/ai

[jeremymeng]

face, vision, speech, document intelligence are parallel services, so current structure looks reasonable to me.

Hmm, there must be a reason why vision is added before face? Also should image-analysis be moved out of vision directory if we follow api-specs structure? I see Face and ImageAnalysis there.

[xirzec]

I see a 'vision' service folder in .NET and Python:

https://github.com/Azure/azure-sdk-for-net/tree/main/sdk/vision
https://github.com/Azure/azure-sdk-for-python/tree/main/sdk/vision

Neither seem to have this version of the SDK yet, only the old version under cognitiveservices.

[jeremymeng]

Docs doesn't seem very consistent, I see Face is listed as one of AI Vision services here: https://learn.microsoft.com/en-us/azure/ai-services/computer-vision/

[Han-msft]

I see a 'vision' service folder in .NET and Python:

https://github.com/Azure/azure-sdk-for-net/tree/main/sdk/vision https://github.com/Azure/azure-sdk-for-python/tree/main/sdk/vision

Neither seem to have this version of the SDK yet, only the old version under cognitiveservices.

Face SDK for the other languages are still under review. All of them are targeting sdk/face folder.
Computer vision and Face API are different service when creating resource in Azure. That's why we put SDK in different folders.
We should have a discussion with our PM if we need go to vision folder with image anaylsis.

.NET PR: Azure/azure-sdk-for-net#43820
Java PR: Azure/azure-sdk-for-java#40023
Python PR: Azure/azure-sdk-for-java#40023

[Han-msft]

Our PM said we should keep branding in vision for now, but it will be pull out of vision in the future.
We'd like to purpose staying at sdk/face to keep the flexibility.
We tried to move our code to sdk/vision in each language but encountered several pipeline issues.
JS got test failure.
Java got TypeSpec generation validation failure.
Since face and image analysis are separate services owned by separate teams, we should not block each other's development.
@xirzec @jeremymeng do you feel it's okay for us to stay at sdk/face?

[qiaozha]

@Han-msft I have a question, do you have independent service tree nodes?

[Han-msft]

@Han-msft I have a question, do you have independent service tree nodes?

Yes, Face API is Cognitive Services Face API under Cognitive Vision.
https://microsoftservicetree.com/services/82b9a1cf-bbe3-488b-bdf9-fee717fa393a
Image analysis is a subset API in Cognitive Services Computer Vision API under Cognitive Vision.
https://microsoftservicetree.com/services/a2eae2da-667d-4b19-8789-9471e346ca84

[qiaozha]

Then I think it's fine to put them into two different folders.

[jeremymeng]

@Han-msft thanks for confirming! I am fine with sdk/face too. It's not customer-facing and more for the consistency of our repo.