Make numba optional dependency / remove numba
telegraphic opened this issue · comments
Danny Price commented
Currently numba is only used in one section of code.
Not quite convinced that speeding this code up warrants an additional dependency, so suggest either making it optional or removing.
Alternatively if more use of numba JIT is planned I could be convinced!
Richard Elkins commented
Point taken.
I ran into a case where numba crashed. The data array was larger than 4 GB (is 32-bits a magic width?). So, I yanked numba out of Waterfall and requirements.txt. Moot issue now.