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

Handle Cohere embedding models #61

Open

jamescalam opened this issue Dec 7, 2023 · 0 comments

Contributor

jamescalam commented Dec 7, 2023

          From testing it a bit seems like they know how to ignore this param if v2.0 is provided. I fine with simply mirroring their behavior, but leave it for your choice if to keep this assertion/remove it/add v2.0 to the list. Any choice would be fine IMO

Originally posted by @acatav in #58 (comment)

Creating an issue to implement more flexible approach to handling the Cohere embedding models (likely just remove validation on pinecone-text side).

The text was updated successfully, but these errors were encountered:

jamescalam changed the title From testing it a bit seems like they know how to ignore this param if v2.0 is provided. I fine with simply mirroring their behavior, but leave it for your choice if to keep this assertion/remove it/add v2.0 to the list. Any choice would be fine IMO Handle Cohere embedding models

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