Skip to content

typo/grammar fixes#11344

Closed
rasbt wants to merge 1 commit intopytorch:masterfrom
rasbt:typofix
Closed

typo/grammar fixes#11344
rasbt wants to merge 1 commit intopytorch:masterfrom
rasbt:typofix

Conversation

@rasbt
Copy link
Contributor

@rasbt rasbt commented Sep 6, 2018

Fixes some minor grammar issues in the code base.

PS: I was actually looking for the following one but couldn't find it via grepping in this repo:

screen shot 2018-09-06 at 3 27 39 pm

Any idea in which file this issue is raised?

@goldsborough
Copy link
Contributor

goldsborough commented Sep 6, 2018

How is "fewer" not grammatically correct? https://www.dictionary.com/e/fewer-vs-less/

screen shot 2018-09-06 at 13 50 08

@rasbt
Copy link
Contributor Author

rasbt commented Sep 6, 2018

How is "fewer" not grammatically correct?

You mean “correct” instead of “not correct”? I learned it this way that you use “fewer” for things that you count (1, 2, 3, ...) and “less” for quantities that you don’t count (eg sth like fewer words but less text)

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.

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

PenghuiCheng pushed a commit to PenghuiCheng/pytorch that referenced this pull request Sep 11, 2018
Summary:
Fixes some minor grammar issues in the code base.

PS: I was actually looking for the following one but couldn't find it via grepping in this repo:

![screen shot 2018-09-06 at 3 27 39 pm](https://user-images.githubusercontent.com/5618407/45184280-1e16a980-b1ec-11e8-9cb1-87a96738bdd1.png)

Any idea in which file this issue is raised?
Pull Request resolved: pytorch#11344

Differential Revision: D9696454

Pulled By: soumith

fbshipit-source-id: 8ffe494b1bf1efb0e35563381d9da2e1e8032a3c
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.

6 participants