-
Notifications
You must be signed in to change notification settings - Fork 648
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.
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
Add PReLU Activation #1570
Add PReLU Activation #1570
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1570 +/- ##
==========================================
+ Coverage 83.07% 83.10% +0.02%
==========================================
Files 69 69
Lines 5826 5836 +10
==========================================
+ Hits 4840 4850 +10
Misses 986 986
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good overall, I just have 2 small comments
71cf644
to
240d974
Compare
b140571
to
67a9558
Compare
225ff22
to
73b9e0c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, Looks good to me!
What does this PR do?
flax.linen.activation.PReLU
discussed in PReLU activation implementation #1419.tests/linen/linen_activation_test.py
Fixes #1419
Checklist
checks if that's the case).
discussion (please add a
link).
documentation guidelines.
(No quality testing = no merge!)