exp: try not to install geos or shapely explicitly

yt-project · Aug 7, 2023 · 984f8b5 · 984f8b5
1 parent fed64eb
commit 984f8b5
Showing 1 changed file with 18 additions and 24 deletions.
diff --git a/tests/ci_install.sh b/tests/ci_install.sh
@@ -10,30 +10,24 @@ if [[ ${dependencies} == "full" || ${dependencies} == "cartopy" ]]; then
     # see https://github.com/yt-project/yt/issues/4270
     python -m pip install 'pip>=21.2'
 
-    # see https://scitools.org.uk/cartopy/docs/latest/installing.html?highlight=install#building-from-source
-    if [[ "${RUNNER_OS}" == "Windows" ]]; then
-        conda install shapely scipy
-    else
-        # geos is needed for shapely, which is needed for cartopy, see
-        case ${RUNNER_OS} in
-        linux|Linux)
-            sudo apt-get -qqy update
-            sudo apt-get -qqy install \
-            libhdf5-serial-dev \
-            libnetcdf-dev \
-            libgeos-dev \
-            libopenmpi-dev \
-            libfuse2
-            ;;
-        osx|macOS)
-            sudo mkdir -p /usr/local/man
-            sudo chown -R "${USER}:admin" /usr/local/man
-            HOMEBREW_NO_AUTO_UPDATE=1 brew install hdf5 geos open-mpi netcdf ccache macfuse
-            ;;
-        esac
-        python -m pip install shapely --no-binary=shapely
-        python -m pip install scipy
-    fi
+    case ${RUNNER_OS} in
+    linux|Linux)
+        sudo apt-get -qqy update
+        sudo apt-get -qqy install \
+        libhdf5-serial-dev \
+        libnetcdf-dev \
+        libopenmpi-dev \
+        libfuse2
+        ;;
+    osx|macOS)
+        sudo mkdir -p /usr/local/man
+        sudo chown -R "${USER}:admin" /usr/local/man
+        HOMEBREW_NO_AUTO_UPDATE=1 brew install hdf5 open-mpi netcdf ccache macfuse
+        ;;
+    esac
+
+    # scipy is an optional dependency to cartopy
+    python -m pip install scipy
 fi
 
 # Step 2: install deps and yt