Skip to content
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

Documentation cleanup #23148

Closed
wants to merge 7 commits into from
Closed

Conversation

z-a-f
Copy link
Contributor

@z-a-f z-a-f commented Jul 22, 2019

Stack from ghstack:

Differential Revision: D16414202

@pytorchbot pytorchbot added the module: nn Related to torch.nn label Jul 22, 2019
@z-a-f z-a-f requested review from dzhulgakov, jspisak and soumith and removed request for dzhulgakov July 22, 2019 00:20
Documentation cleanup

gh-metadata: pytorch pytorch 23148 gh/zafartahirov/14/head
@z-a-f
Copy link
Contributor Author

z-a-f commented Jul 22, 2019

@pytorchbot retest this please

Documentation cleanup

gh-metadata: pytorch pytorch 23148 gh/zafartahirov/14/head
torch/nn/quantized/modules/conv.py Outdated Show resolved Hide resolved
torch/nn/quantized/modules/conv.py Outdated Show resolved Hide resolved
:attr:`inplace` is not supported for the quantized relu.

Applies the rectified linear unit function element-wise. See
:class:`~torch.nn.quantized.ReLU` for more details.
Copy link
Member

Choose a reason for hiding this comment

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

again, torch.nn.functional.ReLU is what you meant to type here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Actually, about that -- don't we want to refer the functional to its module counterpart? The documentation in the torch.nn.quantized.ReLU is more relevant to this than the non-quantized counterpart

@z-a-f z-a-f requested a review from soumith July 26, 2019 19:16
@zou3519 zou3519 deleted the gh/zafartahirov/14/head branch July 31, 2019 18:33
@facebook-github-bot
Copy link
Contributor

@zafartahirov merged this pull request in 5e4c24b.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Merged module: nn Related to torch.nn
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants