Support for ELU and tanh Activation Functions

Question

Support for ELU and tanh Activation Functions

LeaE opened this issue 5 years ago · comments

Hi Avanti,

as far as I can tell from the README and Code, DeepLIFT does currently not support the ELU and tanh activations. Are there any plans to add these features in the near future?

Thanks a lot,
Lea

Av Shrikumar · Answer 1 · Mon Jan 27 2020 04:02:35 GMT+0800 (China Standard Time)

Hi @LeaE, just wanted to quickly confirm that this DeepLIFT implementation does not currently support ELU or tanh, but the DeepLIFT implementations listed here do: https://github.com/kundajelab/deeplift#my-model-architecture-is-not-supported-by-this-deeplift-implementation-what-should-i-do (the caveat is that they don't support the RevealCancel rule). I can try to find time to add them in this week, but if you were interested in making a pull request, I would be also be happy to accept it!

LeaE · Answer 2 · Fri Feb 07 2020 20:27:18 GMT+0800 (China Standard Time)

Hi Avanti,

thanks for the info! Sorry for my late response, I will have a look at these links and get back to you.

Kind reards,
Lea