|
@@ -118,17 +118,19 @@ caution.</p>
|
|
|
<p>We really heavily on Eigen. Nearly all inputs and outputs are Eigen matrices of
|
|
|
some kind. However, we currently <em>only</em> support Eigen’s default column-major
|
|
|
ordering. That means, we <strong>do not</strong> expect our code to work for matrices using
|
|
|
-the <code>Eigen::RowMajor</code> flag. If you can, change definitions like:
|
|
|
-<code>cpp
|
|
|
-Eigen::Matrix<double, Eigen::Dynamic, 3, Eigen::RowMajor> A;
|
|
|
-</code>
|
|
|
-to
|
|
|
-<code>cpp
|
|
|
-Eigen::Matrix<double, Eigen::Dynamic, 3, Eigen::ColMajor> A;
|
|
|
+the <code>Eigen::RowMajor</code> flag. If you can, change definitions like:</p>
|
|
|
+
|
|
|
+<pre><code class="cpp">Eigen::Matrix<double, Eigen::Dynamic, 3, Eigen::RowMajor> A;
|
|
|
+</code></pre>
|
|
|
+
|
|
|
+<p>to</p>
|
|
|
+
|
|
|
+<pre><code class="cpp">Eigen::Matrix<double, Eigen::Dynamic, 3, Eigen::ColMajor> A;
|
|
|
// or simply
|
|
|
Eigen::Matrix<double, Eigen::Dynamic, 3> A;
|
|
|
-</code>
|
|
|
-We hope to fix this, or at least identify which functions are safe (many of
|
|
|
+</code></pre>
|
|
|
+
|
|
|
+<p>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 <em>todo</em> for our development.</p>
|
|
|
|