Skip to content

Conversation

soumith
Copy link
Member

@soumith soumith commented Jan 30, 2018

once this is covered, using torch.set_num_threads(x) will consistently work without needing environment variables.

@fmassa
Copy link
Member

fmassa commented Jan 30, 2018

Are there people using PyTorch with OpenBLAS? Because OpenBLAS doesn't change the number of threads using omp_num_threads, so this might not be enough for those people to avoid setting the environment variable.

@soumith
Copy link
Member Author

soumith commented Jan 30, 2018

I guess we need to define BLAS_IS_OPEN and then use the corresponding openblas prototype. I'll do that in a follow-up PR

@soumith soumith merged commit 2aaeec0 into master Jan 31, 2018
@soumith soumith deleted the mklthreads branch January 31, 2018 16:59
ssnl added a commit to ssnl/pytorch that referenced this pull request Jan 31, 2018
soumith pushed a commit that referenced this pull request Jan 31, 2018
* Revert "Clarify grad_input_mask documentation in derivatives.yaml (#4963)"

This reverts commit 6f3266b.

* Revert "fix triu and tril for zero-strided inputs on gpu (#4962)"

This reverts commit 6c197c2.

* Revert "Add mutex for CPU RNG and move TH to C++ (#4041)"

This reverts commit 96239dd.

* Revert "Support multivariate TransformedDistributions (#4937)"

This reverts commit ca5071d.

* Revert "Only check that arguments are Variables in VariableType (#4943)"

This reverts commit d444379.

* Revert "torch.set_num_threads sets MKL option too (#4949)"

This reverts commit 2aaeec0.
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.

2 participants