8000 cleanup: upgrade vllm version to v0.7.3 by nayihz · Pull Request #289 · InftyAI/llmaz · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

cleanup: upgrade vllm version to v0.7.3 #289

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Mar 6, 2025
Merged

cleanup: upgrade vllm version to v0.7.3 #289

merged 1 commit into from
Mar 6, 2025

Conversation

nayihz
Copy link
Contributor
@nayihz nayihz commented Mar 6, 2025

What this PR does / why we need it

Which issue(s) this PR fixes

Fixes #261

Special notes for your reviewer

Does this PR introduce a user-facing change?

@nayihz
Copy link
Contributor Author
nayihz commented Mar 6, 2025

/assign @kerthcet

kerthcet
kerthcet reviewed Mar 6, 2025
View reviewed changes
Copy link
Member
@kerthcet kerthcet left a comment

Choose a reason for hiding this comment

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

just one nit.
/approve

@InftyAI-Agent InftyAI-Agent added needs-triage Indicates an issue or PR lacks a label and requires one. needs-priority Indicates a PR lacks a label and requires one. do-not-merge/needs-kind Indicates a PR lacks a label and requires one. labels Mar 6, 2025
@kerthcet
Copy link
Member
kerthcet commented Mar 6, 2025

/kind cleanup

@InftyAI-Agent InftyAI-Agent added cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. and removed do-not-merge/needs-kind Indicates a PR lacks a label and requires one. labels Mar 6, 2025
@nayihz nayihz force-pushed the update_vllm_version branch from 8aaf43d to cb8c2ba Compare March 6, 2025 06:47
kerthcet
kerthcet reviewed Mar 6, 2025
View reviewed changes
Copy link
Member
@kerthcet kerthcet left a comment

Choose a reason for hiding this comment

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

/lgtm

Thanks @nayihz Welcome on board!

@kerthcet
Copy link
Member< 8000 /span>
kerthcet commented Mar 6, 2025 •
edited
Loading

/lgtm

@InftyAI-Agent InftyAI-Agent added the lgtm Looks good to me, indicates that a PR is ready to be merged. label Mar 6, 2025
@kerthcet
Copy link
Member
kerthcet commented Mar 6, 2025

/approve

@InftyAI-Agent InftyAI-Agent added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 6, 2025
@InftyAI-Agent InftyAI-Agent merged commit 917ba59 into InftyAI:main Mar 6, 2025
19 checks passed
@nayihz nayihz deleted the update_vllm_version branch March 6, 2025 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kerthcet kerthcet kerthcet left review comments

Assignees

@kerthcet kerthcet

Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. lgtm Looks good to me, indicates that a PR is ready to be merged. needs-priority Indicates a PR lacks a label and requires one. needs-triage Indicates an issue or PR lacks a label and requires one.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

upgrade vllm to v0.7.1 as default version
3 participants
@nayihz @kerthcet @InftyAI-Agent
0