aeon icon indicating copy to clipboard operation
aeon copied to clipboard

[ENH] Split `RocketClassifier` and `RocketRegressor` into multiple classes

Open MatthewMiddlehurst opened this issue 5 months ago • 3 comments

Describe the feature or idea you want to propose

Currently, Rocket, Mini-Rocket and Multi-Rocket are condensed into a single estimator. This is not very consistent with other files we have for similar algorithms i.e. SAST/RSAST, WEASEL/WEASELV2/MUSE etc.

Describe your proposed solution

Split RocketClassifier and RocketRegressor into:

  • [ ] RocketClassifier
  • [ ] MiniRocketClassifier
  • [ ] MultiRocketClassifier
  • [ ] RocketRegressor
  • [ ] MiniRocketRegressor
  • [ ] MultiRocketRegressor

Describe alternatives you've considered, if relevant

There is an argument to be made that we should condense these similar algorithms into a single class instead, but this is simpler for now.

Additional context

No response

MatthewMiddlehurst avatar Sep 24 '24 10:09 MatthewMiddlehurst