Skip to content

Conversation

@muthuArivoli
Copy link
Contributor

@muthuArivoli muthuArivoli commented Sep 15, 2020

Fixes #43669

The bool will still link to https://docs.python.org/3/library/functions.html#bool.
Tested using bmm:
image

@muthuArivoli
Copy link
Contributor Author

@mruberry @zasdfgbnm PTAL

Copy link
Collaborator

@zasdfgbnm zasdfgbnm left a comment

Choose a reason for hiding this comment

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

Super cool. Thanks!

@codecov
Copy link

codecov bot commented Sep 15, 2020

Codecov Report

Merging #44686 into master will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #44686      +/-   ##
==========================================
- Coverage   67.95%   67.94%   -0.01%     
==========================================
  Files         384      384              
  Lines       49743    49743              
==========================================
- Hits        33801    33798       -3     
- Misses      15942    15945       +3     
Impacted Files Coverage Δ
torch/_torch_docs.py 100.00% <ø> (ø)
torch/utils/_benchmark/utils/common.py 77.68% <0.00%> (-2.48%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e107ef5...3b08cf0. Read the comment docs.

@mruberry mruberry added module: docs Related to our documentation, both in docs/ and docblocks triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module labels Sep 15, 2020
@mruberry mruberry self-requested a review September 15, 2020 09:40
Copy link
Collaborator

@mruberry mruberry left a comment

Choose a reason for hiding this comment

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

Thank you!

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.

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

@facebook-github-bot
Copy link
Contributor

@mruberry merged this pull request in 9c364da.

xuzhao9 pushed a commit that referenced this pull request Sep 18, 2020
Summary:
Fixes #43669

The bool will still link to https://docs.python.org/3/library/functions.html#bool.
Tested using bmm:
![image](https://user-images.githubusercontent.com/16063114/93156438-2ad11080-f6d6-11ea-9b81-96e02ee68d90.png)

Pull Request resolved: #44686

Reviewed By: ngimel

Differential Revision: D23703823

Pulled By: mruberry

fbshipit-source-id: 7286afad084f5ab24a1254ad84e5d01907781c85
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 open source triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

sphinx sometimes can not find the correct bool

5 participants