You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
FAILED tests/test_external_packages.py::test_xugrid_opendataset_ugridplot_contourf - ValueError: triangles are indices into the points and must be in the range 0 <= i < 680 but found value -1
FAILED tests/test_external_packages.py::test_xugrid_opendataset_ugridplot_contour_with_colorbar - ValueError: triangles are indices into the points and must be in the range 0 <= i < 680 but found value -1
update dfm_tools to accomodate for this new version (see below)
run testbank: pytest tests/test_dfm_tools.py tests/test_get_nc.py tests/test_xugrid_helpers.py tests/test_external_packages.py
When running the tests with the branch in Deltares/xugrid#299, there are two other testcases that fail:
test_uda_edges_to_faces_interfaces_to_centers: there seems to be something wrong here for “mesh2d_edge_type”, not for the first var >> postprocess_interpolate_edgevar_tofaces.py maybe dcsm and mwra case are useful for testing >> We need to set fill_value=-1 in dfmt.uda_to_faces()
test_xarray_decode_default_fillvals: updating the expected value to -1 should do the trick
The text was updated successfully, but these errors were encountered:
veenstrajelmer
changed the title
Fix xugrid tests in testbank
Fix xugrid tests in testbank (contourf)
Sep 5, 2024
Failing testcases:
Todo:
pytest tests/test_dfm_tools.py tests/test_get_nc.py tests/test_xugrid_helpers.py tests/test_external_packages.py
When running the tests with the branch in Deltares/xugrid#299, there are two other testcases that fail:
test_uda_edges_to_faces_interfaces_to_centers
: there seems to be something wrong here for “mesh2d_edge_type”, not for the first var >> postprocess_interpolate_edgevar_tofaces.py maybe dcsm and mwra case are useful for testing >> We need to set fill_value=-1 in dfmt.uda_to_faces()test_xarray_decode_default_fillvals
: updating the expected value to -1 should do the trickThe text was updated successfully, but these errors were encountered: