I finally changed the type of H in the default constructor to SparseMatrixCSC{T, Int}. I think the tril! function is mandatory because if the user inputs a Symmetric{T, SparseMatrixCSC{I, Int}} matrix, it is not possible to know if this matrix was constructed with a lower triangular matrix (maybe I am mistaken).
I finally changed the type of
H
in the default constructor toSparseMatrixCSC{T, Int}
. I think thetril!
function is mandatory because if the user inputs aSymmetric{T, SparseMatrixCSC{I, Int}}
matrix, it is not possible to know if this matrix was constructed with a lower triangular matrix (maybe I am mistaken).