Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Support n_jobs, other changes #21

Closed
wants to merge 2 commits into from
Closed

Support n_jobs, other changes #21

wants to merge 2 commits into from

Conversation

bibikar
Copy link
Contributor

@bibikar bibikar commented Mar 14, 2020

@oleksandr-pavlyk

This quick PR addresses the following

  • Allow n_jobs in sklearn and only use it if daal is not present
  • Makefile: use rbf kernel for SVM by default
  • Use LinearSVC when not using daal and linear kernel is selected in SVM (benchmarking the linear kernel in SVC #14)
  • specify OMP_NUM_THREADS in Makefile when single-threaded execution is specified, so stock benchmarks will not be implicitly threaded while we want them to run serially.
  • specify USE_DAAL4PY_SKLEARN = yes in makefile so daal will be used when needed
  • decrease SVM problem size

@bibikar
Add n_jobs support to benchmarks, use rbf kernel for SVC, automatical…
211ec76 
…ly use LinearSVC
@bibikar
Makefile: Don't change number of threads used by default
d9a8683 
Also, we now specify OMP_NUM_THREADS when needed so stock execution of
benchmarks won't be threaded without us knowing it
@napetrov napetrov closed this May 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bibikar @napetrov