Skip to content

Conversation

@gramalingam
Copy link
Collaborator

Add option to clear metadata in ort fusion. Otherwise, we lose all metadata after running ORT fusion

@codecov
Copy link

codecov bot commented Nov 5, 2025

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 70.14%. Comparing base (971f9bb) to head (8d6a4e5).
⚠️ Report is 2 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
onnxscript/rewriter/ort_fusions/_core.py 0.00% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2685      +/-   ##
==========================================
- Coverage   70.15%   70.14%   -0.01%     
==========================================
  Files         225      225              
  Lines       27039    27044       +5     
  Branches     2716     2718       +2     
==========================================
+ Hits        18968    18969       +1     
- Misses       7134     7136       +2     
- Partials      937      939       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: Ganesan Ramalingam <[email protected]>
@gramalingam gramalingam enabled auto-merge (squash) November 7, 2025 00:19
@github-project-automation github-project-automation bot moved this from Todo to Done in ONNX Script Review Board Nov 7, 2025
@gramalingam gramalingam merged commit ea8cb3e into main Nov 7, 2025
33 checks passed
@gramalingam gramalingam deleted the rama/keepmeta branch November 7, 2025 00:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

4 participants