[custom] EIGEN_ALIGN16 bad idea
メンテナーはふだん 4 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 20/100
- issue の種類
- リファクタリング
- 明瞭さ
- 説明が足りない
- 活発さ
- 停滞
- 技術スタック
- cpp
調査の方向性
Issue に示されている point の定義とマクロ PCL_ADD_UNION_POINT4D から始め、それらでの EIGEN_ALIGN16 と union の使用が Eigen の要件とどのように関係するかを確認します。完了の条件は、設定可能なアライメント、またはアライメントの削除について合意されたアプローチがあり、関連する point の動作を維持し、キャッシュ効率とマルチスレッドに関する懸念に対処できることです。
索引モデルが issue の本文から書いたものです。
説明
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
- 主要言語
- C++
- スター
- 11.2k
- フォーク
- 4.7k
- 平均マージ
- 5日 8時間
- マージ済み PR（30日）
- 10
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
PointCloudLibrary/pcl のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
PointCloudLibrary/pcl#6509 · コメント 1 件 ·
メンテナーはふだん 4 日以内に返信
-
kind: bug status: triage
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
PointCloudLibrary/pcl#6395 ·
メンテナーはふだん 4 日以内に返信
-
[sample_consensus] Migrated the SSE and AVX implementations of countWithinDistance to RISC-V RVV 1.0オープンkind: request status: triage
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
PointCloudLibrary/pcl#6385 · リアクション 1 件 ·
メンテナーはふだん 4 日以内に返信
-
2 tests fail対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 3/5 1〜2日 初心者へのやさしさ 25/100
PointCloudLibrary/pcl#6373 · コメント 1 件 · リアクション 1 件 ·
メンテナーはふだん 4 日以内に返信
-
kind: question kind: request
難易度 4/5 3〜5日 初心者へのやさしさ 28/100
PointCloudLibrary/pcl#6370 · コメント 2 件 ·
メンテナーはふだん 4 日以内に返信
PointCloudLibrary/pcl の issue をすべて見る
似ている issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
ROCm/FastFlowLM#887 ·
メンテナーはふだん 1 日以内に返信
-
bug medium priority openmp tokenizer wrong result
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
good first issue invalid
難易度 1/5 1時間未満 初心者へのやさしさ 72/100
DanielJerolm/gpcc#74 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
patrick-hudson/put.gd-desktop#808 ·
メンテナーはふだん 1 日以内に返信
-
DPI change is not fully handled in the cascading window on Windows対応中かも @yukawa が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
メンテナーはふだん 4 日以内に返信