Matthew Middlehurst

I prefer the panel for multithreading, I imagine the overhead from swapping between formats using the transformer base class is pretty miniscule. Good point with unequal length, I didn't consider...

> if we change the parameter signature of the old transformer, there needs to be deprecation do you mean n_jobs?

the only parameter change is n_jobs -1 to 1. features is new (see \_\_init__). I can deprecate -1 to use the old functionality for now? >I think it should be...

I have deprecated the n_jobs value and included the new parameter options. Changing the internal data type can wait for another PR I think, it is listed as a bonus...

Has this appeared since the original issue? The link no longer shows the broken test, so it is difficult to assess.

The tests excluded are different from the one in this issue, we know why those break, and it's (mostly) reasonable that they do so. ``` # Early classifiers intentionally retain...

I don't think this has ever reappeared, and we don't have any information to reproduce it. Most likely just an issue with the node or similar. Suggesting we close.

This was fixed in the refactor, correct?

Flashbacks to catch22... Hope the features aren't different between them.

After some testing, this appears to be fixed in `tsfresh` `0.19.0`. I don't think this issue is serious enough to change the version bounds for the dependency, however.