Skip to content

doc TokenLanguageModelKNN: Add (passage|query)_prefix option#2663

Merged
komainu8 merged 2 commits intogroonga:mainfrom
abetomo:docs-lm-knn-prefix
Nov 28, 2025
Merged

doc TokenLanguageModelKNN: Add (passage|query)_prefix option#2663
komainu8 merged 2 commits intogroonga:mainfrom
abetomo:docs-lm-knn-prefix

Conversation

@abetomo
Copy link
Copy Markdown
Member

@abetomo abetomo commented Nov 28, 2025

No description provided.

@abetomo abetomo mentioned this pull request Nov 28, 2025
21 tasks

```

Some models such as multilingual-e5 require prefix for search target texts and query texts.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[nits]

Suggested change
Some models such as multilingual-e5 require prefix for search target texts and query texts.
Some models such as multilingual-e5 require prefix used for search-target texts and query texts.


Some models such as multilingual-e5 require prefix for search target texts and query texts.

`passage_prefix` is for prefix for search target text.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
`passage_prefix` is for prefix for search target text.
`passage_prefix` specifies the prefix for search target text.


`passage_prefix` is for prefix for search target text.

For example, we can set `passage: ` prefix in search target text and `query: ` prefix in query text as below.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
For example, we can set `passage: ` prefix in search target text and `query: ` prefix in query text as below.
For example, you can set `passage: ` prefix in search target text and `query: ` prefix in query text as below.

@komainu8 komainu8 merged commit 03dba48 into groonga:main Nov 28, 2025
19 checks passed
@abetomo abetomo deleted the docs-lm-knn-prefix branch November 28, 2025 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants