Commit fbd4e08c authored by Sébastien Villemot's avatar Sébastien Villemot
Browse files

Document why I don't use Eigen's QR decomposition

parent a4e40340
...@@ -23,9 +23,12 @@ ...@@ -23,9 +23,12 @@
using namespace Eigen; using namespace Eigen;
#include <dynlapack.h> #include <dynlapack.h>
/* Note that Eigen provides built-in QR decomposers (classes *HouseHolderQR).
However, we need to have our own because we need the ability to
left-multiply arbitrary matrices by Q, and this is not offered in a
convenient way by the Eigen classes. */
class QRDecomposition class QRDecomposition
{ {
private: private:
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment