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

Devmerge #2646

Merged
merged 15 commits into from
Nov 22, 2024
Merged

Devmerge #2646

merged 15 commits into from
Nov 22, 2024

Conversation

javitonino
Copy link
Contributor

Description

Describe the proposed changes made in this PR.

How was this PR tested?

Describe how you tested this PR.

javitonino and others added 15 commits November 22, 2024 09:59
* Migrate date filter test

* Migrate more search tests

* Copy remaining search tests: pending to implement suggest

* Port suggest to nidx

* Missing shard_suggest module

* Add basic tests for shards with/without vectorsets

* Implement add and list vectorsets

* Remove unneeded allow

* Remove rayon from relations and don't parallelize sugggest

* Implement remove vectorsets

* cargo fmt

* Change error string

Co-authored-by: Javier Torres <javier@javiertorres.eu>

---------

Co-authored-by: Joan Antoni RE <joanantoni.re16@gmail.com>
* Convert ParagraphSearchRequest to SearchRequest

* Return appropiate paragraph response

* No more RelationSearch either

* Not ready to replace paragraph search request

* Don't call RelationSearch anymore (py)

* Remove RelationSearch

* Let's try again with paragraph search :)

* Typo in type

* Replace last direct call to ParagraphSearch (tests)

* No more ParagraphSearch (grpc)

* No more DocumentSearch

* No more VectorSearch

* Remove test using document search

* Update protos

* Fix test

* Cleanup query_paragraph_shard

* Remove ignored params from resource search endpoint

* Fix test

* Fix unit tests

* Cleanup relations_shard

* Remove paragraph and relation search methods
@javitonino javitonino merged commit b2b8e7a into main Nov 22, 2024
114 of 169 checks passed
@javitonino javitonino deleted the devmerge branch November 22, 2024 09:47
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