Skip to content

feat: enable one2many batch distance for ip and l2#113

Open
kgeg401 wants to merge 3 commits intoalibaba:mainfrom
kgeg401:feat/issue-97-one2many-ip-l2
Open

feat: enable one2many batch distance for ip and l2#113
kgeg401 wants to merge 3 commits intoalibaba:mainfrom
kgeg401:feat/issue-97-one2many-ip-l2

Conversation

@kgeg401
Copy link

@kgeg401 kgeg401 commented Feb 14, 2026

Summary

  • enable one-to-many batch distance path for:
    • InnerProduct metric
    • SquaredEuclidean (L2) metric
  • switch dense/int8/int4 batch-distance wiring from 1,1 to 12,2
  • add regression tests that compare scalar distance vs batch distance for FP32 and INT8

Why

Issue #97 requests enabling one2many for IP and L2. Existing wiring in these metrics used single-lane batch settings, while cosine already uses one2many-capable settings.

Validation

  • added unit tests in:
    • ests/core/metric/inner_product_metric_test.cc
      
    • ests/core/metric/euclidean_metric_test.cc
      
  • note: local environment lacks a usable C/C++ compiler toolchain, so test execution is deferred to CI

Fixes #97

@CLAassistant
Copy link

CLAassistant commented Feb 14, 2026

CLA assistant check
All committers have signed the CLA.

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.

[Enhance]: enable one2many calculation for l2/ip metrics

4 participants