Skip to content

Fix shape tuple syntax in tensorqs_tutorial.py#3843

Open
horno1337 wants to merge 2 commits intopytorch:mainfrom
horno1337:patch-1
Open

Fix shape tuple syntax in tensorqs_tutorial.py#3843
horno1337 wants to merge 2 commits intopytorch:mainfrom
horno1337:patch-1

Conversation

@horno1337
Copy link
Copy Markdown

removed unnecessary coma from the 'shape' tuple ;)

Description

removed an unnecessary coma from a tuple, no issue has been raised for this so I didn't reference any.

Checklist

  • The issue that is being fixed is referred in the description (see above "Fixes #ISSUE_NUMBER")
  • Only one issue is addressed in this pull request
  • Labels from the issue that this PR is fixing are added to this pull request
  • No unnecessary issues are included into this pull request.

removed unnecessary coma from the 'shape' tuple ;)
@pytorch-bot
Copy link
Copy Markdown

pytorch-bot Bot commented Apr 28, 2026

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/tutorials/3843

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit 36b676d with merge base 49c2c81 (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@meta-cla meta-cla Bot added the cla signed label Apr 28, 2026
@sekyondaMeta
Copy link
Copy Markdown
Contributor

@claude

@claude
Copy link
Copy Markdown

claude Bot commented Apr 30, 2026

Claude finished @sekyondaMeta's task in 32s —— View job


Review of PR #3843

  • Gather context and review the diff
  • Analyze the change for correctness
  • Post review feedback

Change: (2,3,)(2,3) on line 66 of beginner_source/basics/tensorqs_tutorial.py

Verdict: This is a valid style improvement. The trailing comma in (2,3,) is syntactically correct Python — it doesn't change behavior — but it's unconventional for short, fixed-length tuples and could confuse beginners reading this tutorial. Removing it makes the code cleaner and more idiomatic for a tutorial context.

The change is minimal, correct, and doesn't affect functionality. LGTM. ✓


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants