Skip to content

docs: add SIMQ index documentation for en and zh#2441

Merged
LHT129 merged 4 commits into
antgroup:mainfrom
Sia-Sheerland:main
Jul 20, 2026
Merged

docs: add SIMQ index documentation for en and zh#2441
LHT129 merged 4 commits into
antgroup:mainfrom
Sia-Sheerland:main

Conversation

@Sia-Sheerland

Copy link
Copy Markdown
Contributor

docs: add SIMQ index documentation for en and zh

Change Type

  • Bug fix
  • New feature
  • Improvement/Refactor
  • Documentation
  • CI/Build/Infra

Linked Issue

Closes: #2291

What Changed

  • Add docs/docs/en/src/indexes/simq.md and docs/docs/zh/src/indexes/simq.md, covering SIMQ algorithm overview, quick-start code, build/search parameter tables, applicable scenarios, and practical guidance for multi-vector (ColBERT / late-interaction) retrieval.
  • Register SIMQ in SUMMARY.md navigation for both en and zh.
  • Add SIMQ row to the overview table in indexes/README.md for both en and zh.

Test Evidence

  • make fmt
  • make lint
  • make test
  • make cov
  • Other: pure Markdown documentation change, no C++ code involved

Test details:

N/A — Markdown-only change, no build or test execution required.

Compatibility Impact

  • API/ABI compatibility: none
  • Behavior changes: none

Performance and Concurrency Impact

  • Performance impact: none
  • Concurrency/thread-safety impact: none

Documentation Impact

  • No docs update needed
  • Updated docs:
    • README.md
    • DEVELOPMENT.md
    • CONTRIBUTING.md
    • Other: docs/docs/{en,zh}/src/indexes/simq.md (new), docs/docs/{en,zh}/src/SUMMARY.md, docs/docs/{en,zh}/src/indexes/README.md

Risk and Rollback

  • Risk level: low — documentation-only addition, no code/build/API changes
  • Rollback plan: revert commit 751fed0b

Checklist

  • I have linked the relevant issue (required for kind/bug and kind/feature; see "Linked Issue" above)
  • I have added/updated tests for new behavior or bug fixes
  • I have considered API compatibility impact
  • I have updated docs if behavior/workflow changed
  • My commit messages follow project conventions (Conventional Commits, optional [skip ci] prefix)

Add dedicated SIMQ index pages to both English and Chinese documentation,
covering algorithm overview, quick-start code, build/search parameter tables,
and practical guidance for multi-vector (ColBERT / late-interaction) retrieval.
Register SIMQ in SUMMARY.md navigation and indexes/README.md overview table.

Signed-off-by: SiaSheerland <x1075956441x@163.com>
Copilot AI review requested due to automatic review settings July 13, 2026 08:05
@vsag-bot

vsag-bot commented Jul 13, 2026

Copy link
Copy Markdown
Collaborator

/label S-ready-to-merge
/waiting-on maintainer
/request-review @jiaweizone
/request-review @wxyucs
/request-review @inabao

@mergify

mergify Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Merge Protections

🟢 All 2 merge protections satisfied — ready to merge.

Show 2 satisfied protections

🟢 Require kind label

  • label~=^kind/

🟢 Require version label

  • label~=^version/

@gemini-code-assist gemini-code-assist Bot left a comment

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.

Code Review

This pull request adds comprehensive English and Chinese documentation for the new SIMQ index, which is designed for multi-vector retrieval using MaxSim scoring (e.g., ColBERT). The feedback highlights a configuration hierarchy inaccuracy in both language versions: the parameters dim, dtype, and metric_type are described as living under index_param, whereas they are actually top-level parameters in the VSAG configuration JSON. Clarifying this distinction will prevent configuration errors for users.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread docs/docs/en/src/indexes/simq.md Outdated
Comment thread docs/docs/zh/src/indexes/simq.md Outdated

Copilot AI left a comment

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.

Pull request overview

Adds end-user documentation for the new SIMQ multi-vector index to the VSAG docs site (English + Chinese), and wires it into the navigation and index overview tables.

Changes:

  • Added new SIMQ index pages: indexes/simq.md (en/zh) with algorithm overview, quick-start, and parameter tables.
  • Registered SIMQ in SUMMARY.md navigation (en/zh).
  • Added SIMQ to the indexes overview table in indexes/README.md (en/zh).

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
docs/docs/en/src/SUMMARY.md Adds SIMQ to the English docs navigation.
docs/docs/en/src/indexes/simq.md New English SIMQ documentation page (overview, quick start, params).
docs/docs/en/src/indexes/README.md Adds SIMQ row to the English indexes overview table.
docs/docs/zh/src/SUMMARY.md Adds SIMQ to the Chinese docs navigation.
docs/docs/zh/src/indexes/simq.md New Chinese SIMQ documentation page (overview, quick start, params).
docs/docs/zh/src/indexes/README.md Adds SIMQ row to the Chinese indexes overview table.

Comment thread docs/docs/en/src/indexes/simq.md
Comment thread docs/docs/en/src/indexes/simq.md
Comment thread docs/docs/en/src/indexes/simq.md Outdated
Comment thread docs/docs/zh/src/indexes/simq.md
Comment thread docs/docs/zh/src/indexes/simq.md
Comment thread docs/docs/zh/src/indexes/simq.md Outdated
@LHT129 LHT129 added version/1.1 kind/documentation Changes to docs, READMEs, or inline comments 文档、说明书、README 或注释的修改 labels Jul 14, 2026
@LHT129 LHT129 self-assigned this Jul 14, 2026
Clarify that dim/dtype/metric_type are top-level build fields while
SIMQ-specific parameters live under index_param. Correct base_io_type
and base_file_path to show their actual defaults from source instead
of marking them as required.

Signed-off-by: SiaSheerland <x1075956441x@163.com>
Copilot AI review requested due to automatic review settings July 20, 2026 03:08

Copilot AI left a comment

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.

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Comment thread docs/docs/en/src/indexes/simq.md Outdated
Comment thread docs/docs/zh/src/indexes/simq.md Outdated
Move verbose parameter descriptions out of markdown tables into bullet
lists below so that all source lines stay within the 100-character
limit required by AGENTS.md.

Signed-off-by: SiaSheerland <x1075956441x@163.com>
Copilot AI review requested due to automatic review settings July 20, 2026 03:42

Copilot AI left a comment

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.

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Comment thread docs/docs/en/src/indexes/README.md Outdated
Signed-off-by: SiaSheerland <x1075956441x@163.com>
Copilot AI review requested due to automatic review settings July 20, 2026 03:53

Copilot AI left a comment

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.

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Comment thread docs/docs/en/src/indexes/simq.md
Comment thread docs/docs/zh/src/indexes/simq.md
Comment thread docs/docs/zh/src/indexes/simq.md

@LHT129 LHT129 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@LHT129
LHT129 merged commit 8153b28 into antgroup:main Jul 20, 2026
19 checks passed
@mergify

mergify Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/documentation Changes to docs, READMEs, or inline comments 文档、说明书、README 或注释的修改 module/docs size/L version/1.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[feat](other): add SIMQ multi-vector index with ColBERT-style two-phase retrieval

4 participants