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

            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!

            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!
            rowen Russell Owen added a comment -

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

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

            People

              rowen Russell Owen
              rowen Russell Owen
              Meredith Rawls
              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.