Skip to content

Conversation

@WuZhuoran
Copy link
Contributor

  1. Add Elu activation function according to this introduction.
  2. Add Elu to README.

@ddbourgin ddbourgin self-assigned this Jul 6, 2019
@ddbourgin ddbourgin mentioned this pull request Jul 6, 2019
@ddbourgin ddbourgin added the enhancement New feature or request label Jul 6, 2019
@ddbourgin
Copy link
Owner

This looks great - thanks a lot! I've added some unit tests in a separate PR (#4). If you agree with the edits, feel free to commit and I'll merge to master :)

@WuZhuoran
Copy link
Contributor Author

This looks great - thanks a lot! I've added some unit tests in a separate PR (#4). If you agree with the edits, feel free to commit and I'll merge to master :)

Thank you! I will commit ASAP.

WuZhuoran and others added 2 commits July 6, 2019 15:41
Co-Authored-By: David Bourgin <ddbourgin@users.noreply.github.com>
Co-Authored-By: David Bourgin <ddbourgin@users.noreply.github.com>
@WuZhuoran
Copy link
Contributor Author

Suggestions Applied.

@ddbourgin ddbourgin merged commit a1e4955 into ddbourgin:master Jul 7, 2019
lew91 pushed a commit to lew91/numpy-ml that referenced this pull request Sep 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants