Sounds fine to me.
Are you sure that is the only change that needs to be made?
On Sun, Mar 28, 2021 at 12:14 PM Kyle Snow <kyle.snow@xxxxxxxxxxxxxxxxxxxxx> 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