Conversation
chienyuanchang
left a comment
There was a problem hiding this comment.
Automated LLM code review (section-based).
LLM usage details:
- Total tokens used: 8124.
- Used deployment: cu-samples-gpt-4.1-mini
- API version: 2024-12-01-preview
| - As of November 2025, the `2025-11-01` API version is now available in a broader range of [regions](https://learn.microsoft.com/en-us/azure/ai-services/content-understanding/language-region-support). | ||
| - We will provide more samples for new functionalities in the GA API versions soon. For details on the updates in the current GA release, please see the [Content Understanding What's New Document page](https://learn.microsoft.com/en-us/azure/ai-services/content-understanding/whats-new). | ||
| - As of November 2025, the `2025-11-01` API version is available in a broader range of [regions](https://learn.microsoft.com/en-us/azure/ai-services/content-understanding/language-region-support). | ||
| - To access sample code for version `2025-05-01-preview`, please check out the corresponding Git tag `2025-05-01-preview` or download it directly from the [release page](https://github.com/Azure-Samples/azure-ai-content-understanding-python/releases/tag/2025-05-01-preview). |
There was a problem hiding this comment.
-
categories: [Formatting]
- change: Removed an extra trailing space at the end of the first line.
- rationale: To clean up unnecessary whitespace and ensure consistent formatting.
- impact: Improves the document’s appearance and prevents potential formatting issues.
-
categories: [Grammar, Clarity]
- change: Added "please" to the sentence linking to the What's New document page.
- rationale: To make the sentence more polite and reader-friendly.
- impact: Enhances readability and provides a more courteous tone.
-
categories: [Grammar, Clarity]
- change: Changed "is now available" to "is available" regarding the API version's regional availability.
- rationale: Because the phrase "as of November 2025" already implies recent availability, "now" is redundant.
- impact: Improves the clarity and precision of the statement.
|
|
||
| 👉 If you are looking for **.NET samples**, check out [this repo](https://github.com/Azure-Samples/azure-ai-content-understanding-dotnet/). | ||
| 👉 If you are looking for **.NET samples**, please check out [this repo](https://github.com/Azure-Samples/azure-ai-content-understanding-dotnet/). | ||
|
|
There was a problem hiding this comment.
- categories: [Grammar, Clarity]
- change: Added the word "please" before "check out" in the sentence directing users to the .NET samples repo.
- rationale: Including "please" makes the sentence more polite and inviting, improving the tone of the documentation.
- impact: Enhances the readability and user-friendliness of the documentation, encouraging users to follow the recommendation.
| 1. Create a new Codespace by selecting the `main` branch, your preferred Codespace region, and the 2-core machine type, as shown in the screenshot below. | ||
|  | ||
|  | ||
| 2. Once the Codespace is ready, open the terminal and follow the instructions in the **Configure Azure AI service resource** section to set up a valid Content Understanding resource. |
There was a problem hiding this comment.
-
categories: [Grammar]
- change: Added the word "please" to the sentence "After clicking the link above, follow these steps to set up the Codespace."
- rationale: Including "please" makes the instruction more polite and reader-friendly.
- impact: Enhances the tone and engagement of the documentation without changing its meaning.
-
categories: [Formatting, Consistency]
- change: Corrected the image alt text from "Create CodeSpace" to "Create Codespace" and added two trailing spaces after the image markdown.
- rationale: The alt text was inconsistent in capitalization; adding trailing spaces ensures proper line breaks in markdown rendering.
- impact: Improves visual formatting and consistency across documentation, ensuring images display correctly and alt text accurately describes the content.
| * [Python 3.11+](https://www.python.org/downloads/) | ||
| * [Azure Developer CLI (azd)](https://aka.ms/install-azd) | ||
| * [Python 3.11+](https://www.python.org/downloads/) | ||
| * [Git LFS](https://git-lfs.com/) |
There was a problem hiding this comment.
-
categories: [Grammar, Clarity]
- change: Added the word "Please" to the instruction "Ensure the following tools are installed" making it "Please ensure the following tools are installed:"
- rationale: This softens the directive, making it more polite and reader-friendly.
- impact: Improves tone and readability, providing a more welcoming instruction.
-
categories: [Formatting]
- change: Added two trailing spaces after the first two list items in the tools list to enable proper Markdown line breaks.
- rationale: In Markdown, two trailing spaces create a line break without starting a new paragraph; this helps maintain proper spacing and readability.
- impact: Enhances the visual presentation of the list, ensuring consistent formatting and spacing in rendered documentation.
|
|
||
| - **Important:** If you use `git clone`, you must install Git LFS and run `git lfs pull` to download sample files in the `data` directory: | ||
| - **Important:** If you use `git clone`, please install Git LFS and run `git lfs pull` to download sample files in the `data` directory: | ||
|
|
There was a problem hiding this comment.
- categories: [Clarity, Tone]
- change: Replaced "you must install Git LFS" with "please install Git LFS"
- rationale: The change softens the instruction, making it more polite and user-friendly.
- impact: Improves the tone of the documentation, making it more approachable without altering the meaning.
|
|
||
| For more information on deploying models, see [Deploy models in Azure AI Foundry](https://learn.microsoft.com/en-us/azure/ai-studio/how-to/deploy-models-openai). | ||
| For more information on deploying models, please see [Deploy models in Azure AI Foundry](https://learn.microsoft.com/en-us/azure/ai-studio/how-to/deploy-models-openai). | ||
|
|
There was a problem hiding this comment.
-
categories: [Grammar, Clarity]
- change: Changed "You must deploy these models before using prebuilt analyzers:" to "Please deploy these models before using prebuilt analyzers:"
- rationale: The phrasing was softened from a directive "must" to a polite request "please," improving readability and tone.
- impact: Makes the instruction friendlier and clearer to users.
-
categories: [Formatting, Consistency]
- change: Moved the list of model requirements (prebuilt analyzers and their required models) from bullet points to inline bullet points with double spaces at line ends to preserve spacing.
- rationale: This aligns the list visually separate from deployment steps and maintains consistent markdown formatting for better readability.
- impact: Enhances document formatting and makes the requirements easier to find and read.
-
categories: [Formatting, Consistency]
- change: Added two spaces at the end of list lines and reordered the steps for deployment instructions with consistent indentation and spacing.
- rationale: Ensures proper markdown rendering with spaced lines and uniform formatting for nested lists across all deployment steps.
- impact: Improves readability and uniformity of the deployment instructions across sections.
-
categories: [Clarity, Consistency]
- change: Added "Note the deployment name (by convention, use
xxx)" after each deployment step instead of only in the GPT-4.1 step. - rationale: Provides consistent important information about deployment naming for all models rather than only the first.
- impact: Ensures users follow a clear naming convention consistently for all deployments.
- change: Added "Note the deployment name (by convention, use
-
categories: [Grammar, Clarity]
- change: Changed "For more information on deploying models, see" to "For more information on deploying models, please see"
- rationale: Politeness addition to the wording to soften the instruction tone.
- impact: Creates a more user-friendly and approachable documentation tone.
| - Replace `<your-resource-name>` and `<your-azure-ai-api-key>` with your actual values. These can be found in your AI Services resource under **Resource Management** > **Keys and Endpoint**. | ||
| 2. Edit `notebooks/.env` and set your credentials. | ||
| - Replace `<your-resource-name>` and `<your-azure-ai-api-key>` with your actual values. These can be found in your AI Services resource under **Resource Management** > **Keys and Endpoint**. | ||
| - If you used different deployment names in Step 2, update the deployment variables accordingly: |
There was a problem hiding this comment.
- categories: [Formatting]
- change: Added a leading space before the hyphen in the second bullet point, changing it from a double hyphen (
--) to a single indented hyphen (-). - rationale: To correct the bullet point formatting, ensuring consistent indentation and proper markdown list structure.
- impact: Improves the readability and visual clarity of the bullet list in the documentation.
- change: Added a leading space before the hyphen in the second bullet point, changing it from a double hyphen (
|
|
||
| > ⚠️ **Note:** If you skip the token authentication step above, you must set `AZURE_AI_API_KEY` in your `.env` file. Get your API key from Azure Portal by navigating to your Foundry resource > **Resource Management** > **Keys and Endpoint**. | ||
| > ⚠️ **Note:** If you skip the token authentication step above, you must set `AZURE_AI_API_KEY` in your `.env` file. You can obtain your API key from Azure Portal by navigating to your Foundry resource > **Resource Management** > **Keys and Endpoint**. | ||
|
|
There was a problem hiding this comment.
- categories: [Grammar, Clarity]
- change: Replaced "Get your API key" with "You can obtain your API key" in the note explaining how to get the API key from Azure Portal.
- rationale: The revised phrasing improves the formality and clarity of the instruction, making it sound more polite and less imperative.
- impact: Enhances readability and professionalism of the documentation, making the instruction clearer and more user-friendly.
|
|
||
| - [Azure Search with Content Understanding](https://github.com/Azure-Samples/azure-ai-search-with-content-understanding-python) | ||
| - [Azure Search with Content Understanding](https://github.com/Azure-Samples/azure-ai-search-with-content-understanding-python) | ||
| - [Azure Content Understanding with OpenAI](https://github.com/Azure-Samples/azure-ai-content-understanding-with-azure-openai-python) |
There was a problem hiding this comment.
- categories: [Formatting]
- change: Added two trailing spaces at the end of the line containing the link to "Azure Search with Content Understanding."
- rationale: In Markdown, two trailing spaces are used to enforce a line break; this ensures that the subsequent list item appears on a new line rather than being rendered inline.
- impact: Improves the readability and formatting of the list by correctly displaying each item on its own line, enhancing the document’s visual structure.
README.md
Outdated
There was a problem hiding this comment.
- categories: Formatting
- change: Removal of a trailing whitespace line at the end of the paragraph.
- rationale: To ensure consistent formatting and eliminate unnecessary blank lines.
- impact: Improves the visual cleanliness and consistency of the documentation without altering its content.
Automated review and documentation improvements for
README.mdon branchmainLLM usage details: