-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
A little bug! #48
Comments
And your code
Your code is missing the default value, resulting in an error at |
And your code
The function in your code is missing an argument |
Maybe I should fork and create pull request 😃 |
And your code
but
right code
|
And
but SoftSign is not implemented in maybe you should delete function |
@Z-zhe - Wow, thanks so much for all these! I haven't had a chance to take a look yet, but should have some time this weekend. In the meantime if you feel like submitting a PR with fixes I'd be happy to review it, otherwise I can try to address these shortly. |
PR is complicated, it is easier for you to modify. 😃 |
your code
I don't think it should be
Please reconsider,thanks. |
Hi, I think there is a little bug at numpy-ml/numpy_ml/neural_nets/activations/activations.py Line 64.
your code
but
self.fn_x
Never definedThe text was updated successfully, but these errors were encountered: