From fbd4e08cd4c4772fba0904d05b930cb349146958 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?S=C3=A9bastien=20Villemot?= <sebastien.villemot@ens.fr>
Date: Mon, 13 Aug 2012 15:08:14 +0200
Subject: [PATCH] Document why I don't use Eigen's QR decomposition
---
mex/sources/estimation/libmat/QRDecomposition.hh | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/mex/sources/estimation/libmat/QRDecomposition.hh b/mex/sources/estimation/libmat/QRDecomposition.hh
index 30e03eb80e..ec5d98d1d3 100644
--- a/mex/sources/estimation/libmat/QRDecomposition.hh
+++ b/mex/sources/estimation/libmat/QRDecomposition.hh
@@ -23,9 +23,12 @@
using namespace Eigen;
-
#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
{
private:
--
GitLab