Skip to content

chore: py-onnxruntime@1.22.2:#45

Merged
wdconinc merged 4 commits intomasterfrom
py-onnxruntime-1.21.0
Jan 3, 2026
Merged

chore: py-onnxruntime@1.22.2:#45
wdconinc merged 4 commits intomasterfrom
py-onnxruntime-1.21.0

Conversation

@wdconinc
Copy link
Contributor

@wdconinc wdconinc commented Nov 19, 2025 •
edited
Loading

Briefly, what does this PR introduce?

This PR updates py-onnxruntime to v1.22.2 or later.

What kind of change does this PR introduce?

  • Bug fix (issue #__)
  • New feature (issue: version bump)
  • Documentation update
  • Other: __

Copilot AI review requested due to automatic review settings November 19, 2025 15:31
Copilot AI reviewed Nov 19, 2025
View reviewed changes
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the py-onnxruntime package version requirement from 1.20.2 to 1.21.0 in the Spack environment configuration.

  • Updated py-onnxruntime version constraint from '@1.20.2' to '@1.21.0'

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

@wdconinc
Copy link
Contributor Author

wdconinc commented Nov 19, 2025

This upgrade causes failures in EICrecon right now, https://eicweb.phy.anl.gov/EIC/benchmarks/physics_benchmarks/-/jobs/6725524.

Copilot AI review requested due to automatic review settings November 27, 2025 20:15
@wdconinc wdconinc force-pushed the py-onnxruntime-1.21.0 branch from 9a2fecf to 7a240c4 Compare November 27, 2025 20:17
@wdconinc wdconinc changed the title chore: py-onnxruntime@1.21.0 chore: py-onnxruntime@1.22.2 Nov 27, 2025
Copilot AI reviewed Nov 27, 2025
View reviewed changes
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

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

@wdconinc wdconinc changed the title chore: py-onnxruntime@1.22.2 chore: py-onnxruntime@1.22.2: Nov 27, 2025
@wdconinc wdconinc force-pushed the py-onnxruntime-1.21.0 branch from 7a240c4 to c13a5c5 Compare December 19, 2025 02:57
Copilot AI review requested due to automatic review settings December 31, 2025 18:52
@wdconinc wdconinc force-pushed the py-onnxruntime-1.21.0 branch from c13a5c5 to deca855 Compare December 31, 2025 18:52
Copilot AI reviewed Dec 31, 2025
View reviewed changes
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

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

@wdconinc
Copy link
Contributor Author

wdconinc commented Jan 1, 2026

microsoft/onnxruntime#25200

@wdconinc wdconinc enabled auto-merge (squash) January 1, 2026 16:50
Copilot AI review requested due to automatic review settings January 1, 2026 18:31
Copilot AI reviewed Jan 1, 2026
View reviewed changes
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@wdconinc wdconinc requested a review from veprbl January 1, 2026 18:33
@wdconinc
Copy link
Contributor Author

wdconinc commented Jan 1, 2026

microsoft/onnxruntime#25200

@veprbl This was probably the reason indeed (dlopen fails on our systems which don't allow executable stack sections). This should now run EICrecon fine again.

veprbl
veprbl approved these changes Jan 3, 2026
View reviewed changes
Copy link
Member

@veprbl veprbl left a comment

Choose a reason for hiding this comment

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

LGTM

@wdconinc wdconinc merged commit 6810d74 into master Jan 3, 2026
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@veprbl veprbl veprbl approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@wdconinc @veprbl