Uploaded image for project: 'Request For Comments'
  1. Request For Comments
  2. RFC-519

Add ginga and ipyevents as third party packages

    XMLWordPrintable

    Details

    • Type: RFC
    • Status: Implemented
    • Resolution: Done
    • Component/s: DM
    • Labels:
      None

      Description

      The display_ginga package cannot be installed with eups distrib install because optional dependencies are not respected (see DM-15299). This means we need to install ginga before the display package. I am also asking to add ipyevents since ginga isn't really usable in notebooks without it.

      Both of these packages install trivially with pip, I do not believe it would be hard to install them from source.

        Attachments

          Issue Links

          is triggering

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-15741 Add ginga and ipyevents as packages

          • Done
          relates to

          Bug - A problem which impairs or prevents the functions of the product. DM-15299 Cannot "rebuild display_ginga"

          • Done

          RFC - An issue whose primary purpose is to advertise an intent and allow stakeholders to comment. RFC-829 Add display_astrowidgets to lsst_distrib

          • Implemented
          mentioned in

          Page Loading...

            Activity

            Ascending order - Click to sort in descending order
            4 older comments
            Hide
            Permalink
            swinbank John Swinbank added a comment -

            Are you suggesting a change in that policy?

            I don't think that's a policy; I think it's a mistake!

            Show
            swinbank John Swinbank added a comment - Are you suggesting a change in that policy? I don't think that's a policy; I think it's a mistake!
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            The fix is to document that the list of python requirements are in the lsstsw package.

            Show
            tjenness Tim Jenness added a comment - The fix is to document that the list of python requirements are in the lsstsw package.
            Hide
            Permalink
            krughoff Simon Krughoff added a comment -

            I can add a link to the documentation if we end up going down that road. Though it looks like neither are vanilla conda installable, so we may need EUPS packages anyway.

            Show
            krughoff Simon Krughoff added a comment - I can add a link to the documentation if we end up going down that road. Though it looks like neither are vanilla conda installable, so we may need EUPS packages anyway.
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Simon Krughoff it looks like you can adopt this RFC. There don't seem to be any objections and it seems like you should use TaP packages.

            Show
            tjenness Tim Jenness added a comment - Simon Krughoff it looks like you can adopt this RFC. There don't seem to be any objections and it seems like you should use TaP packages.
            Hide
            Permalink
            krughoff Simon Krughoff added a comment -

            Adopting with implementation ticket: DM-15741.

            Show
            krughoff Simon Krughoff added a comment - Adopting with implementation ticket: DM-15741 .

              People

              Assignee:
              krughoff Simon Krughoff
              Reporter:
              krughoff Simon Krughoff
              Watchers:
              John Swinbank, Joshua Hoblitt, Simon Krughoff, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Planned End:

                  Jenkins

                  No builds found.