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

Randomize KNN vector format parameters in tests #14047

Closed
msokolov opened this issue Dec 6, 2024 · 1 comment
Closed

Randomize KNN vector format parameters in tests #14047

msokolov opened this issue Dec 6, 2024 · 1 comment
Milestone

Comments

@msokolov
Copy link
Contributor

msokolov commented Dec 6, 2024

Description

I discovered while doing some sweeping refactoring changes that we lack coverage of many vector field features, like scalar compression and concurrent merging. We even lack randomization for basic settings like graph connectivity (we always test with the default = 32).

The idea here is to add randomization to RandomCodec similar to what we do for other field types. I made an initial, but many tests do make assumptions that were broken. I think initially we will need a way to "opt out" of such randomization. Is there a standard way to do that (for a specific format, or for the whole codec)?

Gradle command to reproduce

No response

@msokolov
Copy link
Contributor Author

msokolov commented Dec 9, 2024

also pushed 4f8035c to address a build failure by disabling randomization in more tests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant