Skip to content

Commit

Permalink
Fix tolerance for HEGVDX tests (#879)
Browse files Browse the repository at this point in the history
As part of PR #810, the tolerances for `sygvdx` and `hegvdx` tests were
set too tight, which made 3 of `hegvdx` daily lapack tests fail.

This PR changes the tolerance to allow all tests to pass on all architectures.
  • Loading branch information
jmachado-amd authored Jan 14, 2025
1 parent b94dba0 commit adeb534
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions clients/common/lapack/testing_sygvdx_hegvdx.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -1108,9 +1108,9 @@ void testing_sygvdx_hegvdx(Arguments& argus)
}

// validate results for rocsolver-test
// using 4 * n * machine_precision as tolerance
// using 5 * n * machine_precision as tolerance
if(argus.unit_check)
ROCSOLVER_TEST_CHECK(T, max_error, 4 * n);
ROCSOLVER_TEST_CHECK(T, max_error, 5 * n);

// output results for rocsolver-bench
if(argus.timing)
Expand Down

0 comments on commit adeb534

Please sign in to comment.