search

microsoft / onnxscript

ONNX Script enables developers to naturally author ONNX functions and models using a subset of Python.
https://onnxscript.ai/
MIT License
271 stars 53 forks source link

[IR] Implement pass to remove unused nodes in graph #1841

Open yichen-li-ucla opened 3 weeks ago

yichen-li-ucla commented 3 weeks ago

@justinchuby Fixes #1474

codecov[bot] commented 3 weeks ago

Codecov Report

Attention: Patch coverage is 91.66667% with 9 lines in your changes missing coverage. Please review.

Project coverage is 75.28%. Comparing base (d7a6411) to head (b69dd83). Report is 18 commits behind head on main.

Files with missing lines Patch % Lines
onnxscript/ir/passes/_remove_unused.py 84.09% 3 Missing and 4 partials :warning:
onnxscript/ir/passes/_remove_unused_test.py 96.87% 1 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1841 +/- ## ========================================== + Coverage 75.20% 75.28% +0.07% ========================================== Files 251 253 +2 Lines 27429 27537 +108 Branches 5032 5047 +15 ========================================== + Hits 20629 20730 +101 - Misses 5828 5830 +2 - Partials 972 977 +5 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

yichen-li-ucla commented 3 weeks ago

@justinchuby

yichen-li-ucla commented 3 weeks ago

Just updated with upstream. No need for rerun.

justinchuby commented 2 weeks ago

I will come back to this later this week. Thanks for your patience!