New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

PR to add LogSoftmax, and also has some code cleanup #30

Open

Arka161 wants to merge 2 commits into Talmaj:master from Arka161:master

Arka161 commented Dec 12, 2021

PR summary:

Add LogSoftmax, used in several SOTA CNN models.
Some hyperparameters for Activation operations were hardcoded, taking the values from the node now (eg: Softplus).
Remove unused variables

Arka161 added 2 commits

December 12, 2021 04:51


          add functionality

3e509f0


          remove unused var

44ccb35

h-grieve mentioned this pull request

Conversion request for op_type=LogSoftmax. #49

Open

Owner

Talmaj commented Sep 11, 2024

Hi! Thanks for contributing. A test that covers the new function would be necessary before I can merge this in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet