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

Remove skypix from stack dependencies

    XMLWordPrintable

    Details

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

      Description

      We think we aren't actually using this, and we'd kind of like to use lsst.geom to mean something else.

        Attachments

          Issue Links

            Activity

            No builds found.
            jbosch Jim Bosch created issue -
            Hide
            jbosch Jim Bosch added a comment -

            I've pushed two branches with a start at this, passing off to Russell Owen to finish up (we did those initial changes together, actually).

            Show
            jbosch Jim Bosch added a comment - I've pushed two branches with a start at this, passing off to Russell Owen to finish up (we did those initial changes together, actually).
            jbosch Jim Bosch made changes -
            Field Original Value New Value
            Team Data Release Production [ 10301 ] Alert Production [ 10300 ]
            Assignee Jim Bosch [ jbosch ] Russell Owen [ rowen ]
            rowen Russell Owen made changes -
            Link This issue is triggered by RFC-458 [ RFC-458 ]
            Hide
            rowen Russell Owen added a comment - - edited

            I fixed all instance I could find. Jenkins passed (except on MacOS due to a github communication error) https://ci.lsst.codes/job/stack-os-matrix/27632 I plan to wait for RFC-458 to be accepted before getting this reviewed and merged.

            Show
            rowen Russell Owen added a comment - - edited I fixed all instance I could find. Jenkins passed (except on MacOS due to a github communication error) https://ci.lsst.codes/job/stack-os-matrix/27632 I plan to wait for RFC-458 to be accepted before getting this reviewed and merged.
            jbosch Jim Bosch made changes -
            Link This issue blocks DM-13065 [ DM-13065 ]
            rowen Russell Owen made changes -
            Summary Remove skypix and lsst.geom from stack dependencies Remove skypix from stack dependencies
            Hide
            rowen Russell Owen added a comment -

            Originally this included removing lsst.geom but that I made that a separate ticket: DM-13790

            Show
            rowen Russell Owen added a comment - Originally this included removing lsst.geom but that I made that a separate ticket: DM-13790
            rowen Russell Owen made changes -
            Reviewers Pim Schellart [ pschella ]
            Status To Do [ 10001 ] In Review [ 10004 ]
            rowen Russell Owen made changes -
            Link This issue relates to DM-13790 [ DM-13790 ]
            rowen Russell Owen made changes -
            Sprint AP S18-4 [ 684 ]
            Story Points 2
            Labels SciencePipelines
            Hide
            pschella Pim Schellart [X] (Inactive) added a comment -

            You may want to change the titles in the PR to reflect the fact that it only applies to removing skypix now.

            Show
            pschella Pim Schellart [X] (Inactive) added a comment - You may want to change the titles in the PR to reflect the fact that it only applies to removing skypix now.
            Hide
            pschella Pim Schellart [X] (Inactive) added a comment -

            Looks good. Feel free to merge without changes.
            On a related note, I'm curious why there is so much duplicate code in the obs packages?

            Show
            pschella Pim Schellart [X] (Inactive) added a comment - Looks good. Feel free to merge without changes. On a related note, I'm curious why there is so much duplicate code in the obs packages?
            pschella Pim Schellart [X] (Inactive) made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            rowen Russell Owen made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]
            rowen Russell Owen made changes -
            Link This issue relates to DM-13798 [ DM-13798 ]
            swinbank John Swinbank made changes -
            Epic Link DM-12728 [ 36327 ]

              People

              Assignee:
              rowen Russell Owen
              Reporter:
              jbosch Jim Bosch
              Reviewers:
              Pim Schellart [X] (Inactive)
              Watchers:
              Jim Bosch, Pim Schellart [X] (Inactive), Russell Owen
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.