Skip to content

Fix torch.any documentation #65310

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.

Sign up for GitHub

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

Jump to bottom
Closed
oToToT wants to merge 1 commit into from
Closed

Fix torch.any documentation #65310

oToToT wants to merge 1 commit into from

Conversation

@oToToT
Copy link
Contributor

@oToToT oToToT commented Sep 19, 2021

Currently, the description of torch.any would be parsed like

param input
the input tensor.

However, it should be

Tests if any element in input evaluates to True.

@oToToT
Fix torch.any documentation
8055140 
Currently, the description of torch.any would be parsed like

```
param input
the input tensor.
```

However, it should be

```
Tests if any element in input evaluates to True.
```
@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Sep 19, 2021 •
edited
Loading

🔗 Helpful links

💊 CI failures summary and remediations

As of commit 8055140 (more details on the Dr. CI page):

✅ None of the CI failures appear to be your fault 💚

❄️ 1 failure tentatively classified as flaky

but reruns have not yet been triggered to confirm:

See GitHub Actions build win-vs2019-cuda11.3-py3 / test (default, 1, 1, windows.8xlarge.nvidia.gpu) (1/1)

Step: "Test" (full log | diagnosis details | 🔁 rerun) ❄️

2021-09-19T11:13:47.7636722Z ModuleNotFoundError: No module named 'torch' 
2021-09-19T11:13:44.3253169Z )  else (python run_test.py --exclude-jit-executor --verbose --determine-from="""" ) 
2021-09-19T11:13:45.0234136Z   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
2021-09-19T11:13:45.0234998Z                                  Dload  Upload   Total   Spent    Left  Speed
2021-09-19T11:13:45.0235390Z 
2021-09-19T11:13:45.2007157Z   0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
2021-09-19T11:13:45.2007819Z 100  3186  100  3186    0     0  17899      0 --:--:-- --:--:-- --:--:-- 17898
2021-09-19T11:13:45.2019153Z 100  3186  100  3186    0     0  17896      0 --:--:-- --:--:-- --:--:-- 17898
2021-09-19T11:13:47.7633022Z Traceback (most recent call last):
2021-09-19T11:13:47.7635303Z   File "C:\actions-runner\_work\pytorch\pytorch\pytorch-1250370299\test\run_test.py", line 15, in <module>
2021-09-19T11:13:47.7636160Z     import torch
2021-09-19T11:13:47.7636722Z ModuleNotFoundError: No module named 'torch'
2021-09-19T11:13:47.7718915Z 
2021-09-19T11:13:47.7720240Z C:\actions-runner\_work\pytorch\pytorch\pytorch-1250370299\test>popd
2021-09-19T11:13:47.7725903Z 
2021-09-19T11:13:47.7726721Z C:\actions-runner\_work\pytorch\pytorch\pytorch-1250370299>if ERRORLEVEL 1 exit /b 1 
2021-09-19T11:13:47.7752733Z + cleanup
2021-09-19T11:13:47.7753659Z + retcode=1
2021-09-19T11:13:47.7754490Z + set +x
2021-09-19T11:13:47.7805574Z ##[error]Process completed with exit code 1.
2021-09-19T11:13:48.0653626Z ##[group]Run # -ir => recursive include all files in pattern
2021-09-19T11:13:48.0654516Z �[36;1m# -ir => recursive include all files in pattern�[0m
This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.

@codecov
Copy link

codecov bot commented Sep 19, 2021

Codecov Report

Merging #65310 (8055140) into master (88032d8) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #65310      +/-   ##
==========================================
- Coverage   66.45%   66.45%   -0.01%     
==========================================
  Files         735      735              
  Lines       93906    93906              
==========================================
- Hits        62402    62401       -1     
- Misses      31504    31505       +1

@soulitzer soulitzer self-requested a review September 22, 2021 02:47
soulitzer
soulitzer approved these changes Sep 22, 2021
View reviewed changes
Copy link
Contributor

@soulitzer soulitzer left a comment

Choose a reason for hiding this comment

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

Looks good, thanks for the fix!

@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Sep 22, 2021

@soulitzer has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Sep 22, 2021

@soulitzer merged this pull request in 0fe86ac.

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

Reviewers

@soulitzer soulitzer soulitzer approved these changes

Assignees

No one assigned

Labels

cla signed Merged open source

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@oToToT @facebook-github-bot @soulitzer @pytorchbot