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 basic documentation to negbin #301
Conversation
Codecov Report
@@ Coverage Diff @@
## master #301 +/- ##
=========================================
Coverage ? 81.17%
=========================================
Files ? 7
Lines ? 632
Branches ? 0
=========================================
Hits ? 513
Misses ? 119
Partials ? 0
Continue to review full report at Codecov.
|
src/negbinfit.jl
Outdated
and keyword arguments will be passed to [`glm`](@ref). | ||
|
||
# Keyword Arguments | ||
- `initialθ::Real=Inf`: Starting values for shape parameter θ. If it is `Inf` |
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.
Singular "value".
Also, better document other arguments here rather than above, that's more logical.
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.
done
Thanks. Note that with #288, the argument names will switch to lower case. |
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.
Thanks. I've rebased against #288.
Thanks for doing that! |
Sorry, this was completely forgotten. Would you mind rebasing against master? |
5247922
to
a038023
Compare
a038023
to
8454c21
Compare
I think it's done, but it's been long enough that I've forgotten everything I knew about this function, and have only glanced at the new keyword arguments. |
No description provided.