-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Parametrize Binomial and Categorical distributions via logit_p #5637
Conversation
Hi @ricardoV94 and @MarcoGorelli please have a look. |
Codecov Report
@@ Coverage Diff @@
## main #5637 +/- ##
==========================================
+ Coverage 87.64% 88.07% +0.43%
==========================================
Files 76 76
Lines 13722 13753 +31
==========================================
+ Hits 12026 12113 +87
+ Misses 1696 1640 -56
|
Looks great. Left a small comment above about the tests |
Thank you @ricardoV94, let me fix that |
Ah and we need a note in the release notes about the new feature! |
Okay, I'm not sure what the proper message should be; could you please advise? |
The easiest is to look at previous entries that are similar for inspiration. I spotted these two: Line 276 in 436f94a
Line 597 in 436f94a
|
I think, I also forgot to add |
Done now : ) |
Thanks for your help @purna135! |
Addressing #5005
Parametrized the Binomial and Categorical distributions via logit_p
Added test in test_distributions_random.py