Skip to content

Switch from MS-MPI to Intel MPI for the Windows installer#1056

Draft
bruno-at-orange wants to merge 15 commits into
mainfrom
intel-mpi
Draft

Switch from MS-MPI to Intel MPI for the Windows installer#1056
bruno-at-orange wants to merge 15 commits into
mainfrom
intel-mpi

Conversation

@bruno-at-orange

@bruno-at-orange bruno-at-orange commented Jun 10, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Windows Packaging (NSIS installer)

  • khiops.nsi — Major rewrite: bundles Intel MPI binaries (mpiexec.exe, impi.dll, hydra_*) directly in the installer instead of relying on MS-MPI as a prerequisite
  • Removed the prerequisites page (KhiopsPrerequisiteFunc.nsh, KhiopsGlobals.nsh) — Intel MPI is now bundled, no separate install needed
  • Added Intel MPI third-party license files (Intel-MPI-LICENSE.txt, README.txt)
  • Removed _khiopsgetprocnumber.exe (no longer needed)

Windows Scripts

  • khiops_env.cmd.in — Updated to discover/use Intel MPI's mpiexec instead of MS-MPI
  • set_proc_number.cmd — Updated for Intel MPI

CMake / Build

  • CMakeLists.txt — Updated MPI detection logic for Windows
  • get_mpi_implementation.cmake — Updated to detect Intel MPI
  • install.cmake, packaging.cmake — Updated install rules to include Intel MPI binaries

CI / GitHub Actions

  • action build-khiops — Build with Intel MPI on Windows
  • action test-khiops-install — Updated install checks for Intel MPI (hardened against cmd quote parsing issues)
  • pack-nsis.yml — Updated to supply INTEL_MPI_BIN_DIR and INTEL_MPI_LICENSES_DIR to the NSIS build
  • run-standard-tests.yml — Switch to Intel MPI

Tests

  • kht_test.py — Added Intel MPI detection so non-regression tests correctly identify the MPI implementation

@bruno-at-orange bruno-at-orange changed the title Switch from MS-MPI to **Intel MPI** for the Windows installer Switch from MS-MPI to Intel MPI for the Windows installer Jun 15, 2026
@bruno-at-orange
Switch to intel-mpi
e97609e
@bruno-at-orange
cmake: Discover MPI implementation on windows
2305469
@bruno-at-orange
Remove _khiopsgetprocnumber usage from Windows khiops_env
fb7b9da 
On Windows with Intel MPI, khiops_env.cmd does not need _khiopsgetprocnumber.
Khiops can be launched with `mpiexec MODL.exe` without the `-n` flag.
We use the same pattern in install.cmake for windows and Linux.
@bruno-at-orange
Windows: package Intel MPI binaries with Khiops
b3edf5b
@bruno-at-orange
add third party license for Intel MPI
6937355
@bruno-at-orange
fix issues with mpiexec path
0360ffe 
- fix khiops_env when proc number <=2
- fix broken path in khiops_env.in
@bruno-at-orange
fix khiops script for Linux
8c0f557
@bruno-at-orange
fix(ci/windows): harden Khiops install checks against cmd quote parsing
24ce429 
- replace fragile Windows %VAR% string comparisons with if defined / if not defined
- avoid parsing failures when MPI command contains quoted paths with spaces
@bruno-at-orange
Remove no-op requirements page from NSIS installer
3d9a8bc
@bruno-at-orange
kht_test: add intel MPI detection
22af9c2
@bruno-at-orange
fix pip packaging
79f4228
@bruno-at-orange
remove _khiopsgetprocnumber.exe from the windows installer
d67a48a
@bruno-at-orange
Use intel-mpi in run-standard-tests
9d0e5a8
@bruno-at-orange
[ci skip]
c45ce9b
@bruno-at-orange
Add reference to Temurin license
183a2cf
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@bruno-at-orange