mirror of
https://github.com/deepseek-ai/DeepEP.git
synced 2026-11-05 00:47:32 +00:00
[GH-ISSUE #226] By default, in megatron, condition fais on ibgda_get_state()->num_rc_per_pe == num_channels || ibgda_get_state()->num_rc_per_pe >= num_sms #177
Labels
No labels
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
deepseek-ai/DeepEP#177
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @kuozhang on GitHub (Jun 19, 2025).
Original GitHub issue: https://github.com/deepseek-ai/DeepEP/issues/226
After updated deepep to 7f97f79bda051d991aa9681c04138007faa0366, it fails:
@kuozhang commented on GitHub (Jun 19, 2025):
the root cause is:
@kuozhang commented on GitHub (Jun 20, 2025):
I printed the values, by default , num_rc_per_pe = 12, num_channels = 10, num_sms = 20, Is this by design ?
In trianing, what the value shoud i set, num_sms = 24 or ? num_rc_per_pe = 10?
@sphish commented on GitHub (Jun 20, 2025):
It is recommended to set this value equal to
num_sms == 20. However, if you are also using low-latency kernels, you need to take into account the assertion associated with low-latency mode.@alpha-baby commented on GitHub (Jun 20, 2025):
github.com/deepseek-ai/DeepEP@9d4f7ef8ee/tests/test_internode.py (L228)you should set config:
num_qps_per_rankcorrectly. you could refer here.@kuozhang