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

ExposureCatalog should support new photoCalib objects

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: afw
    • Labels:
      None

      Description

      Currently, ExposureCatalog supports old Calib calibration objects.  However, for use with jointcal or fgcmcal these need to support new PhotoCalib calibration objects.  This will also allow multiple PhotoCalib objects to be bundled into one persistence file (per visit, for example, the number of required inodes would be reduced by a factor of 100 for HSC data for fgcmcal outputs).

        Attachments

          Issue Links

            Activity

            Hide
            jbosch Jim Bosch added a comment -

            jointcal isn't doing anything different from what meas_mosaic did in this regard. Replacing Calib with PhotoCalib is also independent of that (though I hope to tackle that soon).

            I suppose that's fair.  I guess I had just envisioned Calib->PhotoCalib happening at the same time.

            And now that I look into it, what exactly is an ExposureCatalog? The header file isn't very instructive.

            ExposureRecord is basically a version of ExposureInfo that you can make a catalog out of.  We use them extensively in coadds to be able to store metadata about what into the coadd, including some things that are necessary for coadd functionality (like CoaddPsf).  And like any other kind of record, you can add other fields to them, too.  There's a sad tale of why ExposureRecord and ExposureInfo can't easily be unified that I think involves BBox ownership, but I forget the details.

            Show
            jbosch Jim Bosch added a comment - jointcal isn't doing anything different from what meas_mosaic did in this regard. Replacing Calib with PhotoCalib is also independent of that (though I hope to tackle that soon). I suppose that's fair.  I guess I had just envisioned Calib->PhotoCalib happening at the same time. And now that I look into it, what exactly is an  ExposureCatalog ? The header file isn't very instructive. ExposureRecord is basically a version of ExposureInfo that you can make a catalog out of.  We use them extensively in coadds to be able to store metadata about what into the coadd, including some things that are necessary for coadd functionality (like CoaddPsf).  And like any other kind of record, you can add other fields to them, too.  There's a sad tale of why ExposureRecord and ExposureInfo can't easily be unified that I think involves BBox ownership, but I forget the details.
            Hide
            Parejkoj John Parejko added a comment -

            There's a sad tale of why ExposureRecord and ExposureInfo can't easily be unified that I think involves BBox ownership, but I forget the details.

            So, DM-7565 then?

            Show
            Parejkoj John Parejko added a comment - There's a sad tale of why ExposureRecord and ExposureInfo can't easily be unified that I think involves BBox ownership, but I forget the details. So, DM-7565 then?
            Hide
            jbosch Jim Bosch added a comment -

            That's at least some of it; might be all of it.

            Show
            jbosch Jim Bosch added a comment - That's at least some of it; might be all of it.
            Hide
            Parejkoj John Parejko added a comment -

            Done as part of DM-10156.

            Show
            Parejkoj John Parejko added a comment - Done as part of DM-10156 .
            Hide
            Parejkoj John Parejko added a comment -

            Now that this is done, Eli Rykoff: you and I should talk about better ways to persist PhotoCalibs from jointcal/fgcm runs to save a bundle of disk space.

            Show
            Parejkoj John Parejko added a comment - Now that this is done, Eli Rykoff : you and I should talk about better ways to persist PhotoCalibs from jointcal/fgcm runs to save a bundle of disk space.

              People

              • Assignee:
                Parejkoj John Parejko
                Reporter:
                erykoff Eli Rykoff
                Watchers:
                Eli Rykoff, Jim Bosch, John Parejko, John Swinbank
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel