Yang
All masks related code in each method should go to `maskers.py`, for instance: https://github.com/dianna-ai/dianna/blob/134ddf388f147602e3edcc6249d8e16b6b51d3ac/dianna/methods/rise_text.py#L96-L00 _Originally posted by @cwmeijer in https://github.com/dianna-ai/dianna/pull/654#discussion_r1375870468_
As a follow-up to PR #554, we can add a naive fully random masking strategy for masking timeseries. For instance, there are two simple ways for it: - Simply flatten...
We want to add a simple test similar to the one from timeseries methods to the tabular methods: https://github.com/dianna-ai/dianna/blob/dec5abaa46d4c5ef22320d251e81de2cc387b9a1/tests/methods/test_rise_timeseries.py#L21-L37 It can be a naive test for a very simple usecase,...
`num_features` does not work in LIME timeseries: https://github.com/dianna-ai/dianna/blob/dec5abaa46d4c5ef22320d251e81de2cc387b9a1/dianna/methods/lime_timeseries.py#L53 This parameter is supposed to let user decide the number of features to include (display) in the explanation. It only exists in...
Current masking module requires `p_keep`, which does not exist in LIME. We need to explore how LIME does masking in this case and adapt the masking module to LIME timeseries....