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

Replace pyfits with astropy.io.fits in all code

    XMLWordPrintable

    Details

    • Story Points:
      2
    • Team:
      Architecture

      Description

      We currently use pyfits in multiple packages: afw, coadd_chisquared, obs_base, meas_astrom, meas_deblender, meas_extensions_psfex, meas_mosaic, obs_cfht, obs_lsstSim, obs_sdss, obs_subaru and obs_test.

      Strangely, we only have explicit dependencies on pyfits listed for afw, obs_base, galsim, healpy and obs_subaru.

      galsim can use astropy.io.fits or pyfits. healpy really does seem to not work with astropy.io.fits – is there a newer version that does?

      Please replace pyfits with astropy.io.fits where appropriate and update the table files to correctly express the dependency (and removing pyfits where inappropriate).

        Attachments

          Issue Links

          duplicates

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-7316 Replace pyfits with astropy in daf_butlerUtils

          • Invalid
          is triggering

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-15092 ci_hsc failed 2018_07_11 night build

          • Done
          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-15793 Replace pyfits with astropy.io.fits in DESDM

          • Done

            Activity

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

            The current status is:

            • healpy does not use pyfits
            • galsim has pyfits in the table file but does not need it.

            The following use pyfits:

            • meas_deblender
            • obs_base
            • obs_subaru
            • afwdata
            • obs_sdss
            • obs_test
            • obs_cfht
            • meas_extensions_psfex
            • meas_extensions_astrometryNet
            • coadd_chisquared
            • obs_lsstSim
            • meas_mosaic

            Only daf_butlerUtils, obs_base, and obs_subaru declare this dependency.

            Show
            tjenness Tim Jenness added a comment - - edited The current status is: healpy does not use pyfits galsim has pyfits in the table file but does not need it. The following use pyfits: meas_deblender obs_base obs_subaru afwdata obs_sdss obs_test obs_cfht meas_extensions_psfex meas_extensions_astrometryNet coadd_chisquared obs_lsstSim meas_mosaic Only daf_butlerUtils, obs_base, and obs_subaru declare this dependency.
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Thank you for reviews from Russell Owen and John Parejko.

            Show
            tjenness Tim Jenness added a comment - Thank you for reviews from Russell Owen and John Parejko .
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            We no longer use pyfits.

            Show
            tjenness Tim Jenness added a comment - We no longer use pyfits.

              People

              Assignee:
              tjenness Tim Jenness
              Reporter:
              tjenness Tim Jenness
              Reviewers:
              Russell Owen
              Watchers:
              John Parejko, John Swinbank, Nate Pease [X] (Inactive), Russell Owen, Simon Krughoff, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.