Rip out the complete test for eigenValuesNonSym and replace it by the Rosser...
Rip out the complete test for eigenValuesNonSym and replace it by the Rosser matrix test from fieldmatrixtest.cc The old test was complete nonsense: an implementation of some sort of strange undocumented shape function set that happened to need eigenvalues for something. Lots of undocumented code, and no checks at all whether the values obtained by eigenValuesNonSym actually make sense. Only, for the particular choice of parameters used, eigenValuesNonSym was never called. Yes, you read this correctly: these hundreds of lines of code didn't actually test anything. The new test is copied form fieldmatrixtest.cc and adapted for DynamicMatrix types. It computes eigenvalues of a given 8x8 matrix and compares with the known results. Unfortunately, the test matrix is only symmetric. Tests with more general matrices are left for another day.
Please register or sign in to comment