This bugzilla service is closed. All entries have been migrated to https://gitlab.com/libeigen/eigen
Bug 635 - Adding an isCompressed() property to MappedSparseMatrix
Summary: Adding an isCompressed() property to MappedSparseMatrix
Status: RESOLVED FIXED
Alias: None
Product: Eigen
Classification: Unclassified
Component: Sparse (show other bugs)
Version: 3.2
Hardware: All All
: Normal Unknown
Assignee: Nobody
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-07-24 20:36 UTC by adam.p.harrison
Modified: 2019-12-04 12:31 UTC (History)
1 user (show)



Attachments
Patch that implements what's proposed in the bug report (52 bytes, patch)
2013-07-24 20:39 UTC, adam.p.harrison
adam.p.harrison: review? (gael.guennebaud)
Details | Diff

Description adam.p.harrison 2013-07-24 20:36:17 UTC
The new SparseLU module checks whether the matrix in question is already in compressed mode. Unfortunately, when decomposing a MappedSparseMatrix, a compile error will occur as the MappedSparseMatrix does not have a isCompressed() function.

If I'm not mistaken, the MappedSparseMatrix is always wrapping compressed data, so it should be correct to add a isCompressed() function that always return true.
Comment 1 adam.p.harrison 2013-07-24 20:39:59 UTC
Created attachment 372 [details]
Patch that implements what's proposed in the bug report

I just added an isCompressed() member function in MappedSparseMatrix that always returns true. I believe that is consistent with the design and intentions of MappedSparseMatrix.
Comment 2 Gael Guennebaud 2013-08-02 11:12:31 UTC
Thank you.

https://bitbucket.org/eigen/eigen/commits/773a8bf23df6/
Changeset:   773a8bf23df6
User:        ggael
Date:        2013-08-02 11:11:21
Summary:     Fix bug 635: add isCompressed to MappedSparseMatrix for compatibility
Affected #:  1 file

https://bitbucket.org/eigen/eigen/commits/9343ccc56542/
Changeset:   9343ccc56542
Branch:      3.2
User:        ggael
Date:        2013-08-02 11:11:21
Summary:     Fix bug 635: add isCompressed to MappedSparseMatrix for compatibility
(transplanted from 773a8bf23df6a7ca7dd0b17e3bb90446c683d7e9)
Comment 3 Nobody 2019-12-04 12:31:14 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to gitlab.com'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: https://gitlab.com/libeigen/eigen/issues/635.

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