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 hard sigmoid #1776
Add hard sigmoid #1776
Conversation
Thanks for opening your first pull request in this repository! Someone will review it when they have a chance. In the mean time, please be sure that you've handled the following things, to make the review process quicker and easier:
Thank you again for your contributions! 👍 |
src/mlpack/methods/ann/activation_functions/hard_sigmoid_function.hpp
Outdated
Show resolved
Hide resolved
src/mlpack/methods/ann/activation_functions/hard_sigmoid_function.hpp
Outdated
Show resolved
Hide resolved
src/mlpack/methods/ann/activation_functions/hard_sigmoid_function.hpp
Outdated
Show resolved
Hide resolved
src/mlpack/methods/ann/activation_functions/hard_sigmoid_function.hpp
Outdated
Show resolved
Hide resolved
src/mlpack/methods/ann/activation_functions/hard_sigmoid_function.hpp
Outdated
Show resolved
Hide resolved
src/mlpack/methods/ann/activation_functions/hard_sigmoid_function.hpp
Outdated
Show resolved
Hide resolved
src/mlpack/methods/ann/activation_functions/hard_sigmoid_function.hpp
Outdated
Show resolved
Hide resolved
src/mlpack/methods/ann/activation_functions/hard_sigmoid_function.hpp
Outdated
Show resolved
Hide resolved
src/mlpack/methods/ann/activation_functions/hard_sigmoid_function.hpp
Outdated
Show resolved
Hide resolved
src/mlpack/methods/ann/activation_functions/hard_sigmoid_function.hpp
Outdated
Show resolved
Hide resolved
src/mlpack/methods/ann/activation_functions/hard_sigmoid_function.hpp
Outdated
Show resolved
Hide resolved
src/mlpack/methods/ann/activation_functions/hard_sigmoid_function.hpp
Outdated
Show resolved
Hide resolved
src/mlpack/methods/ann/activation_functions/hard_sigmoid_function.hpp
Outdated
Show resolved
Hide resolved
src/mlpack/methods/ann/activation_functions/hard_sigmoid_function.hpp
Outdated
Show resolved
Hide resolved
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.
Some minor corrections.
src/mlpack/methods/ann/activation_functions/hard_sigmoid_function.hpp
Outdated
Show resolved
Hide resolved
src/mlpack/methods/ann/activation_functions/hard_sigmoid_function.hpp
Outdated
Show resolved
Hide resolved
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.
Just a minor comment, rest all looks good.
src/mlpack/methods/ann/activation_functions/hard_sigmoid_function.hpp
Outdated
Show resolved
Hide resolved
src/mlpack/methods/ann/activation_functions/hard_sigmoid_function.hpp
Outdated
Show resolved
Hide resolved
@Bk073 It seems you have also committed the binary files for VSCode? Can you remove them? |
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.
Second approval provided automatically after 24 hours. 👍
* The hard sigmoid function, defined by | ||
* | ||
* @f{eqnarray*}{ | ||
* f(x) &=& \min(1, max(0, {0.2 * x + 0.5}) \\ |
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.
Missing \
before max, also no need to use {}
inside the max function. Sorry for being so picky.
@zoq Made suggested changes. |
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.
No further comments from my side, thanks for the contribution.
Hello there! Thanks for your contribution. I see that this is your first contribution to mlpack. If you'd like to add your name to the list of contributors in In addition, if you'd like some stickers to put on your laptop, I'd be happy to help get them in the mail for you. Just send an email with your physical mailing address to stickers@mlpack.org, and then one of the mlpack maintainers will put some stickers in an envelope for you. It may take a few weeks to get them, depending on your location. 👍 |
Thanks for the contribution @Bk073! |
I wanted to make a review of my work. I have added a hard sigmoid function. Thank you so much