Skip to content

Conversation

@wxyucs
Copy link
Collaborator

@wxyucs wxyucs commented May 12, 2025

fixes: #696

SIMD instructions (avx,sse, ..), in diskann, are not used by vsag now, so we can disable these codes to avoid illegal instructions happened on some old platforms.

@codecov
Copy link

codecov bot commented May 12, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

@@           Coverage Diff           @@
##             main     #698   +/-   ##
=======================================
  Coverage   91.30%   91.30%           
=======================================
  Files         189      189           
  Lines       11437    11440    +3     
=======================================
+ Hits        10442    10445    +3     
  Misses        995      995           
Flag Coverage Δ
cpp 91.30% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
common 93.02% <ø> (ø)
datacell 92.41% <ø> (ø)
index 89.20% <ø> (+<0.01%) ⬆️
simd 100.00% <ø> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b64ca43...84b07b9. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@wxyucs wxyucs force-pushed the remove-diskann-simd branch from 6579d6a to d578ba2 Compare May 13, 2025 13:41
Copy link
Collaborator

@jiaweizone jiaweizone left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Xiangyu Wang <wxy407827@antgroup.com>
@wxyucs wxyucs force-pushed the remove-diskann-simd branch from d578ba2 to 84b07b9 Compare May 14, 2025 03:00
Copy link
Collaborator

@inabao inabao left a comment

Choose a reason for hiding this comment

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

LGTM

@wxyucs wxyucs merged commit 15e053c into main May 14, 2025
23 of 24 checks passed
@wxyucs wxyucs deleted the remove-diskann-simd branch May 14, 2025 10:04
wxyucs added a commit that referenced this pull request May 14, 2025
Signed-off-by: Xiangyu Wang <wxy407827@antgroup.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

diskann static library has unexpected AVX instructions may cause SIGILL on machines without AVX

4 participants