Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update k-NN build artifact script to enable SIMD on ARM for Faiss #1543

Conversation

naveentatikonda
Copy link
Member

Description

Update k-NN build artifact script to enable SIMD on ARM for Faiss because CentOS is supporting gcc version 9.3.x

gcc --version
gcc (GCC) 9.3.1 20200408 (Red Hat 9.3.1-2)
Copyright (C) 2019 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Issues Resolved

[List any issues this PR will resolve]

Check List

  • New functionality includes testing.
  • All tests pass
  • Commits are signed as per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@naveentatikonda
Copy link
Member Author

@peterzhuamazon Can you pls review this PR ?

Copy link

codecov bot commented Mar 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.17%. Comparing base (2b0f5a3) to head (bc9322e).

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1543      +/-   ##
============================================
+ Coverage     85.08%   85.17%   +0.09%     
  Complexity     1299     1299              
============================================
  Files           171      171              
  Lines          5295     5295              
  Branches        506      506              
============================================
+ Hits           4505     4510       +5     
+ Misses          577      572       -5     
  Partials        213      213              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@peterzhuamazon peterzhuamazon left a comment

Choose a reason for hiding this comment

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

Thanks!

@naveentatikonda naveentatikonda merged commit 2959d06 into opensearch-project:main Mar 15, 2024
65 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Mar 15, 2024
naveentatikonda added a commit that referenced this pull request Mar 15, 2024
)

(cherry picked from commit 2959d06)
Signed-off-by: Naveen Tatikonda <[email protected]>
naveentatikonda added a commit that referenced this pull request Mar 15, 2024
) (#1544)

(cherry picked from commit 2959d06)
Signed-off-by: Naveen Tatikonda <[email protected]>

Co-authored-by: Naveen Tatikonda <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.x Infrastructure Changes to infrastructure, testing, CI/CD, pipelines, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants