add custom easyblock for FreeFEM#1969
Conversation
|
@boegel is there a difference in performance between setting |
| except ValueError as err: | ||
| raise EasyBuildError("Failed to parse 'parallel' value '%s' as integer: %s", parallel, err) | ||
|
|
||
| # use number of threads equal to 1/4th of available cores (and ensure at least one) |
| # run tests unless they're disabled explicitly | ||
| if self.cfg['runtest'] is None or self.cfg['runtest']: | ||
| # avoid oversubscribing, by using both OpenMP threads and having OpenBLAS use threads as well | ||
| env.setvar('OPENBLAS_NUM_THREAD', '1') |
There was a problem hiding this comment.
@boegel I think this should be OPENBLAS_NUM_THREADS ? (added 'S')
|
@boegel also, according to the OpenBLAS documentation https://github.com/xianyi/OpenBLAS |
|
Precisely, one should use OMP_NUM_THREAD and nothing else. |
|
@lexming Suggestions (finally!) tackled, can you take another look? |
|
Positive test in https://gist.github.com/eaee20c537c572ba0cdfa1a542bfd8fb |
|
Going in, thanks for the update @boegel ! |
This is an updated version of #1488.