AstrometryTask.run return not consistent with ANetAstrometryTask

Description

ANetAstrometryTask.run returns matchMetadata but AstrometryTask.run returns matchMeta. The two must agree. It turns out that matchMeta is more widely used, so I'll standardize on that.

Checklist

Lucidchart Diagrams

Issue Matrix

hide

Activity

Show:
Russell Owen
August 13, 2015 at 12:40 PM

Thank you for the quick review. After rebasing pipe_tasks to an updated master it passed Jenkins Ci so I merged to master.

Simon Krughoff
August 13, 2015 at 11:43 AM

Looks good. Free to merge.

Russell Owen
August 13, 2015 at 11:18 AM

Could you please take a look? This adds minor changes in meas_astrom and pipe_tasks both on tickets/https://rubinobs.atlassian.net/browse/DM-3453#icft=DM-3453. I have confirmed that it works by running HSC data with it. I will also get a CI run before merging, of course, but I don't think the CI system exercises this code.

Done
Pinned fields
Click on the next to a field label to start pinning.

Details

Assignee

Reporter

Reviewers

Simon Krughoff

Story Points

RubinTeam

Sprint

None

Checklist

Created August 13, 2015 at 9:22 AM
Updated August 13, 2015 at 12:40 PM
Resolved August 13, 2015 at 12:40 PM