-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[Feature request] Implement n_jobs=-2 like scikit-learn #817
Comments
That is a very good suggestion! Would you like to do a PR? |
I can try it. However it might take some weeks because I'm quite busy right now.
Do you mean this one: https://github.com/blue-yonder/tsfresh/blob/main/tsfresh/utilities/distribution.py#L401?
Do you mean this one: https://github.com/blue-yonder/tsfresh/blob/main/tsfresh/feature_selection/relevance.py#L31?
A grep search over the full repo returns this:
|
That would be awesome! If this is not fast enough for you, I can also try to have a look - but more contributors is always better :-)
Yes and yes. Sorry, I was on the smartphone - thanks for providing the links. These two code parts are basically the only two where the
Here are the docstrings that one would need to fix (the rest is not relevant, as only variables are passed).
|
You could have a look into https://github.com/blue-yonder/tsfresh/pull/852/files to get some starter :-) |
When working with sklearn (scikit-learn) I am used to setting the parameter
n_jobs=-2
. As explained at https://scikit-learn.org/stable/glossary.html#term-n_jobs this means:When I set the parameter
n_jobs=-2
in theextract_features()
function I get an error:ValueError: Number of processes must be at least 1
.If tsfresh would be able to accept the parameter
n_jobs=-2
it would be possible to write code for different kinds of CPUs and tell tsfresh "use all CPU cores except for one core". Therefore the code adapts to the CPU it's running on which might be an older Intel 4-core CPU or a newer Ryzen 8, 12 or 16-core CPU.The text was updated successfully, but these errors were encountered: