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

Update starlink_ast

    Details

    • Story Points:
      0.25
    • Sprint:
      Alert Production S17 - 5
    • Team:
      Alert Production

      Description

      starlink_ast before 2017-04-24 fixes has a bug whereby astFitsChan strips TIMESYS. Update our starlink_ast to this version.

        Attachments

          Issue Links

            Activity

            Hide
            rowen Russell Owen added a comment -

            You have already looked at this, so perhaps you would be willing to give formal sign-off? I may hold off for a day or two because there is a small bug in astPolyTran (that I'm quite happy to work around, as I think my workaround makes for cleaner code), but I figure if it gets fixed I might as well include it.

            Note that AST includes some changes to astTimeAdd which required updating the docs and unit test for TimeMap on astshim.

            Show
            rowen Russell Owen added a comment - You have already looked at this, so perhaps you would be willing to give formal sign-off? I may hold off for a day or two because there is a small bug in astPolyTran (that I'm quite happy to work around, as I think my workaround makes for cleaner code), but I figure if it gets fixed I might as well include it. Note that AST includes some changes to astTimeAdd which required updating the docs and unit test for TimeMap on astshim.
            Hide
            tjenness Tim Jenness added a comment -

            It looked good to me.

            Show
            tjenness Tim Jenness added a comment - It looked good to me.
            Hide
            rowen Russell Owen added a comment -

            Thank you for the quick review and for updating the settings for the git repo. Merged and pushed.

            Show
            rowen Russell Owen added a comment - Thank you for the quick review and for updating the settings for the git repo. Merged and pushed.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel