Skip to content

Commit 16f64e7

Browse files
authored
Fix signature in generic cholesky! (#1412)
1 parent 1ffbf9d commit 16f64e7

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

src/cholesky.jl

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -413,7 +413,7 @@ end
413413
# cholesky!. Destructive methods for computing Cholesky factorization of real symmetric
414414
# or Hermitian matrix
415415
## No pivoting (default)
416-
function cholesky!(A::SelfAdjoint, ::NoPivot = NoPivot(); check::Bool = true)
416+
function cholesky!(A::RealSymHermitian, ::NoPivot = NoPivot(); check::Bool = true)
417417
C, info = _chol!(A.data, A.uplo == 'U' ? UpperTriangular : LowerTriangular)
418418
check && checkpositivedefinite(info)
419419
return Cholesky(C.data, A.uplo, info)
@@ -455,7 +455,7 @@ end
455455

456456
## With pivoting
457457
### Non BLAS/LAPACK element types (generic).
458-
function cholesky!(A::SelfAdjoint, ::RowMaximum; tol = 0.0, check::Bool = true)
458+
function cholesky!(A::RealSymHermitian, ::RowMaximum; tol = 0.0, check::Bool = true)
459459
AA, piv, rank, info = _cholpivoted!(A.data, A.uplo == 'U' ? UpperTriangular : LowerTriangular, tol, check)
460460
C = CholeskyPivoted(AA, A.uplo, piv, rank, tol, info)
461461
check && chkfullrank(C)

src/symmetric.jl

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -224,6 +224,7 @@ const RealHermSymSymTri{T<:Real} = Union{RealHermSym{T}, SymTridiagonal{T}}
224224
const RealHermSymComplexHerm{T<:Real,S} = Union{Hermitian{T,S}, Symmetric{T,S}, Hermitian{Complex{T},S}}
225225
const RealHermSymComplexSym{T<:Real,S} = Union{Hermitian{T,S}, Symmetric{T,S}, Symmetric{Complex{T},S}}
226226
const RealHermSymSymTriComplexHerm{T<:Real} = Union{RealHermSymComplexSym{T}, SymTridiagonal{T}}
227+
const RealSymHermitian{S} = Union{Symmetric{<:Real,S}, Hermitian{<:Any,S}}
227228
const SelfAdjoint = Union{SymTridiagonal{<:Real}, Symmetric{<:Real}, Hermitian}
228229

229230
wrappertype(::Union{Symmetric, SymTridiagonal}) = Symmetric

0 commit comments

Comments
 (0)