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

feat: add support for llama 3.3 via together AI #9222

Merged
merged 3 commits into from
Dec 11, 2024

Conversation

fontanierh
Copy link
Contributor

@fontanierh fontanierh commented Dec 9, 2024

Description

Adds support for TogetherAI as an LLM provider. Currently only in dust apps, and only for llama 3.3 (the new 70b model).

Will follow-up with managed key setup if we decide to do it.

Risk

There's some code changes that could affect Open AI use.

Deploy Plan

Deploy core and front

@fontanierh fontanierh force-pushed the feat/support-together-ai branch from 55c1ed3 to d73a178 Compare December 10, 2024 15:40
@fontanierh fontanierh changed the title WIP together ai feat: add support for llama 3.3 via together AI Dec 10, 2024
@fontanierh fontanierh requested a review from spolu December 10, 2024 15:43
Copy link
Contributor

@spolu spolu left a comment

Choose a reason for hiding this comment

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

LGTM \o/

@@ -287,12 +287,19 @@ pub enum OpenAIContentBlock {
}

#[derive(Debug, Serialize, Deserialize, PartialEq, Clone)]
pub struct OpenAIContentBlockVec(Vec<OpenAIContentBlock>);
pub struct OpenAIContentBlockVec(pub Vec<OpenAIContentBlock>);
Copy link
Contributor

Choose a reason for hiding this comment

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

OpenAIContentBlocks maybe?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah kept as is but your suggested name is indeed better 👍

Copy link
Contributor Author

Choose a reason for hiding this comment

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

tbf i'm not sure we need this intermediary struct to begin with

@fontanierh fontanierh force-pushed the feat/support-together-ai branch from 9103659 to 4c99b02 Compare December 11, 2024 09:38
@fontanierh fontanierh force-pushed the feat/support-together-ai branch from 4c99b02 to 993faa6 Compare December 11, 2024 09:51
@fontanierh fontanierh merged commit c678d5f into main Dec 11, 2024
3 checks passed
@fontanierh fontanierh deleted the feat/support-together-ai branch December 11, 2024 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants