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

Remove unused ra/dec angle handling methods from afw and utils

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: afw, obs_ctio0m9, utils
    • Labels:
      None
    • Story Points:
      1
    • Sprint:
      Arch 2018-09-24
    • Team:
      Architecture

      Description

      Following RFC-528, remove the unused sexagesimal angle routines from utils and afw. obs_ctio0m9 needs fixing since it is the only package that uses the utils routines. Replace that with astropy usage.

        Attachments

          Issue Links

          is triggered by

          RFC - An issue whose primary purpose is to advertise an intent and allow stakeholders to comment. RFC-528 Remove RA/Dec angle handling from utils package

          • Implemented
          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-9220 Delegate to Astropy for coordinate stringification

          • Invalid

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            mfisherlevine Merlin Fisher-Levine added a comment -

            Have reviewed on Github.

            Show
            mfisherlevine Merlin Fisher-Levine added a comment - Have reviewed on Github.
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Russell Owen would you mind reviewing this? It's mostly code removal. Merlin Fisher-Levine has reviewed the obs_ctio0m9 change already so this is the afw and utils changes.

            Show
            tjenness Tim Jenness added a comment - Russell Owen would you mind reviewing this? It's mostly code removal. Merlin Fisher-Levine has reviewed the obs_ctio0m9 change already so this is the afw and utils changes.
            Hide
            Permalink
            rowen Russell Owen added a comment -

            Your utils and afw changes look perfect to me. Thank you very much for removing this code.

            Show
            rowen Russell Owen added a comment - Your utils and afw changes look perfect to me. Thank you very much for removing this code.
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            All code removed.

            Show
            tjenness Tim Jenness added a comment - All code removed.

              People

              Assignee:
              tjenness Tim Jenness
              Reporter:
              tjenness Tim Jenness
              Reviewers:
              Russell Owen
              Watchers:
              Merlin Fisher-Levine, Russell Owen, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.