Ed Anderson Bug Fix - Jan 26 2015
authorjulie <julielangou@users.noreply.github.com>
Fri, 30 Jan 2015 07:26:22 +0000 (07:26 +0000)
committerjulie <julielangou@users.noreply.github.com>
Fri, 30 Jan 2015 07:26:22 +0000 (07:26 +0000)
commit9e3dc66dbaa792145e53273228bfbaaa4d26d585
tree94e36771ae31c933189265981d3a8bec214aee1d
parent93ccfe64e2f49cac82a8bd87c61dcab3865575b5
Ed Anderson Bug Fix - Jan 26 2015

In TESTING/LIN, the test routines xchkqr kept track of the number of tests performed in the variable NT, which could be 2, 6, or 7.  Recently another test was added at the end and the loop that checks for failing tests was changed to run from 1 to NT to 1 to NTESTS, including tests that were never performed.  As a result, with THRESH = 0., the number of tests failing exceeds the number of tests performed (for example, with n = 1 as the only N value and THRESH = 0.0 you get “604 out of 514 tests failed to pass the threshold”.)  The right way to fix this would be to renumber the tests so that the new test is test #3 and the loop can run from 1 to NT again, but I just changed the count of tests performed from NT to NTESTS.  For consistency between schkqr and schkrq and other test routines in the QR family, it would have been better for the new tests to go in a separate test module.
TESTING/LIN/cchkqr.f
TESTING/LIN/dchkqr.f
TESTING/LIN/schkqr.f
TESTING/LIN/zchkqr.f