-
Notifications
You must be signed in to change notification settings - Fork 90
[Rewriter]: introduce remove_optional_bias #2635
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
Conversation
|
|
||
|
|
||
| class RemoveOptionalBiasQLinearConvTest(_RemoveOptionalBiasTestBase): | ||
| def _get_test_model(self, zero_bias): |
Check warning
Code scanning / CodeQL
Signature mismatch in overriding method Warning
_RemoveOptionalBiasTestBase._get_test_model
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #2635 +/- ##
==========================================
+ Coverage 70.32% 70.46% +0.13%
==========================================
Files 222 224 +2
Lines 26438 26572 +134
Branches 2632 2637 +5
==========================================
+ Hits 18593 18723 +130
- Misses 6926 6928 +2
- Partials 919 921 +2 ☔ View full report in Codecov by Sentry. |
9735091 to
4547bd2
Compare
Removes optional bias when it is all zero from Conv, ConvTranspose, Gemm and QLinearConv operations.
4547bd2 to
920a5d3
Compare
Fixes #2547. I've kept the same ops as in #2555.