After some design review, we want to make two tweaks to the tracer classes in sacc_support.py before moving them to SACC.
The instance variable names should be changed to support generic programming (“duck typing”): the various x_upper and x_lower should become upper and lower; we can rely on the type of the object to know that BinZTracer.lower refers to a value of z. This allows functions to manipulate different kinds of bins when that is appropriate
The center variable in any bin should be documented as the location to use for a single point estimate, since it not required to be the arithmetic or geometric mean of the bin. Rather, it is the value that should be used if only one value is wanted for a point estimate of the location of that bin.
After some design review, we want to make two tweaks to the tracer classes in sacc_support.py before moving them to SACC.
x_upperandx_lowershould becomeupperandlower; we can rely on the type of the object to know thatBinZTracer.lowerrefers to a value of z. This allows functions to manipulate different kinds of bins when that is appropriatecentervariable in any bin should be documented as the location to use for a single point estimate, since it not required to be the arithmetic or geometric mean of the bin. Rather, it is the value that should be used if only one value is wanted for a point estimate of the location of that bin.