pynapple-org / pynapple

PYthon Neural Analysis Package :pineapple:
https://pynapple.org/
MIT License
262 stars 61 forks source link

Urgent Fix: kwargs #301

Closed BalzaniEdoardo closed 2 months ago

BalzaniEdoardo commented 3 months ago

There are some methods like IntervalSet.__init__ that has unused **kwargs. This can result in an unwanted behavior if one, for example, misspell a keyword argument it won't be set and no error will be raised.

We should go through all the defs that includes **kwargs and think if we can remove any and fix the logic.

gviejo commented 3 months ago
gviejo commented 3 months ago

TsGroup test for time_support if not None

gviejo commented 3 months ago

check kwargs list for count