Skip to content
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.

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

fix some reST syntax warnings (#393) #394

Closed
wants to merge 1 commit into from
Closed

fix some reST syntax warnings (#393) #394

wants to merge 1 commit into from

Conversation

9bow
Copy link
Contributor

@9bow 9bow commented Dec 19, 2018

fix some reST syntax warnings while building html (using make html or make html-noplot).

Now, syntax warnings that occur are as below:

...
reading sources... [100%] intermediate/spatial_transformer_tutorial
/Users/reserve/Workspace/tutorials/advanced/cpp_extension.rst:2: WARNING: Duplicate explicit target name: "here".
/Users/reserve/Workspace/tutorials/advanced/cpp_extension.rst:2: WARNING: Duplicate explicit target name: "here".
/Users/reserve/Workspace/tutorials/advanced/torch_script_custom_ops.rst:2: WARNING: Duplicate explicit target name: "here".
/Users/reserve/Workspace/tutorials/beginner/audio_classifier_tutorial.rst:11: WARNING: Duplicate explicit target name: "here".
looking for now-outdated files... none found
pickling environment... done
checking consistency... /Users/reserve/Workspace/tutorials/beginner/audio_classifier_tutorial.rst: WARNING: document isn't included in any toctree
/Users/reserve/Workspace/tutorials/beginner/former_torchies_tutorial.rst: WARNING: document isn't included in any toctree
/Users/reserve/Workspace/tutorials/beginner/hybrid_frontend_tutorial.rst: WARNING: document isn't included in any toctree
/Users/reserve/Workspace/tutorials/beginner/nn_basics.rst: WARNING: document isn't included in any toctree
/Users/reserve/Workspace/tutorials/beginner/ptcheat.rst: WARNING: document isn't included in any toctree
done
...

The remained 4 Duplicate explicit target name warnings cannot fix because of their cause.
Please see this issue(sphinx #3921) for more information about this warning.

Thanks.

@netlify
Copy link

netlify bot commented Dec 19, 2018

Deploy preview for pytorch-tutorials-preview ready!

Built with commit 67a654b

https://deploy-preview-394--pytorch-tutorials-preview.netlify.com

@9bow
Copy link
Contributor Author

9bow commented Jan 14, 2019

Rebased to avoid conflicts. Please review this PR.

@9bow
Copy link
Contributor Author

9bow commented Jun 26, 2019

I think these reST syntax warnings do not matter, but it seems to be a burden (among many open PRs) - because those would not affect how the tutorials look.
If you do not care about these minor warnings, I may close this PR.

What do you think?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants