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

Several names for reduced X^2 #91

Open
Catarina-Alves opened this issue Mar 25, 2019 · 5 comments
Open

Several names for reduced X^2 #91

Catarina-Alves opened this issue Mar 25, 2019 · 5 comments
Assignees

Comments

@Catarina-Alves
Copy link
Collaborator

Catarina-Alves commented Mar 25, 2019

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 141
  • rcs - snfeatures line 594
  • chi2 - 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 150
  • reduced_chi_squared - sndata line 474
  • reducedChi2 - gps line 289

I propose to call it reduced_chi2 reduced_chi_squared for variable name.

@Catarina-Alves Catarina-Alves added the question General codebase questions about functionality/implementation choices label Mar 25, 2019
@Catarina-Alves
Copy link
Collaborator Author

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.

@Catarina-Alves Catarina-Alves self-assigned this Mar 25, 2019
@Catarina-Alves Catarina-Alves added help wanted Open call for help on a particular issue and removed help wanted Open call for help on a particular issue question General codebase questions about functionality/implementation choices labels Mar 25, 2019
@Catarina-Alves
Copy link
Collaborator Author

Catarina-Alves commented Mar 26, 2019

Files changed to this convention:

  • gps_test.py (except line 30 as it uses something from sndata)

@tallamjr
Copy link
Collaborator

I personally would prefer to not have numbers in variable names, so something like reduced_chi_square is preferable to me

@Catarina-Alves
Copy link
Collaborator Author

I personally would prefer to not have numbers in variable names, so something like reduced_chi_square is preferable to me

I am okay with that. It is a bit longer and it was because of that that I choose the other.

@tallamjr tallamjr added this to the reduced-chi-squared milestone Apr 29, 2019
tallamjr added a commit that referenced this issue May 16, 2019
Reduced chi squared

Closes #120 , #91 , #85  and part of #107
@tallamjr tallamjr reopened this May 20, 2019
@tallamjr
Copy link
Collaborator

Can we close this, or does this relate to this #122 (comment) ?

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

No branches or pull requests

2 participants