|
@@ -100,15 +100,19 @@ We really heavily on Eigen. Nearly all inputs and outputs are Eigen matrices of
|
|
|
some kind. However, we currently _only_ support Eigen's default column-major
|
|
|
ordering. That means, we **do not** expect our code to work for matrices using
|
|
|
the `Eigen::RowMajor` flag. If you can, change definitions like:
|
|
|
+
|
|
|
```cpp
|
|
|
Eigen::Matrix<double, Eigen::Dynamic, 3, Eigen::RowMajor> A;
|
|
|
```
|
|
|
+
|
|
|
to
|
|
|
+
|
|
|
```cpp
|
|
|
Eigen::Matrix<double, Eigen::Dynamic, 3, Eigen::ColMajor> A;
|
|
|
// or simply
|
|
|
Eigen::Matrix<double, Eigen::Dynamic, 3> A;
|
|
|
```
|
|
|
+
|
|
|
We hope to fix this, or at least identify which functions are safe (many of
|
|
|
them probably work just fine). This requires setting up unit testing, which is
|
|
|
a major _todo_ for our development.
|