Make matplotlib dependency optional. (#594)

* Make matplotlib dependency optional.

* Require new mocpy.

* Add requirement to RTD build. Use similar importorskip logic for sphgeom tests.

* Give more general install instructions.

* Autoformat.

* Add full dependency in setup and smoke tests.

Author: delucchi-cmu

.github/workflows/smoke-test.yml

@@ -34,14 +34,22 @@ jobs:
       run: |
         sudo apt-get update
         uv pip install --system -e .[dev]
-        uv pip install --system -e .[full]
         if [ -f requirements.txt ]; then uv pip install --system -r requirements.txt; fi
     - name: List dependencies
       run: |
         pip list
     - name: Run unit tests with pytest
       run: |
         python -m pytest
+    - name: Install FULL dependencies
+      run: |
+        uv pip install --system -e .[full]
+    - name: List FULL dependencies
+      run: |
+        pip list
+    - name: Run FULL unit tests with pytest
+      run: |
+        python -m pytest
     - name: Send status to Slack app
       if: ${{ failure() && github.event_name != 'workflow_dispatch' }}
       id: slack

.github/workflows/testing-and-coverage.yml

@@ -30,9 +30,14 @@ jobs:
       run: |
         sudo apt-get update
         uv pip install --system -e .[dev]
-        uv pip install --system -e .[full]
         if [ -f requirements.txt ]; then uv pip install --system -r requirements.txt; fi
     - name: Run unit tests with pytest
+      run: |
+        python -m pytest
+    - name: Install FULL dependencies
+      run: |
+        uv pip install --system -e .[full]
+    - name: Run FULL unit tests with pytest
       run: |
         python -m pytest --cov=hats --cov-report=xml
     - name: Upload coverage report to codecov

.setup_dev.sh

@@ -41,6 +41,8 @@ python -m pip install -e . > /dev/null
 
 echo "Installing developer dependencies in local environment"
 python -m pip install -e .'[dev]' > /dev/null
+python -m pip install -e .'[full]' > /dev/null
+if [ -f requirements.txt ]; then python -m pip install -r requirements.txt > /dev/null; fi
 if [ -f docs/requirements.txt ]; then python -m pip install -r docs/requirements.txt > /dev/null; fi
 
 echo "Installing pre-commit"

docs/requirements.txt

@@ -1,6 +1,7 @@
 ipykernel
 ipython
 jupytext
+matplotlib
 myst_parser
 nbconvert
 nbsphinx

pyproject.toml

@@ -23,8 +23,7 @@ dependencies = [
     "cdshealpix>=0.7.0",
     "fsspec>=2023.10.0", # Used for abstract filesystems
     "jproperties>=2.0.0",
-    "matplotlib>=3.10.1",
-    "mocpy>=0.17.1",
+    "mocpy>=0.19.0",
     "nested-pandas>=0.4.1",
     "numba>=0.58",
     "numpy>=2,<3", 
@@ -55,9 +54,13 @@ dev = [
     "pytest-mock", # Used to mock objects in tests
     "pytest-timeout", # Used to test for code efficiency
 ]
+plotting = [
+    "matplotlib>=3.10.1",
+]
 full = [
     "fsspec[full]", # complete file system specs.
     "lsst-sphgeom ; sys_platform == 'darwin' or sys_platform == 'linux'", # To handle spherical sky polygons, not available on Windows
+    "matplotlib>=3.10.1",
 ]
 
 [build-system]