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

Remove skypix from stack dependencies

    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

            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).
            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.
            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
            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?

              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:

                  Summary Panel