You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The current version of pylint on the CI pipeline (2.12.2) requires several ignore statements which are not needed by newer pylint versions (e.g. 2.17, 3.0.3, etc).
For example, a common pattern in ART is to use a # pylint: disable=W0221 for when the number of parameters change in a method between the parent and child classes. However, this suppression is not needed for newer pylint versions as long as a default argument is supplied. There are also examples of unneeded disable comments in the tensorflow derandamized smoothing ablator.
We propose to increase the pylint version and see how many of the existing pylint disable comments are required under the newer tools.
The text was updated successfully, but these errors were encountered:
The current version of pylint on the CI pipeline (2.12.2) requires several ignore statements which are not needed by newer pylint versions (e.g. 2.17, 3.0.3, etc).
For example, a common pattern in ART is to use a
# pylint: disable=W0221
for when the number of parameters change in a method between the parent and child classes. However, this suppression is not needed for newer pylint versions as long as a default argument is supplied. There are also examples of unneeded disable comments in the tensorflow derandamized smoothing ablator.We propose to increase the pylint version and see how many of the existing pylint disable comments are required under the newer tools.
The text was updated successfully, but these errors were encountered: