Skip to content

Conversation

@weslleyspereira
Copy link

Add tests for all xGGQRCS new routines.
Also, fix bugs Travis found at Reference-LAPACK#406

@codecov-commenter
Copy link

Codecov Report

Merging #3 (0e327d6) into add-xGGQRCS (8a0bef1) will increase coverage by 28.61%.
The diff coverage is 75.00%.

Impacted file tree graph

@@               Coverage Diff                @@
##           add-xGGQRCS       #3       +/-   ##
================================================
+ Coverage        53.40%   82.01%   +28.61%     
================================================
  Files             1904     1904               
  Lines           191702   191704        +2     
================================================
+ Hits            102374   157235    +54861     
+ Misses           89328    34469    -54859     
Impacted Files Coverage Δ
SRC/sggqrcs.f 74.85% <75.00%> (+45.26%) ⬆️
SRC/slapll.f 0.00% <0.00%> (-100.00%) ⬇️
SRC/slags2.f 50.68% <0.00%> (-47.95%) ⬇️
SRC/stgsja.f 69.92% <0.00%> (-23.31%) ⬇️
SRC/sggsvd3.f 79.41% <0.00%> (-17.65%) ⬇️
SRC/sggsvp3.f 90.90% <0.00%> (-8.27%) ⬇️
SRC/sorcsd2by1.f 90.94% <0.00%> (+0.37%) ⬆️
SRC/slasq4.f 96.64% <0.00%> (+0.67%) ⬆️
SRC/dlasd4.f 76.71% <0.00%> (+1.03%) ⬆️
SRC/slasv2.f 98.76% <0.00%> (+1.23%) ⬆️
... and 531 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8a0bef1...0e327d6. Read the comment docs.

christoph-conrads pushed a commit that referenced this pull request May 23, 2021
christoph-conrads pushed a commit that referenced this pull request Jun 6, 2021
Comments in xGTT02 declared that it uses norm(A) but its code implemented 1-norm of op(A).
christoph-conrads pushed a commit that referenced this pull request Jun 6, 2021
comments fixed: synced with code in xGTT02 (#3 from PR#562 discussion)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants