Skip to content

Add max_cores to the sklearn fit methods in estimators to limit the number of CPU cores allocated to the training - #619

Open
tramora wants to merge 2 commits into
mainfrom
max-cores-in-sklearn-api
Open

Add max_cores to the sklearn fit methods in estimators to limit the number of CPU cores allocated to the training#619
tramora wants to merge 2 commits into
mainfrom
max-cores-in-sklearn-api

Conversation

@tramora

@tramora tramora commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator
  • this technical parameter was not added to the estimator initializer to allow a different value per call

Fixes #605 and #585


TODO Before Asking for a Review

  • Rebase your branch to the latest version of main (or main-v10)
  • Make sure all CI workflows are green
  • When adding a public feature/fix: Update the Unreleased section of CHANGELOG.md (no date)
  • Self-Review: Review "Files Changed" tab and fix any problems you find
  • API Docs (only if there are changes in docstrings, rst files or samples):
    • Check the docs build without warning: see the log of the API Docs workflow
    • Check that your changes render well in HTML: download the API Docs artifact and open index.html
    • If there are any problems it is faster to iterate by building locally the API Docs

…he number of CPU cores allocated to the training

- this technical parameter was not added to the estimator initializer to allow a different value per call
@tramora
tramora requested review from popescu-v and removed request for popescu-v August 28, 2026 17:06
@tramora
tramora force-pushed the max-cores-in-sklearn-api branch from 2f1c808 to 78785b2 Compare August 28, 2026 17:17
…value of `max_cores`

- the allocated number of CPU cores can never be greater that the value of `max_cores`
@tramora
tramora force-pushed the max-cores-in-sklearn-api branch from 78785b2 to addb1ad Compare August 28, 2026 17:34
@tramora
tramora requested a review from popescu-v August 28, 2026 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Issue with core limit option in Khiops Python library not working with scikit-learn interface or environment variable

1 participant