Matthew Middlehurst

Results 287 comments of Matthew Middlehurst

Hi, we are planning (another) refactor for anomaly detection which is likely to make this more complicated. I would prefer to get this in first, so may push some changes...

Hello, please fix the failing pre-commit test and add a unit test if you can.

see https://github.com/aeon-toolkit/aeon/pull/2690#issuecomment-2757785421

Similar to the other draft PR, classification now has better coverage through the general testing. The clustering PR above does not, so though are still useful though.

Hi I don't think this is a good issue currently as I think @hadifawaz1999 and @chrisholder want to develop a prototyping module which would impact this. This also seems to...

@Cyril-Meyer 1. I think your merged resolution is fine. We don't support different number of channels, so just raising an error instead of adding a parameter is ok. 2. It...

I feel this could be improved still, i.e. moved to the top level and link to resources for accepted data types. There are also some aspects that are still not...

Unfortunately these tests are redundant with out general testing from the `testing/` module which is run on all estimators. I do appreciate the effort in ensuring code is covered by...