[Perf] Add Performance Test for Qwen-Image Step-Level Execution#2707
Conversation
Signed-off-by: Didan Deng <33117903+wtomin@users.noreply.github.com>
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
| "name": "1536x1536_steps35", | ||
| "dataset": "random", | ||
| "task": "t2i", | ||
| "width": 1536, |
There was a problem hiding this comment.
Could we cover this issue #845 in the tests? Although it's a functional issue actually.
There was a problem hiding this comment.
It depends on whether we want to monitor its performance in the long run. I think currently, 1328x1328 is not our priority.
|
PR description incomplete. Test Plan and Test Result sections are empty. Please update with:
|
It has been updated. |
|
@asukaqaq-s Do this performance test and results look good to you? The results shown in this PR may not be compared directly to the results in #1769, because they might be obtained on different machines. |
I think this conclusion is consistent with my tests: there is some performance improvement at low resolutions. |
|
I think |
#2644 this issue |
|
I think it's ready for merge. PTAL @hsliuustc0106 @Gaohan123 @fhfuih |
Signed-off-by: Didan Deng <33117903+wtomin@users.noreply.github.com>
…-project#2707) Signed-off-by: Didan Deng <33117903+wtomin@users.noreply.github.com>
…-project#2707) Signed-off-by: Didan Deng <33117903+wtomin@users.noreply.github.com>
…-project#2707) Signed-off-by: Didan Deng <33117903+wtomin@users.noreply.github.com>
…-project#2707) Signed-off-by: Didan Deng <33117903+wtomin@users.noreply.github.com>
…-project#2707) Signed-off-by: Didan Deng <33117903+wtomin@users.noreply.github.com>
PLEASE FILL IN THE PR DESCRIPTION HERE ENSURING ALL CHECKLIST ITEMS (AT THE BOTTOM) HAVE BEEN CONSIDERED.
Purpose
To monitor the step-level execution feature's performance on Qwen-Image, we need to include this test case in nightly perf test.
Test Plan
Running it on CI machines (H100) . Results in buildkite.
Test Result
Conclusion:
step_executiondoes not change end-to-end latency/throughput at the large resolution, and gives a small gain at the small resolution.Essential Elements of an Effective PR Description Checklist
supported_models.mdandexamplesfor a new model. Please runmkdocs serveto sync the documentation editions to./docs.BEFORE SUBMITTING, PLEASE READ https://github.com/vllm-project/vllm-omni/blob/main/CONTRIBUTING.md (anything written below this line will be removed by GitHub Actions)