-
-
Notifications
You must be signed in to change notification settings - Fork 11
Add plot_rank_dist #253
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
Add plot_rank_dist #253
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #253 +/- ##
==========================================
+ Coverage 69.30% 70.96% +1.65%
==========================================
Files 41 42 +1
Lines 4721 4859 +138
==========================================
+ Hits 3272 3448 +176
+ Misses 1449 1411 -38 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Not sure how to fix the warnings. And I can't build docs locally, sphinx is giving me a "segmentation fault" (I thought I had solved this) |
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.
looks good, left a comment to fix the docs warning.
To fix the doc build locally you might need to regenerate the env (it might have arviz-base installed from an old commit for example) so add -r
when doing tox -e docs -r
(after running the cleandocs job to make sure old caches aren't interfering)
Co-authored-by: Oriol Abril-Pla <[email protected]>
closes #16
This also add stats_kwargs argument to
plot_rank
.📚 Documentation preview 📚: https://arviz-plots--253.org.readthedocs.build/en/253/