Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: afw
    • Story Points:
      20
    • Epic Link:
    • Sprint:
      Alert Production F17 - 11, AP S18-1
    • Team:
      Alert Production

      Description

      As part of DM-10765 it became apparently that SkyWcs needed many changes, including:

      • Do not inherit from Transform
      • Add FITS binary table persistence
      • Add missing functionality as methods or free functions

      In addition, it is possible to greatly simplify the code using the new ast::FrameDict class.

      Jim Bosch suggested, and I agree, that it is better to do the SkyWcs overhaul on a ticket separate than DM-10765, making DM-10765 itself smaller and more focused.

        Attachments

          Issue Links

            Activity

            Hide
            rowen Russell Owen added a comment -

            As suggested by John Swinbank, I would like you two to review this as follows:

            Changes include:

            • Functions that returned a SkyWcs or Transform now return a shared_ptr. This includes readString, readStream, then and makeWcsPairTransform
            • New functions to construct a (shared pointer to) SkyWcs, such as makeTanSipWcs and makeModifiedWcs
            • SkyWcs(crpix, crval, ...) now has a projection argument which defaults to Tan
            • A new set of functions in wcsUtils

            All this work was done in support of DM-10765, which makes use of this new functionality.

            Show
            rowen Russell Owen added a comment - As suggested by John Swinbank , I would like you two to review this as follows: Detailed review by Krzysztof Findeisen Examine the API by Jim Bosch Changes include: Functions that returned a SkyWcs or Transform now return a shared_ptr. This includes readString, readStream, then and makeWcsPairTransform New functions to construct a (shared pointer to) SkyWcs, such as makeTanSipWcs and makeModifiedWcs SkyWcs(crpix, crval, ...) now has a projection argument which defaults to Tan A new set of functions in wcsUtils All this work was done in support of DM-10765 , which makes use of this new functionality.
            Hide
            krzys Krzysztof Findeisen added a comment -

            Done with my portion.

            Show
            krzys Krzysztof Findeisen added a comment - Done with my portion.

              People

              • Assignee:
                rowen Russell Owen
                Reporter:
                rowen Russell Owen
                Reviewers:
                Jim Bosch
                Watchers:
                Jim Bosch, Krzysztof Findeisen, Russell Owen
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel