Skip to content

Remove trailing whitespace from the repo#9065

Merged
ghpvnist merged 1 commit intopytorch:masterfrom
ghpvnist:whitespace
May 8, 2025
Merged

Remove trailing whitespace from the repo#9065
ghpvnist merged 1 commit intopytorch:masterfrom
ghpvnist:whitespace

Conversation

@ghpvnist
Copy link
Copy Markdown
Collaborator

@ghpvnist ghpvnist commented Apr 30, 2025

Nuking trailing whitespaces using regex s/\s+$//

closes #9101

@tengyifei
Copy link
Copy Markdown
Collaborator

@zpcore ptal, ty!

@ghpvnist ghpvnist self-assigned this May 6, 2025
Copy link
Copy Markdown
Collaborator

@bhavya01 bhavya01 left a comment

Choose a reason for hiding this comment

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

Thanks!

@ghpvnist
Copy link
Copy Markdown
Collaborator Author

ghpvnist commented May 8, 2025

@tengyifei Do we need to increase tolerance for this particular GPU test? I think I've seen this test fail for multiple PRs.

@tengyifei
Copy link
Copy Markdown
Collaborator

@tengyifei Do we need to increase tolerance for this particular GPU test? I think I've seen this test fail for multiple PRs.

I don't think so. That looks like a legit bug to me. Because the delta is not small.

In any case, we can merge this without blocking on GPU tests. If we want to do something with the tolerance, that should be done in a separate PR.

@tengyifei
Copy link
Copy Markdown
Collaborator

@ghpvnist feel free to squash and merge

@tengyifei
Copy link
Copy Markdown
Collaborator

Specifically the number is very close to 113/125

@ghpvnist ghpvnist merged commit 262ca1e into pytorch:master May 8, 2025
23 of 24 checks passed
zhanyong-wan pushed a commit to zhanyong-wan/pytorch-xla that referenced this pull request May 9, 2025
@ghpvnist ghpvnist deleted the whitespace branch May 10, 2025 05:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove trailing whitespace from the repo

5 participants