[BUG] mpdist window length

[TonyBagnall]

fixes #1498

the window length (called m for some reason) has as default value of 0, which means its set to a quarter of the length of the series. However, this check was happening after forming the windows, so it was trying to form windows with 0 length.

I think it might be better to just use stumpy for this, m = None should be the default and k/threshold should definitely be a parameter. I will see if it makes a difference in a separate PR

[aeon-actions-bot]

Thank you for contributing to aeon

I have added the following labels to this PR based on the title: [ $\color{#d73a4a}{\textsf{bug}}$ ].
I would have added the following labels to this PR based on the changes made: [ $\color{#5209C9}{\textsf{distances}}$ ], however some package labels are already present.

The Checks tab will show the status of our automated tests. You can click on individual test runs in the tab or "Details" in the panel below to see more information if there is a failure.

If our pre-commit code quality check fails, any trivial fixes will automatically be pushed to your PR unless it is a draft.

Don't hesitate to ask questions on the aeon Slack channel if you have any.

[MatthewMiddlehurst]

could you make a change to run the CI again? If there are other problems would be good to create an issue.

[baraline]

Overall I agree that having m=0 just feels weird as default value, None would be better !
Using stumpy would also make sense as we have it as softdep already.

At some point the same functionality should be available with similairty search if you want something "in house". Advantage would be that you could choose the internal distance to use (ED, DTW, etc...)

[baraline]

LGTM to me now !