-
Notifications
You must be signed in to change notification settings - Fork 41
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.
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
Batch support with vllm #105
Batch support with vllm #105
Conversation
Can you rebase this PR on |
Thanks @russellb, let's hold on to the reviews, we are still testing the changes. Will request the review once it is ready. |
6685898
to
5492d58
Compare
59d4276
to
c973f95
Compare
Co-authored-by: Nikhil Palaskar <[email protected]> Co-authored-by: Nick Hill <[email protected]> Signed-off-by: Aakanksha Duggal <[email protected]>
c973f95
to
1182c80
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tested offline - lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @njhill @npalaska for adding batched VLLM support and @aakankshaduggal for testing this out
PR instructlab#105 needs some new words added to the spelling dictionary. Signed-off-by: JJ Asghar <[email protected]>
…ling Update spelling per instructlab#105
Update knowledge-submissions-past-wikipedia.md
This PR adds batch support with vllm