[geometry/common/class] Implement Matrix<r, c, T>

[Luzhiled]

概要 / Overview

幾何用に特殊化された行列の実装

関連 issue, PR / Related issues, PRs

• 145

• 147

TODO

• [x] 実装
• [x] ドキュメント作成

CI 関連項目

verify-check / verify

(cd $(git rev-parse --show-toplevel) && oj-verify all)

clang-format

(cd $(git rev-parse --show-toplevel) && clang-format -i $(find src/ test/ unit-test/ -name "*.hpp" -or -name "*.cpp"))

[Luzhiled]

あんまり何も考えていなかったけど array をネストしない場合は mat[a][b] みたいな書き方ができないか

[Luzhiled]

内部にもう一つクラス作ったりして参照返せばええな

[Luzhiled]

爆発して死んだので .at(i, j) を書いて終わります　〜完〜

[Luzhiled]

point vector のためにも matrix を部分特殊化したくて、そうなると基底クラスを作ったほうがよさそうだな

[Luzhiled]

scalar * Matrix が動かねえかも

[Luzhiled]

代入演算子周りがうまいこといってなくて、Matrix に InnerMatrix を代入したりということをしたい (as コピーするだけなので) んだけど、なんかずっと動かない

[ei1333]

やはり行列ではないか

[ei1333]

    Matrix(const InternalMatrix< r, c, T > &m) {
       this=m.らずひるど;
    }

みたいに書くとかではないのかな

[Luzhiled]

んーやってみる

[Luzhiled]

@ei1333 おい　助けてくれ　main.cpp のコンパイルを通すことはできますか

[Luzhiled]

TODO: internal を別ファイルに分ける(?)

[Luzhiled]

最終的に std::array のコピー周りでも死んだりして困ったので std::vector を使うことにしました まあサイズに変更があったりするわけじゃないから多分速度で問題になることはないんじゃないかな

[Luzhiled]

行列積の実装は必要になってからでいいかな

[Luzhiled]

TODO

• [x] ファイルの切り分け
• [ ] ベクトルのコンストラクタとして (x: 1, y: 2, z: 3) みたいなのを用意する
• [x] 不要な行の削除
• [x] main.cpp 消す
• [x] InternalMatrix -> Mat

[Luzhiled]

geometry
├── 2d
│   ├── common
│   ├── r2
│   └── z2
└── common
    └── class
        ├── column-vector.hpp
        ├── internal
        │   └── matrix.hpp
        └── matrix.hpp

にしようかな？

[Luzhiled]

整数か整数じゃないかよりも何次元かのほうが大事だと思っているが、まあこれには諸説あるな

[Luzhiled]

• 147

[Luzhiled]

namespace を

• luz::geometry::common
• luz::geometry::r2
• luz::geometry::z2 に分けます

[Luzhiled]

2d
├── common
│   └── class
│       └── vector-2d.hpp
├── r2
│   └── class
│       └── vector-2d.hpp
└── z2
    └── class
        └── vector-2d.hpp
common
└── class
    ├── column-vector.hpp
    ├── internal
    │   └── matrix.hpp
    └── matrix.hpp

現状です

[Luzhiled]

eps どこに置くかとか決めないと r2::Vector2d の operator== は実装できないのでこいつは一旦常に false を返すようにしています この PR ではないかな

[Luzhiled]

飽きてきた　merge していい？