Skip to content

remove obsolete encoding headers #3361

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 19, 2025
Merged

remove obsolete encoding headers #3361

merged 1 commit into from
May 19, 2025

Conversation

partev
Copy link
Contributor

@partev partev commented May 18, 2025

header "# -- coding: utf-8 --" is only needed for Python 2.x.

Since Python 3.x default encoding is utf-8 and is header is not necessary. It gets displayed in the tutorials here: https://docs.pytorch.org/tutorials/beginner/pytorch_with_examples.html

and could confuse newcomers who might think that encoding header is necessary

Copy link

pytorch-bot bot commented May 18, 2025

🔗 Helpful Links

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

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

✅ No Failures

As of commit f1e06f5 with merge base bb88078 (image):
💚 Looks good so far! There are no failures yet. 💚

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

@partev partev changed the title remove obsolete encoding header remove obsolete encoding headers May 18, 2025
@svekars svekars merged commit 001e1a5 into pytorch:main May 19, 2025
19 checks passed
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.

3 participants