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

Document the need for abstract base tasks for tasks

    XMLWordPrintable

    Details

    • Story Points:
      2
    • Team:
      Architecture

      Description

      As part of RFC-183 document the fact that variant tasks should have a common abstract base class that defines the API. If we add future tasks that we feel are likely to have variants, then we should create an abstract base class.

      Candidates include star selectors, PSF determiners and ISR tasks.

      Note that this applies to tasks LSST provides in its stack, not to variants users produce and other obscure one-off code.

      Also document the desire that tasks with anticipated many variants, such as star selectors, and PSF determiners should be in registries. This explicitly excludes tasks such as ISR where only one task is likely to be useful for a given set of data.

        Attachments

          Issue Links

            Activity

            Hide
            swinbank John Swinbank added a comment -

            Reviewed by DM-CCB, 2020-01-15.

            Show
            swinbank John Swinbank added a comment - Reviewed by DM-CCB, 2020-01-15.
            Hide
            ktl Kian-Tat Lim added a comment -

            I decided the easiest way to get rid of this ticket was to do it. I put the text in the pipe_base task-writing document.

            Show
            ktl Kian-Tat Lim added a comment - I decided the easiest way to get rid of this ticket was to do it. I put the text in the pipe_base task-writing document.
            Hide
            rowen Russell Owen added a comment -

            Reviewed on github. Looks very helpful.

            Show
            rowen Russell Owen added a comment - Reviewed on github. Looks very helpful.
            Hide
            ktl Kian-Tat Lim added a comment -

            Oops, accidentally set back to In Review.

            Show
            ktl Kian-Tat Lim added a comment - Oops, accidentally set back to In Review.
            Hide
            ktl Kian-Tat Lim added a comment -

            Used Russell's suggestions and merged.

            Show
            ktl Kian-Tat Lim added a comment - Used Russell's suggestions and merged.

              People

              Assignee:
              ktl Kian-Tat Lim
              Reporter:
              rowen Russell Owen
              Reviewers:
              Russell Owen
              Watchers:
              Jim Bosch, John Swinbank, Kian-Tat Lim, Krzysztof Findeisen, Paul Price, Russell Owen
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.