Skip to content

Conversation

@yangw-dev
Copy link
Collaborator

No description provided.

Signed-off-by: Yang Wang <elainewy@meta.com>
Signed-off-by: Yang Wang <elainewy@meta.com>
Signed-off-by: Yang Wang <elainewy@meta.com>
Signed-off-by: Yang Wang <elainewy@meta.com>
Signed-off-by: Yang Wang <elainewy@meta.com>
Signed-off-by: Yang Wang <elainewy@meta.com>
Signed-off-by: Yang Wang <elainewy@meta.com>
{% endif %}
{% endfor %}

- group: "vllm against torch nightly"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just curious if we can use the matrix parameter here to run stable and nightly side by side instead of copy/paste the CI steps. It seems doable if we could set soft_fail dynamically. What do you think?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmm, I wonder if it's possible,so we only want the nightlyto be triggered manually only, let me try it to see if it's possible

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add macro functions to wrap configs that can be used for both vllm against torch nightly and normal vllm.

Notice, kevin is working on use python script to generate the yaml file for test, so this jinja template thing will be deprecated in h2

Signed-off-by: Yang Wang <elainewy@meta.com>
Signed-off-by: Yang Wang <elainewy@meta.com>
Signed-off-by: Yang Wang <elainewy@meta.com>
Signed-off-by: Yang Wang <elainewy@meta.com>
Signed-off-by: Yang Wang <elainewy@meta.com>
Signed-off-by: Yang Wang <elainewy@meta.com>
@yangw-dev yangw-dev requested a review from khluu April 22, 2025 05:47
yangw-dev and others added 4 commits April 21, 2025 23:04
Signed-off-by: Yang Wang <elainewy@meta.com>
Signed-off-by: Yang Wang <elainewy@meta.com>
Signed-off-by: Yang Wang <elainewy@meta.com>
@khluu khluu marked this pull request as ready for review April 23, 2025 03:59
@khluu khluu merged commit 00b30d9 into main Apr 23, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants