Skip to content

Commit

Permalink
Add comment on why we duplicate test_basics_np_required for float16
Browse files Browse the repository at this point in the history
  • Loading branch information
raulcd committed Aug 30, 2024
1 parent a10801b commit 937d0aa
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions python/pyarrow/tests/test_scalars.py
Original file line number Diff line number Diff line change
Expand Up @@ -100,6 +100,9 @@ def test_basics(value, ty, klass, pickle_module):
assert wr() is None


# This test is a copy of test_basics but only for float16 (HalfFloatScalar)
# which currently requires a numpy scalar to create it. The test collection
# fails if numpy is used on the parametrization when not present.
@pytest.mark.numpy
def test_basics_np_required(pickle_module):
value, ty, klass = np.float16(1.0), pa.float16(), pa.HalfFloatScalar
Expand Down

0 comments on commit 937d0aa

Please sign in to comment.