-
Notifications
You must be signed in to change notification settings - Fork 102
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
Make params exclusive to COp
s
#513
Conversation
539124e
to
5fce452
Compare
@@ -803,7 +809,10 @@ def code_gen(self): | |||
|
|||
sub = dict(failure_var=failure_var) | |||
|
|||
params = node.run_params() | |||
try: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We raise 4 lines above if it's not a ClinkerOp
5fce452
to
e7139e6
Compare
Also removes them from the signature of perform
e7139e6
to
1b242b9
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #513 +/- ##
==========================================
- Coverage 80.78% 80.78% -0.01%
==========================================
Files 162 162
Lines 46172 46161 -11
Branches 11283 11279 -4
==========================================
- Hits 37302 37293 -9
+ Misses 6640 6637 -3
- Partials 2230 2231 +1
|
Would like to get this merged before the next release |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice simplification. I guess the COps still need params?
Yup |
Closes #511