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

instantiate uint32 bit Images and MaskedImages for LSST

    XMLWordPrintable

    Details

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

      Description

      The LSST camera will apparently write 18 bits of data per pixel, so we may end up storing images as uint32. We might as well be prepared by instantiating them.

      This affects afw::math::Statistics and our SWIG interfaces as well. Note that we presently use these one-character suffixes in our SWIG wrappers: U=uint16, L=uint64. I'm not sure what we'd use for uint32, but if you want it to be obvious to the users I strongly suggest changing to U16, U32 and U64 for those cases where there is more than one option (hence not masks, which should stay U as long as we only have one mask pixel type).

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            I assume we no longer have any plans for this given that we are taking data with LSSTCam and immediately converting to floating point when we read the raw?

            Show
            tjenness Tim Jenness added a comment - I assume we no longer have any plans for this given that we are taking data with LSSTCam and immediately converting to floating point when we read the raw?
            Hide
            jbosch Jim Bosch added a comment -

            Calling this Won't Fix, as I think we've settled into using signed 32-bit ints in places where unsigned 32-bit ints might be more natural but 31 unsigned bits are sufficient.

            Show
            jbosch Jim Bosch added a comment - Calling this Won't Fix, as I think we've settled into using signed 32-bit ints in places where unsigned 32-bit ints might be more natural but 31 unsigned bits are sufficient.

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              rowen Russell Owen
              Watchers:
              Jim Bosch, John Parejko, Kian-Tat Lim, Robyn Allsman [X] (Inactive), Russell Owen, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.