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

community: Escapes AzureCosmosDBNoSqlVectorStore hybrid & full text queries that contain single quotes #29339

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

BenMcH
Copy link

@BenMcH BenMcH commented Jan 21, 2025

Thank you for contributing to LangChain!

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Jan 21, 2025
Copy link

vercel bot commented Jan 21, 2025

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

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Visit Preview Jan 23, 2025 5:53pm

@dosubot dosubot bot added community Related to langchain-community Ɑ: vector store Related to vector store module 🤖:bug Related to a bug, vulnerability, unexpected error with an existing feature labels Jan 21, 2025
@BenMcH
Copy link
Author

BenMcH commented Jan 21, 2025

It seems that most if not all of these failures are due to failures in other packages

Edit: Turns out I was using a python feature not supported by older versions. This is fixed now!

@BenMcH BenMcH force-pushed the fix-cosmosdb-hybrid-search branch from 0a035e3 to 490afb0 Compare January 22, 2025 18:22
@BenMcH
Copy link
Author

BenMcH commented Jan 23, 2025

@efriis any chance I could get a review on this? Should be a really quick one 🤠

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖:bug Related to a bug, vulnerability, unexpected error with an existing feature community Related to langchain-community size:S This PR changes 10-29 lines, ignoring generated files. Ɑ: vector store Related to vector store module
Projects
Status: Triage
Development

Successfully merging this pull request may close these issues.

1 participant