JuliaMath / Hadamard.jl

Fast Walsh-Hadamard transforms for the Julia language

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

hadamard(1200) should work

stevengj opened this issue · comments

hadamard(12) and hadamard(100) are both supported, but hadamard(1200) gives an error.

The problem is the order in which we look for factors: finds the factor of 120 first (the biggest factor for which we have a tabulated Hadamard matrix), but what is left is then a factor of 10 (which does not have a Hadamard matrix because it is not a multiple of 4).

We should probably search all factors of sizes 4*odd first.