Add support for configurable number of threads per process

golemfactory / gumpi

MPI on Golem Unlimited PoC

GNU Affero General Public License v3.0

2 stars 3 forks source link

Add support for configurable number of threads per process #54

Open marmistrz opened 5 years ago

marmistrz commented 5 years ago

Closes #51.

Tasks:

[x] assign slots based on the number of threads
[x] update the docs
[ ] set the OMP_NUM_THREADS variable on the provider node [needs-gu-changes]
[x] handle not enough slots

marmistrz commented 5 years ago

We won't currently implement automatic configuration of OMP_NUM_THREADS because it requires changes in golem-unlimited and the applications of our current interest support setting the number of OpenMP threads from the command line.

marmistrz commented 5 years ago

@kubkon thanks for the review, I have addressed your remarks, see if it's fine now.

kubkon commented 5 years ago

@kubkon thanks for the review, I have addressed your remarks, see if it's fine now.

Fallible instead of panics looks great!