Skip to content

Commit

Permalink
Fix SparseArrays test failures
Browse files Browse the repository at this point in the history
  • Loading branch information
christiangnrd authored Jan 27, 2024
1 parent e569283 commit e8f01da
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions src/rulesets/SparseArrays/sparsematrix.jl
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,8 @@ function rrule(::typeof(findnz), v::AbstractSparseVector)
end

if Base.USE_GPL_LIBS # Don't define rrules for sparse determinants if we don't have CHOLMOD from SuiteSparse.jl

using SparseInverseSubset

Check warning on line 55 in src/rulesets/SparseArrays/sparsematrix.jl

View workflow job for this annotation

GitHub Actions / format

[JuliaFormatter] reported by reviewdog 🐶 Raw Output: src/rulesets/SparseArrays/sparsematrix.jl:55:- src/rulesets/SparseArrays/sparsematrix.jl:65:+
if VERSION < v"1.7"
#=
The method below for `logabsdet(F::UmfpackLU)` is required to calculate the (log)
Expand Down Expand Up @@ -82,8 +83,6 @@ if Base.USE_GPL_LIBS # Don't define rrules for sparse determinants if we don't h
end
return ifelse(isodd(result), -1, 1)
end

using SparseInverseSubset

function LinearAlgebra.logabsdet(F::UmfpackLU{T, TI}) where {T<:Union{Float64,ComplexF64},TI<:Union{Int32, Int64}}
n = checksquare(F)
Expand Down

0 comments on commit e8f01da

Please sign in to comment.