Skip to content

Commit f044228

Browse files
committed
apply suggestions
1 parent a2b7e9a commit f044228

4 files changed

Lines changed: 3 additions & 8 deletions

File tree

docs/src/user_interface/decompositions.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -270,7 +270,7 @@ true
270270
## Null Spaces
271271

272272
Similarly, it can be convenient to obtain an orthogonal basis for the kernel or cokernel of a matrix.
273-
These are the complements of the coimage and image, respectively, and can be computed using the [`left_null`](@ref) and [`right_null`](@ref) functions.
273+
These are the orthogonal complements of the coimage and image, respectively, and can be computed using the [`left_null`](@ref) and [`right_null`](@ref) functions.
274274

275275
### Overview
276276

src/algorithms.jl

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,6 @@ end
5555

5656
# Algorithm traits
5757
# ----------------
58-
5958
"""
6059
does_truncate(alg::AbstractAlgorithm) -> Bool
6160

src/implementations/orthnull.jl

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ copy_input(::typeof(right_null), A) = copy_input(lq_null, A) # do we ever need a
77

88
check_input(::typeof(left_orth!), A, VC, alg::AbstractAlgorithm) =
99
check_input(left_orth!, A, VC, left_orth_alg(alg))
10-
1110
check_input(::typeof(left_orth!), A, VC, alg::LeftOrthViaQR) =
1211
check_input(qr_compact!, A, VC, alg.alg)
1312
check_input(::typeof(left_orth!), A, VC, alg::LeftOrthViaPolar) =
@@ -16,7 +15,6 @@ check_input(::typeof(left_orth!), A, VC, alg::LeftOrthViaSVD) = nothing
1615

1716
check_input(::typeof(right_orth!), A, CVᴴ, alg::AbstractAlgorithm) =
1817
check_input(right_orth!, A, CVᴴ, right_orth_alg(alg))
19-
2018
check_input(::typeof(right_orth!), A, VC, alg::RightOrthViaLQ) =
2119
check_input(lq_compact!, A, VC, alg.alg)
2220
check_input(::typeof(right_orth!), A, VC, alg::RightOrthViaPolar) =
@@ -39,7 +37,6 @@ check_input(::typeof(right_null!), A, Nᴴ, alg::RightNullViaSVD) = nothing
3937
# -------
4038
initialize_output(::typeof(left_orth!), A, alg::AbstractAlgorithm) =
4139
initialize_output(left_orth!, A, left_orth_alg(alg))
42-
4340
initialize_output(::typeof(left_orth!), A, alg::LeftOrthViaQR) =
4441
initialize_output(qr_compact!, A, alg.alg)
4542
initialize_output(::typeof(left_orth!), A, alg::LeftOrthViaPolar) =
@@ -48,7 +45,6 @@ initialize_output(::typeof(left_orth!), A, alg::LeftOrthViaSVD) = nothing
4845

4946
initialize_output(::typeof(right_orth!), A, alg::AbstractAlgorithm) =
5047
initialize_output(right_orth!, A, right_orth_alg(alg))
51-
5248
initialize_output(::typeof(right_orth!), A, alg::RightOrthViaLQ) =
5349
initialize_output(lq_compact!, A, alg.alg)
5450
initialize_output(::typeof(right_orth!), A, alg::RightOrthViaPolar) =

src/interface/orthnull.jl

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ the chosen decomposition type. By default, the supported kinds are:
6060
and is roughly equivalent to:
6161
```julia
6262
V, S, C = svd_trunc(A; trunc, kwargs...)
63-
C = S * C
63+
C = lmul!(S, C)
6464
```
6565
6666
### `alg::AbstractAlgorithm`
@@ -141,7 +141,7 @@ the chosen decomposition type. By default, the supported kinds are:
141141
and is roughly equivalent to:
142142
```julia
143143
C, S, Vᴴ = svd_trunc(A; trunc, kwargs...)
144-
C = C * S
144+
C = rmul!(C, S)
145145
```
146146
147147
### `alg::AbstractAlgorithm`

0 commit comments

Comments
 (0)