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

CModel has unflagged flux=0 fluxSigma=NAN for 1% of sources

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: meas_modelfit
    • Labels:
    • Team:
      External

      Description

      NAOJ report that in the S19A production run with hscPipe 7, about 1% of objects have CModel flux=0 fluxsigma=NaN, without the flag set.

        Attachments

          Activity

          Hide
          price Paul Price added a comment -

          The problem is present in the unforced "meas" catalog (from measureCoaddSources) from S19A.

          I'm not sure whether to implement my fix without a test and be done with it, or go to greater lengths to reproduce the problem.

          Show
          price Paul Price added a comment - The problem is present in the unforced "meas" catalog (from measureCoaddSources ) from S19A. I'm not sure whether to implement my fix without a test and be done with it, or go to greater lengths to reproduce the problem.
          Hide
          price Paul Price added a comment -

          In the interests of getting more stuff done in advance of the next HSC production run, I've decided to push forward without actually reproducing the problem. We know these occur, and so we need to flag them. I'm happy to invest more effort to reproduce them if you think it's important, but let's start with this.

          Show
          price Paul Price added a comment - In the interests of getting more stuff done in advance of the next HSC production run, I've decided to push forward without actually reproducing the problem. We know these occur, and so we need to flag them. I'm happy to invest more effort to reproduce them if you think it's important, but let's start with this.
          Hide
          jbosch Jim Bosch added a comment -

          Looks good to me.  I agree that it seems expedient to just get this in; it may then be easier for NAOJ to see if it makes the problem go away than to help you reproduce it.

          Show
          jbosch Jim Bosch added a comment - Looks good to me.  I agree that it seems expedient to just get this in; it may then be easier for NAOJ to see if it makes the problem go away than to help you reproduce it.
          Hide
          price Paul Price added a comment -
          Show
          price Paul Price added a comment - Jenkins is green .
          Hide
          price Paul Price added a comment -

          Merged to master.

          Thanks Jim!

          Show
          price Paul Price added a comment - Merged to master. Thanks Jim!

            People

            • Assignee:
              price Paul Price
              Reporter:
              price Paul Price
              Reviewers:
              Jim Bosch
              Watchers:
              Hisanori Furusawa, Jim Bosch, Masayuki Tanaka, Paul Price
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: