-
-
Notifications
You must be signed in to change notification settings - Fork 20.5k
[MM][Perf][CG] Support ViT full cudagraphs for mllama4 #40660
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 7 commits
06e53c0
a8b3228
9445614
dd51d8d
6da9c10
347e2bb
ecbccd9
77f770e
0f856b8
c4bcfe9
a6271c7
147048e
ee8ad7e
c1633fc
44eec75
ef7dcc7
9ee6e22
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -10,7 +10,8 @@ | |
| from vllm.platforms import current_platform | ||
|
|
||
| from ....conftest import IMAGE_ASSETS, VIDEO_ASSETS | ||
| from ...utils import dummy_hf_overrides | ||
|
|
||
| from ....utils import create_new_process_for_each_test, multi_gpu_marks, dummy_hf_overrides | ||
| from .vlm_utils.builders import sample_frames_with_video_metadata | ||
|
|
||
|
|
||
|
|
@@ -51,6 +52,23 @@ def step3_vl_chat_template(content: str) -> str: | |
|
|
||
|
|
||
| MODEL_CONFIGS: dict[str, VitCudagraphTestConfig] = { | ||
| "llama4": VitCudagraphTestConfig( | ||
| model="meta-llama/Llama-4-Scout-17B-16E-Instruct", | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm not sure if this 17B model would cause OOM issues in the CI. FYI, #43082.
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. yea... this is tricky. llama4 cg test is probably too heavy. I can shrink the config more but it looks like the only options are moving the llama4 cg testing to a different hardware config or not testing llama4. what r ur thoughts?
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You can use dummy weights to test this model, because we only test basic functionality here. You can refer to Step3-VL below to reduce num layers in config. |
||
| modalities=["image"], | ||
| image_prompt=( | ||
| "<|begin_of_text|><|header_start|>user<|header_end|>\n\n" | ||
| "<|image|>What is in this image?<|eot|>" | ||
| "<|header_start|>assistant<|header_end|>\n\n" | ||
| ), | ||
|
This conversation was marked as resolved.
|
||
| max_model_len=4096, | ||
| max_tokens=32, | ||
| max_num_seqs=2, | ||
| vllm_runner_kwargs={ | ||
| "distributed_executor_backend": "mp", | ||
| "tensor_parallel_size": 4, | ||
| }, | ||
| marks=[pytest.mark.core_model, *multi_gpu_marks(num_gpus=4)], | ||
| ), | ||
| "qwen2_5_vl": VitCudagraphTestConfig( | ||
| model="Qwen/Qwen2.5-VL-3B-Instruct", | ||
| image_prompt=qwen_vl_chat_template( | ||
|
|
||
|
This conversation was marked as resolved.
|
Uh oh!
There was an error while loading. Please reload this page.