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
DM-25450: Add Rowe statistics to SQuaSH #23
Conversation
8f26cac
to
2fc0187
Compare
|
||
--- | ||
# Specification partial | ||
id: "RhoStat_smallScale-base" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was looking for consistency as well. If all can be referred to (here and in the metric names and definitions) as rhoStat
, then use that everywhere. If you feel that the abbreviation is not appropriate, the they should all be rhoStatistics
. A mix-n-match is just confusing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed. I use rhoStatistics
in full everywhere in this file.
2fc0187
to
193f686
Compare
This PR defines new metrics related to Rho Statistics computed from star catalogs in
lsst-dm/pipe_analysis
. There is an associated PR which includes the code for measurement of these metrics.