Skip to content

Fix docstring of torch.jit.createResolutionCallback#11921

Closed
zasdfgbnm wants to merge 2 commits intopytorch:masterfrom
zasdfgbnm:patch-1
Closed

Fix docstring of torch.jit.createResolutionCallback#11921
zasdfgbnm wants to merge 2 commits intopytorch:masterfrom
zasdfgbnm:patch-1

Conversation

@zasdfgbnm
Copy link
Collaborator

The sample code in the docstring of torch.jit.createResolutionCallback is not working:

createResolutionCallback() gets the frame of bar. In order to get the frame of baz, one need to use createResolutionCallback(1)

@pytorchbot pytorchbot added the oncall: jit Add this issue/PR to JIT oncall triage queue label Sep 21, 2018
@soumith
Copy link
Collaborator

soumith commented Sep 21, 2018

./torch/jit/init.py:532:1: W293 blank line contains whitespace
./torch/jit/init.py:541:1: W293 blank line contains whitespace

@zasdfgbnm
Copy link
Collaborator Author

@soumith fixed

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.

iotamudelta pushed a commit to ROCm/pytorch that referenced this pull request Sep 21, 2018
Summary:
The sample code in the docstring of `torch.jit.createResolutionCallback` is not working:

`createResolutionCallback()` gets the frame of `bar`. In order to get the frame of  `baz`, one need to use `createResolutionCallback(1)`
Pull Request resolved: pytorch#11921

Differential Revision: D9989123

Pulled By: soumith

fbshipit-source-id: a7166defdccbbf6979f7df4c871298e6b9a2b415
@zasdfgbnm zasdfgbnm deleted the patch-1 branch January 24, 2019 21:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

oncall: jit Add this issue/PR to JIT oncall triage queue open source

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants