Don't add $CONDA_PREFIX/bin to the PATH on Linux and MacOS

Instead, add the path to Conda environment bin directory to the
environment, *as a specific environment variable*.

Thus, we match "Conda-based" environments as much as possible.

A "Conda-based" environment is a Conda environment whose packages are
launched outside of the environment, as e.g. when using the Khiops Python
library from a JuPyter notebook which:
- is installed outside the Conda environment where Khiops is installed
- uses a Python kernel (via the `ipykernel` Conda package) which is
  installed inside the Conda envirovnment.

This configuration results in two particularities:
- `sys.path` is correctly set according to the Conda environment (as
  the Python kernel is installed therein);
- `PATH` does not contain `$CONDA_PREFIX/bin`.

Author: popescu-v

.github/actions/test-khiops-install/action.yml

@@ -5,6 +5,8 @@ runs:
   using: composite
   steps:
     - name: Test that the executables are installed (Linux/macOS)
+      # We compute the directory of khiops_env if khiops_env is in the PATH,
+      # otherwise we get this directory from the environment
       # We call khiops_env a first time to test the calling of _khiopsgetprocnumber
       # then we play with KHIOPS_PROC_NUMBER:
       # - with KHIOPS_PROC_NUMBER=2, khiops should not use MPI, we check that KHIOPS_MPI_COMMAND is empty
@@ -19,13 +21,28 @@ runs:
         OMPI_MCA_rmaps_base_oversubscribe: true
         KHIOPS_MPI_VERBOSE: true
       run: |
+        # Compute or get the khiops_env directory
+        if type -P khiops_env &> /dev/null
+        then
+          KHIOPS_ENV_DIR=$(dirname $(type -P khiops_env))
+        elif [[ -n "$CONDA_ENV_BIN_DIR" ]]
+        then
+          KHIOPS_ENV_DIR=$CONDA_ENV_BIN_DIR
+        else
+          echo "::error::This action expects CONDA_ENV_BIN_DIR to be defined"
+          false
+        fi
+        echo
+        echo --- khiops_env directory $KHIOPS_ENV_DIR
         echo
         echo --- Khiops environment with KHIOPS_PROC_NUMBER not set
-        khiops_env --env
+
+        # khiops_env is not in the path anymore, for Conda environments
+        $KHIOPS_ENV_DIR/khiops_env --env
         echo
         export KHIOPS_PROC_NUMBER=2
         echo --- Khiops environment with KHIOPS_PROC_NUMBER=$KHIOPS_PROC_NUMBER
-        source khiops_env --env
+        source $KHIOPS_ENV_DIR/khiops_env --env
         if [ ! -z "$KHIOPS_MPI_COMMAND" ]; 
         then 
           echo "::error::MPI is used even though there are only 2 procs available"
@@ -34,7 +51,7 @@ runs:
         echo
         export KHIOPS_PROC_NUMBER=8
         echo --- Khiops environment with KHIOPS_PROC_NUMBER=$KHIOPS_PROC_NUMBER
-        source khiops_env --env
+        source $KHIOPS_ENV_DIR/khiops_env --env
         if [ ! -z "$KHIOPS_MPI_ERROR" ]; 
         then 
           echo "::error::Unexpected MPI error: $KHIOPS_MPI_ERROR"
@@ -47,10 +64,10 @@ runs:
         fi
         echo
         echo --- Khiops status
-        khiops -s
+        $KHIOPS_ENV_DIR/khiops -s
         echo
         echo --- Khiops Coclustering status
-        khiops_coclustering -s
+        $KHIOPS_ENV_DIR/khiops_coclustering -s
     - name: Test that the executables are installed (windows)
       if: runner.os == 'Windows'
       shell: cmd /C call {0}
@@ -96,8 +113,19 @@ runs:
         OMPI_MCA_rmaps_base_oversubscribe: true
         KHIOPS_MPI_VERBOSE: true
       run: |
-        khiops -s
-        khiops -v &> output
+        # Compute or get the khiops_env directory
+        if type -P khiops_env &> /dev/null
+        then
+          KHIOPS_ENV_DIR=$(dirname $(type -P khiops_env))
+        elif [[ -n "$CONDA_ENV_BIN_DIR" ]]
+        then
+          KHIOPS_ENV_DIR=$CONDA_ENV_BIN_DIR
+        else
+          echo "::error::This action expects CONDA_ENV_BIN_DIR to be defined"
+          false
+        fi
+        $KHIOPS_ENV_DIR/khiops -s
+        $KHIOPS_ENV_DIR/khiops -v &> output
         LINE_NUMBER=$(wc -l < output)
         if [ $LINE_NUMBER -gt 1 ] ;
         then 
@@ -120,8 +148,19 @@ runs:
       if: runner.os != 'Windows'
       shell: bash
       run: |-
-        khiops_coclustering -s
-        khiops_coclustering -v &> output
+        # Compute or get the khiops_env directory
+        if type -P khiops_env &> /dev/null
+        then
+          KHIOPS_ENV_DIR=$(dirname $(type -P khiops_env))
+        elif [[ -n "$CONDA_ENV_BIN_DIR" ]]
+        then
+          KHIOPS_ENV_DIR=$CONDA_ENV_BIN_DIR
+        else
+          echo "::error::This action expects CONDA_ENV_BIN_DIR to be defined"
+          false
+        fi
+        $KHIOPS_ENV_DIR/khiops_coclustering -s
+        $KHIOPS_ENV_DIR/khiops_coclustering -v &> output
         LINE_NUMBER=$(wc -l < output)
         if [ $LINE_NUMBER -gt 1 ] ;
         then 
@@ -147,7 +186,18 @@ runs:
         KHIOPS_PROC_NUMBER: 4
         OMPI_MCA_rmaps_base_oversubscribe: true
       run: |-
-        PROC_NUMBER=$(khiops -s | grep "Logical processes" | awk '{print $NF}')
+        # Compute or get the khiops_env directory
+        if type -P khiops_env &> /dev/null
+        then
+          KHIOPS_ENV_DIR=$(dirname $(type -P khiops_env))
+        elif [[ -n "$CONDA_ENV_BIN_DIR" ]]
+        then
+          KHIOPS_ENV_DIR=$CONDA_ENV_BIN_DIR
+        else
+          echo "::error::This action expects CONDA_ENV_BIN_DIR to be defined"
+          false
+        fi
+        PROC_NUMBER=$($KHIOPS_ENV_DIR/khiops -s | grep "Logical processes" | awk '{print $NF}')
         if [ "$PROC_NUMBER" != "$KHIOPS_PROC_NUMBER" ] ;
         then
           echo "::error::Wrong proc number ($PROC_NUMBER vs $KHIOPS_PROC_NUMBER)"

.github/actions/test-khiops-on-iris/action.yml

@@ -44,8 +44,19 @@ runs:
           echo "KHIOPS_SCRIPT=$KHIOPS_SCRIPT" >> "$GITHUB_ENV"
           echo "KHIOPS_CC_SCRIPT=$KHIOPS_CC_SCRIPT" >> "$GITHUB_ENV"
         else
-          echo "KHIOPS_SCRIPT=khiops" >> "$GITHUB_ENV"
-          echo "KHIOPS_CC_SCRIPT=khiops_coclustering" >> "$GITHUB_ENV"
+          # Compute or get the khiops_env directory
+          if type -P khiops_env &> /dev/null
+          then
+            KHIOPS_ENV_DIR=$(dirname $(type -P khiops_env))
+          elif [[ -n "$CONDA_ENV_BIN_DIR" ]]
+          then
+            KHIOPS_ENV_DIR=$CONDA_ENV_BIN_DIR
+          else
+            echo "::error::This action expects CONDA_ENV_BIN_DIR to be defined"
+            false
+          fi
+          echo "KHIOPS_SCRIPT=$KHIOPS_ENV_DIR/khiops" >> "$GITHUB_ENV"
+          echo "KHIOPS_CC_SCRIPT=$KHIOPS_ENV_DIR/khiops_coclustering" >> "$GITHUB_ENV"
         fi
     - name: Run Khiops tests
       shell: bash

.github/workflows/conda.yml

@@ -119,9 +119,14 @@ jobs:
       - name: Install the Khiops executables in the Conda environment
         run: |
           conda install --channel conda-forge --channel ./build/conda khiops-core
-      - name: Add  CONDA_PREFIX to shared PATH
+      - name: Add  CONDA_PREFIX to shared PATH (Windows)
+        if: runner.os == 'Windows'
         run: |
           echo "$CONDA_PREFIX/bin" >> $GITHUB_PATH
+      - name: Add Conda environment binary directory to the environment (Linux and
+          MacOS)
+        if: runner.os != 'Windows'
+        run: echo "CONDA_ENV_BIN_DIR=$CONDA_PREFIX/bin" >> $GITHUB_ENV
       - name: Checkout sources
         uses: actions/checkout@v4
       - name: Test that the Khiops executables are installed