tutorials icon indicating copy to clipboard operation
tutorials copied to clipboard

💡 [REQUEST] - Update CONTRIBUTING.md to include CI steps (e.g. pyspelling)

Open mgs28 opened this issue 1 year ago • 3 comments

🚀 Describe the improvement or the new tutorial

Update CONTRIBUTING.md to include steps for testing CI steps (e.g. pyspelling) so that PRs are a little faster. The make html step doesn't test these items

https://github.com/pytorch/tutorials/blob/main/CONTRIBUTING.md

Existing tutorials on this topic

No response

Additional context

No response

mgs28 avatar Jul 10 '24 16:07 mgs28

Pretty much just run pyspelling from the root of the repo. It takes about 3 minutes to run. To make it faster:

  1. To check only python files, run pyspelling -n python (This still takes like 3 minutes)
  2. to check .rst only: pyspelling -n reST. Currently, for .rst files only the beginner/ dir is included. PRs a welcome to fix spelling in other dirs for .rst.

We could add this to README.md.

svekars avatar Jul 10 '24 20:07 svekars

Thanks! I am happy to do it after I collect all the dings from my PR.

What time limit guidance should we put in the documentation? My original submission took 30 minutes on CPU but I reduced it to 70 seconds. Is 10 minutes on a local device a good number of 5 minutes better?

mgs28 avatar Jul 11 '24 22:07 mgs28

It ran about 3 minutes for me. 5 minutes should be fine.

svekars avatar Jul 15 '24 17:07 svekars

/assigntome

krishnakalyan3 avatar Jun 03 '25 19:06 krishnakalyan3