Skip to content

Commit be4e6d7

Browse files
aporialiaofacebook-github-bot
authored andcommitted
Fix Unit Test SkipIf Worldsize check (#3098)
Summary: Pull Request resolved: #3098 These unit tests actually require at least 4 GPUs - due to world size requirements. Updating skipif to match Created from CodeHub with https://fburl.com/edit-in-codehub Reviewed By: aliafzal Differential Revision: D76621861 fbshipit-source-id: 09f9b04c4d3cb7b10736fbbaff3886a8534b96fa
1 parent 65b82f9 commit be4e6d7

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

torchrec/distributed/tests/test_dynamic_sharding.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -401,8 +401,8 @@ def _run_ebc_resharding_test(
401401
)
402402

403403
@unittest.skipIf(
404-
torch.cuda.device_count() <= 1,
405-
"Not enough GPUs, this test requires at least two GPUs",
404+
torch.cuda.device_count() <= 3,
405+
"Not enough GPUs, this test requires at least four GPUs",
406406
)
407407
@given( # pyre-ignore
408408
num_tables=st.sampled_from([2, 3, 4]),
@@ -445,8 +445,8 @@ def test_dynamic_sharding_ebc_tw(
445445
)
446446

447447
@unittest.skipIf(
448-
torch.cuda.device_count() <= 1,
449-
"Not enough GPUs, this test requires at least two GPUs",
448+
torch.cuda.device_count() <= 3,
449+
"Not enough GPUs, this test requires at least four GPUs",
450450
)
451451
@given( # pyre-ignore
452452
num_tables=st.sampled_from([2, 3, 4]),

0 commit comments

Comments
 (0)