Skip to content

Commit 60e83ed

Browse files
authored
Apply suggestions from code review
Co-authored-by: Oscar Dowson <odow@users.noreply.github.com>
1 parent db4a366 commit 60e83ed

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

Project.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ MathOptInterfaceLDLFactorizationsExt = "LDLFactorizations"
2828

2929
[compat]
3030
BenchmarkTools = "1"
31-
CliqueTrees = "1.17"
31+
CliqueTrees = "1.13"
3232
CodecBzip2 = "0.6, 0.7, 0.8"
3333
CodecZlib = "0.6, 0.7"
3434
ForwardDiff = "1"

src/Utilities/sparse_square_root.jl

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,15 +44,15 @@ If unsuccessful, this function returns `nothing`.
4444
4545
## Extensions
4646
47-
By default, this function attempts to use a `LinearAlgebra.cholesky`. If that
47+
By default, this function attempts to use a Cholesky decomposition. If that
4848
fails, it may optionally use various extension packages.
4949
5050
These extension packages must be loaded before calling `compute_sparse_sqrt`.
5151
5252
The extensions currently supported are:
5353
5454
* The LDL routine in `LDLFactorizations.jl`
55-
* The pivoted cholesky in `CliqueTrees.jl`
55+
* The pivoted Cholesky in `CliqueTrees.jl`
5656
"""
5757
function compute_sparse_sqrt(Q::AbstractMatrix)
5858
# There's a big try-catch here because Cholesky can fail even if

0 commit comments

Comments
 (0)