Skip to content

Together AI Client#2919

Merged
sonichi merged 22 commits intomainfrom
togetherai_client
Jun 21, 2024
Merged

Together AI Client#2919
sonichi merged 22 commits intomainfrom
togetherai_client

Conversation

@marklysze
Copy link
Copy Markdown
Contributor

@marklysze marklysze commented Jun 11, 2024

This is a Together AI Client that provides a way to use the Together AI cloud inference service, which provides options to use many LLMs, such as Google's Gemma, Meta's Llama 2/3, Qwen, Mistral.AI's Mistral, and Mixtral, NousResearch's Hermes models.

This client supports text generation and function/tool calling (when using certain models as noted by Together AI).

Token costs are also implemented.

Will be used in the upcoming blog post #2965.

Related issue number

Can be used for #2916.

Checks

@marklysze marklysze self-assigned this Jun 11, 2024
@marklysze marklysze added the models Pertains to using alternate, non-GPT, models (e.g., local models, llama, etc.) label Jun 11, 2024
@marklysze
Copy link
Copy Markdown
Contributor Author

Converting from draft, utilising the updated client_utils, now. I think this is ready for review.

Copy link
Copy Markdown
Contributor

@Hk669 Hk669 left a comment

Choose a reason for hiding this comment

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

@marklysze
Copy link
Copy Markdown
Contributor Author

we can retire this notebook https://microsoft.github.io/autogen/docs/topics/non-openai-models/cloud-togetherai/

Yes, I think you're right. I'll go through and tidy up references to it.

Can someone provide guidance on what we do if we remove a document page, e.g. External links may break

@Hk669
Copy link
Copy Markdown
Contributor

Hk669 commented Jun 19, 2024

we can retire this notebook https://microsoft.github.io/autogen/docs/topics/non-openai-models/cloud-togetherai/

Yes, I think you're right. I'll go through and tidy up references to it.

Can someone provide guidance on what we do if we remove a document page, e.g. External links may break

@qingyun-wu @sonichi @yiranwu0

Comment thread autogen/oai/together.py
Comment thread autogen/oai/together.py
Comment thread autogen/oai/together.py Outdated
Comment thread autogen/oai/together.py Outdated
Comment thread autogen/oai/together.py
Comment thread autogen/oai/together.py Outdated
Comment thread test/oai/test_together.py Outdated
Comment thread test/oai/test_together.py
@marklysze
Copy link
Copy Markdown
Contributor Author

Thanks @yiranwu0 for working through the code. I've updated most items, just a couple of conversations to discuss and resolve.

@marklysze
Copy link
Copy Markdown
Contributor Author

@Hk669, I'll wait for feedback on removing the old notebook. I'm thinking perhaps we could just replace the old notebook for this and Mistral with the new ones. What do you think?

@Hk669
Copy link
Copy Markdown
Contributor

Hk669 commented Jun 19, 2024

@Hk669, I'll wait for feedback on removing the old notebook. I'm thinking perhaps we could just replace the old notebook for this and Mistral with the new ones. What do you think?

Yeah sounds good.

@marklysze
Copy link
Copy Markdown
Contributor Author

@Hk669, I'll wait for feedback on removing the old notebook. I'm thinking perhaps we could just replace the old notebook for this and Mistral with the new ones. What do you think?

Yeah sounds good.

All done :)... same in both Mistral and Together PRs

Copy link
Copy Markdown
Contributor

@qingyun-wu qingyun-wu left a comment

Choose a reason for hiding this comment

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

Looks great! Thanks @marklysze!

Comment thread autogen/oai/together.py Outdated
@Hk669
Copy link
Copy Markdown
Contributor

Hk669 commented Jun 20, 2024

I'm waiting for a few clarifications on the comments made by yiran. Otherwise it looks good to me.

cc @marklysze @qingyun-wu

Copy link
Copy Markdown
Contributor

@yiranwu0 yiranwu0 left a comment

Choose a reason for hiding this comment

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

LGTM!

@marklysze
Copy link
Copy Markdown
Contributor Author

LGTM!

Thanks for all your help!!

@yiranwu0
Copy link
Copy Markdown
Contributor

Resolve the conflict and we are good to go!

Copy link
Copy Markdown
Contributor

@Hk669 Hk669 left a comment

Choose a reason for hiding this comment

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

lgtm. Thanks @marklysze

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

models Pertains to using alternate, non-GPT, models (e.g., local models, llama, etc.)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants