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

Enable CModel in CalibrateTask prior to PhotoCal

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: meas_modelfit
    • Labels:
      None
    • Story Points:
      1
    • Sprint:
      Science Pipelines DM-W16-6, DRP X16-2, DRP X16-3
    • Team:
      Data Release Production

      Description

      CModel needs to run in CalibrateTask before PhotoCal in order to compute aperture corrections, but it also needs a Calib objects as input, and that isn't available until after PhotoCal is run.

      On the HSC side, we dealt with this by adding preliminary PhotoCal run before CModel is run, but we could also deal with it by removing the need for a Calib as input, at least in some situations.

        Attachments

          Issue Links

            Activity

            Hide
            swinbank John Swinbank added a comment -

            At sprint planning of 2016-04-27, Jim Bosch suggests that we can choose an arbitrary zero point which will be an adequate solution until & unless we have magnitude dependent priors. That makes this a much simpler ~0.5 SP issue.

            Show
            swinbank John Swinbank added a comment - At sprint planning of 2016-04-27, Jim Bosch suggests that we can choose an arbitrary zero point which will be an adequate solution until & unless we have magnitude dependent priors. That makes this a much simpler ~0.5 SP issue.
            Hide
            jbosch Jim Bosch added a comment -

            Review complete.

            Code in afw looks fine, but please reword the git commit: saying "flux" when you mean "fluxMag0" is a bit confusing in this context.

            Some minor comments on the meas_modelfit PR; only non-trivial one is a unit test request.

            Show
            jbosch Jim Bosch added a comment - Review complete. Code in afw looks fine, but please reword the git commit: saying "flux" when you mean "fluxMag0" is a bit confusing in this context. Some minor comments on the meas_modelfit PR; only non-trivial one is a unit test request.

              People

              Assignee:
              nlust Nate Lust
              Reporter:
              jbosch Jim Bosch
              Reviewers:
              Jim Bosch
              Watchers:
              Jim Bosch, John Swinbank
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: