Replies: 6 comments
-
Of course we're open to docstrings; that would be much appreciated! |
Beta Was this translation helpful? Give feedback.
-
Would like to explore and see what you would consider to be highest priority at the moment. For now, I have seen the following places where I might be able to add docstrings:
For the module-level docstrings for |
Beta Was this translation helpful? Give feedback.
-
Yes, that sounds great; however, I don't know the extent to which we can detail all of the intent and architecture, since—for at least the latter—that's still being worked-out. |
Beta Was this translation helpful? Give feedback.
-
Got it. What is your take on docstrings for the log prob functions, @brandonwillard? Necessary or unnecessary? |
Beta Was this translation helpful? Give feedback.
-
Probably not necessary, because the parameters are just the ones used by the There might be a few notes worth adding about the exact choice of log-probabilities, but none that I can think of right now. |
Beta Was this translation helpful? Give feedback.
-
Thanks @brandonwillard. Let me comb through the log-prob implementations to see if there's anything noteworthy there. I'll start off with that. |
Beta Was this translation helpful? Give feedback.
-
I'm happy to work on docstrings if you find this task to be tedium. 😄
Let me know, @brandonwillard!
Beta Was this translation helpful? Give feedback.
All reactions