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

error in formula 29, linear correlation #10

Open
timothyyu opened this issue Jun 10, 2019 · 3 comments
Open

error in formula 29, linear correlation #10

timothyyu opened this issue Jun 10, 2019 · 3 comments
Assignees

Comments

@timothyyu
Copy link
Owner

error brought to my attention by S. Mamont for formula 29, linear correlation:
image

@timothyyu timothyyu changed the title error in formula 29, linear corellation error in formula 29, linear correlation Jun 10, 2019
@timothyyu
Copy link
Owner Author

related #3

@timothyyu
Copy link
Owner Author

formula in question, 29:
image

@timothyyu timothyyu self-assigned this Jul 11, 2019
@timothyyu timothyyu added priority-3 question Further information is requested formula labels Jul 12, 2019
@timothyyu
Copy link
Owner Author

Error in formula 29, linear correlation, noted in formulas.py. The Scipy.stats module implementation of pearson r is going to be used instead:
80ff7da#diff-cb33a52a1b5551c0fa2f0477a49de96b

import numpy as np
import math
import scipy.stats

# Formula 29, Linear Correlation 
    # Acknowledgements: Error in formula in source article brought to my attention by S. Mamont.
    # Use scipy.stats.pearsonr
        # scipy.stats.pearsonr(x, y)
        # https://docs.scipy.org/doc/scipy/reference/generated/scipy.stats.pearsonr.html```

@timothyyu timothyyu removed the question Further information is requested label Aug 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant