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

Remove skypix from stack dependencies

    XMLWordPrintable

Details

    • Story
    • Status: Done
    • Resolution: Done
    • None
    • None
    • 2
    • AP S18-4
    • Alert Production

    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

            jbosch Jim Bosch added a comment -

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

            jbosch Jim Bosch added a comment - I've pushed two branches with a start at this, passing off to rowen to finish up (we did those initial changes together, actually).
            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.

            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.
            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 added a comment - Originally this included removing lsst.geom but that I made that a separate ticket: DM-13790

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

            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.

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

              rowen Russell Owen
              jbosch Jim Bosch
              Pim Schellart [X] (Inactive)
              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.