GETRF: Integrate IPIV array in the interface and testings
All threads resolved!
All threads resolved!
-
Fix some issues in the actual getrf testing:
- seed incorrectly initialized
- Diagonal not dominant by default => Since LU NoPiv, diagonal dominant is the default for the generation.
- Integrate the swap locally on the home node 0 if needed.
-
Add the IPIV to all getrf interfaces
This is part of the MR !378 (merged) and is a follow-up of !392 (merged)
Merge request reports
Activity
changed milestone to %Chameleon 1.3.0
added API label
requested review from @fpruvost
assigned to @faverge
added 6 commits
-
ae95a471 - 1 commit from branch
solverstack:master
- cc4c9499 - desc: allows descriptors to store matrices of integers
- ce7b73c3 - testing/getrf: Fix incorrect parameters in the testing
- 89bbca9f - zgetrf: Add IPIV to the prototype of the function
- bb7f959c - zgetrf: Add a function using map to initialize the IPIV array (may be unused in the future)
- f95be9ac - testing/getrf: Integrate the IPIV in the interface to call getrf algorithm
Toggle commit list-
ae95a471 - 1 commit from branch
added 8 commits
-
f95be9ac...2c450498 - 3 commits from branch
solverstack:master
- 0b837738 - desc: allows descriptors to store matrices of integers
- e4861ed5 - testing/getrf: Fix incorrect parameters in the testing
- abb9f173 - zgetrf: Add IPIV to the prototype of the function
- dfee81a1 - zgetrf: Add a function using map to initialize the IPIV array (may be unused in the future)
- 144e6e10 - testing/getrf: Integrate the IPIV in the interface to call getrf algorithm
Toggle commit list-
f95be9ac...2c450498 - 3 commits from branch
Ping @kuhn
- Resolved by Mathieu Faverge
Can I please have some review on this one ? I would like to push some other PR that relies on this one. Thanks :)
Edited by Mathieu Faverge
mentioned in commit fc1dc041
Please register or sign in to reply