Commit 68a9391
UT reduce AllReduce duplicate test cases
Summary:
1. `commInt32` and `commFloat` has same width (for nranks, we in fact don't care
about width, just leaving int8 is sufficient to verify different width here.)
2. `4/6 rank` test cases v.s. `2` is not quite different, we have `8` vs `2` already.
3. `SufficientElements_LargeMessage` is duplicate with the base CtranAllReduce
positive test case `CtranAllReduceTest::BasicRunAbortEnabled`
4. `InsufficientElements_1Element` test case is not different from `InsufficientElements_NRanksMinus1` for the purpose of these
tests.
Before: (< 7 min)
https://www.internalfb.com/intern/testinfra/testrun/14073748959875707
After: (< 3 min)
https://www.internalfb.com/intern/testinfra/testrun/1125900364950557
Reviewed By: saifhhasan
Differential Revision: D86159506
fbshipit-source-id: 5c26ddc822f2e30a64540e5eb36efd78dc07704b1 parent 615798d commit 68a9391
1 file changed
+1
-24
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
363 | 363 | | |
364 | 364 | | |
365 | 365 | | |
366 | | - | |
367 | | - | |
368 | | - | |
369 | | - | |
370 | | - | |
371 | | - | |
372 | | - | |
373 | 366 | | |
374 | 367 | | |
375 | | - | |
| 368 | + | |
376 | 369 | | |
377 | 370 | | |
378 | 371 | | |
| |||
382 | 375 | | |
383 | 376 | | |
384 | 377 | | |
385 | | - | |
386 | | - | |
387 | 378 | | |
388 | 379 | | |
389 | 380 | | |
| |||
400 | 391 | | |
401 | 392 | | |
402 | 393 | | |
403 | | - | |
404 | | - | |
405 | | - | |
406 | | - | |
407 | | - | |
408 | | - | |
409 | 394 | | |
410 | 395 | | |
411 | 396 | | |
412 | 397 | | |
413 | 398 | | |
414 | | - | |
415 | 399 | | |
416 | | - | |
417 | | - | |
418 | | - | |
419 | | - | |
420 | | - | |
421 | | - | |
422 | 400 | | |
423 | | - | |
424 | 401 | | |
425 | 402 | | |
426 | 403 | | |
| |||
0 commit comments