Skip to content

Conversation

titaiwangms
Copy link
Contributor

No description provided.

Copy link

codecov bot commented Jun 24, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.32%. Comparing base (ff0a132) to head (23949ca).
Report is 14 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2416      +/-   ##
==========================================
+ Coverage   69.31%   69.32%   +0.01%     
==========================================
  Files         204      204              
  Lines       25850    25854       +4     
  Branches     2694     2696       +2     
==========================================
+ Hits        17917    17923       +6     
+ Misses       7002     7001       -1     
+ Partials      931      930       -1     

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

@titaiwangms titaiwangms marked this pull request as draft June 24, 2025 22:12
@justinchuby
Copy link
Collaborator

Looks like we need to expose

@justinchuby
Copy link
Collaborator

But it is, I wonder why even for the onnx-ir-git workflow

@justinchuby
Copy link
Collaborator

oh. We should switch to import onnx-ir by now

@titaiwangms
Copy link
Contributor Author

I thought it was the version of ir.

@justinchuby
Copy link
Collaborator

No I think it’s because we re exposed the passes under onnxscript ir

@justinchuby
Copy link
Collaborator

#2417

@titaiwangms titaiwangms marked this pull request as ready for review June 30, 2025 22:58
@titaiwangms titaiwangms merged commit 87d6f11 into microsoft:main Jun 30, 2025
26 of 33 checks passed
@justinchuby
Copy link
Collaborator

Looks like the pass was not actually added? @titaiwangms

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.

2 participants