Skip to content

Commit d3a78c4

Browse files
authored
Make ipywidgets and scipy properly optional (#100)
Typo on `ipywidgets` was causing the whole jupyter stack to get installed for the base stackstac installation! Scipy is only needed for `reproject_array`, which isn't in the core API IMO. And xarray already raises a decent error when it's missing.
1 parent 0239a5d commit d3a78c4

File tree

2 files changed

+10
-9
lines changed

2 files changed

+10
-9
lines changed

poetry.lock

Lines changed: 7 additions & 7 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

pyproject.toml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ pystac-client = {version = "^0.3", optional = true}
3838
python = "^3.8"
3939
rasterio = "^1.2.3"
4040
sat-search = {version = "^0.3.0", optional = true}
41-
scipy = "^1.6.1"
41+
scipy = {version = "^1.6.1", optional = true}
4242
sphinx-autodoc-typehints = {version = "^1.11.1", optional = true}
4343
xarray = ">=0.18,<1"
4444

@@ -66,7 +66,7 @@ binder = [
6666
"distributed",
6767
"geogif",
6868
"ipyleaflet",
69-
"ipywidets",
69+
"ipywidgets",
7070
"jupyterlab-geojson",
7171
"jupyter-server-proxy",
7272
"jupyterlab-system-monitor",
@@ -95,6 +95,7 @@ viz = [
9595
"matplotlib",
9696
"mercantile",
9797
"Pillow",
98+
"scipy",
9899
]
99100

100101
[build-system]

0 commit comments

Comments
 (0)