Skip to content

Commit

Permalink
rename instruction_set to something less ambiguous
Browse files Browse the repository at this point in the history
It is a set of instruction sets, and "instruction_set" is not great for that
  • Loading branch information
h-vetinari committed Jan 29, 2021
1 parent 68d0d0c commit 13289c4
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions faiss/python/loader.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,9 @@
import logging


def instruction_set():
def supported_instruction_sets():
"""
Returns the set of supported instruction sets, see
Returns the set of supported CPU features, see
https://github.com/numpy/numpy/blob/master/numpy/core/src/common/npy_cpu_features.h
for the list of features that this set may contain per architecture.
Expand Down Expand Up @@ -46,7 +46,7 @@ def instruction_set():
logger = logging.getLogger(__name__)

try:
has_AVX2 = "AVX2" in instruction_set()
has_AVX2 = "AVX2" in supported_instruction_sets()
if has_AVX2:
logger.info("Loading faiss with AVX2 support.")
from .swigfaiss_avx2 import *
Expand Down

0 comments on commit 13289c4

Please sign in to comment.