-
Notifications
You must be signed in to change notification settings - Fork 7
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
264 improve normality test #267
Conversation
Normality tests accepts now numpy ndarrays. Added test for ndarray. Added check for empty np.ndarray.
Added checks and tests for non-numeric data.
Created new exception class, checks and tests for maximum sample size. Max set to 5000 samples.
Added missing data handling, and tests for it.
Changed dataframe column validation for consistency.
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.
Hey, just a few typing/documentation related things and should be good to go then
data: Dataframe or numpy array containing the input data. | ||
columns: Optional Columns to be used for testing. |
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.
data: Dataframe or numpy array containing the input data. | |
columns: Optional Columns to be used for testing. | |
data: Dataframe or Numpy array containing the input data. | |
columns: Optional columns to be used for testing. |
- Fixed typos. - Clarified return types, and improved docstring. - Removed check and test for numeric data for Numpy array.
Better formatting for autogenerated docs.
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.
Looks good to me now! Merging
Made improvements for normality test as requested in issue#264 .