Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Introduce OpSignature to IR #1838

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Warn once

ce95d91
Select commit
Loading
Failed to load commit list.
Open

Introduce OpSignature to IR #1838

Warn once
ce95d91
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Oct 23, 2024 in 0s

74.14% (-1.32%) compared to 2b60939

View this Pull Request on Codecov

74.14% (-1.32%) compared to 2b60939

Details

Codecov Report

Attention: Patch coverage is 65.38462% with 108 lines in your changes missing coverage. Please review.

Project coverage is 74.14%. Comparing base (2b60939) to head (ce95d91).

Files with missing lines Patch % Lines
onnxscript/ir/_schemas.py 61.60% 79 Missing and 12 partials ⚠️
onnxscript/values.py 70.58% 12 Missing and 3 partials ⚠️
onnxscript/ir/_schemas_test.py 90.00% 1 Missing and 1 partial ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #1838      +/-   ##
==========================================
- Coverage   75.46%   74.14%   -1.32%     
==========================================
  Files         252      254       +2     
  Lines       27335    27635     +300     
  Branches     3171     3224      +53     
==========================================
- Hits        20628    20491     -137     
- Misses       5743     6126     +383     
- Partials      964     1018      +54

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

View more details on Codecov