| [eigen] special matrix classes: template parameters |
[ Thread Index |
Date Index
| More lists.tuxfamily.org/eigen Archives
]
- To: eigen <eigen@xxxxxxxxxxxxxxxxxxx>
- Subject: [eigen] special matrix classes: template parameters
- From: Benoit Jacob <jacob.benoit.1@xxxxxxxxx>
- Date: Sat, 14 Nov 2009 10:42:51 -0500
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:received:date:message-id:subject :from:to:content-type; bh=hK32ZMqY2asxmX3WHbF5Eysy3vTV9E6vbd0Tq6TH2tM=; b=rdjd6VgAOkC13e3nLjqNCrMWoBBwPAw2XdTmclsH/G1QzLwbGmfQ+uJbkA9+kf+gEf NQpGg3r0/pdhcSPflChmaCVpaLSPVaqZBnJv6lPTcmAwpPdt76t97q0gOzm0ZZYydiPq 6CVT8qBgWuA4VUCAjSFnFnc8aNsnBlVwsh6xs=
- Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:date:message-id:subject:from:to:content-type; b=KDhF4gdXOV9l42oKYKqfmLE2uufKxBJOoZUq6a5YjG16lyI/j0T9bGlpPhqoeVkYId l53hY8z0LqU/uSFXKGHCdVVAdyARgBqtEWmMgIZv0XaZMiaFxM90UuTZLkZNMBZc9rES MvhpQ81e7AYCxb1VwfFR3Gq45wgecvAAnlYec=
Hi,
Currently the BandMatrix class takes Scalar, Rows,Cols template
parametes that could be replaced by a single parameter DenseMatrixType
representing the return value of toDense(). Not only this would allow
to simplify the template parameters considerably, this would also be
more future proof (it could read more data from the DenseMatrixType),
and would allow to benefit from the MaxRowsAtCompileTime notion (for
dynamic-size-inside-fixed-size-storage --- that is required to ensure
"no malloc with fixed-size objects" in complicated use cases).
So OK to let it take a DenseMatrixType template parameter instead? And
in the Options, the RowMajor stuff would also move there, so that's a
further simplification.
Benoit