Skip to content

Add README to tools, delete defunct scripts.#13621

Closed
ezyang wants to merge 1 commit intopytorch:masterfrom
ezyang:pr/doc-tools
Closed

Add README to tools, delete defunct scripts.#13621
ezyang wants to merge 1 commit intopytorch:masterfrom
ezyang:pr/doc-tools

Conversation

@ezyang
Copy link
Copy Markdown
Contributor

@ezyang ezyang commented Nov 6, 2018

Some extra documentation for other bits too.

Signed-off-by: Edward Z. Yang ezyang@fb.com

Some extra documentation for other bits too.

Signed-off-by: Edward Z. Yang <ezyang@fb.com>
@facebook-github-bot facebook-github-bot added the oncall: jit Add this issue/PR to JIT oncall triage queue label Nov 6, 2018
Copy link
Copy Markdown
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.

@ezyang is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@kalisp
Copy link
Copy Markdown

kalisp commented Nov 8, 2018

I think this commit is wrong, at least https://github.com/pytorch/pytorch/blob/master/tools/README.md seems so.

Its doubling 'tools' in all urls, I guess it takes its parent folder (tools) as a basedir and adding tools/ to links its doubling that, resulting in links like https://github.com/pytorch/pytorch/blob/master/tools/tools/autograd

@ezyang ezyang added the merged label Jun 25, 2019
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants