Skip to content

Update ONNX Runtime to 1.28.0 - #11

Open
ericstj wants to merge 1 commit into
mainfrom
bump-onnxruntime-1.28.0
Open

Update ONNX Runtime to 1.28.0#11
ericstj wants to merge 1 commit into
mainfrom
bump-onnxruntime-1.28.0

Conversation

@ericstj

@ericstj ericstj commented Jul 26, 2026

Copy link
Copy Markdown
Owner

Updates Microsoft.ML.OnnxRuntime from 1.27.1 to 1.28.0, the current stable release (and the version the Python onnxruntime wheel is on).

Validation

  • dotnet build PrivacyFilter.Net.sln -c Release — clean (0 warnings, 0 errors)
  • dotnet test PrivacyFilter.Net.sln -c Release13/13 passing

Perf note

Benchmarked fp32 CPU on a Tesla T4 host: 1.28.0 is on par with 1.26/1.27.1 (no measurable regression), and matches Python + ONNX Runtime 1.28 within ~1% on the long input, confirming the managed binding adds no meaningful model-execution overhead.

Matches the current onnxruntime stable release. Build and all tests pass.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 84fe6252-dcf1-44dd-8257-eb6473b88d81
Copilot AI review requested due to automatic review settings July 26, 2026 18:22

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the centrally-managed NuGet package version for ONNX Runtime so the solution builds/tests against Microsoft.ML.OnnxRuntime 1.28.0, aligning with the current stable ONNX Runtime release referenced in the PR description.

Changes:

  • Bump Microsoft.ML.OnnxRuntime from 1.27.1 to 1.28.0 via central package management (Directory.Packages.props).
Show a summary per file
File Description
Directory.Packages.props Updates the centrally pinned Microsoft.ML.OnnxRuntime package version to 1.28.0.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Low

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants