-
-
Notifications
You must be signed in to change notification settings - Fork 46.9k
Adds Gaussian Function in maths section #1051
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
Conversation
This is cool! In this PR, please focus only on adding the Gaussian function. Please drop the .md files that are not directly tied to the Gaussian function. In general, we have the autogenerated DIRECTORY.md for users to find algorithms so if the .md files do not contain explanations, we would rather not have them. Awesome that you have added doctests but how about a few more? Both good parameters and bad parameters: Large numbers, negative numbers, strings, lists. |
|
All submissions will be tested with mypy so we encourage to add Python type hints where it makes sense to do so. |
OPTIONAL: Extra credit if you run your code through python/black for readability. OPTIONAL: You can also try to minimize parens: return 1 / sqrt(2 * pi * sigma ** 2) * exp(-(x - mu) ** 2 / 2 * sigma ** 2) |
Lowercase directory and Python filenames please:
|
Output did not match... |
The test failed because in Python 3.7.3 |
|
Fixed the error with |
No description provided.