lapack icon indicating copy to clipboard operation
lapack copied to clipboard

Fix error on LAPACKE_*tpmqrt_work for ROW MAJOR matrices

Open weslleyspereira opened this issue 3 years ago • 1 comments

Fixes #404

Description There are many bugs in LAPACKE_*tpmqrt_work routines for ROW MAJOR matrices, as pointed out by @haldaas in #404.

Checklist

  • [x] Fix LAPACKE_dtpmqrt_work
  • [ ] Fix the remaining LAPACKE_*tpmqrt_work whenever my solution is validated for the double algorithm
  • [ ] Fix similar issues in other methods (if someone identifies it)

weslleyspereira avatar Apr 22 '21 13:04 weslleyspereira

Codecov Report

Base: 82.37% // Head: 82.37% // No change to project coverage :thumbsup:

Coverage data is based on head (1c33cda) compared to base (4b3c7c2). Patch has no changes to coverable lines.

:exclamation: Current head 1c33cda differs from pull request most recent head 64a0db9. Consider uploading reports for the commit 64a0db9 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #540   +/-   ##
=======================================
  Coverage   82.37%   82.37%           
=======================================
  Files        1894     1894           
  Lines      190681   190681           
=======================================
  Hits       157067   157067           
  Misses      33614    33614           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

codecov[bot] avatar Apr 22 '21 13:04 codecov[bot]

Thanks a lot for reviewing this PR, @angsch! I applied all your fixes in this last commit.

I will now apply the changes to the other LAPACKE_*tpmqrt_work.

weslleyspereira avatar Oct 10 '22 19:10 weslleyspereira