Repository navigation
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Update build, platform, dependency, and contribution guidance; remove obsolete Static API references; and rename the cookbook sample test class. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
Build prerequisites, command paths, and platform configuration guidance need corrections so contributors can follow the refreshed documentation successfully.
Review effort: Balanced
Findings: 6
Open (6)
Clone the repository before running the dependency script · New Avoid running Homebrew bundle as root on macOS · New Add OpenMP prerequisites to the dependency script · New Document setting MLNET_BACKEND before application startup · New Use forward-slash project paths in dotnet commands · New Document x86 restrictions and LightGBM inference support · New
What changed in this PR
Refreshes ML.NET’s user and contributor documentation while preserving historical release notes and ROADMAP.md.
Changes:
- Removes obsolete READMEs and Static API references.
- Updates build, platform, TensorFlow, contribution, and release guidance.
- Preserves oneDAL guidance and renames the cookbook sample test class.
| File | Description |
|---|---|
| test/Microsoft.ML.Tests/Scenarios/Api/CookbookSamples/CookbookSamples.cs | Renames the class and cleans up comments. |
| README.md | Refreshes installation, platform, and CI information. |
| README.ko-KR.md | Removes the obsolete localized README. |
| README-oneDAL.md | Removes the standalone oneDAL README. |
| docs/README.md | Updates macOS terminology. |
| docs/project-docs/release-process.md | Updates terminology and installation command. |
| docs/project-docs/platform-limitations.md | Revises architecture and native-component limitations. |
| docs/project-docs/developer-guide.md | Refreshes build and test instructions. |
| docs/project-docs/contributing.md | Updates contribution guidance link. |
| docs/project-docs/components-and-dependencies.md | Preserves oneDAL activation guidance. |
| docs/code/MlNetCookBook.md | Removes obsolete Static API references. |
| docs/building/windows-instructions.md | Updates Windows build prerequisites. |
| docs/building/unix-instructions.md | Replaces outdated dependency instructions. |
| docs/api-reference/tensorflow-usage.md | Refreshes CPU and GPU runtime guidance. |
| CONTRIBUTING.md | Updates contributor resources and headings. |
| .github/copilot-instructions.md | Clarifies SDK selection and CI roles. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Split Windows x64 and x86 support and identify the 64-bit-only TensorFlow, LightGBM training, and TorchSharp functionality. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Correct Unix setup ordering and commands, document oneDAL startup requirements, and use cross-platform project paths. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7760 +/- ##
==========================================
+ Coverage 70.25% 70.26% +0.01%
==========================================
Files 1419 1419
Lines 272569 272569
Branches 27938 27938
==========================================
+ Hits 191493 191529 +36
+ Misses 73661 73632 -29
+ Partials 7415 7408 -7
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
Add an ML.NET-owned wrapper that installs OpenMP on supported platforms, warns where manual setup is required, and use it across CI and contributor documentation. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
🟡 Changes recommended
The cookbook contains an invalid, unbounded snippet, and TensorFlow GPU guidance incorrectly limits the packages to inference.
2 open findings
6 resolved since last review
Document x86 restrictions and LightGBM inference support Use forward-slash project paths in dotnet commands Document setting MLNET_BACKEND before application startup Add OpenMP prerequisites to the dependency script Avoid running Homebrew bundle as root on macOS Clone the repository before running the dependency script
🧠 Review effort: Balanced
Clarified instructions for adding GPU package for TensorFlow. Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Summary
Historical release notes and
ROADMAP.mdwere intentionally left unchanged.Validation
CookbookSamplestests onnet8.0