Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

Added documentation for together ai embeddings endpoint #223

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

Govind-S-B
Copy link

This is related to chroma-core/chroma#1828 where I added support for together ai embeddings endpoint

Copy link

vercel bot commented Mar 6, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 16, 2024 9:56am

Copy link
Contributor

@atroyn atroyn left a comment

Choose a reason for hiding this comment

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

Please also add to the table of embeddings integrations:

https://github.com/chroma-core/docs/blob/main/docs/embeddings.md?plain=1#L14

And to the list in the sidebar:

'embeddings/openai',

@Govind-S-B
Copy link
Author

@atroyn resolved

@Govind-S-B Govind-S-B requested a review from atroyn March 11, 2024 06:15
Copy link
Contributor

@jeffchuber jeffchuber left a comment

Choose a reason for hiding this comment

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

couple small changes, thank you!

docs/embeddings/together-ai-embeddings.md Outdated Show resolved Hide resolved
@tazarov
Copy link
Contributor

tazarov commented Apr 11, 2024

@Govind-S-B, can you resolve the conflicts?

@tazarov
Copy link
Contributor

tazarov commented Apr 16, 2024

@Govind-S-B, can you resolve the conflicts?

Bump for reminder.

@Govind-S-B
Copy link
Author

@Govind-S-B, can you resolve the conflicts?

Bump for reminder.

Hey thanks for the reminder.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants