Skip to content

Conversation

shanjiaz
Copy link
Collaborator

SUMMARY:
"please provide a brief summary"

TEST PLAN:
"please outline how the changes were tested"

Copy link

👋 Hi! Thank you for contributing to llm-compressor. Please add the ready label when the PR is ready for review.

Note: This is required to complete the testing suite, please only add the label once the PR is code complete and local testing has been performed.

Copy link
Collaborator

@dsikka dsikka left a comment

Choose a reason for hiding this comment

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

FYI - this may fail in vllm as I think tyler reverted the PR to support
vllm-project/vllm#25607

@shanjiaz
Copy link
Collaborator Author

FYI - this may fail in vllm as I think tyler reverted the PR to support vllm-project/vllm#25607

Ah! That makes sense it was failing for me locally. I was planning on trying to serve the model in vllm directly.

Copy link
Collaborator

@dsikka dsikka left a comment

Choose a reason for hiding this comment

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

I was wrong. Should now work with this PR: vllm-project/vllm#25219

@dsikka
Copy link
Collaborator

dsikka commented Oct 1, 2025

@shanjiaz can we get this in soon

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.

2 participants