search

fzi-forschungszentrum-informatik / TSInterpret

An Open-Source Library for the interpretability of time series classifiers
BSD 3-Clause "New" or "Revised" License
115 stars 8 forks source link

Figsize argument not taken into account #41

Closed hugo-rddi closed 1 year ago

hugo-rddi commented 1 year ago

Hi again, Very minor issue but the exp.plot(..., figsize=()) is not taken into account in your plot function.

Code here : https://github.com/fzi-forschungszentrum-informatik/TSInterpret/blob/main/TSInterpret/InterpretabilityModels/FeatureAttribution.py#L190

Nothing that causes error or anything, but will be useful to have this hyperparameter working for dense and large time series for example. Thanks in advance !

JHoelli commented 1 year ago

Good Point. Thanks for bringing it to my attention. Fixed with the new Release (v.0.3.4).