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

Add rubin-env-developer conda-forge variant

    XMLWordPrintable

Details

    • RFC
    • Status: Implemented
    • Resolution: Done
    • DM
    • None

    Description

      The developer guide now refers to black, isort, and pre-commit but those packages are not easily available to people using a read-only conda environment because they are not part of rubin-env. This RFC proposes a new rubin-env-dev package that includes developer tools that will not be needed at pipelines run time. This should include:

      • mypy (plus types extension packages)
      • isort
      • pre-commit
      • black
      • pydocstyle
      • build (to be able to test pypi distribution building locally)

      Since these are used by middleware. We should also consider add on packages that are used by the shared stack and decide on extras vs dev.

      Additionally a --dev option should be added to lsstinstall and lsstsw deploy to install these packages rather than the default rubin-env. The shared stack should be built with this option and if we are using CernVMFS weeklies for development at USDF we should ask for that stack to be using the option as well.

      Attachments

        Issue Links

          Activity

            ktl Kian-Tat Lim added a comment -

            pdbpp requires fancycompleter, which is not yet py310-compatible.

            In conda-forge, snake-viz is spelled snakeviz and line-profiler is spelled line_profiler.

            ktl Kian-Tat Lim added a comment - pdbpp requires fancycompleter , which is not yet py310-compatible. In conda-forge, snake-viz is spelled snakeviz and line-profiler is spelled line_profiler .
            Parejkoj John Parejko added a comment -

            I'm trying to help move the pdbpp/fancycompleter fixes along.

            Parejkoj John Parejko added a comment - I'm trying to help move the pdbpp/fancycompleter fixes along.
            tjenness Tim Jenness added a comment -

            obs_base is about to need types-python-dateutil (it's used by the curated calibration reading code)

            tjenness Tim Jenness added a comment - obs_base is about to need types-python-dateutil (it's used by the curated calibration reading code)
            ktl Kian-Tat Lim added a comment -

            How do people feel about adding a rubin-env-build that would be needed for source builds (but could make the main rubin-env for binaries considerably lighter)? rubin-env-developer would then add useful tools on top of -build. Is this excessive proliferation?

            ktl Kian-Tat Lim added a comment - How do people feel about adding a rubin-env-build that would be needed for source builds (but could make the main rubin-env for binaries considerably lighter)? rubin-env-developer would then add useful tools on top of -build . Is this excessive proliferation?
            tjenness Tim Jenness added a comment -

            Was adopted with rubin-env-developer.

            Unfortunately, it doesn't look like we recorded whether we approved the rubin-env-build variant or not.

            tjenness Tim Jenness added a comment - Was adopted with rubin-env-developer. Unfortunately, it doesn't look like we recorded whether we approved the rubin-env-build variant or not.

            People

              tjenness Tim Jenness
              tjenness Tim Jenness
              Colin Slater, Eli Rykoff, Frossie Economou, Jim Bosch, John Parejko, Jonathan Sick, Kian-Tat Lim, Leanne Guy, Steve Pietrowicz, Tim Jenness, Wil O'Mullane, Yusra AlSayyad
              Votes:
              0 Vote for this issue
              Watchers:
              12 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                Planned End:

                Jenkins

                  No builds found.