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

pyfits 3.4 upgrade broke obs_subaru build

    Details

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

      Description

      The upgrade to pyfits 3.4 appears to have broken the defect list construction, which is part of the obs_subaru build:

      bin.src/genDefectFits.py:56: PyfitsDeprecationWarning: The new_table function is deprecated as of version 3.3 and may be removed in a future version.
       
              Use :meth:`BinTableHDU.from_columns` for new BINARY tables or :meth:`TableHDU.from_columns` for new ASCII tables instead.
        table = pyfits.new_table(cols)
      Traceback (most recent call last):
        File "bin.src/genDefectFits.py", line 86, in <module>
          genDefectFits(args.cameraName, args.defectsFile, args.targetDir)
        File "bin.src/genDefectFits.py", line 58, in genDefectFits
          table.header.update('NAME', ccd)
        File "/tigress/pprice/lsstsw/stack/Linux64/pyfits/3.4.0/lib/python/pyfits-3.4-py2.7-linux-x86_64.egg/pyfits/header.py", line 1017, in update
          'value, and comment string.' % idx)
      ValueError: Header update sequence item #0 is invalid; the item must either be a 2-tuple containing a keyword and value, or a 3-tuple containing a keyword, value, and comment string.
      scons: *** [hsc/defects/2013-01-31/defects.dat-fits] Error 1
      scons: building terminated because of errors.
      

      When I hack the environment to use pyfits 3.3, it works fine.

        Attachments

          Container Issues

            Issue Links

              Activity

                People

                • Assignee:
                  swinbank John Swinbank
                  Reporter:
                  price Paul Price
                  Reviewers:
                  Paul Price
                  Watchers:
                  John Swinbank, Paul Price, Tim Jenness
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Summary Panel