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

PhotoCalTask ignores all input errors if any are NaN

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: meas_photocal
    • Labels:
      None
    • Team:
      Data Release Production

      Description

      If any of the input flux errors are NaN, the photoCalTask ignores all of them (and uses sqrt(flux) instead)

        Attachments

          Activity

          Hide
          jbosch Jim Bosch added a comment -

          Robert, where's the code to review for this issue? I'm guessing it's in meas_astrom, but is it an HSC-side branch or an LSST-side branch?

          Show
          jbosch Jim Bosch added a comment - Robert, where's the code to review for this issue? I'm guessing it's in meas_astrom, but is it an HSC-side branch or an LSST-side branch?
          Hide
          rhl Robert Lupton added a comment -

          Sorry. This wasn't labelled HSC, so tickets/DM-824 on the lsst side

          Show
          rhl Robert Lupton added a comment - Sorry. This wasn't labelled HSC, so tickets/ DM-824 on the lsst side
          Hide
          jbosch Jim Bosch added a comment -

          Looks fine!

          I think that wins (at least ties) the most trivial review request ever.

          Show
          jbosch Jim Bosch added a comment - Looks fine! I think that wins (at least ties) the most trivial review request ever.
          Hide
          ktl Kian-Tat Lim added a comment -

          I think there have been one-character changes, so this may be 50% less trivial than the most .

          Show
          ktl Kian-Tat Lim added a comment - I think there have been one-character changes, so this may be 50% less trivial than the most .

            People

            • Assignee:
              rhl Robert Lupton
              Reporter:
              rhl Robert Lupton
              Reviewers:
              Jim Bosch
              Watchers:
              Jim Bosch, Kian-Tat Lim, Robert Lupton
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Summary Panel