haystack icon indicating copy to clipboard operation
haystack copied to clipboard

Support `return_embedding` param in DocumentStore's `get_document_by_id`

Open tstadel opened this issue 2 years ago • 0 comments

Is your feature request related to a problem? Please describe. BaseDocumentStore's get_all_documents supports return_embedding as method param but get_document_by_id does not. This is inconsistent and a bit confusing.

Describe the solution you'd like Add return_embedding param to get_doucment_by_id of BaseDocumentStore and all its subclasses.

Describe alternatives you've considered None

Additional context Besides API consistency this could be handy if you want to check whether an individual document has (correct) embeddings.

tstadel avatar Aug 09 '22 13:08 tstadel