Skip to content
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

Bernoulli distribution doctests #132

Merged
merged 3 commits into from
Sep 27, 2023
Merged

Bernoulli distribution doctests #132

merged 3 commits into from
Sep 27, 2023

Conversation

chammika-become
Copy link
Contributor

#58 - doctests for bernoulli distribution

Bug-fix:
@avhz I made a bug fix to kurtosis of Bernoulli distribution based on Wolfram-alpa

It appeared + 3.0 was missing.

avhz
avhz requested changes Sep 23, 2023
View reviewed changes
Copy link
Owner

@avhz avhz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, thanks a bunch for more doc tests, really helpful. I just gotta change the name of the function to excess_kurtosis.

src/statistics/distributions/bernoulli.rs Outdated Show resolved Hide resolved
@avhz avhz merged commit 7084b8a into avhz:main Sep 27, 2023
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@avhz avhz avhz approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@chammika-become @avhz