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

Change our interpretation of FITS HDUs to be 0-indexed

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: afw
    • Labels:
      None
    • Team:
      DM Science

      Description

      See RFC-304.

      I'll do something to support the desire to keep -1 to mean "the last HDU", although I do not plan to support this now (a fits file doesn't declare how many HDUs it contains, so we'd have to count them). In C++ using the most negative int makes sense as the default value, and I assume that pybind11 can keep that as a default value.

        Attachments

          Issue Links

          is triggered by

          RFC - An issue whose primary purpose is to advertise an intent and allow stakeholders to comment. RFC-304 Change interpretation of HDU for fits I/O routines

          • Implemented
          is triggering

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-10091 Fix problems left over from DM-9952

          • Done
          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-10206 Fix obs_decam compatibility with 0-indexed HDUs

          • Done

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-9854 Disable code that skips an empty PDU when reading metadata

          • Won't Fix

            Activity

            Ascending order - Click to sort in descending order
            rhl Robert Lupton created issue -
            rhl Robert Lupton made changes -
            Field Original Value New Value
            Link This issue is triggered by RFC-304 [ RFC-304 ]
            swinbank John Swinbank made changes -
            Team Project Science [ 11800 ]
            rhl Robert Lupton made changes -
            Link This issue relates to DM-9854 [ DM-9854 ]
            rhl Robert Lupton made changes -
            Reviewers John Parejko [ parejkoj ]
            Status To Do [ 10001 ] In Review [ 10004 ]
            Parejkoj John Parejko made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            rhl Robert Lupton made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]
            tjenness Tim Jenness made changes -
            Link This issue is triggering DM-10091 [ DM-10091 ]
            mrawls Meredith Rawls made changes -
            Link This issue relates to DM-10206 [ DM-10206 ]
            swinbank John Swinbank made changes -
            Team Project Science [ 11800 ] DM Science [ 12218 ]

              People

              Assignee:
              rhl Robert Lupton
              Reporter:
              rhl Robert Lupton
              Reviewers:
              John Parejko
              Watchers:
              John Parejko, Meredith Rawls, Robert Lupton
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: