mstump: description of idx in private functions #867
NimaSarajpoor
started this conversation in
General
Replies: 1 comment 1 reply
-
I think you're right. Though, I think it's more like:
Is this better? |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
In module mstump.py:
The function
_mstump
calls the function_compute_multi_D
which has the argumentidx
with the following description:The function
_mstump
also calls the functioncore._compute_multi_PI
that has the same argumentidx
with the same description.However, I think this description might be incorrect. If
T
is multi-dimensional, i.e. 2D numpy array, where each row corresponds to one time series, then the description ofidx
should be:This is based on what I can see, regarding
idx
, in the following lines of code:stumpy/stumpy/mstump.py
Lines 800 to 814 in 76cb980
Beta Was this translation helpful? Give feedback.
All reactions