Skip to content

Commit 78b89f6

Browse files
maarten-icolivhoenen
authored andcommitted
Add xarray as optional imas-python dependency
1 parent 09fb355 commit 78b89f6

File tree

2 files changed

+8
-4
lines changed

2 files changed

+8
-4
lines changed

.github/workflows/test_with_pytest.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ jobs:
2828
python -m venv venv
2929
source venv/bin/activate
3030
pip install --upgrade pip setuptools wheel
31-
pip install .[h5py,netcdf,test]
31+
pip install .[test]
3232
3333
- name: Run tests
3434
run: |

pyproject.toml

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -71,8 +71,7 @@ dependencies = [
7171
[project.optional-dependencies]
7272
# these self-dependencies are available since pip 21.2
7373
all = [
74-
"imas-python[test,docs,netcdf,h5py]"
75-
# "imas-python[test,docs,imas-core,netcdf,h5py]" TODO enable when imas-core is available on pypi
74+
"imas-python[test,docs]"
7675
]
7776
docs = [
7877
"sphinx>=6.0.0,<7.0.0",
@@ -90,6 +89,9 @@ netcdf = [
9089
h5py = [
9190
"h5py",
9291
]
92+
xarray = [
93+
"xarray",
94+
]
9395
test = [
9496
"pytest>=5.4.1",
9597
"pytest-cov>=0.6",
@@ -101,7 +103,9 @@ test = [
101103
"virtualenv",
102104
# Pint and xarray are used in training snippets
103105
"pint",
104-
"xarray",
106+
# Optional dependencies
107+
# TODO add imas-core when it is available on pypi
108+
"imas-python[netcdf,h5py,xarray]",
105109
]
106110

107111
[project.scripts]

0 commit comments

Comments
 (0)