Instantiate SIMD Ops, not distance implementations
Los mantenedores suelen responder en 5 días
Nadie ha tomado este issue todavía.
- #199 de @copilot-swe-agent — cerrado sin fusionar
Evaluación
Este issue todavía no se ha evaluado.
Descripción
For ISA dispatching, we instantiate the distance implementations (L2Impl, IPImpl, CosineSimilarityImpl). Since there are template parameters N, Ea, Eb, AVX_AVAILABILITY, corresponding to dimensionality, element type A, element type B, and AVX availability flag, respectively, many combinations of explicit template values are required. Crucially, also one for N, the fixed-/dynamic-dimensionality support.
It is not the best place to create the AVX-specific instantiations, because only the actual compute ops benefit from ISA-optimization.
The current implementations creates a lot of code
- at the end of the distance headers (
cosine.h,euclidean.h,inner_product.h) to define the symbols asextern; and - to produce the instantiations
multi-arch/avx2.cppandmulti-arch/avx512.cpp;
all of which makes heavy use of preprocessor macros and is therefore hard to understand and debug.
Completion of #183 should allow to lower the explicit instantiations to the various compute ops (IPFloatOp, IPVNNIOp, L2FloatOp, etc.), for which we already use fixed values of N, eliminating combinatorial complexity in the instantiations.
The task is to understand if is after completion of #183 such a simplification is possible, and, if so, implement it.
AS A maintainer of SVS
I WANT TO lower the instantiation of architecture-specific functions to the actual compute operations
SO THAT I can reduce maintenance and future development cost, as well as have a more optimal solution in general.
- Lenguaje dominante
- C++
- Estrellas
- 241
- Forks
- 48
- Merge medio
- 4 d 21 h
- PR fusionados (30 d)
- 11
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de intel/ScalableVectorSearch
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
intel/ScalableVectorSearch#356 ·
Los mantenedores suelen responder en 5 días
-
Dificultad 4/5 3-5 días Aptitud para principiantes 42/100
intel/ScalableVectorSearch#469 ·
Los mantenedores suelen responder en 5 días
-
Add CI test for C API on ARMAbiertoenhancement
Dificultad 3/5 1-2 días Aptitud para principiantes 32/100
intel/ScalableVectorSearch#460 ·
Los mantenedores suelen responder en 5 días
-
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
intel/ScalableVectorSearch#438 ·
Los mantenedores suelen responder en 5 días
-
CI: tier the test matrix into PR / main / nightly to cut Actions minutesPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 32/100
intel/ScalableVectorSearch#432 · 1 comentario ·
Los mantenedores suelen responder en 5 días
Todos los issues de intel/ScalableVectorSearch
Issues similares
-
runtime: lf_f128_mod is documented as fmodq (truncating) but implements floor-based modulo; unusedAbiertointernal correctness low priority rt_lib
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 87/100
-
PriorityNotificationQueue::wakeUpAll() loses the wake-up for a thread that is not waiting yetPosiblemente ocupada @matejk la tomó hoy. Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
pocoproject/poco#5674 ·
Los mantenedores suelen responder en 2 días
-
bug docs GUI
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
sonic-pi-net/sonic-pi#3609 ·
-
Unreachable code warningAbiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
SanderMertens/flecs#2395 ·
Los mantenedores suelen responder en 1 día