-
Notifications
You must be signed in to change notification settings - Fork 935
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
doc: enable multiprocessing for pyspelling #12751
Conversation
doc/.sphinx/spellingcheck.yaml
Outdated
@@ -1,3 +1,4 @@ | |||
jobs: 8 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why 8?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's the example they used in their announcement. ;)
Does another value make more sense?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we maybe use something equivalent to $$nproc
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we are using GH runners with 2 CPUs currently so 2 would probably be best.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MusicDin thats a good idea
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
$$nproc
How do I get that in the config file?
Current time is around 6 mins. |
Pyspelling has a new option to run jobs in parallel. This should make the spelling check considerably faster. Signed-off-by: Ruth Fuchss <ruth.fuchss@canonical.com>
e84f69f
to
d0c61c3
Compare
@ru-fu nice! does the docs build itself support a multi core mode? |
3.50 with 2 jobs (could just be normal variation though)
I don't know - I doubt it, otherwise someone would probably have suggested already to speed it up. 😉 |
Pyspelling has a new option to run jobs in parallel. This should make the spelling check considerably faster.