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.

Sign up for GitHub

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

Jump to bottom

parters: langchain-voyageai package #18702

Closed
wants to merge 38 commits into from
Closed

parters: langchain-voyageai package #18702

wants to merge 38 commits into from

Conversation

thomas0809
Copy link
Contributor 

- **Description:** Implementing a new langchain-voyageai package under `partners`
- **Issue:** 
- **Dependencies:** 
- **Twitter handle:**

fodizoltan and others added 30 commits February 15, 2024 21:16
@fodizoltan
Change model to a required argument
0ff1332 
Change the default batch size to 72 for model "voyage-2"/"voyage-02", and to 7 for all other models. (However, still allow the user to specify batch size)
@fodizoltan
Removed the model attribute declaration as it isn't use
be7d6cc
@fodizoltan
Referring to the documentation
146f0f1
@fodizoltan
- model:str
4de3553 
 -make batch_size optional
@thomas0809
Merge pull request #3 from voyage-ai/voyage_2_model
6e53bf9 
community: model attribute is required, set default batch size
@fodizoltan
Reformatting voyage
2275b99
@thomas0809
Merge pull request #4 from voyage-ai/voyage_2_model
1041892 
Reformatting voyageai.ipynb
@fodizoltan
Correct validator and log a warning if no model is defined
af41f94
@fodizoltan
Add test for handling the default value and small code correction
0d3ae1a
@fodizoltan
Add test for handling the default value and small code correction
6a789d3
@fodizoltan
Corrections, due to comments (and more)
59efab5
@fodizoltan
VoyageAI package init
e9bac0e
@fodizoltan
v0.0.1
7fd85d1
@fodizoltan
Corrections
e37f8af
@fodizoltan
Corrections
5617a10
@fodizoltan
Adding tests
de61f28
@fodizoltan
Loop and remove bad test
83da26e
@fodizoltan
Finalizing the package
9bfb0a0
@thomas0809
Merge branch 'langchain-ai:master' into master
2b9a1d7
@thomas0809
Merge pull request #5 from voyage-ai/voyage_2_model
853293c 
Correcting due to the comments
@thomas0809
Update truncation default value
9eddafd
@thomas0809
Update argument type for lint
9d59a2f
@fodizoltan
Implemented async
c463b1d
@fodizoltan
Revert changes in pyproject.toml
969cbb2
@fodizoltan
Change in LICENSE
cf71a5f
@fodizoltan
Revert poetry.lock changes
c8f12d2
@fodizoltan
Revert pyproject.toml changes
1966be3
@fodizoltan
Simple for loop
b908f18 
Documentation
@fodizoltan
Remove default values
8599e6a 
Reformat
@fzowl
Merge branch 'master' into voyageai_package
a0c6b06
@efriis efriis added the partner label Mar 6, 2024
@efriis efriis self-assigned this Mar 6, 2024
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Mar 6, 2024
@vercel Vercel
Copy link

vercel bot commented Mar 6, 2024 •
edited

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

Name Status Preview Comments Updated (UTC)
langchain ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 7, 2024 0:27am

@dosubot dosubot bot added Ɑ: embeddings Related to text embedding models module 🤖:enhancement A large net-new component, integration, or chain. Use sparingly. The largest features labels Mar 6, 2024
@efriis
Copy link
Member

efriis commented Mar 15, 2024

Going to edit a few things and merge from #19098! Don't have edit access to this branch. See this FAQ for more info!

https://python.langchain.com/docs/contributing/faq#how-do-i-allow-maintainers-to-edit-my-pr

@efriis efriis closed this Mar 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@efriis efriis

Labels
Ɑ: embeddings Related to text embedding models module 🤖:enhancement A large net-new component, integration, or chain. Use sparingly. The largest features partner size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@thomas0809 @efriis @fodizoltan @fzowl