[V1] Remove additional_config check#16710
Conversation
Signed-off-by: wangxiyuan <wangxiyuan1007@gmail.com>
|
👋 Hi! Thank you for contributing to the vLLM project. 💬 Join our developer Slack at https://slack.vllm.ai to discuss your PR in #pr-reviews, coordinate on features in #feat- channels, or join special interest groups in #sig- channels. Just a reminder: PRs would not trigger full CI run by default. Instead, it would only run Once the PR is approved and ready to go, your PR reviewer(s) can run CI to test the changes comprehensively before merging. To run CI, PR reviewers can either: Add 🚀 |
|
Hey @robertgshaw2-redhat, I assume this check was intentional. Can you take a look? |
|
@robertgshaw2-redhat hi, may I ask for your review? we test V1 engine with additional_config. It works well. Anything I missed? |
|
Good by me. I was not sure what this was for, so I disabled out of an abundance of caution. |
|
Thanks for the PR! |
|
Test failure with lora error. Looks doesn't relate to this PR |
Signed-off-by: wangxiyuan <wangxiyuan1007@gmail.com> Signed-off-by: Frieda (Jingying) Huang <jingyingfhuang@gmail.com>
Signed-off-by: wangxiyuan <wangxiyuan1007@gmail.com>
Signed-off-by: wangxiyuan <wangxiyuan1007@gmail.com>
Signed-off-by: wangxiyuan <wangxiyuan1007@gmail.com> Signed-off-by: Agata Dobrzyniewicz <adobrzyniewicz@habana.ai>
Signed-off-by: wangxiyuan <wangxiyuan1007@gmail.com> Signed-off-by: Mu Huai <tianbowen.tbw@antgroup.com>
additional_config is contained in vllm_config and can be used by custom platform. It doesn't affect any V1 feature or function and can be used by V1 correctly. This PR drop the related check for V1 engine.