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

modelfit_CModel code occassionally tries to use uninitialized pointer

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Won't Fix
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: meas_modelfit
    • Labels:
      None
    • Team:
      Data Release Production

      Description

      startDetermineFitRegion returns an uninitialized PTR(footprint) where it handles the condition: !footprint.contains(pixel). This subsequently causes an assert.

        Attachments

          Issue Links

          is triggering

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-4202 Revert temporary disabling of CModel in config override files

          • Done
          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-4768 Port W16 CModel improvements from HSC

          • Done

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            swinbank John Swinbank added a comment -

            Looks like the correct thing to do here might be to throw a MeasurementError rather than to set the flag and return, but I'd need to refresh my memory of the framework to be certain.

            Show
            swinbank John Swinbank added a comment - Looks like the correct thing to do here might be to throw a MeasurementError rather than to set the flag and return, but I'd need to refresh my memory of the framework to be certain.
            Hide
            Permalink
            jbosch Jim Bosch added a comment -

            This bug will go away with DM-4768, as the code in question has been completely be rewritten and {Footprint is no longer used in determining the fit region.

            Show
            jbosch Jim Bosch added a comment - This bug will go away with DM-4768 , as the code in question has been completely be rewritten and { Footprint is no longer used in determining the fit region.

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              pgee Perry Gee
              Watchers:
              Jim Bosch, John Swinbank, Perry Gee
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: