Support for ELU and tanh Activation Functions

[LeaE]

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

[AvantiShri]

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]

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