Skip to content

Commit e2e5401

Browse files
Fixing ctd unit test
Requires Parcels-code/Parcels#2348 to work
1 parent 6f3307d commit e2e5401

File tree

2 files changed

+6
-13
lines changed

2 files changed

+6
-13
lines changed

tests/instruments/test_ctd.py

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -91,9 +91,9 @@ def test_simulate_ctds(tmpdir) -> None:
9191
s[:, 0, 1, 0] = ctd_exp[1]["maxdepth"]["salinity"]
9292

9393
lons, lats = (
94-
np.linspace(-1, 2, dims[2]),
95-
np.linspace(-1, 2, dims[3]),
96-
) # TODO set to (0, 1) once Parcels can interpolate on domain boundaries
94+
np.linspace(0, 1, dims[2]),
95+
np.linspace(0, 1, dims[3]),
96+
)
9797
ds = xr.Dataset(
9898
{
9999
"U": (["time", "depth", "YG", "XG"], u),
@@ -151,9 +151,6 @@ def test_simulate_ctds(tmpdir) -> None:
151151
assert len(results.trajectory) == len(ctds)
152152
assert np.min(results.z) == -1000.0
153153

154-
pytest.skip(
155-
reason="Parcels v4 can't interpolate on grid boundaries, leading to NaN values in output."
156-
)
157154
for ctd_i, (traj, exp_bothloc) in enumerate(
158155
zip(results.trajectory, ctd_exp, strict=True)
159156
):

tests/instruments/test_ctd_bgc.py

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -153,9 +153,9 @@ def test_simulate_ctd_bgcs(tmpdir) -> None:
153153
nppv[:, 0, 1, 0] = ctd_bgc_exp[1]["maxdepth"]["nppv"]
154154

155155
lons, lats = (
156-
np.linspace(-1, 2, dims[2]),
157-
np.linspace(-1, 2, dims[3]),
158-
) # TODO set to (0, 1) once Parcels can interpolate on domain boundaries
156+
np.linspace(0, 1, dims[2]),
157+
np.linspace(0, 1, dims[3]),
158+
)
159159
ds = xr.Dataset(
160160
{
161161
"U": (["time", "depth", "YG", "XG"], u),
@@ -225,10 +225,6 @@ def test_simulate_ctd_bgcs(tmpdir) -> None:
225225
assert len(results.trajectory) == len(ctd_bgcs)
226226
assert np.min(results.z) == -1000.0
227227

228-
pytest.skip(
229-
reason="Parcels v4 can't interpolate on grid boundaries, leading to NaN values in output."
230-
)
231-
232228
for ctd_i, (traj, exp_bothloc) in enumerate(
233229
zip(results.trajectory, ctd_bgc_exp, strict=True)
234230
):

0 commit comments

Comments
 (0)