Skip to content

Clarifying LAUUM documentation - #1422

Open
jprhyne wants to merge 1 commit into
Reference-LAPACK:masterfrom
jprhyne:lauum_documentation
Open

jprhyne wants to merge 1 commit into
Reference-LAPACK:masterfrom
jprhyne:lauum_documentation

Conversation

@jprhyne

@jprhyne jprhyne commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Description
Updating the documentation of LAUUM routines to state the unused strictly triangular component is not referenced

Closes #1415
Checklist

  • The documentation has been updated.
  • If the PR solves a specific issue, it is set to be closed on merge.

Comment thread SRC/clauum.f
*> overwritten with the upper triangle of the product U * U**H;
*> if UPLO = 'L', the lower triangle of A is overwritten with
*> the lower triangle of the product L**H * L.
*> overwritten with the upper triangle of the product U * U**T,

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the unrelated symbol change from H to T intentional ? H is used in section "Purpose" right above the change, so this would introduce an inconsistency in the documentation

@codecov

codecov Bot commented Sep 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.36%. Comparing base (a6c6e74) to head (a6da54c).
✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1422   +/-   ##
=======================================
  Coverage   69.36%   69.36%           
=======================================
  Files        6122     6122           
  Lines      486711   486711           
  Branches    23268    23268           
=======================================
  Hits       337584   337584           
  Misses     148689   148689           
  Partials      438      438           
Components Coverage Δ
BLAS 97.94% <ø> (ø)
CBLAS 96.98% <ø> (ø)
LAPACK 82.38% <ø> (ø)
LAPACKE 2.17% <ø> (ø)
TMGLIB 55.69% <ø> (ø)
BLAS testing 88.33% <ø> (ø)
CBLAS testing 89.63% <ø> (ø)
LAPACK testing 82.20% <ø> (ø)
LAPACKE testing ∅ <ø> (∅)
Files with missing lines Coverage Δ
SRC/clauum.f 83.87% <ø> (ø)
SRC/dlauum.f 83.87% <ø> (ø)
SRC/slauum.f 83.87% <ø> (ø)
SRC/zlauum.f 83.87% <ø> (ø)

Continue to review full report in Codecov by Harness.

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

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.

Unused Triangular component in LAUUM

2 participants