-
Notifications
You must be signed in to change notification settings - Fork 75
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
Release 0.9.56 #1276
Release 0.9.56 #1276
Conversation
* spec dec config * add optional dict of trt llm configs * fix bad merge * add extensions support * fix fixture * cli push fixes * constants * fix ordering * fix merge * refactor interface * add tp validation error * self review * use constant * fix tests * fix tests * add request_default_max_tokens * fix default on trtllm runtime * update copy * bump to 54rc0 * add total token limit to toplevel config * bump briton to 0.3.10 * fix import * 54rc2 * fix rc3 * rc4 * bump briton server image * bump rc6 for briton 0.3.12.dev3 * bump rc7 * revert trtllm serialization changes * bump briton * interface refactor * add validation + tests * 56rc0 * reduce property
* bump briton to briton==0.3.12.dev8 * bump truss to 0.9.56rc1
@@ -49,6 +49,8 @@ async def run_remote(self) -> AsyncIterator[bytes]: | |||
) | |||
print("Yield") | |||
yield streamer.yield_item(data) | |||
if cause_error and i > 2: |
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.
@joostinyi @tianshuc0731 This is not written by me, but seems an odd style to have a variable that is not default that causes an error?
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.
Good point, it's a bit dirty to have this here in /examples
. I'll move it into /tests
in a follow up. But not blocking the release.
Updating Truss from "0.9.55" to 0.9.56. PLEASE ENSURE YOU MERGE, NOT SQUASH