-
Notifications
You must be signed in to change notification settings - Fork 227
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
Pylint2.10 update #937
Pylint2.10 update #937
Conversation
Can one of the admins verify this patch? |
4 similar comments
Can one of the admins verify this patch? |
Can one of the admins verify this patch? |
Can one of the admins verify this patch? |
Can one of the admins verify this patch? |
Jenkins please retry a build |
nncf/torch/layers.py
Outdated
@@ -96,6 +96,7 @@ def _custom_forward_fn(self, input_): | |||
proxy_weight = self.weight | |||
return self._conv_forward(input_, proxy_weight, proxy_padding_value) | |||
|
|||
# pylint: disable=W0237 |
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.
I think the _conv_forward
function shouldn't have the same name as in the original implementation of Conv2d.
The warning should be resolved by simply renaming this method to something different, e.g. _conv_forward_proxy
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.
Ok, i'll change the name of this function
…nto pylint2.10_update
Jenkins please retry a build |
* fixed all issues in examples folder for pylint 2.10 update * fixed some pylint warnings in tests and nncf folders * fixed pylint warnings in tools folder + setup.py file * fixed parameter naming in synthetic.py * forgot to pass file in json.load() * added pylint flags * disabled high-priority warnings in tests and fixed low-priority warnings * fixed issues with iteration * fixed issue with checkpoint_loading * tried to fix pylint issues with bad option values * fixed disable flag in examples/tensorflow/common/callbacks.py * fixed disable flag in examples/tensorflow/common/callbacks.py
* fixed all issues in examples folder for pylint 2.10 update * fixed some pylint warnings in tests and nncf folders * fixed pylint warnings in tools folder + setup.py file * fixed parameter naming in synthetic.py * forgot to pass file in json.load() * added pylint flags * disabled high-priority warnings in tests and fixed low-priority warnings * fixed issues with iteration * fixed issue with checkpoint_loading * tried to fix pylint issues with bad option values * fixed disable flag in examples/tensorflow/common/callbacks.py * fixed disable flag in examples/tensorflow/common/callbacks.py
Made fixes to resolve pylint 2.10 warnings.