weaviate
weaviate copied to clipboard
FR: Can I specify `ef` at search time?
Instead of updating the schema definition, could I specify an ef argument at search time? Is this something you'd be open to adding?
Seems reasonable, I've added the backlog label. This means upvotes are counted on our Roadmap page.
Maybe this could fit in the gRPC API which is used by the new v4 python client (cc @tsmith023, @dirkkul).
One thing that we need to be aware of: ef is specific to hnsw. WIth other vector types in the future, this property won't make sense. So it should be a conditional or optional property. Maybe even namespaced under something like hnsw.
I am interested to start my open source contribution journey, Can you please assign me this issue?
please just start working on it :)
We usually do not assign issues as people sometimes just disappear
IIRC, we can add a ef field to the gRPC search request, and pass it through all the way to hnsw SearchByVector https://sourcegraph.com/github.com/weaviate/weaviate@master/-/blob/adapters/repos/db/vector/hnsw/search.go?L64. This SearchByVector is also an interface method, so adding an argument will need changes in many places.
@dirkkul @etiennedi does the above sounds reasonable?
Just wondering if this issue is done, per #3730. If not, let me know if any help is needed.