Skip to content

Added submodules for normality tests #43

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

RafeyIqbalRahman
Copy link
Contributor

These submodules are used to test the distribution of the data using hypothesis testing (inferential statistics), i.e., whether the distribution of the data points is normal (Gaussian) at specific confidence intervals.

These submodules are used to test the distribution of the data using hypothesis testing (inferential statistics), i.e., whether the distribution of the data points is normal (Gaussian) at specific confidence intervals.
@tkrabel
Copy link
Collaborator

tkrabel commented Aug 24, 2020

Hi @RafeyIqbalRahman - thanks for the PR :)

We already import stats from scipy and if I am not mistaken, then this is the usual way to use those submodules (see e.g. here).

Would it work for you to call the modules such as e.g. stats.normaltest or is there a good reason for importing them the way you suggest?

Cheers

@RafeyIqbalRahman
Copy link
Contributor Author

Hi @RafeyIqbalRahman - thanks for the PR :)

We already import stats from scipy and if I am not mistaken, then this is the usual way to use those submodules (see e.g. here).

Would it work for you to call the modules such as e.g. stats.normaltest or is there a good reason for importing them the way you suggest?

Cheers

stats.normaltest is appropriate but an ideal practice would be to import normaltest separately. I've seen some tutorials that discourage the use of stats.normaltest as a bad practice. According to them, you should minimize the number of '.' (dots) in your code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants