Fix issues on new_testing
Try to fix issues reported by @furmento
- Enable the possibility to give both gemm and [sdcz]gemm as operator
- Add the operation option in the
.in
files - Fix issue when calling
./dnew-testing -h
- Move
CHAMELEON_My_Mpi_Rank()
as deprecated in profit ofCHAMELEON_Comm_rank()
- Add the missing
-l --niter
option to set the number of iteration of each test. - Fix issue with the conversions rules that were not converting alpha/beta parameters, as well as read/print functions
Merge request reports
Activity
added Feature Tests/CTest labels
added 3 commits
-
7d441eb9...41060486 - 2 commits from branch
solverstack:master
- b61201cf - Enable both [sdcz]xxxx and xxxx as operations, and to declare the operator withinh the input file
-
7d441eb9...41060486 - 2 commits from branch
added 1 commit
- 3feb9ade - Fix conversion rules, and issue with the niter
- About the return parameter, it is not printed when no check is performed to avoid confusion. We could print SUCCESS/FAILURE when check is not enabled to say the testing returned a correct value, but I think this could be confusing for users. We would think that some checks have been performed when it is not the case. (@ltaief, @agullo, @thibault, @fpruvost what do you think ?)
This other question is do you really need this column ? or you suggest we should print that no check has been performed ?
- The niter parameter has been restored in this PR.
Edited by Mathieu FavergeNo that's the mistake, the number of columns will vary. For now, not all testings have it, but the checks are adding information like the norm of the residual, the norm of the input matrix, the value of the scaled residual that is used to check the validity, ... You can try the QR testings, and the LU testings. You'll see that the number of columns will vary with and without check.
The idea is to detect the columns by their header, not by their place in the print :)
so let's merge!
Edited by Nathalie Furmentomentioned in commit f2c3f027
mentioned in commit debfb49a
Fixed in commit debfb49a