Skip to content

[PyTorch] Fix linter warnings from unused args - #816

Merged
ksivaman merged 2 commits into
NVIDIA:mainfrom
timmoon10:fix-linter-errors
Apr 30, 2024
Merged

[PyTorch] Fix linter warnings from unused args#816
ksivaman merged 2 commits into
NVIDIA:mainfrom
timmoon10:fix-linter-errors

Conversation

@timmoon10

Copy link
Copy Markdown
Member

We're seeing Pylint failures in the nightly CI because we are defining some PyTorch autograd functions that don't access the autograd context. This PR silences those warnings by renaming the args with a leading underscore.

Signed-off-by: Tim Moon <tmoon@nvidia.com>
@timmoon10timmoon10 added the bug Something isn't working label Apr 26, 2024
@timmoon10
timmoon10 requested a review from ksivamanApril 26, 2024 18:05
@timmoon10

Copy link
Copy Markdown
MemberAuthor

/te-ci pytorch

Comment thread.gitignore Outdated
Co-authored-by: Kirthi Shankar Sivamani <ksivamani@nvidia.com>
Signed-off-by: Tim Moon <4406448+timmoon10@users.noreply.github.com>

@ksivamanksivaman left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ksivaman
ksivaman merged commit 1f36c2c into NVIDIA:mainApr 30, 2024
@timmoon10
timmoon10 deleted the fix-linter-errors branch April 30, 2024 18:14
phu0ngng pushed a commit to phu0ngng/TransformerEngine that referenced this pull request May 3, 2024
* Fix linter warnings from unused args
Signed-off-by: Tim Moon <tmoon@nvidia.com>
* Update .gitignore
Co-authored-by: Kirthi Shankar Sivamani <ksivamani@nvidia.com>
Signed-off-by: Tim Moon <4406448+timmoon10@users.noreply.github.com>
---------
Signed-off-by: Tim Moon <tmoon@nvidia.com>
Signed-off-by: Tim Moon <4406448+timmoon10@users.noreply.github.com>
Co-authored-by: Kirthi Shankar Sivamani <ksivamani@nvidia.com>
pggPL pushed a commit to pggPL/TransformerEngine that referenced this pull request May 23, 2024
* Fix linter warnings from unused args
Signed-off-by: Tim Moon <tmoon@nvidia.com>
* Update .gitignore
Co-authored-by: Kirthi Shankar Sivamani <ksivamani@nvidia.com>
Signed-off-by: Tim Moon <4406448+timmoon10@users.noreply.github.com>
---------
Signed-off-by: Tim Moon <tmoon@nvidia.com>
Signed-off-by: Tim Moon <4406448+timmoon10@users.noreply.github.com>
Co-authored-by: Kirthi Shankar Sivamani <ksivamani@nvidia.com>
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugSomething isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@timmoon10@ksivaman