Skip to content

[rewriter] Enable llama rule sets #2124

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Mar 24, 2025

fix tests

33966b1
Select commit
Loading
Failed to load commit list.
Merged

[rewriter] Enable llama rule sets #2124

fix tests
33966b1
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Mar 24, 2025 in 0s

66.76% (-6.27%) compared to cde945d

View this Pull Request on Codecov

66.76% (-6.27%) compared to cde945d

Details

Codecov Report

Attention: Patch coverage is 83.33333% with 1 line in your changes missing coverage. Please review.

Project coverage is 66.76%. Comparing base (cde945d) to head (33966b1).

Files with missing lines Patch % Lines
onnxscript/rewriter/_ir_utils.py 50.00% 1 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (cde945d) and HEAD (33966b1). Click for more details.

HEAD has 22 uploads less than BASE
Flag BASE (cde945d) HEAD (33966b1)
24 2
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2124      +/-   ##
==========================================
- Coverage   73.02%   66.76%   -6.27%     
==========================================
  Files         219      219              
  Lines       29147    29145       -2     
  Branches     3440     3440              
==========================================
- Hits        21286    19460    -1826     
- Misses       6708     8566    +1858     
+ Partials     1153     1119      -34     

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