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

Change RADECSYS to RADESYS in our code base

    Details

      Description

      The FITS keyword RADESYS is often misspelled as RADECSYS. Both appear to be accepted by our FITS reading code, but the latter is incorrect and we should not be using it. I found 31 instances across 27 files. I will attach the search results.

        Attachments

          Issue Links

            Activity

            Hide
            rowen Russell Owen added a comment -

            A few trivial changes to a few packages. This only affects unit tests and they still pass.

            Show
            rowen Russell Owen added a comment - A few trivial changes to a few packages. This only affects unit tests and they still pass.
            Hide
            mrawls Meredith Rawls added a comment -

            A bit of a fascinating rabbit hole to see how these FITS keywords have propagated... At any rate, these changes look fine, especially since you say all the unit tests still pass. Thanks for working on technical debt!

            Show
            mrawls Meredith Rawls added a comment - A bit of a fascinating rabbit hole to see how these FITS keywords have propagated... At any rate, these changes look fine, especially since you say all the unit tests still pass. Thanks for working on technical debt!

              People

              • Assignee:
                rowen Russell Owen
                Reporter:
                rowen Russell Owen
                Reviewers:
                Meredith Rawls
                Watchers:
                Meredith Rawls, Russell Owen, Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel