aeon icon indicating copy to clipboard operation
aeon copied to clipboard

[ENH] Tidy up the rocket transformers

Open TonyBagnall opened this issue 8 months ago • 4 comments

Describe the feature or idea you want to propose

Its time to tidy up the convolutional transformers, will collate all issues here and make tasks for smaller PRs. Replaces #208

To Do

  • [ ] Combine minirocket and minirocket variable length
  • [ ] GPU implementations, benchmark vs CPU #313 #1248
  • [ ] Profile memory usage #1126
  • [ ] Tidy up multirocket implementation, remove duplicate methods and repeated code from minirocket
  • [ ] Decorate numba functions in rocket, test if any speed up
  • [ ] Test for numpy 2 compatibility, and optimise (e.g. G = _X + _X + _X)

Done

  • [x] Minirocket has hard coded arrays that reduce readability. Replace with a numba function Done: #1698
  • [x] Combine minirocket and minirocket multivariate #1708
  • [x] Benchmark variants against alternative implementations #214
  • [x] Combine two multirocket variants into one #1711

TonyBagnall avatar Jun 18 '24 16:06 TonyBagnall