|
24 | 24 | "soc_version, enable_expert_parallel, world_size, num_tokens, mc2_tokens_capacity, quant_type, expected_method",
|
25 | 25 | [
|
26 | 26 | # Case 1: Expert parallel is disabled, should always be 'allgather'
|
27 |
| - ("A2", False, 8, 100, 256, None, "allgather"), |
28 |
| - ("A3", False, 16, 500, 256, None, "allgather"), |
| 27 | + ("A2", False, 8, 100, 256, None, MoECommType.ALLGATHER), |
| 28 | + ("A3", False, 16, 500, 256, None, MoECommType.ALLGATHER), |
29 | 29 |
|
30 | 30 | # Case 2: A2 SOC with w4a8_dynamic -> use alltoall when not mc2
|
31 |
| - ("A2", True, 8, 100, 256, "w4a8_dynamic", "alltoall"), |
32 |
| - ("A2", True, 16, 257, 256, "w4a8_dynamic", "alltoall"), |
33 |
| - ("A2", True, 16, 100, 256, "w4a8_dynamic", "mc2"), # meets mc2 condition |
| 31 | + ("A2", True, 8, 100, 256, "w4a8_dynamic", MoECommType.ALLTOALL), |
| 32 | + ("A2", True, 16, 257, 256, "w4a8_dynamic", MoECommType.ALLTOALL), |
| 33 | + ("A2", True, 16, 100, 256, "w4a8_dynamic", MoECommType.MC2), # meets mc2 condition |
34 | 34 |
|
35 | 35 | # Case 3: A2 SOC without w4a8_dynamic -> fallback to allgather
|
36 |
| - ("A2", True, 8, 100, 256, None, "allgather"), |
37 |
| - ("A2", True, 16, 257, 256, None, "allgather"), |
| 36 | + ("A2", True, 8, 100, 256, None, MoECommType.ALLGATHER), |
| 37 | + ("A2", True, 16, 257, 256, None, MoECommType.ALLGATHER), |
38 | 38 |
|
39 | 39 | # Case 4: A3 SOC
|
40 |
| - ("A3", True, 8, 100, 256, None, "mc2"), |
41 |
| - ("A3", True, 8, 257, 256, None, "alltoall"), |
| 40 | + ("A3", True, 8, 100, 256, None, MoECommType.MC2), |
| 41 | + ("A3", True, 8, 257, 256, None, MoECommType.ALLTOALL), |
42 | 42 |
|
43 | 43 | # Case 5: P3 SOC
|
44 |
| - ("310P", True, 8, 100, 256, None, "allgather"), |
45 |
| - ("310P", True, 8, 257, 256, None, "allgather"), |
| 44 | + ("310P", True, 8, 100, 256, None, MoECommType.ALLGATHER), |
| 45 | + ("310P", True, 8, 257, 256, None, MoECommType.ALLGATHER), |
46 | 46 | ])
|
47 | 47 | # yapf: enable
|
48 | 48 | def test_select_moe_comm_method(soc_version, enable_expert_parallel,
|
|
0 commit comments