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.

Sign up for GitHub

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

Jump to bottom

fix: p is ρ (greek rho) #10

Merged

storopoli merged 2 commits into main from js/rho

Jan 21, 2024

Collaborator

storopoli commented Jan 21, 2024 •

edited

Loading

Renames the Multiple Regression p to rho (ρ).
Refactor docstrings from struct to proper fields.
Some rustfmt autoformatting (we can drop this if you want).
Rearrangement of the tests so that the t-tests are together.

storopoli requested a review from rikhuijzer

January 21, 2024 11:27

storopoli self-assigned this


          fix: p is ρ (greek rho) and some rustfmt

07acf3e

storopoli force-pushed the js/rho branch from c495cea to 07acf3e Compare

January 21, 2024 11:29


          refactor: move docstring to struct field

b17bd24

rikhuijzer approved these changes

View reviewed changes

storopoli merged commit fb0c3bb into main

2 checks passed

storopoli deleted the js/rho branch

January 21, 2024 13:18

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet