Skip to content
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

DOC: Documentation cleanup for stats tutorials. #9953

Merged
merged 2 commits into from
Mar 28, 2019

Conversation

pvanmulbregt
Copy link
Contributor

@pvanmulbregt pvanmulbregt commented Mar 18, 2019

  • Be more explicit about the support of a distribution. Have it agree with the implementation.
  • Added definitions of functions used (I -> incomplete beta, or incomplete gamma or bessel,...; airy's Hh_v)
  • Removed indicator functions (inconsistently used, confusible with the I's above)
  • Replace G(\alpha;...) with G(q; ...) and make follow-on changes:
    gengamma, levy_l, nakagami, dgamma, chi-squared, loggamma, gamma.
  • Fixed ordering of arguments (student t)
  • Reformatted many long (>1500chars) lines for maintainability.
  • Fixed m_n median for double weibull.

Be more explicit about the support of a distribution.  Have it agree with
the implementation.
Added definitions of functions used (I -> incomplete beta, or incomplete
gamma or bessel,...;  airy's Hh_v)
Removed indicator functions (inconsistently used, confusible with the I's above)
Replace G(\alpha;...) with G(q; ...) and make follow-on changes:
 gengamma, levy_l,  nakagami, dgamma, chi-squared, loggamma, gamma.
Fixed ordering of arguments (student t)
Reformatted many long (>1500chars) lines for maintainability.
Fixed m_n median for double weibull.
@pvanmulbregt pvanmulbregt added scipy.stats Documentation Issues related to the SciPy documentation. Also check https://github.com/scipy/scipy.org labels Mar 18, 2019
@rgommers
Copy link
Member

Be more explicit about the support of a distribution. Have it agree with
the implementation.

Good idea, thanks for reviewing all that.

The doc build CI failed due to some unresolved references.

@pvanmulbregt
Copy link
Contributor Author

The unresolved symbols turned out to be a consequence of a failure in an earlier step of the build, due to some long-standing extraneous alignment tab characters.

@rgommers rgommers added this to the 1.3.0 milestone Mar 28, 2019
@rgommers
Copy link
Member

I reviewed a few bits in detail and sanity checked the rest. All looked great. The reformatting of the equations makes things more readable as well.

Merging. Thanks @pvanmulbregt!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Issues related to the SciPy documentation. Also check https://github.com/scipy/scipy.org scipy.stats
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants