Skip to content

Conversation

@jthomson04
Copy link
Contributor

Similar to MR from @ishandhanani, but TRTLLM: #3134

@ishandhanani
Copy link
Contributor

I'm curious on if we can consolidate any logic here? Throw something into dynamo.common?

@jthomson04
Copy link
Contributor Author

I'm curious on if we can consolidate any logic here? Throw something into dynamo.common?

Hmm. Level of reuse here would probably be pretty small. The only ones that come to mind are:

  1. `_build_sampling_params
  2. _process_text_stream

Signed-off-by: jthomson04 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants