Skip to content

Conversation

@dhirupandey
Copy link

Added Coherence VectorStore Integration

Dependencies: The integration has a dependency on coherence-client package. See langchain/libs/partners/coherence/pyproject.toml

Integration test added at langchain/libs/partners/coherence/tests/integration_tests/test_coherence_store.py. However they need Coherence server to be run in order to work

Added documentation notebook at at docs/docs/integrations/vectorstores/coherence.ipynb

@vercel
Copy link

vercel bot commented Jul 28, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
langchain Ready Ready Preview Comment Sep 12, 2025 9:33pm

@codspeed-hq
Copy link

codspeed-hq bot commented Jul 28, 2025

CodSpeed WallTime Performance Report

Merging #32289 will improve performances by 15.04%

Comparing dhirupandey:master (d95802f) with master (338d3d2)

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

Summary

⚡ 1 improvement
✅ 12 untouched

Benchmarks breakdown

Benchmark BASE HEAD Change
test_import_time[RunnableLambda] 560.7 ms 487.4 ms +15.04%

@codspeed-hq
Copy link

codspeed-hq bot commented Jul 28, 2025

CodSpeed Instrumentation Performance Report

Merging #32289 will not alter performance

Comparing dhirupandey:master (d95802f) with master (338d3d2)

Summary

✅ 14 untouched

@dhirupandey dhirupandey changed the title feat(Coherence): add CoherenceVectorStore integration feat: add partner CoherenceVectorStore integration Jul 28, 2025
@mdrxy
Copy link
Member

mdrxy commented Aug 11, 2025

Hi thank you for the work so far. We are not adding new integrations to this monorepo, but instead to separate repositories. Would you be able to migrate this?

@dhirupandey
Copy link
Author

Hi thank you for the work so far. We are not adding new integrations to this monorepo, but instead to separate repositories. Would you be able to migrate this?

Hi @mdrxy Where does this need to be migrated ? Can you provide details and instructions on how to do that.

@mdrxy mdrxy self-assigned this Sep 8, 2025
@mdrxy mdrxy changed the title feat: add partner CoherenceVectorStore integration docs: add CoherenceVectorStore integration Sep 11, 2025
@mdrxy mdrxy added the documentation Improvements or additions to documentation label Sep 11, 2025
@mdrxy
Copy link
Member

mdrxy commented Sep 15, 2025

Hi @dhirupandey, apologies for the delays. We've written new contributing guides that should hopefully address your question. Happy to answer any questions as they arise!

@mdrxy mdrxy closed this Sep 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants