hnfc fails on a 3x3 matrix, with det == 10: AssertionError: U is not unimodular.

[glwhart]

I've made good use of NormalForms.jl since the SNF bug I exposed was fixed. Thanks. I was surprised to run into this error in the hnfc function on a 3x3 matrix.

julia> A = [-7 -2 10; -1 0 0; 0 -1 0]
3×3 Matrix{Int64}:
 -7  -2  10
 -1   0   0
  0  -1   0

julia> hnfc(A)
ERROR: AssertionError: U is not unimodular.
Stacktrace:
 [1] ColumnHermite(H::Matrix{Int64}, U::Matrix{Int64}, info::Int64)
   @ NormalForms ~/.julia/packages/NormalForms/oFF45/src/hermite.jl:63
 [2] hnfc! (repeats 2 times)
   @ ~/.julia/packages/NormalForms/oFF45/src/hermite.jl:162 [inlined]
 [3] hnfc(M::Matrix{Int64})
   @ NormalForms ~/.julia/packages/NormalForms/oFF45/src/hermite.jl:173
 [4] top-level scope
   @ REPL[74]:1

[glwhart]

Here are a few more cases that fail:
[-9 -2 12; -1 0 0; 0 -1 0]
[-7 -2 12; 6 1 -12; 0 1 0]
[-11 -3 13; 10 2 -13; 0 1 0]
[-11 -3 14; -1 0 0; 0 -1 0]

[glwhart]

Brandon, I didn't realize that I was using v0.1.9. When I updated to 0.1.10 then all my test cases (about 3760 3x3 matrices up to det==32) ran without error.

I'm going to close the issue.

[brainandforce]

I missed this, thanks for reporting though!