Modifications to make the pip deployment (hopefully) work (#837)

.github/workflows/deploy.yml

@@ -48,7 +48,7 @@ jobs:
       name: pypi-deployment
       url: https://pypi.org/p/cosmos-synthesizer
     permissions:
-      id-token: write 
+      id-token: write
 
     steps:
       - name: Download all the dists

.github/workflows/python-app.yml

@@ -35,7 +35,9 @@ jobs:
           # is not set up properly and depends on numpy and cython, the former
           # we depend on, the latter we need to install)
           pip install cython
-          WITH_OPENMP=1 pip install .[bluetides,eagle,illustris,dense_basis]
+          WITH_OPENMP=1 pip install .[bluetides,eagle]
+          pip install "illustris_python @ git+https://github.com/illustristng/illustris_python.git@master"
+          pip install "dense_basis@git+https://github.com/kartheikiyer/dense_basis"
           # Output the compilation report so it can be viewed in an action log
           cat build_synth.log
       - uses: astral-sh/ruff-action@v1 # Lint with Ruff

.github/workflows/static.yml

@@ -34,7 +34,8 @@ jobs:
         run: |
           python -m pip install --upgrade pip
           pip install wheel
-          pip install -e .[docs,dense_basis]
+          pip install -e .[docs]
+          pip install "dense_basis@git+https://github.com/kartheikiyer/dense_basis"
           sudo apt install pandoc
       - name: Download test data
         run: |

pyproject.toml

@@ -126,17 +126,6 @@ bluetides = [
 eagle = [
     "schwimmbad",
 ]
-illustris = [
-    "illustris_python @ git+https://github.com/illustristng/illustris_python.git@master",
-]
-
-# Optional dependency for using dense_basis (note that we 
-# currently have to use the main branch since the PyPi version 
-# hasn't been updated with a fix we required)
-dense_basis = [
-  "dense_basis@git+https://github.com/kartheikiyer/dense_basis",
-]
-
 
 # Project urls
 [project.urls]

src/synthesizer/load_data/load_illustris.py

@@ -35,8 +35,8 @@
 except ImportError:
     raise UnmetDependency(
         "The `illustris_python` module is required to load IllustrisTNG data. "
-        "Install synthesizer with the `illustris` extra dependencies:"
-        " `pip install .[illustris]`."
+        "Install it via `pip install 'illustris_python @ "
+        "git+https://github.com/illustristng/illustris_python.git@master'"
     )