Skip to content

Conversation

@ACSimon33
Copy link
Contributor

@ACSimon33 ACSimon33 commented Oct 6, 2022

Description

Following up on #722, I've implemented the LAPACKE_?langb interfaces. I've used the same trick as in LAPACKE_?lange where you interpret the matrix as transposed in row-major layout. Although the matrix is in band layout here, it works because it is a square matrix (correct me if I'm wrong). kl and ku are swapped in the row-major case due to the implicit transposition.

Does the documentation update itself via oxygen?

Closes #722

Checklist

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

@langou
Copy link
Contributor

langou commented Oct 6, 2022

Following up on #722, I've implemented the LAPACKE_?langb interfaces. I've used the same trick as in LAPACKE_?lange where you interpret the matrix as transposed in row-major layout. Although the matrix is in band layout here, it works because it is a square matrix (correct me if I'm wrong). kl and ku are swapped in the row-major case due to the implicit transposition.

Well done!

Does the documentation update itself via [d]oxygen?

Good question. No, it does not update itself. When we release the new release (probably 3.11), then we (@julielangou) will update the documentation. We do not have any automagic way to update the doxygen documentation from the GitHub repo at each merge.

@langou langou merged commit 64023f5 into Reference-LAPACK:master Oct 6, 2022
@ACSimon33 ACSimon33 deleted the LAPACKE_xlangb branch October 6, 2022 21:48
@julielangou julielangou added this to the LAPACK 3.11.0 milestone Nov 12, 2022
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.

Why is there no LAPACKE_?langb interface?

4 participants