-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add KHIOPS_MPI_VERBOSE in khiops_env (UNIXes only) #393
Add KHIOPS_MPI_VERBOSE in khiops_env (UNIXes only) #393
Conversation
5e163af
to
7250e88
Compare
7250e88
to
9d82f04
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comments.
601d96c
to
87358bf
Compare
87358bf
to
f80c026
Compare
299afcb
to
ca9e3a0
Compare
4e5ba4e
to
75cc143
Compare
75cc143
to
e579801
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are 2 comments to address.
- Add the variable KHIOPS_MPI_VERBOSE to disable the flag "--quiet" in the mpirun command. In khiops_env, its default value is true and the mpirun is run without --quiet. if its value is different from 'true', '--quiet' is added to the command line. - On the contrary, KHIOPS_MPI_VERBOSE default value is false in the script 'khiops': we don't want to see MPI messages when running khiops in standalone. - some variables are mandatory to run MPI correctly (like OMPI_MCA_btl_vader_single_copy_mechanism), these variables are displayed when running `khiops-env --env`. In that way, all variables needed to run Khiops are avalaible for khiops' wrappers. - Add KHIOPS_MPI_VERBOSE=true to the tests on CI: test-khiops-on-iris and test-khiops-install in order to check that there is no MPI error
The environment modules systematically: - undloads current mpi module - loads the desired mpi module In that way we are sure to launch the appropriate mpiexec.
e579801
to
e2b0b70
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Add the variable KHIOPS_MPI_VERBOSE to disable the flag "--quiet" in the mpirun command. Its default value in
khiops_env
is true and the mpirun is verbose. if its value is different from 'true', '--quiet' is added to the command line.Close #383
Close #310