Repository navigation
Python: docs: fix dead Learn link to the Rubric evaluators guide - #9906
Merged
Eduard van Valkenburg (eavanvalkenburg) merged 1 commit intoNov 2, 2026
Conversation
kun654
requested review from
SergeyMenshykh,
Eduard van Valkenburg (eavanvalkenburg) and
Roger Barreto (rogerbarreto)
as code owners
October 31, 2026 03:31
kun654
temporarily deployed
to
github-app-auth
October 31, 2026 03:31 — with
GitHub Actions
Inactive
kun654
requested review from
Darren Cohen (dargilco),
Jose Alvarez (jpalvarezl),
sophia-ramsey and
westey (westey-m)
as code owners
October 31, 2026 03:31
kun654
temporarily deployed
to
github-app-auth
October 31, 2026 03:31 — with
GitHub Actions
Inactive
Contributor
There was a problem hiding this comment.
🟢 Approval recommended
No active review issues remain for the Rubric evaluators guide link update.
No open issues 🎉
What changed in this PR
Fixes the Rubric evaluators guide link in the Python evaluation sample’s module docstring.
Changes:
- Updates the Microsoft Learn URL from the
ai-foundrypath to thefoundrypath. - Changes documentation only; sample behavior, configuration, and public APIs remain unchanged.
| File | Description |
|---|---|
python/samples/05-end-to-end/evaluation/foundry_evals/evaluate_with_rubric_sample.py |
Updates the docstring’s link to the Rubric evaluators guide. |
Review effort: Balanced
Eduard van Valkenburg (eavanvalkenburg)
approved these changes
Nov 2, 2026
This branch was previously deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation & Context
The
evaluate_with_rubric_sample.pymodule docstring points readers at the Rubric evaluators guide, but that Learn URL no longer resolves.https://learn.microsoft.com/azure/ai-foundry/concepts/evaluation-evaluators/rubric-evaluatorsanswers302to/en-us/azure/ai-foundry/..., and that target returns404. The page is now published under thefoundrydocs root, so theSee:reference in the sample should follow it.This is a dead-link fix in documentation text; it does not change sample behavior, configuration, or public API.
Description & Review Guide
ai-foundrypath to the currentfoundrypath.See:link land on the live guide instead of a 404.https://learn.microsoft.com/azure/foundry/concepts/evaluation-evaluators/rubric-evaluatorsis the target you prefer; the page title is "Rubric evaluators - Microsoft Foundry | Microsoft Learn", and it is the page the sample's prose describes.Related Issue
N/A - trivial change: a single broken documentation link inside a docstring, where the intended destination is not in dispute.
AI Assistance
AI assistance details: The link replacement and the URL checks were prepared with an AI coding assistant. I verified the result myself: the old URL 404s after its redirect, the new URL returns 200 and serves the Rubric evaluators page, the file still parses with
ast.parse, and the diff is a single line in one file.Contribution Checklist
breaking changelabel (or add "[BREAKING]" to the title prefix, before or after any language prefix) — a workflow keeps the label and title prefix in sync automatically.The build and test boxes are left unchecked because the change only touches a docstring string; no Python source, test, or packaging file is modified.