This bugzilla service is closed. All entries have been migrated to
Bug 629 - eliminate redundant counter & fix rowmajor version of reserve()
Summary: eliminate redundant counter & fix rowmajor version of reserve()
Status: NEW
Alias: None
Product: Eigen
Classification: Unclassified
Component: Sparse (show other bugs)
Version: 3.1
Hardware: All All
: Normal enhancement
Assignee: Nobody
Depends on:
Reported: 2013-07-14 19:53 UTC by cDc
Modified: 2019-12-04 12:28 UTC (History)
0 users


Description cDc 2013-07-14 19:53:33 UTC
In SparseMatrix.h, function reserveInnerVectors90, line 294, change:

        Index count = 0;
        for(Index j=0; j<m_outerSize; ++j)
          newOuterIndex[j] = count;
          count += reserveSizes[j] + (m_outerIndex[j+1]-m_outerIndex[j]);
          totalReserveSize += reserveSizes[j];


        for(Index j=0; j<m_outerSize; ++j)
          newOuterIndex[j] = totalReserveSize;
          totalReserveSize += reserveSizes[j];

Please enhance this function to support rowmajor matrices as well.
Comment 1 Nobody 2019-12-04 12:28:23 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance:

Note You need to log in before you can comment on or make changes to this bug.