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

Replace pyfits with astropy.io.fits in all code

    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

            Activity

            Hide
            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
            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
            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, Russell Owen, Simon Krughoff, Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                6 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel