-
-
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
Updating docstrings of distributions #5998
Updating docstrings of distributions #5998
Conversation
Codecov Report
@@ Coverage Diff @@
## main #5998 +/- ##
==========================================
- Coverage 89.36% 89.29% -0.07%
==========================================
Files 73 72 -1
Lines 13253 12884 -369
==========================================
- Hits 11843 11505 -338
+ Misses 1410 1379 -31
|
…lo as a contributor. Co-authored-by: pibieta <ibieta.pablo@gmail.com>
Regarding the |
Also, if you look at the preview, the |
Thanks @vitaliset! |
What is this PR about?
In response to issue #5459. Me and @pibieta updated docstrings for some distribution classes inside the multivariate.py module, namely:
Attention points
covs
,chols
andevds
were left aslist of arrays
andlist of tuples
because the proper type is being developed as @OriolAbril told us.sigma
should befloat
as far as I understand but we were not sure because the old docstring called it avariable
.Numpy matrix
input that we changed tondarray of int
. Does this make sense?nxn array
that we changed tonxn tensor_like of float
. Does that make sense or we should put something else?Docs / Maintenance
updated docstrings to follow the numpydoc standard
#DataUmbrellaPyMCSprint
cc: @reshamas