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

Replace use of makeVisitInfo(... with VisitInfo(...)

    Details

    • Story Points:
      0.5
    • Sprint:
      Alert Production S17 - 3
    • Team:
      Alert Production

      Description

      lsst.afw.image.makeVisitInfo(...) is superseded by lsst.afw.image.VisitInfo(...). Remove existing usage of makeVisitInfo.

      makeVisitInfo is used in one or two places in most obs_ packages and a few other packages as well.

      Note that this can be done on a package by package basis as time permits. Once all usage is gone we can get rid of makeVisitInfo.

        Attachments

          Issue Links

            Activity

            Hide
            rowen Russell Owen added a comment -

            I somehow managed to get a ticket branch pushed that contained some incorrect changes. Corrected, but the result is an invalid pull request in addition to the good one.

            Show
            rowen Russell Owen added a comment - I somehow managed to get a ticket branch pushed that contained some incorrect changes. Corrected, but the result is an invalid pull request in addition to the good one.
            Hide
            rowen Russell Owen added a comment -

            Do you have time to look at this? It's a lot of packages, but the changes are completely trivial.

            Note that I accidentally pushed the change to pipe_drivers to master instead of the ticket branch. I can undo that if you want.

            Show
            rowen Russell Owen added a comment - Do you have time to look at this? It's a lot of packages, but the changes are completely trivial. Note that I accidentally pushed the change to pipe_drivers to master instead of the ticket branch. I can undo that if you want.
            Hide
            price Paul Price added a comment -

            Trivial changes, no worries.

            Don't worry about the accidental push to pipe_drivers master.

            Show
            price Paul Price added a comment - Trivial changes, no worries. Don't worry about the accidental push to pipe_drivers master.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel