Skip to content

Conversation

@garybadwal
Copy link
Contributor

  1. Adds integration for Comet API for Chat Generator.
  2. This integration enable users to leverage chat models to enhance RAG (Retrieval-Augmented Generation) pipelines using Comet API.

@garybadwal garybadwal requested a review from a team as a code owner October 19, 2025 05:46
Copy link
Contributor

@bilgeyucel bilgeyucel left a comment

Choose a reason for hiding this comment

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

Thanks for the PR @garybadwal. Can you double check the links? It seems like there's no such integration

Comment on lines +18 to +20
repo: https://github.com/deepset-ai/haystack-core-integrations/tree/main/integrations/cometapi
type: Model Provider
report_issue: https://github.com/deepset-ai/haystack-core-integrations/issues
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you provide the correct links for the repo here?

```

## Usage
You can use [CometAPIChatGenerator](https://docs.haystack.deepset.ai/docs/cometapichatgenerator) as standalone, within a [pipeline](https://docs.haystack.deepset.ai/docs/pipelines) or with the [Agent component](https://docs.haystack.deepset.ai/docs/agent).
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
You can use [CometAPIChatGenerator](https://docs.haystack.deepset.ai/docs/cometapichatgenerator) as standalone, within a [pipeline](https://docs.haystack.deepset.ai/docs/pipelines) or with the [Agent component](https://docs.haystack.deepset.ai/docs/agent).
You can use `CometAPIChatGenerator` as standalone, within a [pipeline](https://docs.haystack.deepset.ai/docs/pipelines) or with the [Agent component](https://docs.haystack.deepset.ai/docs/agent).

@garybadwal
Copy link
Contributor Author

garybadwal commented Nov 10, 2025

Hi @bilgeyucel,

Thank you for the review. Regarding the links, I’ve submitted an integration PR in Haystack Core IntegrationsComet API Integration PR. The PR is currently under review, and I’ll update the links once it’s merged. Would that be okay?

@mpangrazzi
Copy link

@garybadwal it's merged now, feel free update this PR! Thanks!

@garybadwal
Copy link
Contributor Author

@garybadwal it's merged now, feel free update this PR! Thanks!

Thank you so much mpangrazzi, I'll update this PR.

@garybadwal
Copy link
Contributor Author

Hi @bilgeyucel, Can you re-review my PR. Now all links are working and updated.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants