albermax / innvestigate

A toolbox to iNNvestigate neural networks' predictions!
Other
1.25k stars 233 forks source link

[Feature Request] - Relative Attributing Propagation #223

Open mrektor opened 3 years ago

mrektor commented 3 years ago

Hi everybody,

I am an enthusiast user of this library, but I found an interesting generalization of LRP-alpha-beta (Relative Attributing Propagation) that would be interesting to be supported by iNNvestigate.

https://github.com/wjNam/Relative_Attributing_Propagation

The implementation is in pytorch, but maybe it would not be too difficult to port it to the iNNvestigate framework. It would be very useful and it will make the library richer. I would try to do it myself but I am still not too practical with the framework...

What do you think about it? PS: Thanks again for this amazing library!!

rachtibat commented 3 years ago

Hi,

Thanks for the suggestion and sorry for the late reply. I think it is a great idea and as I can see you already implemented it. Hopefully albermax will merge it soon.

Best

rachtibat commented 3 years ago

Ah I see, the pull request is not from you, sorry. Yeah, we will have a look at the implementation. But right now, we are still working on the basic structure of Innvestigate 2.0 for tf 2.0