Trilinos: Ifpack2: UnitTestSingleProcessRILUK's tests for (alpha, beta) cases tests only default case.

Created on 14 Aug 2017  路  1Comment  路  Source: trilinos/Trilinos

@trilinos/ifpack2: The unit test calls

crsmatrix->apply (x_copy, y_copy, Teuchos::NO_TRANS, alpha, beta);

for the various cases, but not

prec.apply(y, x, mode, alpha, beta);

I believe the second is what was intended. Fix this to get coverage of these non-default cases. This was discovered in the course of PR #1594.

Ifpack2 bug

Most helpful comment

New test would have caught the RILUK::apply() bug. Will soon run checkin script.

>All comments

New test would have caught the RILUK::apply() bug. Will soon run checkin script.

Was this page helpful?
0 / 5 - 0 ratings