Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-4454

Fix multiple patch catalog sorting for forcedPhotCcd.py

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: meas_base
    • Labels:
      None

      Description

      forcedPhotCcd.py is currently broken due to the requirement of the lsst.afw.table.getChildren() function that the SourceCatalog is sorted by the parent key (i.e. lsst.afw.table.SourceTable.getParentKey()). This occurs naturally in the case of SourceCatalogs produced by the detection and deblending tasks, but it may not be true when concatenating multiple such catalogs. This is indeed the case for forcedPhotCcd.py as a given CCD can be overlapped by multiple patches, thus requiring a concatenation of the reference catalogs of all overlapping patches.

      There two places in the running of forcedPhotCcd.py where calls to getChildren() can cause a failure: one in the subset() function in references.py, and the other in the run function of SingleFrameMeasurementTask in sfm.py.

        Attachments

          Activity

          lauren Lauren MacArthur created issue -
          lauren Lauren MacArthur made changes -
          Field Original Value New Value
          Status To Do [ 10001 ] In Progress [ 3 ]
          lauren Lauren MacArthur made changes -
          Attachment forcedCcdCat.png [ 27325 ]
          lauren Lauren MacArthur made changes -
          Reviewers Paul Price [ price ]
          Status In Progress [ 3 ] In Review [ 10004 ]
          swinbank John Swinbank made changes -
          Epic Link DM-3585 [ 19682 ]
          swinbank John Swinbank made changes -
          Sprint Science Pipelines DM-W16-3 [ 182 ] Science Pipelines DM-W16-3, Science Pipelines DM-W16-4 [ 182, 187 ]
          swinbank John Swinbank made changes -
          Rank Ranked higher
          price Paul Price made changes -
          Status In Review [ 10004 ] Reviewed [ 10101 ]
          lauren Lauren MacArthur made changes -
          Resolution Done [ 10000 ]
          Status Reviewed [ 10101 ] Done [ 10002 ]

            People

            • Assignee:
              lauren Lauren MacArthur
              Reporter:
              lauren Lauren MacArthur
              Reviewers:
              Paul Price
              Watchers:
              Lauren MacArthur, Paul Price
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Summary Panel