-
Notifications
You must be signed in to change notification settings - Fork 5
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
Several names for reduced X^2 #91
Comments
This change can be gradual but once we touch a function/ class that has one of these names, we can change it to the accorded one. |
Files changed to this convention:
|
I personally would prefer to not have numbers in variable names, so something like |
I am okay with that. It is a bit longer and it was because of that that I choose the other. |
Can we close this, or does this relate to this #122 (comment) ? |
Over
snmachine
there are different references to the reduced X^2. I think it would be better to always call it in the same way.So far I have seen (this is not an exhaustive list):
rchi2
-snfeatures
line 141rcs
-snfeatures
line 594chi2
-snfeatures
line 597 (this appears in several places and I think it is dangerous as X^2 is different from reduced X^2)redChi2
-gps
line 150reduced_chi_squared
-sndata
line 474reducedChi2
-gps
line 289I propose to call it
reduced_chi2
reduced_chi_squared
for variable name.The text was updated successfully, but these errors were encountered: