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

Simplify Default MPI Command Arguments According to Recent khiops-env Changes #192

Closed
popescu-v opened this issue Jun 20, 2024 · 0 comments · Fixed by #193
Closed

Simplify Default MPI Command Arguments According to Recent khiops-env Changes #192

popescu-v opened this issue Jun 20, 2024 · 0 comments · Fixed by #193
Assignees
Labels
Status/Done The issue has been addressed and merged to the dev branch
Milestone

Comments

@popescu-v
Copy link
Collaborator

popescu-v commented Jun 20, 2024

Description

Recently (pending for 10.2.2), khiops-env has evolved the arguments that are passed to the mpiexec command.

Most notably, the bindings have been removed, for OpenMPI and MPICH alike.

Questions/Ideas

@popescu-v popescu-v self-assigned this Jun 20, 2024
@popescu-v popescu-v added Status/InDevelopment The issue is in development by one or more team members Priority/0-High To do now labels Jun 20, 2024
@popescu-v popescu-v changed the title Simplify MPI Command According to Recent (and Pending) khiops-env Changes Simplify MPI Command According to Recent khiops-env Changes Jun 20, 2024
popescu-v added a commit that referenced this issue Jun 20, 2024
…ckages

Thusly, functional parity is kept with the `khiops-env` script which is
part of the native Khiops binary packages.

closes #192
popescu-v added a commit that referenced this issue Jun 20, 2024
@popescu-v popescu-v changed the title Simplify MPI Command According to Recent khiops-env Changes Simplify Default MPI Command Arguments According to Recent khiops-env Changes Jun 20, 2024
popescu-v added a commit that referenced this issue Jun 20, 2024
…ckages

Thusly, functional parity is kept with the `khiops-env` script which is
part of the native Khiops binary packages.

closes #192
popescu-v added a commit that referenced this issue Jun 20, 2024
popescu-v added a commit that referenced this issue Jun 24, 2024
…ckages

Thusly, functional parity is kept with the `khiops-env` script which is
part of the native Khiops binary packages.

closes #192
popescu-v added a commit that referenced this issue Jun 24, 2024
popescu-v added a commit that referenced this issue Jun 24, 2024
…ckages

Thusly, functional parity is kept with the `khiops-env` script which is
part of the native Khiops binary packages.

Also honor the `KHIOPS_MPI_VERBOSE` environment variable which controls
the verbosity of the MPI command.

closes #192
popescu-v added a commit that referenced this issue Jun 24, 2024
…ckages

Thusly, functional parity is kept with the `khiops-env` script which is
part of the native Khiops binary packages.

Also honor the `KHIOPS_MPI_VERBOSE` environment variable which controls
the verbosity of the MPI command.

closes #192
popescu-v added a commit that referenced this issue Jun 24, 2024
…ckages

Thusly, functional parity is kept with the `khiops-env` script which is
part of the native Khiops binary packages.

Also honor the `KHIOPS_MPI_VERBOSE` environment variable which controls
the verbosity of the MPI command.

closes #192
popescu-v added a commit that referenced this issue Jun 25, 2024
…ckages

Thusly, functional parity is kept with the `khiops-env` script which is
part of the native Khiops binary packages.

Also honor the `KHIOPS_MPI_VERBOSE` environment variable which controls
the verbosity of the MPI command.

closes #192
@folmos-at-orange folmos-at-orange added Status/Done The issue has been addressed and merged to the dev branch and removed Status/InDevelopment The issue is in development by one or more team members labels Jul 3, 2024
@folmos-at-orange folmos-at-orange added this to the 10.2.2.0 milestone Jul 3, 2024
@folmos-at-orange folmos-at-orange removed the Priority/0-High To do now label Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status/Done The issue has been addressed and merged to the dev branch
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants