Closed
Description
openedon May 10, 2013
The current logdet implementation in linalg/dense.jl,
logdet(A::Matrix) = 2.0 * sum(log(diag(cholfact(A)[:U])))
would probably be cleaner if implemented as:
logdet(A::Matrix) = logdet(cholfact(A))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment