Skip to content

Commit

Permalink
Simplify the list of support numpy dtypes with DTYPES_NUMERIC
Browse files Browse the repository at this point in the history
  • Loading branch information
seisman committed Oct 31, 2024
1 parent e5ecee9 commit d31c2ff
Show file tree
Hide file tree
Showing 6 changed files with 13 additions and 6 deletions.
3 changes: 2 additions & 1 deletion pygmt/tests/test_clib_put_matrix.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
import pytest
import xarray as xr
from pygmt import clib
from pygmt.clib.sessions import DTYPES_NUMERIC
from pygmt.exceptions import GMTCLibError
from pygmt.helpers import GMTTempFile
from pygmt.tests.test_clib import mock
Expand All @@ -17,7 +18,7 @@ def fixture_dtypes():
"""
List of supported numpy dtypes.
"""
return "int8 int16 int32 int64 uint8 uint16 uint32 uint64 float32 float64".split()
return [dtype for dtype in DTYPES_NUMERIC if dtype != np.timedelta64]


@pytest.mark.benchmark
Expand Down
3 changes: 2 additions & 1 deletion pygmt/tests/test_clib_put_vector.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@
import numpy.testing as npt
import pytest
from pygmt import clib
from pygmt.clib.sessions import DTYPES_NUMERIC
from pygmt.exceptions import GMTCLibError, GMTInvalidInput
from pygmt.helpers import GMTTempFile

Expand All @@ -18,7 +19,7 @@ def fixture_dtypes():
"""
List of supported numpy dtypes.
"""
return "int8 int16 int32 int64 uint8 uint16 uint32 uint64 float32 float64".split()
return [dtype for dtype in DTYPES_NUMERIC if dtype != np.timedelta64]


@pytest.mark.benchmark
Expand Down
3 changes: 2 additions & 1 deletion pygmt/tests/test_clib_virtualfile_from_matrix.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
import numpy as np
import pytest
from pygmt import clib
from pygmt.clib.sessions import DTYPES_NUMERIC
from pygmt.helpers import GMTTempFile


Expand All @@ -13,7 +14,7 @@ def fixture_dtypes():
"""
List of supported numpy dtypes.
"""
return "int8 int16 int32 int64 uint8 uint16 uint32 uint64 float32 float64".split()
return [dtype for dtype in DTYPES_NUMERIC if dtype != np.timedelta64]


@pytest.mark.benchmark
Expand Down
3 changes: 2 additions & 1 deletion pygmt/tests/test_clib_virtualfile_from_vectors.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
import pandas as pd
import pytest
from pygmt import clib
from pygmt.clib.sessions import DTYPES_NUMERIC
from pygmt.exceptions import GMTInvalidInput
from pygmt.helpers import GMTTempFile

Expand All @@ -17,7 +18,7 @@ def fixture_dtypes():
"""
List of supported numpy dtypes.
"""
return "int8 int16 int32 int64 uint8 uint16 uint32 uint64 float32 float64".split()
return [dtype for dtype in DTYPES_NUMERIC if dtype != np.timedelta64]


@pytest.fixture(scope="module", name="dtypes_pandas")
Expand Down
3 changes: 2 additions & 1 deletion pygmt/tests/test_clib_virtualfiles.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
import numpy as np
import pytest
from pygmt import clib
from pygmt.clib.sessions import DTYPES_NUMERIC
from pygmt.exceptions import GMTCLibError, GMTInvalidInput
from pygmt.helpers import GMTTempFile
from pygmt.tests.test_clib import mock
Expand All @@ -28,7 +29,7 @@ def fixture_dtypes():
"""
List of supported numpy dtypes.
"""
return "int8 int16 int32 int64 uint8 uint16 uint32 uint64 float32 float64".split()
return [dtype for dtype in DTYPES_NUMERIC if dtype != np.timedelta64]


@pytest.fixture(scope="module", name="dtypes_pandas")
Expand Down
4 changes: 3 additions & 1 deletion pygmt/tests/test_grdimage_image.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,10 @@
Test Figure.grdimage on 3-band RGB images.
"""

import numpy as np
import pytest
from pygmt import Figure
from pygmt.clib.session import DTYPES_NUMERIC
from pygmt.datasets import load_blue_marble

rioxarray = pytest.importorskip("rioxarray")
Expand Down Expand Up @@ -43,7 +45,7 @@ def test_grdimage_image_dataarray(xr_image):

@pytest.mark.parametrize(
"dtype",
["int8", "uint16", "int16", "uint32", "int32", "float32", "float64"],
[dtype for dtype in DTYPES_NUMERIC if dtype != np.uint8],
)
def test_grdimage_image_dataarray_unsupported_dtype(dtype, xr_image):
"""
Expand Down

0 comments on commit d31c2ff

Please sign in to comment.