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

VoyageAI embedding function #226

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

VoyageAI embedding function #226

wants to merge 21 commits into from

Conversation

fzowl
Copy link

@fzowl fzowl commented Mar 14, 2024

No description provided.

Copy link

vercel bot commented Mar 14, 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 May 8, 2024 5:17pm

@jeffchuber
Copy link
Contributor

@fzowl thanks! are the changes to js_reference necessary in this PR? We've been wanting to improve our autogen docs, because the output is not very user friendly. Anyways - I think it's probably out of scope for this PR?

Copy link
Contributor

@tazarov tazarov left a comment

Choose a reason for hiding this comment

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

@fzowl, thank you for adding the docs. Can you update the reference to point to the correct embedding in embeddings.md, rebase to resolve the conflicts, and add ESM import in the JS example? I would do it myself, but seems you haven't granted access to maintainers in your fork.

docs/embeddings.md Outdated Show resolved Hide resolved
docs/embeddings/voyageai.md Show resolved Hide resolved
@fzowl
Copy link
Author

fzowl commented Apr 22, 2024

Hello @tazarov ! I just made the requested changes (and added some correction in our sample code as well).
Can you please take a look?

@fzowl fzowl requested a review from tazarov April 22, 2024 15:30
@fzowl
Copy link
Author

fzowl commented May 8, 2024

@tazarov Can you please check this and approve?

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.

5 participants