TDAmeritrade / stumpy

STUMPY is a powerful and scalable Python library for modern time series analysis
https://stumpy.readthedocs.io/en/latest/
Other
3.67k stars 320 forks source link

Numba JIT decorator is missing in `stumpy/maamp.py:: _maamp` and `stumpy/mstump.py::_mstump` #999

Closed NimaSarajpoor closed 4 months ago

NimaSarajpoor commented 4 months ago

The NJIT decorator is missing for the functions _maamp (in stumpy/maamp.py) and _mstump(in stumpy/mstump.py)

seanlaw commented 4 months ago

Actually, this is not needed. Both _maamp and _mstump do not need to be njit-ed as it is the inner function, _compute_multi_D that is njit-ed

NimaSarajpoor commented 4 months ago

Going to close this issue and its corresponding PR #1000.