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

Support EQUINOX being "J2000.0" as well as 2000.0

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: afw
    • Labels:
      None

      Description

      I share the concern that we're going to spend a lot of effort on handling illegal WCS headers, but this one's easy, and we already have a partial fix for the SDSS headers which use a string but no "J" prefix.

      Skymapper uses "J2000.0" not 2000.0 for its EQUINOX. Our code silently converts this to float using atof, and thus interprets it as 0.0 and deduces FK4 (which we don't support).

        Attachments

          Activity

          Hide
          rhl Robert Lupton added a comment -

          A one liner.

          No test, but I checked that it works on Skyserver files so I think it's safe.

          Show
          rhl Robert Lupton added a comment - A one liner. No test, but I checked that it works on Skyserver files so I think it's safe.
          Hide
          tjenness Tim Jenness added a comment -

          Looks okay, but can you add a comment before the code explaining the insanity. Otherwise someone is bound to come along in a couple of years and think we are crazy.

          Show
          tjenness Tim Jenness added a comment - Looks okay, but can you add a comment before the code explaining the insanity. Otherwise someone is bound to come along in a couple of years and think we are crazy.
          Hide
          rhl Robert Lupton added a comment -

          Added comment and merged to master

          Show
          rhl Robert Lupton added a comment - Added comment and merged to master

            People

            Assignee:
            rhl Robert Lupton
            Reporter:
            rhl Robert Lupton
            Reviewers:
            Tim Jenness
            Watchers:
            Robert Lupton, Tim Jenness
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: