DOC: Clarify definition of kurtosis used #9437
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Trivial PR to better explain how kurtosis is computed. When checking the calculation, I noticed 2 lines that were unused to also removes those. |
| 'Return unbiased kurtosis over requested axis\nNormalized by N-1', | ||
| 'Return unbiased kurtosis over requested axis using Fisher''s ' | ||
| 'definition of\nkurtosis (kurtosis of normal == 0.0). Normalized ' | ||
| 'by N-1\n'', |
Added explation that kurtosis returned is Fisher's kurtosis, also known as excess kurtosis. Also cleans up the kurtosis function to remove some unused computations.
bashtage
force-pushed
the
doc-kurtosis
branch
from
24e788c to
e00524b
Compare
|
Indeed on the first use of |
jreback
added
Docs
Numeric Operations
jreback
added a commit
that referenced
this pull request
Feb 10, 2015
DOC: Clarify definition of kurtosis used
|
@bashtage thanks |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added explation that kurtosis returned is Fisher's kurtosis, also
known as excess kurtosis.
Also cleans up the kurtosis function to remove some unused computations.