[v1.x] Work around two pylint false positives in mxnet/io/io.py#18269
[v1.x] Work around two pylint false positives in mxnet/io/io.py#18269nickguletskii wants to merge 1 commit intoapache:v1.xfrom
Conversation
|
Hey @nickguletskii , Thanks for submitting the PR
CI supported jobs: [centos-gpu, miscellaneous, clang, unix-gpu, windows-cpu, windows-gpu, edge, website, unix-cpu, sanity, centos-cpu] Note: |
|
@mxnet-bot run ci [centos-gpu] |
|
Jenkins CI successfully triggered : [centos-gpu] |
|
It's better to fix the root-cause of the issue by backporting |
|
This has already been resolved in my PR #18220 @nickguletskii please rebase. |
|
@ChaiBapchya Sorry, didn't know about your PR. It seems that it has only been merged into v1.7.x, but not v1.x. Could you please create a second one for the v1.x branch? Or should I update this PR/create a new one instead? |
|
It's just that we have so many branches to actively maintain. Tough to track all the actively maintained branches. Creating separate cherry-pick. Plz close this one. Thanks @nickguletskii |
Description
With the latest pylint, lint checks fail due to the following errors:
Original CI log: http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%2Fsanity/detail/PR-18268/2/pipeline
You may reproduce this by running
python3 -m pylint --rcfile=./ci/other/pylintrc --ignore-patterns=".*\.so$$,.*\.dll$$,.*\.dylib$$" python/mxnet/io/io.py.This PR simply disables the rule that is giving false positives on these lines.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.
Changes
Comments