Skip to content

Conversation

nairbv
Copy link
Collaborator

@nairbv nairbv commented May 22, 2019

was reading the README on github and came across a couple of typos.

@pytorchbot pytorchbot added module: docs Related to our documentation, both in docs/ and docblocks module: operators labels May 22, 2019
frontend:
a) For an inplace operations such as `self.abs_()`
frontend:
a) For an inplace operations such as `self.abs_()`
Copy link
Contributor

Choose a reason for hiding this comment

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

There's some extra white space at the end of this line.

Copy link
Contributor

Choose a reason for hiding this comment

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

Just FYI.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It's intentional, two whitespace characters cause the rendering to break to the next line. Otherwise it puts the a) and b) sections all on one line, which looks kind of messy. You can see what it looks like on master here:
https://github.com/pytorch/pytorch/blob/master/aten/src/ATen/native/README.md

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@nairbv is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

zdevito pushed a commit to zdevito/ATen that referenced this pull request May 23, 2019
Summary:
was reading the README on github and came across a couple of typos.
Pull Request resolved: pytorch/pytorch#20819

Differential Revision: D15469603

Pulled By: nairbv

fbshipit-source-id: 0ed7868de2d4e6d82557a8c170783966f8a1afd7
@facebook-github-bot
Copy link
Contributor

@nairbv merged this pull request in 80aed36.

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

Labels

Merged module: docs Related to our documentation, both in docs/ and docblocks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants