Skip to content

Revert version converter support on local function#2815

Closed
titaiwangms wants to merge 3 commits intomicrosoft:mainfrom
titaiwangms:titaiwang/revert_vc
Closed

Revert version converter support on local function#2815
titaiwangms wants to merge 3 commits intomicrosoft:mainfrom
titaiwangms:titaiwang/revert_vc

Conversation

@titaiwangms
Copy link
Copy Markdown
Contributor

version converter does not support local function currently, and it needs inliner within the call. The PRs broke benchmakring and Olive.

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 18, 2026

Codecov Report

❌ Patch coverage is 84.61538% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 70.42%. Comparing base (e6f79e1) to head (be63ea7).

Files with missing lines Patch % Lines
onnxscript/version_converter/_version_converter.py 60.00% 3 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2815      +/-   ##
==========================================
- Coverage   70.52%   70.42%   -0.10%     
==========================================
  Files         228      228              
  Lines       27135    27077      -58     
  Branches     2727     2723       -4     
==========================================
- Hits        19137    19070      -67     
- Misses       7065     7071       +6     
- Partials      933      936       +3     

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

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.

1 participant