Add a Python tutorial template #2264
Conversation
✅ Deploy Preview for pytorch-tutorials-preview ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
beginner_source/template_tutorial.py
Outdated
| </div> | ||
|
|
||
| To test your tutorial locally, make a copy of this repo and add only your | ||
| tutorial removing everything. Then run ``make html``. Verify that all outputs were generated correctly |
There was a problem hiding this comment.
Suggest that we add "else": "removing everything" -> "removing everything else"
| # | ||
| # Example code (the output below is generated automatically): | ||
| # | ||
| import torch |
There was a problem hiding this comment.
[Non-blocking comment]: Will we know which libraries they import just by inspecting their code? I think that is probably the most efficient way. If we ask them to list their imports in a second location--such as above--then we have the possibility that list will get out of sync with what they actually end up importing.
In other words, I think we are all good here as is . . . :-)
beginner_source/template_tutorial.py
Outdated
|
|
||
| To test your tutorial locally, make a copy of this repo and add only your | ||
| tutorial removing everything. Then run ``make html``. Verify that all outputs were generated correctly | ||
| in the created HTML. |
There was a problem hiding this comment.
[Non-blocking comment]: Do we want to start requiring that people run pylint or flake8 or black? Let's discuss--but not a reason to wait on merging.
Address feedback and add another option to build locally
Minor fixes.
* Add a Python tutorial template
Add a tutorial template for interactive tutorials stored in Python files.