Skip to content

Conversation

@zihyunting
Copy link
Contributor

Update package name and links for new SDK. The new SDK is under reviewing: Azure/azure-sdk-for-python#35384.

@prmerger-automator
Copy link
Contributor

@zihyunting : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit a3f1686:

✅ Validation status: passed

File Status Preview URL Details
articles/ai-services/computer-vision/includes/quickstarts-sdk/identity-python-sdk.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@prmerger-automator
Copy link
Contributor

PRMerger Results

Issue Description
File Change Percent This PR contains file(s) with more than 30% file change.

@Jak-MS
Copy link
Contributor

Jak-MS commented May 3, 2024

@PatrickFarley

  • Can you review this PR?
  • IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator bot added the aq-pr-triaged tracking label for the PR review team label May 3, 2024

To delete the **PersonGroup** you created in this quickstart, run the following code in your script:

[!code-python[](~/cognitive-services-quickstart-code/python/Face/FaceQuickstart.py?name=snippet_deletegroup)]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are we removing these instructions?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PersonGroup deletion is included in the new sample code (FaceQuickstart-single.py) now. The PersonGroup will be automatically deleted, so I think it is fine to remove these instructions in Clean up resources section.
image

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit f6561ce:

✅ Validation status: passed

File Status Preview URL Details
articles/ai-services/computer-vision/includes/quickstarts-sdk/identity-python-sdk.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@prmerger-automator
Copy link
Contributor

PRMerger Results

Issue Description
File Change Percent This PR contains file(s) with more than 30% file change.

@Court72
Copy link
Contributor

Court72 commented May 17, 2024

Thanks, @PatrickFarley!

When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

@PatrickFarley
Copy link
Contributor

#sign-off

@Court72
Copy link
Contributor

Court72 commented May 21, 2024

Due to the upcoming Build conference, all merging and publishing is on hold until 10 AM on 5/21.

This PR is ready for merge on 5/21.

#label: "event-triaged-to-10AM"
#label: "missed-content-complete"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants