Skip to content

Add loaders for ELU, RELU activation layers (#78) #128

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
Merged
merged 3 commits into from
Jun 22, 2021
Merged

Add loaders for ELU, RELU activation layers (#78) #128

merged 3 commits into from
Jun 22, 2021

Conversation

kokorins
Copy link
Contributor

  • missing activation layers loaders added
  • two distinct examples with save/load added to examples folder (trying to reach 0.7 accuracy)

@kokorins
Add loaders for ELU, RELU activation layers (Kotlin#78)
b42c934 
- missing activation layers loaders added
- two distinct examples with save/load added to examples folder (trying to reach 0.7 accuracy)
@kokorins kokorins closed this Jun 18, 2021
@kokorins
Copy link
Contributor Author

For some reason I cant reopen the old one #115

@kokorins kokorins reopened this Jun 18, 2021
@kokorins kokorins mentioned this pull request Jun 18, 2021
Closed
@zaleslaw zaleslaw added the Review This PR is under review label Jun 21, 2021
@zaleslaw zaleslaw linked an issue Jun 21, 2021 that may be closed by this pull request
Add missed loaders for the ReLU and ELU activation layers #78
Closed
5 tasks
@zaleslaw
Copy link
Collaborator

zaleslaw commented Jun 21, 2021 •
edited
Loading

So, @kokorins, I'd like your PR, especially the custom callback!

Could you please update your PR by adding trainability property (like in the pr )

@zaleslaw zaleslaw added LGTM PR reviewed and is ready to merge and removed Review This PR is under review labels Jun 21, 2021
@zaleslaw zaleslaw merged commit 03c1890 into Kotlin:master Jun 22, 2021
@zaleslaw
Copy link
Collaborator

Congrats @kokorins ! Thanks for the contribution

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
LGTM PR reviewed and is ready to merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add missed loaders for the ReLU and ELU activation layers
2 participants
@kokorins @zaleslaw