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.

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

remove restriction on number of threads #36778

Merged
merged 1 commit into from
Aug 24, 2020
Merged

remove restriction on number of threads #36778

merged 1 commit into from
Aug 24, 2020

Conversation

vtjnash
Copy link
Sponsor Member

@vtjnash vtjnash commented Jul 23, 2020

Closes #36469

@vtjnash vtjnash closed this Jul 23, 2020
@vtjnash vtjnash reopened this Jul 23, 2020
@JeffBezanson JeffBezanson added the triage This should be discussed on a triage call label Jul 26, 2020
@JeffBezanson JeffBezanson added the multithreading Base.Threads and related functionality label Aug 13, 2020
@JeffBezanson
Copy link
Sponsor Member

Triage approves. But there should be an error if the number of threads is > cpu threads and JULIA_EXCLUSIVE is set.

@JeffBezanson JeffBezanson removed the triage This should be discussed on a triage call label Aug 13, 2020
@vtjnash vtjnash merged commit 5ef6d0f into master Aug 24, 2020
@vtjnash vtjnash deleted the jn/36469 branch August 24, 2020 20:15
simeonschaub pushed a commit to simeonschaub/julia that referenced this pull request Aug 29, 2020
@KristofferC KristofferC added the needs news A NEWS entry is required for this change label Sep 13, 2020
DilumAluthge added a commit that referenced this pull request Jan 6, 2021
DilumAluthge added a commit that referenced this pull request Jan 6, 2021
vtjnash pushed a commit that referenced this pull request Jan 11, 2021
staticfloat pushed a commit that referenced this pull request Jan 15, 2021
staticfloat pushed a commit that referenced this pull request Dec 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
multithreading Base.Threads and related functionality needs news A NEWS entry is required for this change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Warn user about number of threads when greater than logical cores
3 participants