-
Notifications
You must be signed in to change notification settings - Fork 2.4k
feat(dspy): add vector, hybrid and fulltext search support in azure ai search module #815
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
feat(dspy): add vector, hybrid and fulltext search support in azure ai search module #815
Conversation
|
@insop @arnavsinghvi11 @okhat , Can I get the review on this PR. |
|
Thanks @HARISHKUMAR1112001 ! left a few minor comments. good to merge after that! |
|
Thanks @arnavsinghvi11 for the quick review. I have updated the code with correct spellings. |
|
Thanks @HARISHKUMAR1112001 ! |
|
@HARISHKUMAR1112001 @arnavsinghvi11 i'm getting the following error with the following code: requirements.txt ` error: [2024-04-18T08:45:17.286Z] Executed 'Functions.f29bot2' (Failed, Id=60fe1453-0587-4435-bd53-525ca750323c, Duration=222ms) already commented this on the discord this morning |
|
I think you are provide wrong arguments to AzureOpenAI. It has below arguments: ` """Construct a new synchronous azure openai client instance. |
|
thanks for ur response @HARISHKUMAR1112001 it was really weird because both the AzureOpenAI and the SearchClient were working alone without DSPy but debugging with a custom AzureRM class i think i found the error it was related to the "search_service_name=search_endpoint" arg I was passing the search_endpoint as a complete url: search_endpoint = "https://f29search.search.windows.net"; [2024-04-19T12:36:57.321Z] Azure Search Request tried then without "https://" search_endpoint = "https://f29search.search.windows.net"; [2024-04-19T12:41:45.467Z] Azure Search Request but should have passed the service name only: search_endpoint = "f29search"; # Add your Azure AI Search endpoint here now it worked! sorry for bothering, was my bad! |
…-azure-ai-search feat(dspy): add vector, hybrid and fulltext search support in azure ai search module
Feature Addition: Updating AzureAISearchRM Class
This pull request aims to enhance the functionality of the
AzureAISearchRMclass by introducing support for additional search capabilities. Specifically, the class is updated to enableVector Search,Hybrid Search, andFull Text SearchwithSemantic ReRanker, providing users with more versatile and powerful search options.Changes Included:
Vector Search,Hybrid Search, andFull Text Searchfunctionalities.Semantic ReRankerto enhance search results with semantic understanding.Checks:
Pre-Commit checks are passing (locally and remotely)
Title of your PR / MR corresponds to the required format
Commit message follows required format {label}(dspy): {message}
If there are any further improvements or adjustments required, please feel free to provide feedback. This pull request aims to enhance the functionality and usability of the
AzureAISearchRMclass, and any suggestions for refinement are highly appreciated.