Renames HybridQuery to AggregateHybridQuery #112
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR updates the class name from HybridQuery to AggregateHybridQuery throughout the vector search tutorial notebooks, aligning with a renamed class in the RedisVL library.
- Updated all imports and instantiations of
HybridQueryto useAggregateHybridQuery - Updated documentation, comments, and markdown cells to reference the new class name
- Ensured consistency across both affected notebooks
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| python-recipes/vector-search/02_hybrid_search.ipynb | Updated all 13 references from HybridQuery to AggregateHybridQuery in documentation, code examples, and markdown cells |
| python-recipes/vector-search/01_redisvl.ipynb | Updated 2 references from HybridQuery to AggregateHybridQuery in import statement and class instantiation |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Probably on your checklist but we will need to change the pin version for redisvl in this notebook for the new release otherwise it'll bomb on import.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <[email protected]>
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.