[custom] EIGEN_ALIGN16 bad idea
Los mantenedores suelen responder en 4 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 20/100
- Tipo de issue
- Refactorización
- Claridad
- Necesita aclaración
- Estado de actividad
- Estancado
- Stack tecnológico
- cpp
- Área
- computer-vision, performance
Línea de trabajo
Comienza con las definiciones de puntos y la macro PCL_ADD_UNION_POINT4D mostradas en el issue, y luego revisa cómo se relaciona su uso de EIGEN_ALIGN16 y de unions con los requisitos de Eigen. El trabajo estaría terminado cuando exista un enfoque acordado para una alineación configurable o eliminada que preserve el comportamiento relevante de los puntos y aborde las cuestiones de eficiencia de la caché y multithreading.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
the use of EIGEN_ALIGN16
seems to be a very bad idea. In particular for cache efficiency and multithreading.
I would either eliminate that and all similar ones or substitute them with something configurable at compile time
giving also the option to use, compile-time-configurable directives like
alignas(std::hardware_destructive_interference_size) (and here the topic of moving to C++17 goes on)
why are the definitions of the points so relying on alignment and they make use of union??
isn't this too hard coding and anti-OOP?
#define PCL_ADD_UNION_POINT4D
union EIGEN_ALIGN16 {
float data[4];
struct {
float x;
float y;
float z;
};
};
i see it is probably "required" by Eigen
- Lenguaje dominante
- C++
- Estrellas
- 11.2k
- Forks
- 4.7k
- Merge medio
- 5 d 8 h
- PR fusionados (30 d)
- 10
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 PointCloudLibrary/pcl
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
PointCloudLibrary/pcl#6509 · 1 comentario ·
Los mantenedores suelen responder en 4 días
-
kind: bug status: triage
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
PointCloudLibrary/pcl#6395 ·
Los mantenedores suelen responder en 4 días
-
[sample_consensus] Migrated the SSE and AVX implementations of countWithinDistance to RISC-V RVV 1.0Abiertokind: request status: triage
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
PointCloudLibrary/pcl#6385 · 1 reacción ·
Los mantenedores suelen responder en 4 días
-
2 tests failPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 25/100
PointCloudLibrary/pcl#6373 · 1 comentario · 1 reacción ·
Los mantenedores suelen responder en 4 días
-
kind: question kind: request
Dificultad 4/5 3-5 días Aptitud para principiantes 28/100
PointCloudLibrary/pcl#6370 · 2 comentarios ·
Los mantenedores suelen responder en 4 días
Todos los issues de PointCloudLibrary/pcl
Issues similares
-
good first issue invalid
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
DanielJerolm/gpcc#74 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
patrick-hudson/put.gd-desktop#808 ·
Los mantenedores suelen responder en 1 día
-
DPI change is not fully handled in the cascading window on WindowsPosiblemente ocupada @yukawa la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 4 días
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
enhancement Ready for final check
Dificultad 2/5 1-3 horas Aptitud para principiantes 79/100
mpogue2/SquareDesk#1818 · 1 comentario ·