Skip to content
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

Consider moving to chat/completions API #87

Open
markmc opened this issue Jul 5, 2024 · 3 comments
Open

Consider moving to chat/completions API #87

markmc opened this issue Jul 5, 2024 · 3 comments
Labels
enhancement New feature or request

Comments

@markmc
Copy link
Contributor

markmc commented Jul 5, 2024

From @gabe-l-hart here

it looks like these prompt templates are largely encoding the chat template for the model. For most models that I've seen, the chat template is actually encoded in the model metadata and applied server-side when using the chat/completions API. Has there been thought around using the chat API rather than applying the template client-side to the lower-level completions API?

@nathan-weinberg nathan-weinberg added the enhancement New feature or request label Aug 20, 2024
Copy link

This issue has been automatically marked as stale because it has not had activity within 90 days. It will be automatically closed if no further activity occurs within 30 days.

@github-actions github-actions bot added the stale label Nov 23, 2024
@bbrowning
Copy link
Contributor

Unstale - some of the research work we're merging in starts moving us over to the chat completions API instead of the generic completions API. So, we will be addressing at least the spirit of this issue.

@bbrowning bbrowning removed the stale label Nov 26, 2024
@bbrowning
Copy link
Contributor

#461 adds an LLMMessagesBlock that uses the chat completions API, as opposed to the LLMBlock in Pipelines that uses the older completions API. However, adding that block is just the first step towards moving us over - we still need to evaluate how/if we'd move our default pipelines over to the chat completions API instead of the completions API.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants