deeplift icon indicating copy to clipboard operation
deeplift copied to clipboard

Published 20 hours ago • verified publisher icon kundajelab

Support for ELU and tanh Activation Functions

Open LeaE opened this issue 4 years ago • 2 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

LeaE avatar Jan 23 '20 09:01 LeaE

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!

AvantiShri avatar Jan 26 '20 20:01 AvantiShri

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

LeaE avatar Feb 07 '20 12:02 LeaE