Details
-
Type:
Story
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: obs_cfht
-
Labels:None
-
Story Points:1
-
Epic Link:
-
Sprint:Science Pipelines DM-W16-1
-
Team:Alert Production
Description
The CFHT-specific CalibrateTask tries to apply aperture correction once just after measuring it (which is too early) and again later, at the right time. The error probably has no effect on the final results, but it is confusing and needlessly divergent from the standard CalibrateTask. The required changes are small. I plan to test by running Dominique Boutigny's CFHT demo.
Attachments
Activity
| Field | Original Value | New Value |
|---|---|---|
| Epic Link |
|
| Description | The CFHT-specific CalibrateTask has some outdated code for handling aperture corrections. The code is incorrect and divergent from the standard CalibrateTask. The required changes are small. I plan to test by running [~boutigny]'s CFHT demo. | The CFHT-specific CalibrateTask tries to apply aperture correction once just after measuring it (which is too early) and again later, at the right time. The error probably has no effect on the final results, but it is confusing and needlessly divergent from the standard CalibrateTask. The required changes are small. I plan to test by running [~boutigny]'s CFHT demo. |
| Reviewers | Ian Sullivan [ sullivan ] | |
| Status | To Do [ 10001 ] | In Review [ 10004 ] |
| Summary | CalibrateTask has some outdated, incorrect for handling aperture corrections | CalibrateTask has some outdated, incorrect code for handling aperture corrections |
| Summary | CalibrateTask has some outdated, incorrect code for handling aperture corrections | CalibrateTask has outdated, incorrect code for handling aperture corrections |
| Status | In Review [ 10004 ] | Reviewed [ 10101 ] |
| Resolution | Done [ 10000 ] | |
| Status | Reviewed [ 10101 ] | Done [ 10002 ] |