Bug 63 - Tests : original LAPACK testsuite
Tests : original LAPACK testsuite
Status: RESOLVED FIXED
Product: Eigen
Classification: Unclassified
Component: Tests
unspecified
All All
: --- Unknown
Assigned To: Nobody
:
Depends on:
Blocks: 3.2
  Show dependency treegraph
 
Reported: 2010-10-16 04:46 UTC by Benoit Jacob
Modified: 2013-07-06 15:10 UTC (History)
4 users (show)



Attachments

Description Benoit Jacob 2010-10-16 04:46:59 UTC
The idea is that once we have a LAPACK library implemented using Eigen, we should be able to run the original LAPACK testsuite directly against it.
Comment 1 Gael Guennebaud 2013-03-20 18:08:43 UTC
Current status:
 - PartialPivLU: getrf, getrs
- LLT: potrf, potrs
- SelfAdjointEigenSolver: syev

For 3.2 would be nice to have at least:
 - HouseholderQR
 - ColPivHouseholderQR
 - JacobiSVD

Then:
 - Tridiagonalization
 - Hessenberg
 - EigenSolver
Comment 2 Gael Guennebaud 2013-03-20 18:10:26 UTC
Currently, the LAPACK testsuite is not shipped with Eigen, and so it is rarely executed. Would it make sense to include it in Eigen's repo? Or perhaps have a cmake script downloading it automatically when doing make lapack_testsuite?
Comment 3 Christoph Hertzberg 2013-07-01 14:25:28 UTC
(In reply to comment #2)
> Currently, the LAPACK testsuite is not shipped with Eigen, and so it is rarely
> executed. Would it make sense to include it in Eigen's repo? Or perhaps have a
> cmake script downloading it automatically when doing make lapack_testsuite?

Either of these would definitely be nice. I would slightly prefer the download script option to keep the release small. I assume the majority of users don't even run the Eigen test suite at all.
Comment 4 Gael Guennebaud 2013-07-06 15:10:55 UTC
https://bitbucket.org/eigen/eigen/commits/8565a9c8a9e5/
Changeset:   8565a9c8a9e5
User:        ggael
Date:        2013-07-06 15:08:42
Summary:     Bug 63: add lapack unit tests. They are automatically downloaded and configured if EIGEN_ENABLE_LAPACK_TESTS is ON.

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