Skip to content

Conversation

@der3318
Copy link
Member

@der3318 der3318 commented Jul 23, 2024

Purpose

A follow-up of #369 and https://github.com/MicrosoftDocs/azure-docs-pr/pull/281502. New samples are now published, so we are re-organizing legacy face code snippets that are not referenced anymore:

  • Keep the "{Language}/Face/" root folders clean, hosting only the top and currently recommended quick start codes.
  • Backup the samples written in old SDKs in a subfolder named "{Language}/Face/cognsvcsdk/".

Take C# for example:

dotnet/Face/
├── cognsvcsdk                  (snippets using old Microsoft.Azure.CognitiveServices.Vision.Face SDK)
│   ├── analyze.cs
│   ├── detect.cs
│   ├── FaceQuickstart.cs
│   ├── FaceQuickstart-single.cs
│   └── identify.cs
├── Detect.cs                   (latest Azure.AI.Vision.Face SDK demoed in official learning docs)
├── FindSimilar.cs
├── Quickstart.cs
└── rest                        (basic usage via raw HttpClient REST API)
    └── detect.cs

Also applying feedback from peers: explicitly indicating (i.e., comment and variable naming) we are interacting with "large" person group to avoid possible confusion.

Does this introduce a breaking change?

  • Yes
  • No

Pull Request Type

What kind of change does this Pull Request introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Documentation content changes
  • Other... Please describe:

How to Test

Visual Studio 2022 | .NET 8.0 (LTS)

$ dotnet run

Python 3.8.9

$ python Quickstart.py

OpenJS.NodeJS.LTS (Version 20.15.0)

$ node Quickstart.js

@PatrickFarley PatrickFarley merged commit 38cb9fa into Azure-Samples:master Jul 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants