-
Notifications
You must be signed in to change notification settings - Fork 43
Review main-README.md
#113
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
chienyuanchang
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- 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: