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

The per-Amplifier flag hasRawInfo is not cleared when the CCD is assembled

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Invalid
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ip_isr
    • Labels:
      None
    • Team:
      Data Release Production

      Description

      There are a pair of methods Amplifier.get/setHasRawInfo but the latter is apparently no longer called.  This means that routines that pull out the data section of an image cannot know whether to call getRawDataBBox or getBBox.  Please fix this, so that trimmed Detectors know that they are trimmed.

      Note that this means that we cannot assume that detectors are immutable, and makes the back-pointer to the camera a little trickier.  I think it'd be OK for the back pointer to point at a camera containing only the Detector in question.

        Attachments

          Issue Links

          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-15410 measureCrosstalk was broken by the move of crosstalk correction to above assembly

          • Done

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-18610 Add fields, limited mutability, and trim/assembly-state tracking to cameraGeom

          • Done
          mentioned in

          Page Loading...

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            I only see one place that calls setHasRawInfo and that is in obs_decam/decam/makeDecamCameraRepository.py. I don't see any call to getHasRawInfo. afw Amplifier only seems to have an hasRawInfo flag and no-one is looking at it.

            Show
            tjenness Tim Jenness added a comment - I only see one place that calls setHasRawInfo and that is in obs_decam/decam/makeDecamCameraRepository.py . I don't see any call to getHasRawInfo . afw Amplifier only seems to have an hasRawInfo flag and no-one is looking at it.
            Hide
            Permalink
            jbosch Jim Bosch added a comment -

            DM-18610 changed how Amplifiers reflect their assembly/trim state pretty substantially since this ticket was filed, so I think it's now best to assume this ticket is irrelevant until/unless someone says otherwise.

            Show
            jbosch Jim Bosch added a comment - DM-18610 changed how Amplifiers reflect their assembly/trim state pretty substantially since this ticket was filed, so I think it's now best to assume this ticket is irrelevant until/unless someone says otherwise.

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              rhl Robert Lupton
              Watchers:
              Christopher Waters, Jim Bosch, Robert Lupton, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.