Please check if the PR fulfills these requirements
[x] The commit message follows our guidelines
[x] Tests for the changes have been added (for bug fixes/features)
[ ] Docs have been added / updated (for bug fixes / features)
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Added the content_type parameter to the embed function and content_type="document" to embed test requests, which is a new field added in https://github.com/marqo-ai/marqo/pull/821 for the prefix feature
What is the current behavior? (You can also link to an open issue here)
Without adding the content_type as a parameter in the embed function, we are not able to pass its value into marqo.
Without defining the field content_type, content_type defaults to "query" prefixes, which makes the tests fail when comparing to embeddings from add_documents since add_documents default to "document" prefixes.
What is the new behavior (if this is a feature change)?
We add content_type as a parameter and are now able to take this field.
By defining content_type defaults="document" we make sure that the prefixes prepended to document chunks are the same for both embed calls and add_document calls.
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
None
Please check if the PR fulfills these requirements
[x] The commit message follows our guidelines
[x] Tests for the changes have been added (for bug fixes/features)
[ ] Docs have been added / updated (for bug fixes / features)
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...) Added the
content_type
parameter to the embed function andcontent_type="document"
to embed test requests, which is a new field added in https://github.com/marqo-ai/marqo/pull/821 for the prefix featureWhat is the current behavior? (You can also link to an open issue here)
Without adding the content_type as a parameter in the embed function, we are not able to pass its value into marqo.
Without defining the field content_type, content_type defaults to "query" prefixes, which makes the tests fail when comparing to embeddings from add_documents since add_documents default to "document" prefixes.
What is the new behavior (if this is a feature change)?
We add content_type as a parameter and are now able to take this field.
By defining content_type defaults="document" we make sure that the prefixes prepended to document chunks are the same for both embed calls and add_document calls.
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?) None
Other information: