aeon icon indicating copy to clipboard operation
aeon copied to clipboard

[ENH] Add KDTW kernel to be used with kernelKmeans algorithm for time series clustering

Open boujida-h opened this issue 1 year ago • 1 comments

Describe the feature or idea you want to propose

The algorithm TimeSeriesKernelKMeans in the API reference for clustering does not take KDTW**, A Regularized Dynamic Time Warping Kernel, as a kernel.

** : https://github.com/pfmarteau/KDTW/tree/master

Describe your proposed solution

Integrate the KDTW given above in https://www.aeon-toolkit.org/en/stable/api_reference/auto_generated/aeon.clustering.TimeSeriesKernelKMeans.html#aeon.clustering.TimeSeriesKernelKMeans

The kernelKmeans algorithm must use the KDTW as a kernel.

boujida-h avatar Jun 26 '24 12:06 boujida-h

great idea, @chrisholder is on it

TonyBagnall avatar Jun 27 '24 20:06 TonyBagnall

great idea, @chrisholder is on it

Hey, Can I work on this. @chrisholder @TonyBagnall

tanishy7777 avatar Mar 16 '25 12:03 tanishy7777

That's fine in principle, but please do not take on too much at once. Getting PRs merged and responding to reviews is the main goal.

MatthewMiddlehurst avatar Mar 16 '25 14:03 MatthewMiddlehurst

That's fine in principle, but please do not take on too much at once. Getting PRs merged and responding to reviews is the main goal.

Hi, currently my other PRs are still being reviewed and I am considerable free so I think I can take up this one. But yep will keep that in mind. Thanks!

tanishy7777 avatar Mar 17 '25 07:03 tanishy7777

@aeon-actions-bot assign @tanishy7777

tanishy7777 avatar Mar 17 '25 07:03 tanishy7777