KEMBAR78
Fix broken hyperlinks by awgu · Pull Request #59425 · pytorch/pytorch · GitHub
Skip to content

Conversation

@awgu
Copy link
Collaborator

@awgu awgu commented Jun 3, 2021

Overview:
A number of the hyperlinks in the CONTRIBUTING.md file are broken since they include an extraneous /torch/. This PR fixes those links.

The files whose links are broken are

  • ProcessGroupNCCL.hpp
  • Store.hpp
  • FileStore.hpp
  • TCPStore.hpp
  • PrefixStore.hpp
  • rref_impl.h
  • rref_context.h

Test Plan:
The CONTRIBUTING.md file is at https://github.com/pytorch/pytorch/blob/master/torch/distributed/CONTRIBUTING.md.

ProcessGroupNCCL.hpp should have link https://github.com/pytorch/pytorch/blob/master/torch/lib/c10d/ProcessGroupGloo.hpp, which is equivalent to ../lib/c10d/ProcessGroupGloo.hpp.

Store.hpp should have link https://github.com/pytorch/pytorch/blob/master/torch/lib/c10d/Store.hpp, which is equivalent to ../lib/c10d/Store.hpp.

FileStore.hpp should have link https://github.com/pytorch/pytorch/blob/master/torch/lib/c10d/FileStore.hpp, which is equivalent to ../lib/c10d/FileStore.hpp.

PrefixStore.hpp should have link https://github.com/pytorch/pytorch/blob/master/torch/lib/c10d/PrefixStore.hpp, which is equivalent to ../lib/c10d/PrefixStore.hpp.

rref_interface.h should have link https://github.com/pytorch/pytorch/blob/master/aten/src/ATen/core/rref_interface.h, which is equivalent to ../../aten/src/ATen/core/rref_interface.h.

rref_context.h should have link https://github.com/pytorch/pytorch/blob/master/torch/csrc/distributed/rpc/rref_context.h, which is equivalent to ../csrc/distributed/rpc/rref_context.h.

@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Jun 3, 2021

💊 CI failures summary and remediations

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


  • 1/1 failures introduced in this PR

🕵️ 1 new failure recognized by patterns

The following CI failures do not appear to be due to upstream breakages:

See GitHub Actions build Lint / clang-tidy (1/1)

Step: "Upload artifact" (full log | diagnosis details | 🔁 rerun)

2021-06-03T23:40:37.4160231Z ModuleNotFoundError: No module named 'numpy'
2021-06-03T23:40:37.3814375Z   Not compiling with NUMA.  Suppress this warning with -DUSE_NUMA=OFF
2021-06-03T23:40:37.3814940Z Call Stack (most recent call first):
2021-06-03T23:40:37.3815610Z   CMakeLists.txt:623 (include)
2021-06-03T23:40:37.3815920Z 
2021-06-03T23:40:37.3816098Z 
2021-06-03T23:40:37.3816661Z -- Using third party subdirectory Eigen.
2021-06-03T23:40:37.4002069Z -- Found PythonInterp: /usr/bin/python3 (found suitable version "3.6.9", minimum required is "3.0") 
2021-06-03T23:40:37.4023778Z -- Found PythonLibs: /usr/lib/libpython3.6m.so.1.0 (found suitable version "3.6.9", minimum required is "3.0") 
2021-06-03T23:40:37.4158146Z Traceback (most recent call last):
2021-06-03T23:40:37.4159056Z   File "<string>", line 1, in <module>
2021-06-03T23:40:37.4160231Z ModuleNotFoundError: No module named 'numpy'
2021-06-03T23:40:37.4190668Z -- Could NOT find NumPy (missing: NUMPY_INCLUDE_DIR NUMPY_VERSION) 
2021-06-03T23:40:37.4191539Z CMake Warning at cmake/Dependencies.cmake:982 (message):
2021-06-03T23:40:37.4192376Z   NumPy could not be found.  Not building with NumPy.  Suppress this warning
2021-06-03T23:40:37.4193178Z   with -DUSE_NUMPY=OFF
2021-06-03T23:40:37.4193656Z Call Stack (most recent call first):
2021-06-03T23:40:37.4194238Z   CMakeLists.txt:623 (include)
2021-06-03T23:40:37.4194575Z 
2021-06-03T23:40:37.4194792Z 
2021-06-03T23:40:37.4200225Z -- Using third_party/pybind11.
2021-06-03T23:40:37.4201536Z -- pybind11 include dirs: /__w/pytorch/pytorch/cmake/../third_party/pybind11/include

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.

Copy link
Member

@H-Huang H-Huang left a comment

Choose a reason for hiding this comment

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

Thanks for fixing!

@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

@andwgu merged this pull request in 6d51a89.

deniskokarev pushed a commit to deniskokarev/pytorch that referenced this pull request Jun 9, 2021
Summary:
**Overview:**
A number of the hyperlinks in the [`CONTRIBUTING.md` file](https://github.com/pytorch/pytorch/blob/master/torch/distributed/CONTRIBUTING.md) are broken since they include an extraneous `/torch/`. This PR fixes those links.

The files whose links are broken are
- `ProcessGroupNCCL.hpp`
- `Store.hpp`
- `FileStore.hpp`
- `TCPStore.hpp`
- `PrefixStore.hpp`
- `rref_impl.h`
- `rref_context.h`

Pull Request resolved: pytorch#59425

Test Plan:
The `CONTRIBUTING.md` file is at https://github.com/pytorch/pytorch/blob/master/torch/distributed/CONTRIBUTING.md.

`ProcessGroupNCCL.hpp` should have link https://github.com/pytorch/pytorch/blob/master/torch/lib/c10d/ProcessGroupGloo.hpp, which is equivalent to `../lib/c10d/ProcessGroupGloo.hpp`.

`Store.hpp` should have link https://github.com/pytorch/pytorch/blob/master/torch/lib/c10d/Store.hpp, which is equivalent to `../lib/c10d/Store.hpp`.

`FileStore.hpp` should have link https://github.com/pytorch/pytorch/blob/master/torch/lib/c10d/FileStore.hpp, which is equivalent to `../lib/c10d/FileStore.hpp`.

`PrefixStore.hpp` should have link https://github.com/pytorch/pytorch/blob/master/torch/lib/c10d/PrefixStore.hpp, which is equivalent to `../lib/c10d/PrefixStore.hpp`.

`rref_interface.h` should have link https://github.com/pytorch/pytorch/blob/master/aten/src/ATen/core/rref_interface.h, which is equivalent to `../../aten/src/ATen/core/rref_interface.h`.

`rref_context.h` should have link https://github.com/pytorch/pytorch/blob/master/torch/csrc/distributed/rpc/rref_context.h, which is equivalent to `../csrc/distributed/rpc/rref_context.h`.

Reviewed By: mruberry

Differential Revision: D28888188

Pulled By: andwgu

fbshipit-source-id: 023219184d42284ea1cbfcf519c1b4277dd5a02b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla signed Merged oncall: distributed Add this issue/PR to distributed oncall triage queue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants