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

Transfer functionality from LSST analysis package script utils.py to sgs-fsbutler

    Details

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

      Description

      The utils.py script in the LSST-DM package sgs-fsbutler calls the function "makeMapperInfo" from the utils.py script in the LSST analysis package. Move "makeMapperInfo" into the sgs-fsbutler utils.py script so that sgs-fsbutler no longer depends on analysis.

        Attachments

          Issue Links

            Activity

            Hide
            vpk24 Vishal Kasliwal [X] (Inactive) added a comment -

            I'd like to get it done - how about I let you know if it begins to look like I can't get it done by the next standup?

            Show
            vpk24 Vishal Kasliwal [X] (Inactive) added a comment - I'd like to get it done - how about I let you know if it begins to look like I can't get it done by the next standup?
            Hide
            swinbank John Swinbank added a comment -

            Sure.

            Show
            swinbank John Swinbank added a comment - Sure.
            Hide
            vpk24 Vishal Kasliwal [X] (Inactive) added a comment -

            DM-9687 filed to take care of removing makeMapperInfo.py once the butler provides equivalent functionality (https://jira.lsstcorp.org/browse/DM-9687).

            DM-9688 filed to update/remove the PsfModelImage class from lsst.analysis.utils (https://jira.lsstcorp.org/browse/DM-9688).

            Show
            vpk24 Vishal Kasliwal [X] (Inactive) added a comment - DM-9687 filed to take care of removing makeMapperInfo.py once the butler provides equivalent functionality ( https://jira.lsstcorp.org/browse/DM-9687 ). DM-9688 filed to update/remove the PsfModelImage class from lsst.analysis.utils ( https://jira.lsstcorp.org/browse/DM-9688 ).
            Hide
            vpk24 Vishal Kasliwal [X] (Inactive) added a comment -

            Hi John,
            I've made the changes suggested by you in the review. Could you please give this ticket one more review? Thanks!

            Show
            vpk24 Vishal Kasliwal [X] (Inactive) added a comment - Hi John, I've made the changes suggested by you in the review. Could you please give this ticket one more review? Thanks!
            Hide
            swinbank John Swinbank added a comment -

            Looks good. A couple of minor points:

            • I don't think you need a30652, do you? Please delete it entirely before you merge.
            • I trust that we've actually got everything that is used by sgs-fsbutler, not just bits that "seem to be". Maybe reword the statement here to be a bit more definitive?

            Otherwise, merge at will.

            Show
            swinbank John Swinbank added a comment - Looks good. A couple of minor points: I don't think you need a30652 , do you? Please delete it entirely before you merge. I trust that we've actually got everything that is used by sgs-fsbutler, not just bits that "seem to be". Maybe reword the statement here to be a bit more definitive? Otherwise, merge at will.

              People

              • Assignee:
                vpk24 Vishal Kasliwal [X] (Inactive)
                Reporter:
                vpk24 Vishal Kasliwal [X] (Inactive)
                Reviewers:
                John Swinbank
                Watchers:
                John Swinbank, Vishal Kasliwal [X] (Inactive)
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel