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

Add display_astrowidgets to lsst_distrib

    XMLWordPrintable

    Details

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

      Description

      display_astrowidgets is an afw.display compatible interface to allow astrowidgets/ginga displays to be created in notebooks.  The prerequisites (ginga in particular) are already installed on the RSP.  I have tested that the stack club tutorial for Firefly runs with the backend switched to astrowidgets, and will attach that notebook to this ticket.

      There are some new changes on the DM-33580 ticket branch for display_astrowidgets, and that ticket will be used for the implementation.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            Christopher Waters do you intend to adopt this RFC?

            Show
            tjenness Tim Jenness added a comment - Christopher Waters do you intend to adopt this RFC?
            Hide
            tjenness Tim Jenness added a comment - - edited

            Kian-Tat Lim did we actually resolve the rubin-env problem in the discussion? It seems like we left it with "astrowidgets" being problematic in that it pulls in jupyterlab but we need that to be there before we can add display_astrowidgets to lsst_distrib.

            Show
            tjenness Tim Jenness added a comment - - edited Kian-Tat Lim did we actually resolve the rubin-env problem in the discussion? It seems like we left it with "astrowidgets" being problematic in that it pulls in jupyterlab but we need that to be there before we can add display_astrowidgets to lsst_distrib.
            Hide
            tjenness Tim Jenness added a comment -

            For reference we have two other open RFCs that talk about installing ginga:

            • RFC-519 wants to add ginga and ipyevents as eups packages to support display_ginga (presumably now as conda-forge packages).
            • RFC-420 wants to add display_ginga to the shared stacks (which sounds like it has been done)
            Show
            tjenness Tim Jenness added a comment - For reference we have two other open RFCs that talk about installing ginga: RFC-519 wants to add ginga and ipyevents as eups packages to support display_ginga (presumably now as conda-forge packages). RFC-420 wants to add display_ginga to the shared stacks (which sounds like it has been done)
            Hide
            ktl Kian-Tat Lim added a comment -

            I don't see why it is necessary to have astrowidgets be added to rubin-env in order to add display_astrowidgets to lsst_distrib.  I would prefer that the latter be done with sufficient protections around all usages that no breakage occurs in Jenkins where astrowidgets is not available, but that everything is usable in the shared stack and RSP where it is.

            Show
            ktl Kian-Tat Lim added a comment - I don't see why it is necessary to have astrowidgets be added to rubin-env in order to add display_astrowidgets to lsst_distrib.  I would prefer that the latter be done with sufficient protections around all usages that no breakage occurs in Jenkins where astrowidgets is not available, but that everything is usable in the shared stack and RSP where it is.
            Hide
            ktl Kian-Tat Lim added a comment -

            But I'm also fine with display_astrowidgets not being added to lsst_distrib and only added via eups distrib install to RSP containers and the shared stack.

            Show
            ktl Kian-Tat Lim added a comment - But I'm also fine with display_astrowidgets not being added to lsst_distrib and only added via eups distrib install to RSP containers and the shared stack.

              People

              Assignee:
              czw Christopher Waters
              Reporter:
              czw Christopher Waters
              Watchers:
              Andrés Alejandro Plazas Malagón, Christopher Waters, Eli Rykoff, Jim Bosch, John Parejko, Kian-Tat Lim, Tim Jenness, Wil O'Mullane, Yusra AlSayyad
              Votes:
              0 Vote for this issue
              Watchers:
              9 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Planned End:

                  Jenkins

                  No builds found.