-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Added moment for geometric distribution along with test for issue #5078 #5158
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
It seems like you picked some commits that are not part of your PR. You will probably need to rebase to fix it. |
Codecov Report
@@ Coverage Diff @@
## main #5158 +/- ##
==========================================
+ Coverage 78.05% 78.07% +0.02%
==========================================
Files 88 88
Lines 14138 14142 +4
==========================================
+ Hits 11035 11042 +7
+ Misses 3103 3100 -3
|
b70f2f8
to
1f20dd8
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.
I rebased your PR and added a couple of unrelated commits that I had in the backlog.
Thanks @sagartomar just merged the PR! Looking forward to your next one :) |
Thanks @ricardoV94 |
Thank your for opening a PR!
Depending on what your PR does, here are a few things you might want to address in the description: