Skip to content

Commit

Permalink
Fix slicing in tests, although not sure it ever has enough workers
Browse files Browse the repository at this point in the history
  • Loading branch information
seberg committed Nov 14, 2023
1 parent ad0c0b6 commit fca015e
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion python/raft-dask/raft_dask/test/test_comms.py
Original file line number Diff line number Diff line change
Expand Up @@ -247,7 +247,9 @@ def test_collectives(client, func, root_location):


@pytest.mark.nccl
@pytest.mark.parametrize("subset", [-1, 1, slice(None, None, -2)])
@pytest.mark.parametrize(
"subset", [slice(-1, None), slice(1), slice(None, None, -2)]
)
def test_comm_init_worker_subset(client, subset):
# Basic test that initializing a subset of workers is fine
cb = Comms(comms_p2p=True, verbose=True)
Expand Down

0 comments on commit fca015e

Please sign in to comment.