Documentation for query_vector base64 parameter#142675
Conversation
Update documentation for query_vector parameter in diversify and knn retrievers to include base64-encoded vector string support and clarify hex-encoded byte vector usage. Relates to elastic#138190
|
Pinging @elastic/es-search-relevance (Team:Search Relevance) |
ℹ️ Important: Docs version tagging👋 Thanks for updating the docs! Just a friendly reminder that our docs are now cumulative. This means all 9.x versions are documented on the same page and published off of the main branch, instead of creating separate pages for each minor version. We use applies_to tags to mark version-specific features and changes. Expand for a quick overviewWhen to use applies_to tags:✅ At the page level to indicate which products/deployments the content applies to (mandatory) What NOT to do:❌ Don't remove or replace information that applies to an older version 🤔 Need help?
|
|
Pinging @elastic/core-docs (Team:Docs) |
benwtrent
left a comment
There was a problem hiding this comment.
Looks mostly good. I am not sure how this versioning stuff works...
docs/reference/elasticsearch/rest-apis/retrievers/diversify-retriever.md
Outdated
Show resolved
Hide resolved
docs/reference/elasticsearch/rest-apis/retrievers/diversify-retriever.md
Outdated
Show resolved
Hide resolved
…triever.md Co-authored-by: Benjamin Trent <ben.w.trent@gmail.com>
Adding big-endianess
leemthompo
left a comment
There was a problem hiding this comment.
Thanks @ah89 and @benwtrent for the versioning vigilance!
Couple suggestions here for clarity and syntax. Also makes me think if we're repeating the same info in multiple spots here, we could factor the common stuff out into a reusable snippet and include it wherever we need it. LMKWYT.
You can use the URL previews to verify everything looks OK
docs/reference/elasticsearch/rest-apis/retrievers/diversify-retriever.md
Outdated
Show resolved
Hide resolved
|
|
||
| Query vector. Must have the same number of dimensions as the vector field you are searching against. Must be either an array of floats or a hex-encoded byte vector. | ||
| Query vector. Must have the same number of dimensions as the vector field you are searching against. | ||
| Must be either an array of floats or a hex-encoded byte vector (one byte per dimension; for `bit`, one byte per 8 dimensions). |
There was a problem hiding this comment.
copy/paste error here and let's use a list like the first suggestion :)
There was a problem hiding this comment.
yeah, @ah89 we need a list here like the diversified retriever and same goes for the knn-query docs please.
docs/reference/query-languages/query-dsl/query-dsl-knn-query.md
Outdated
Show resolved
Hide resolved
docs/reference/query-languages/query-dsl/query-dsl-knn-query.md
Outdated
Show resolved
Hide resolved
docs/reference/query-languages/query-dsl/query-dsl-knn-query.md
Outdated
Show resolved
Hide resolved
docs/reference/query-languages/query-dsl/query-dsl-knn-query.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Liam Thompson <leemthompo@gmail.com>
…triever.md Co-authored-by: Liam Thompson <leemthompo@gmail.com>
Co-authored-by: Liam Thompson <leemthompo@gmail.com>
Co-authored-by: Liam Thompson <leemthompo@gmail.com>
Co-authored-by: Liam Thompson <leemthompo@gmail.com>
Eliminates an unnecessary code block for improved clarity and readability in the documentation. No content functionality is affected.
Improves documentation by explicitly listing supported input formats for the kNN retriever and query APIs. Enhances clarity on when to use array, hex, or base64-encoded vectors and the applicable product versions for each format.
Vale Linting ResultsSummary: 1 suggestion found 💡 Suggestions (1)
The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale. |
* Documentation for query_vector parameter Update documentation for query_vector parameter in diversify and knn retrievers to include base64-encoded vector string support and clarify hex-encoded byte vector usage. Relates to elastic#138190 * Added tag * Update docs/reference/elasticsearch/rest-apis/retrievers/diversify-retriever.md Co-authored-by: Benjamin Trent <ben.w.trent@gmail.com> * Update documentation for query_vector parameter Adding big-endianess * Update docs/reference/query-languages/query-dsl/query-dsl-knn-query.md Co-authored-by: Liam Thompson <leemthompo@gmail.com> * Update docs/reference/elasticsearch/rest-apis/retrievers/diversify-retriever.md Co-authored-by: Liam Thompson <leemthompo@gmail.com> * Update docs/reference/query-languages/query-dsl/query-dsl-knn-query.md Co-authored-by: Liam Thompson <leemthompo@gmail.com> * Update docs/reference/query-languages/query-dsl/query-dsl-knn-query.md Co-authored-by: Liam Thompson <leemthompo@gmail.com> * Update docs/reference/query-languages/query-dsl/query-dsl-knn-query.md Co-authored-by: Liam Thompson <leemthompo@gmail.com> * Removes redundant code block from documentation Eliminates an unnecessary code block for improved clarity and readability in the documentation. No content functionality is affected. --------- Co-authored-by: Benjamin Trent <ben.w.trent@gmail.com> Co-authored-by: Liam Thompson <leemthompo@gmail.com>
…locations * upstream/main: (51 commits) ESQL: Remaining serialization tests (elastic#143470) Eagerly release resources in `TransportAwaitClusterStateVersionAppliedAction` (elastic#143477) Stop and relocate sliced reindex on shutdown (elastic#143183) Documentation for query_vector base64 parameter (elastic#142675) ES|QL: Fix LIMIT after all columns are dropped (elastic#143463) Update docs-build.yml (elastic#142958) Fix KnnIndexTester to work with byte vectors (elastic#143493) Fix IndexInputUtils.withSlice to produce native-safe MemorySegments on Java 21 (elastic#143479) CPS fix: include only relevant projects in the search response metadata (elastic#143367) apm-data: explicit map of timestamp.us to long (elastic#143173) [Inference API] Add custom headers for Azure OpenAI Service (elastic#142969) ESQL: Add name IDs to golden tests and fix synthetic names (elastic#143450) Add getUnavailableShards to BaseBroadcastResponse (elastic#143406) Add description to reindex API without sensitive info (elastic#143112) SQL: fix CLI tests (elastic#143451) ES|QL: Add note of future removal of FORK implicit LIMIT (elastic#143457) [Test] Randomly disable doc values skippers in time-series indices (elastic#143389) Improve pattern text downgrade license test (elastic#143102) [Transform] Stop transforms at the end of tests (elastic#139783) Mute org.elasticsearch.compute.lucene.read.ValueSourceReaderTypeConversionTests testLoadAll elastic#143471 ...
* Documentation for query_vector parameter Update documentation for query_vector parameter in diversify and knn retrievers to include base64-encoded vector string support and clarify hex-encoded byte vector usage. Relates to elastic#138190 * Added tag * Update docs/reference/elasticsearch/rest-apis/retrievers/diversify-retriever.md Co-authored-by: Benjamin Trent <ben.w.trent@gmail.com> * Update documentation for query_vector parameter Adding big-endianess * Update docs/reference/query-languages/query-dsl/query-dsl-knn-query.md Co-authored-by: Liam Thompson <leemthompo@gmail.com> * Update docs/reference/elasticsearch/rest-apis/retrievers/diversify-retriever.md Co-authored-by: Liam Thompson <leemthompo@gmail.com> * Update docs/reference/query-languages/query-dsl/query-dsl-knn-query.md Co-authored-by: Liam Thompson <leemthompo@gmail.com> * Update docs/reference/query-languages/query-dsl/query-dsl-knn-query.md Co-authored-by: Liam Thompson <leemthompo@gmail.com> * Update docs/reference/query-languages/query-dsl/query-dsl-knn-query.md Co-authored-by: Liam Thompson <leemthompo@gmail.com> * Removes redundant code block from documentation Eliminates an unnecessary code block for improved clarity and readability in the documentation. No content functionality is affected. --------- Co-authored-by: Benjamin Trent <ben.w.trent@gmail.com> Co-authored-by: Liam Thompson <leemthompo@gmail.com>
Update documentation for query_vector parameter in diversify and knn retrievers to include base64-encoded vector string support and clarify hex-encoded byte vector usage.
Relates to #138190