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.

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

WIP : Add cohere chat generator #6423

Conversation

sunilkumardash9
Copy link
Contributor

Proposed Changes:

add CohereChatGenerator

How did you test it?

@sunilkumardash9 sunilkumardash9 requested a review from a team as a code owner November 27, 2023 12:46
@sunilkumardash9 sunilkumardash9 requested review from julian-risch and removed request for a team November 27, 2023 12:46
@github-actions github-actions bot added the type:documentation Improvements on the docs label Nov 27, 2023
@sunilkumardash9 sunilkumardash9 marked this pull request as draft November 27, 2023 12:47
@julian-risch
Copy link
Member

Hi @sunilkumardash9 great to see that you are working on a CohereChatGenerator. I see this PR is in draft mode. Do you have any questions or need any support at the moment? If not I'll wait until you say this PR is ready to be reviewed.

@sunilkumardash9
Copy link
Contributor Author

Hi @julian-risch, thanks for the offer of help. I have a query actually. Cohere chat has params like connectors like web-search and documents for RAG. Should we also explicitly handle them?

@masci
Copy link
Contributor

masci commented Dec 4, 2023

Hi @sunilkumardash9 I moved the cohere code to our core-integrations repo where I created a "cohere integration", see https://github.com/deepset-ai/haystack-core-integrations/tree/main/integrations/cohere

Do you mind moving this PR there? I'll do the same with the PR shipping embeddings support #5831

@masci masci closed this Dec 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:documentation Improvements on the docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants