[azure parallel] Remove cimport openmp and improve comment

lesteve · lesteve · commit e9c23ccd2642 · 2023-01-13T14:04:03.000+01:00
diff --git a/build_tools/azure/install.sh b/build_tools/azure/install.sh
@@ -104,7 +104,11 @@ scikit_learn_install() {
         # Without openmp, we use the system clang. Here we use /usr/bin/ar
         # instead because llvm-ar errors
         export AR=/usr/bin/ar
-        # delete omp.h
+        # Make sure omp.h is not present in the conda environment, so that
+        # using an unprotected "cimport openmp" will make this build fail. At
+        # the time of writing, on OSX, blas (mkl or openblas) brings in openmp
+        # so that you end up having the omp.h include inside the conda
+        # environment.
         find $CONDA_PREFIX -name omp.h -delete -print
     fi
 
diff --git a/sklearn/_isotonic.pyx b/sklearn/_isotonic.pyx
@@ -9,7 +9,6 @@ cimport numpy as cnp
 from cython cimport floating
 
 cnp.import_array()
-cimport openmp
 
 def _inplace_contiguous_isotonic_regression(floating[::1] y, floating[::1] w):
     cdef:
