Skip to content

Add docs for matrix functions and exponential#251

Merged
lkdvos merged 7 commits into
QuantumKitHub:mainfrom
sanderdemeyer:docs
Jul 2, 2026
Merged

Add docs for matrix functions and exponential#251
lkdvos merged 7 commits into
QuantumKitHub:mainfrom
sanderdemeyer:docs

Conversation

@sanderdemeyer

Copy link
Copy Markdown
Contributor

Add documentation for general matrix functions, specifically for exponential.

Add documentation for general matrix functions, specifically for `exponential`.
@codecov

codecov Bot commented Jun 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@lkdvos lkdvos added the documentation Improvements or additions to documentation label Jun 30, 2026

@lkdvos lkdvos left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for getting this started! Left some small comments/suggestions, but overall this looks good!

Comment thread docs/src/user_interface/matrix_functions.md Outdated

```@autodocs; canonical=false
Modules = [MatrixAlgebraKit]
Filter = t -> t isa Type && (t <: MatrixAlgebraKit.MatrixFunctionViaLA || t <: MatrixAlgebraKit.MatrixFunctionViaEig || t <: MatrixAlgebraKit.MatrixFunctionViaEigh)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think here we either want to add a union type for matrixfunction algorithms, or just replace this with a @docs block, it is probably overkill to search through all algorithms and then only filter for 3 specific ones, since in that case we can just manually list them :)

| [`SVDViaPolar`](@ref) | SVD | `fixgauge`, `tol` |
| [`PolarViaSVD`](@ref) | polar | positional `svd_alg` argument |
| [`PolarNewton`](@ref) | polar | `maxiter`, `tol` |
| [`MatrixFunctionViaLA`](@ref) | exponential | |

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do we want to specifically separate out the matrix functions from the decompositions? It might be more clear to keep these separate

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

That's a good idea. A suggestion on how to do this is in the latest commit.

Comment thread docs/src/user_interface/matrix_functions.md Outdated
Comment thread docs/src/user_interface/matrix_functions.md Outdated
sanderdemeyer and others added 2 commits July 1, 2026 10:40
Co-authored-by: Jutho <Jutho@users.noreply.github.com>
Co-authored-by: Jutho <Jutho@users.noreply.github.com>
@lkdvos lkdvos merged commit 11096c1 into QuantumKitHub:main Jul 2, 2026
86 of 91 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants