Limit factorization size method to Circulant (#130)

JuliaLinearAlgebra · Jan 25, 2024 · 131933a · 131933a
1 parent 135d562
commit 131933a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/ToeplitzMatrices.jl b/src/ToeplitzMatrices.jl
@@ -99,8 +99,6 @@ struct ToeplitzFactorization{T,A<:AbstractToeplitz{T},S<:Number,P<:Plan{S}} <: F
     tmp::Vector{S}
     dft::P
 end
-Base.size(T::ToeplitzFactorization) = (s = size(T.dft,1); (s, s))
-Base.size(T::ToeplitzFactorization, i::Integer) = size(T)[i]
 
 include("toeplitz.jl")
 include("special.jl")

diff --git a/src/special.jl b/src/special.jl
@@ -257,3 +257,7 @@ end
 function Base.replace_in_print_matrix(A::LowerTriangularToeplitz, i::Integer, j::Integer, s::AbstractString)
     i >= j ? s : Base.replace_with_centered_mark(s)
 end
+
+# size for factorize
+size(T::ToeplitzFactorization{<:Any, <:Circulant}) = (s = size(T.dft,1); (s, s))
+size(T::ToeplitzFactorization{<:Any, <:Circulant}, i::Integer) = size(T)[i]