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

Change RADECSYS to RADESYS in our code base

    XMLWordPrintable

    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:

                  Jenkins

                  No builds found.