[DOC] Fixing n_jobs doc param in multiclass.py#17489
[DOC] Fixing n_jobs doc param in multiclass.py#17489thomasjpfan merged 7 commits intoscikit-learn:masterfrom
Conversation
sklearn/multiclass.py
Outdated
| The number of jobs to use for the computation, where each job computes | ||
| a classifier in parallel. |
There was a problem hiding this comment.
thanks for the PR @annejeevan
I feel like the current formulation may suggest that the internal implementation of the classifier is parallelized which is usually not the case
How about
The number of jobs to use for the computation: the
n_classesone-vs-rest problems are computed in parallel.
There was a problem hiding this comment.
Thank you for the feedback @NicolasHug .
Yes, it looks like. Sure. That looks good. I will try to modify the doc string.
There was a problem hiding this comment.
Hey @NicolasHug, I have modified accordingly. Can you please check the PR? Thanks!
sklearn/multiclass.py
Outdated
|
|
||
| n_jobs : int or None, optional (default=None) | ||
| The number of jobs to use for the computation. | ||
| The number of jobs to use for the computation: the n_classes |
There was a problem hiding this comment.
| The number of jobs to use for the computation: the n_classes | |
| The number of jobs to use for the computation: the `n_classes` |
There was a problem hiding this comment.
Hey @NicolasHug, I have modified accordingly. Can you please check the PR? Thanks!
sklearn/multiclass.py
Outdated
|
|
||
| n_jobs : int or None, optional (default=None) | ||
| The number of jobs to use for the computation. | ||
| The number of jobs to use for the computation: the n_classes |
There was a problem hiding this comment.
In this case there are n_classes * (n_classes - 1) / 2 OVO problems
There was a problem hiding this comment.
Hey @NicolasHug, I have modified accordingly. Can you please check the PR? Thanks!
sklearn/multiclass.py
Outdated
| The number of jobs to use for the computation: the code_size | ||
| Output-Code multiclass problems are computed in parallel. |
There was a problem hiding this comment.
looks like the actual number is tricky here so let's just go with "the multiclass problems are computed in parallel."
There was a problem hiding this comment.
Hey @NicolasHug, I have modified accordingly. Can you please check the PR? Thanks!
sklearn/multiclass.py
Outdated
| n_jobs : int or None, optional (default=None) | ||
| The number of jobs to use for the computation, where each job computes | ||
| a classifier in parallel. | ||
| The number of jobs to use for the computation: the n_classes one-vs-rest problems are computed in parallel. |
There was a problem hiding this comment.
This needs to rewrap to make sure we are under 80 characters per line.
There was a problem hiding this comment.
Sure @thomasjpfan. I think I have updated it already. Can you please verify the new commit.
thomasjpfan
left a comment
There was a problem hiding this comment.
LGTM Thank you @emdupre and @annejeevan !
* Fixing n_jobs doc param in multiclass.py * Fixing n_jobs doc param in multiclass.py * Fixing n_jobs doc param in multiclass.py * Better documenting n_jobs param in multiclass * Better documenting n_jobs param in multiclass * Better documenting n_jobs param in multiclass * Better documenting n_jobs param in multiclass
* Fixing n_jobs doc param in multiclass.py * Fixing n_jobs doc param in multiclass.py * Fixing n_jobs doc param in multiclass.py * Better documenting n_jobs param in multiclass * Better documenting n_jobs param in multiclass * Better documenting n_jobs param in multiclass * Better documenting n_jobs param in multiclass
Reference Issues/PRs
Reference Issue #14228
What does this implement/fix? Explain your changes.
This fixes n_jobs param documentation in multiclass.py
Any other comments?
Submitted with @emdupre for the #DataUmbrella June sprint