[custom] EIGEN_ALIGN16 bad idea
I maintainer di solito rispondono entro 4 giorni
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 20/100
- Tipo di issue
- Refactoring
- Chiarezza
- Da chiarire
- Stato di attività
- Ferma
- Stack tecnologico
- cpp
- Ambito
- computer-vision, performance
Direzione di ricerca
Inizia con le definizioni dei punti e la macro PCL_ADD_UNION_POINT4D mostrate nell’issue, quindi esamina il rapporto tra il loro uso di EIGEN_ALIGN16 e delle union e i requisiti di Eigen. Il lavoro sarà completo quando sarà stato concordato un approccio per un allineamento configurabile o rimosso che preservi il comportamento rilevante dei punti e affronti le problematiche relative all’efficienza della cache e al multithreading.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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
- Lingua principale
- C++
- Stelle
- 11.2k
- Fork
- 4.7k
- Merge medio
- 5g 8h
- PR unite (30g)
- 10
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di PointCloudLibrary/pcl
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100
PointCloudLibrary/pcl#6509 · 1 commento ·
I maintainer di solito rispondono entro 4 giorni
-
kind: bug status: triage
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
PointCloudLibrary/pcl#6395 ·
I maintainer di solito rispondono entro 4 giorni
-
[sample_consensus] Migrated the SSE and AVX implementations of countWithinDistance to RISC-V RVV 1.0Apertakind: request status: triage
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
PointCloudLibrary/pcl#6385 · 1 reazione ·
I maintainer di solito rispondono entro 4 giorni
-
2 tests failForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 25/100
PointCloudLibrary/pcl#6373 · 1 commento · 1 reazione ·
I maintainer di solito rispondono entro 4 giorni
-
kind: question kind: request
Difficoltà 4/5 3-5 giorni Idoneità per principianti 28/100
PointCloudLibrary/pcl#6370 · 2 commenti ·
I maintainer di solito rispondono entro 4 giorni
Tutte le issue di PointCloudLibrary/pcl
Issue simili
-
good first issue invalid
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 72/100
DanielJerolm/gpcc#74 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
patrick-hudson/put.gd-desktop#808 ·
I maintainer di solito rispondono entro 1 giorno
-
DPI change is not fully handled in the cascading window on WindowsForse già presa @yukawa l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
I maintainer di solito rispondono entro 4 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 77/100
dasher-project/Dasher-GTK#97 ·
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
I maintainer di solito rispondono entro 1 giorno