Skip to content

Navigation Menu

Sign in
Sign up

enchancemet - #179

Open
moparthiamrutha wants to merge 3 commits into
microsoft:master from
moparthiamrutha:master
Open

enchancemet #179
moparthiamrutha wants to merge 3 commits into
microsoft:master from
moparthiamrutha:master

Conversation

@moparthiamrutha

@moparthiamrutha moparthiamrutha commented Apr 14, 2026

Copy link
Copy Markdown

bcnvv

@ManyaS-Git Manya Sharma (ManyaS-Git) left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This change is mostly a regression and shouldn't be merged as-is. The intent seems to be a grammar cleanup of .docs/python_scoring.md, but it introduces a broken link, breaks a paragraph, and leaves genuine grammar errors.

1. (Blocker) It breaks a URL.

-...[AzureML Pipelines w/GPU](https://github.com/Azure/Batch-Scoring-Deep-Learning-Models-With-AML)
+...[AzureML Pipeline w/GPU](https://github.com/Azure/Batch-Scoring-Deep-Learning-Model-With-AML)

You changed the link target from ...-Models-With-AML to ...-Model-With-AML (singular). I verified via the GitHub API: Azure/Batch-Scoring-Deep-Learning-Models-With-AML exists (it now redirects to microsoft/az-deep-batch-score), while Azure/Batch-Scoring-Deep-Learning-Model-With-AML returns 404. Please never change a URL while editing prose — restore the original link.

2. Mid-sentence line breaks break the paragraph.

+... For a tutorial on deploying classical ML model
+
+
+ for real time scoring look [AKS](...)...

The two blank lines split one sentence across three fragments, which renders as detached lines. The original was a single flowing paragraph.

3. The "grammar fix" is internally inconsistent.

  • "When deploying ML model in Python" (singular) but "For deploying deep learning models" (plural) a few words later — same inconsistency now exists between "ML model" and "deep learning models".
  • "deep learning model that require real time" — subject/verb disagreement introduced by making "model" singular while keeping "require".
  • "with GPU" vs "with GPUs" and "Pipeline" vs "Pipelines" are now inconsistent within the same paragraph.

4. It doesn't fix the actual bug in this file.
The image path <img src="../images/decision_python_scoring.png"> is still the broken images/ path (issue #128 — the directory is .images/, and PR #129 already fixes this). This PR is based on older master, so it will conflict with #129 and undo nothing useful here.

5. Hygiene.

  • The PR ends the file with no trailing newline again (\ No newline at end of file).
  • Title has a typo ("enchancemet") and the description is placeholder text ("bcnvv").

Recommendation: request changes — restore the original URL and the single-paragraph structure, keep the existing "models/GPUs" plural wording, and rebase on top of the images/.images/ fix from #129 (or just close this and leave the prose as-is; the original text wasn't wrong).

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

Reviewers

1 more reviewer
Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /