aeon-toolkit / aeon

A toolkit for machine learning from time series
https://aeon-toolkit.org/
BSD 3-Clause "New" or "Revised" License
882 stars 93 forks source link

[ENH] REDCOMETS tweaks / fixes #1722

Open zy18811 opened 5 days ago

zy18811 commented 5 days ago

What does this implement/fix? Explain your changes.

Several small tweaks / fixes to REDCOMETS, improving performance.

image

aeon-actions-bot[bot] commented 5 days ago

Thank you for contributing to aeon

I have added the following labels to this PR based on the title: [ $\color{#FEF1BE}{\textsf{enhancement}}$ ]. I have added the following labels to this PR based on the changes made: [ $\color{#BCAE15}{\textsf{classification}}$ ]. Feel free to change these if they do not properly represent the PR.

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.

PR CI actions

These checkboxes will add labels to enable/disable CI functionality for this PR. This may not take effect immediately, and a new commit may be required to run the new configuration.

zy18811 commented 4 days ago

LGTM, but need the current test results used for comparison updated. This is all in testing/.

If you need some help with this, let me know, I can get around to it eventually if you are busy.

Last commit should've now done this, please have a look and double check :)

patrickzib commented 4 days ago

Not blocking, but would be nice to convert it to SFA_FAST at some point, which is much faster.

TonyBagnall commented 1 day ago

Not blocking, but would be nice to convert it to SFA_FAST at some point, which is much faster.

maybe make an issue, same as with TDE?

zy18811 commented 1 day ago

Not blocking, but would be nice to convert it to SFA_FAST at some point, which is much faster.

Yeah, I did briefly look into doing this but ran into some numba weirdness so put it on the back burner for now.

I’ll make an issue like @TonyBagnall suggested.

patrickzib commented 6 hours ago

@zy18811 if you need help with SFA_FAST in the future, drop me a message.