[GH-ISSUE #327] low latency test error with Separate profiling #229

Closed
opened 2026-10-26 09:57:03 +00:00 by bsakaguchi · 5 comments
Owner

Originally created by @shifangx on GitHub (Jul 25, 2025).
Original GitHub issue: https://github.com/deepseek-ai/DeepEP/issues/327

Assert len(durations) % num_kernels_per_period == 0 in bench_kineto() failed for GB200, even for 1 node with 2 processes.

File "/DeepEP/tests/utils.py", line 212, in bench_kineto
assert len(durations) % num_kernels_per_period == 0

Originally created by @shifangx on GitHub (Jul 25, 2025). Original GitHub issue: https://github.com/deepseek-ai/DeepEP/issues/327 Assert len(durations) % num_kernels_per_period == 0 in bench_kineto() failed for GB200, even for 1 node with 2 processes. File "/DeepEP/tests/utils.py", line 212, in bench_kineto assert len(durations) % num_kernels_per_period == 0
Author
Owner

@sphish commented on GitHub (Jul 29, 2025):

We haven't encountered this issue. This assertion indicates that the number of events collected by torch profiler doesn't match the number of test iterations. Perhaps you can print all the profiled events to see where the problem occurs.

<!-- gh-comment-id:3131309129 --> @sphish commented on GitHub (Jul 29, 2025): We haven't encountered this issue. This assertion indicates that the number of events collected by torch profiler doesn't match the number of test iterations. Perhaps you can print all the profiled events to see where the problem occurs.
Author
Owner

@MaoZiming commented on GitHub (Sep 15, 2025):

@shifangx I have seen similar issues. Did you manage to resolve it?

<!-- gh-comment-id:3290122369 --> @MaoZiming commented on GitHub (Sep 15, 2025): @shifangx I have seen similar issues. Did you manage to resolve it?
Author
Owner

@shifangx commented on GitHub (Sep 15, 2025):

This issue has been solved by update to the newest commit of DeepEP.

<!-- gh-comment-id:3290137933 --> @shifangx commented on GitHub (Sep 15, 2025): This issue has been solved by update to the newest commit of DeepEP.
Author
Owner

@MaoZiming commented on GitHub (Sep 15, 2025):

Thank you. @shifangx Do you know which commit fixes this (or do you mean the latest commit)?

<!-- gh-comment-id:3292744037 --> @MaoZiming commented on GitHub (Sep 15, 2025): Thank you. @shifangx Do you know which commit fixes this (or do you mean the latest commit)?
Author
Owner

@MaoZiming commented on GitHub (Sep 15, 2025):

github.com/deepseek-ai/DeepEP@c5facf5c6f (diff-9e08e8c776)
The fix seems to be the torch.cuda.synchronize() in bench_kineto.
Thank you

<!-- gh-comment-id:3293305465 --> @MaoZiming commented on GitHub (Sep 15, 2025): https://github.com/deepseek-ai/DeepEP/commit/c5facf5c6fef042e56627c33817feebaf97aee26#diff-9e08e8c7769db7b58089fd7659d75ca96df40bb7b5d50299d3a30abd54da9ad6R174 The fix seems to be the `torch.cuda.synchronize()` in `bench_kineto`. Thank you
Sign in to join this conversation.
No labels
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
deepseek-ai/DeepEP#229
No description provided.