Our [logdet](https://github.com/JuliaDiff/ChainRules.jl/blob/78a0028184af3cd5fa3a93e3130fe7ed247db18d/src/rulesets/LinearAlgebra/dense.jl#L86) implementation is too general. In particular, it captures any and all `AbstractMatrix`s, which is bad for the reasons discussed in #232 . We should at the very least constrain it to be for `StridedMatrix{<:Real}`s.
Our logdet implementation is too general. In particular, it captures any and all
AbstractMatrixs, which is bad for the reasons discussed in #232 .We should at the very least constrain it to be for
StridedMatrix{<:Real}s.