Re: [eigen] [Review] Pull request 66, Huge Tensor module improvements

[ Thread Index | Date Index | More Archives ]

On Mon, Jun 16, 2014 at 11:51 AM, Christian Seiler <christian@xxxxxxxx> wrote:
I generally like your new names, but I don't think I can reuse
DenseBase at the moment as a base class, since EigenBase,
DenseCoeffsBase and DenseBase currently only support two indices.
That's why I suggested copying but not reusing Eigen's own class

I don't see why this is a problem: MultiDimBase would add the needed multi-dim accessor overloads.


Mail converted by MHonArc 2.6.19+