Skip to content

Fix Ambiguity in transformer_tutorial.py #2382

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

Conversation

QasimKhan5x
Copy link
Contributor

Fixes #1037

Description

Changed the text,

we would divide the alphabet into 4 sequences of length 6

to

we would divide the alphabet into sequences of length 6, resulting in 4 such sequences

With this, the reader knows that a tensor of shape (seq_length, batch_size) is being created, consistent with the shape of the tensor returned by the batchify function

@QasimKhan5x QasimKhan5x changed the title Qasim fix typo transformer tutorial Fix Ambiguity in transformer_tutorial.py Jun 1, 2023
@netlify
Copy link

netlify bot commented Jun 1, 2023

Deploy Preview for pytorch-tutorials-preview ready!

Name Link
🔨 Latest commit 721bde8
🔍 Latest deploy log https://app.netlify.com/sites/pytorch-tutorials-preview/deploys/6478497a83bb99000896f81d
😎 Deploy Preview https://deploy-preview-2382--pytorch-tutorials-preview.netlify.app/beginner/transformer_tutorial
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@github-actions github-actions bot added Text Issues relating to text tutorials docathon-h1-2023 A label for the docathon in H1 2023 easy and removed cla signed labels Jun 1, 2023
@QasimKhan5x QasimKhan5x closed this Jun 1, 2023
@QasimKhan5x QasimKhan5x deleted the qasim_fix_typo_transformer_tutorial branch June 1, 2023 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docathon-h1-2023 A label for the docathon in H1 2023 easy Text Issues relating to text tutorials
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ambiguity in transformer_tutorial.py
2 participants