Sounds fine to me.

Are you sure that is the only change that needs to be made?

MR: https://gitlab.com/libeigen/eigen/-/merge_requests/453


On Sun, Mar 28, 2021 at 12:14 PM Kyle Snow <[email protected]>
wrote:

> Dear Eigen Developers:
>
> I was able to use sparse matrices of larger dimension than otherwise
> possible by changing a variable type from int to Index in the macro defined
> in line 16 of PlainObjectBase.h, which I have copied below for reference.
>
> Would you kindly consider making this change in the next Eigen release? I
> would like to keep my code in sync with the Eigen repository rather than
> use locally modified copies of it.
>
> Thank you,
> Kyle
>
> # define EIGEN_INITIALIZE_COEFFS_IF_THAT_OPTION_IS_ENABLED for(Index
> i=0;i<base().size();++i) coeffRef(i)=Scalar(0); // changed int to Index
>
> *Kyle Snow, PhD *
>
> President
> P: 614-323-0573 <(614)%20323-0573>
> polarisgeospatial.com
>
>
>

Reply via email to