You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This is plea for grouping the three arguments that parameterize the windowing step into a single argument, that would take a SlidingWindow argument. This would greatly de-bloat some of the functions while retaining the same functionalities.
This SlidingWindow could be, for instance, a dataclass, with the same default parameters as already defined everywhere in the package:
# default values are the same as already defined in the functions headers@dataclassclassSlidingWindow:
length: float=0.25hop: float=0.10wtype : WindowType="hamming"# bad idea to shadow "type" keyword
Each function making use of a sliding window could then be like this
This is plea for grouping the three arguments that parameterize the windowing step into a single argument, that would take a
SlidingWindow
argument. This would greatly de-bloat some of the functions while retaining the same functionalities.This
SlidingWindow
could be, for instance, a dataclass, with the same default parameters as already defined everywhere in the package:Each function making use of a sliding window could then be like this
What does @hbredin thinks of this?
The text was updated successfully, but these errors were encountered: