Repository navigation
[Cherrypick] Update ONNX extras (#40821) - #40884
Conversation
* Update ONNX extras * Trigger postcommits * Fix py311-ml onnx bounds * add onnxscript to tox environment * Add onnxscript to ml_base
There was a problem hiding this comment.
📋 Review Summary
This PR cleanly cherry-picks #40821 into release-2.78, modernizing the ONNX optional dependencies in setup.py and re-enabling the ONNX inference unit test suite across tox.ini and Gradle.
🔍 General Feedback
- The cherry-pick matches the original commit in #40821 with no merge discrepancies.
- Keras 3 compatibility adjustments, PyTorch opset updates, and tox dependency bounds are well-structured and properly aligned.
💡 Tip: Comment /gemini review on this pull request to trigger a new Gemini code review.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## release-2.78 #40884 +/- ##
=================================================
+ Coverage 0 79.81% +79.81%
=================================================
Files 0 494 +494
Lines 0 83573 +83573
=================================================
+ Hits 0 66707 +66707
- Misses 0 16866 +16866
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Assigning reviewers: R: @damccorm for label python. Note: If you would like to opt out of this review, comment Available commands:
The PR bot will only process comments in the main thread (not review comments). |
Update ONNX extras
Trigger postcommits
Fix py311-ml onnx bounds
add onnxscript to tox environment
Add onnxscript to ml_base