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

203 conda #240

Closed
wants to merge 8 commits into from
Closed

203 conda #240

wants to merge 8 commits into from

Conversation

folmos-at-orange
Copy link
Member

No description provided.

Build new docker images with MPICH and OpenMPI. They are named with "-new"
e.g. khiopsdev-new-ubuntu22.04

Generates DEB and RPM packages for multiple mpi implementations (mpich and openMPI)
- Add mpi implemntation discovering in cmake
- Add a suffix to package names corresponding to the mpi implementation (as it advised in fedora packaging guidline)
- Add mpi implementation name in packages names
- Add mpich vs openMPI option in debian and rpm workflows (openMPI is the default)
- Use ctest to run unit tests (issue #23)

Cherry pick LearningTestTool and test hierarchy from dev
Switch to openMPI in kht_test and standard tests

notes:
OpenMpi is used without problem on the linux host. There are some changes:
- the flags used with mpiexec are different from mpich, we have to add:
        - --oversubscribe to launch more process than physical cores
        - --allow-run-as-root
return 0 on success, 1 on fatal error
If there is any error or warning in the log file, the exit code is 0
fix issue #216
When the environment variable KhiopsFileServerActivated is set to true, an MPI file server process
is launched on a single machine. These process are used on clusters to exchange temporary files.
This feature allows to test the file server on a common laptop.
@folmos-at-orange folmos-at-orange changed the base branch from dev to dev-v10 April 16, 2024 13:43
@folmos-at-orange folmos-at-orange changed the base branch from dev-v10 to 203-switch-to-openmpi-for-linux-packages April 16, 2024 13:44
@folmos-at-orange folmos-at-orange changed the base branch from 203-switch-to-openmpi-for-linux-packages to dev-v10 April 16, 2024 13:45
@folmos-at-orange folmos-at-orange force-pushed the 203-conda branch 7 times, most recently from ad2dd2a to ba82001 Compare April 16, 2024 15:24
The kht_test script handles the new naming of th eparallel binaries:
MODL_openmpi or MODL_mpich or MODL (the last one has priority)
@folmos-at-orange folmos-at-orange force-pushed the 203-conda branch 8 times, most recently from 1e79c27 to b9a98d6 Compare April 18, 2024 09:07
packages khiops-core-openmpi with a version >> 10.2.1 replace khiops-core-10.2.1 without trouble with apt and dnf

Update of the Khiops Version to 10.2.2-b0
@folmos-at-orange folmos-at-orange force-pushed the 203-conda branch 6 times, most recently from 60db078 to cff4e80 Compare April 19, 2024 12:29
@folmos-at-orange folmos-at-orange force-pushed the 203-conda branch 26 times, most recently from f39323e to 3dafb0d Compare April 24, 2024 10:32
@folmos-at-orange
Copy link
Member Author

Closed because it is a test branch.

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