vLLM support for DocSum #885
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: sgurunat <[email protected]>
…rt for vllm endpoints
for more information, see https://pre-commit.ci
Codecov ReportAll modified and coverable lines are covered by tests ✅
|
Signed-off-by: sgurunat <[email protected]>
…into vllm-docsum
for more information, see https://pre-commit.ci
Collaborator
|
Please update comps/llms/summarization/vllm/langchain/Dockerfile to .github/workflows/docker/compose/llms-compose-cd.yaml. The yaml is used for release images build. |
…workflows Signed-off-by: sgurunat <[email protected]>
Contributor
Author
|
@lvliang-intel - Updated the llm-compose-cd.yaml file. Its showing conflict |
Collaborator
Please resolve the conflict. I can't help to resolve it. |
Signed-off-by: sgurunat <[email protected]>
Signed-off-by: sgurunat <[email protected]>
Contributor
Author
|
@chensuyue @lvliang-intel Updated the git workflows to include vllm docsum build in llm-compose.yaml file |
Signed-off-by: sgurunat <[email protected]>
lvliang-intel
approved these changes
Nov 13, 2024
Spycsh
approved these changes
Nov 13, 2024
madison-evans
pushed a commit
to SAPD-Intel/GenAIComps
that referenced
this pull request
May 12, 2025
* Add model parameter for DocSumGateway in gateway.py file Signed-off-by: sgurunat <[email protected]> * Add langchain vllm support for DocSum along with authentication support for vllm endpoints * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Updated docker_compose_llm.yaml and README file with vLLM information Signed-off-by: sgurunat <[email protected]> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Updated docsum-vllm Dockerfile into llm-compose-cd.yaml under github workflows Signed-off-by: sgurunat <[email protected]> * Updated llm-compose.yaml file to include vllm sumarization docker build Signed-off-by: sgurunat <[email protected]> --------- Signed-off-by: sgurunat <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: lvliang-intel <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Added langchain vLLM support for Summarization. Added new folder called vllm under summarization and have maintained the same file structure similar to tgi. Dockerfile, compose yaml and README contains details about how to build and use summarization with vLLM.
This also contains changes related to authentication support for remote vLLM endpoints in llm.py file.
Since model is mandatory parameter for vLLM updated the DocsumGateway to include the new parameter called "model" in Mega service gateway.py file
Issues
N/A
Type of change
Dependencies
N/A