Status: Won't Fix
Fix Version/s: None
GtransfoPoly(gtransfo, frame, degree, nPoint) needs a unittest and should probably be refactored, though I think it's only used by the SimpleAstrometryModel. It came with a commented out #ifdef TO_BE_FIXED too.
- mentioned in
Reviewing old tickets for CCB: this still looks relevant, even though the classes involved have been renamed (GtransoPoly->AstrometryTransformPolynomial and gtransfo->AstrometryTransform), and this does seemed to be used by ConstrainedAstrometryModel (https://github.com/lsst/jointcal/blob/7f1c91f61beaa663a6ddc2696f0071bad96cbf6f/src/ConstrainedAstrometryModel.cc#L95-L97) as well as SimpleAstrometryModel. That makes this a bit more important (the constrained model is what we use in production), but it also means we've got pretty good indirect test coverage, so I still don't think this is a high priority.