-
-
Notifications
You must be signed in to change notification settings - Fork 118
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
slopes
documentation
#701
slopes
documentation
#701
Conversation
…uilding summary dataframe
…mproved parsing of uncertainty dict
…e values to other methods
…ing when len(wrt) > 2
merge slopes branch
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Codecov Report
@@ Coverage Diff @@
## main #701 +/- ##
==========================================
+ Coverage 89.57% 89.73% +0.16%
==========================================
Files 43 43
Lines 3522 3499 -23
==========================================
- Hits 3155 3140 -15
+ Misses 367 359 -8
... and 1 file with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
excellent work @GStechschulte. Just left some comments. I don't think further additions are needed, only some removals and maybe simplifications. Thanks a lot! |
Thanks @tomicapretto , and also for the review! |
@@ -0,0 +1,2503 @@ | |||
{ |
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.
instead of having to do two imports we should just import bambi as bmb and then call
bmb.slopes, bmb.plot_slopes, etc
Reply via ReviewNB
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 will change this in PR #699 and then merge into this branch 👍🏼
This is fantastic! |
Thank you @aloctavodia! And thanks for the feedback. Much appreciated :) |
@GStechschulte there are some conflicts now 😅 @aloctavodia if you don't have any further comments i will merge once the conflicts are resolved |
Good to go now 😄 |
This PR provides documentation for PR #699 (
slopes
andplot_slopes
). The notebook provides the following sections:slopes
andplot_slopes
, and the benefits under the following: