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

Please document the appropriate location for tasks

    XMLWordPrintable

    Details

      Description

      On DM-6631, Russell Owen asserts:

      Command-line tasks are encouraged to be in the package that makes most sense for them, so IsrTask belongs in ip_isr. pipe_tasks is intended for tasks (command-line and otherwise) that need so many packages that they don't make sense being in a lower-level package.

      I certainly don't disagree – seems perfectly reasonable – but I've not seen it written down before. Please capture this in the developer documentation.

        Attachments

          Issue Links

            Activity

            Hide
            rowen Russell Owen added a comment -

            John Swinbank I suggest you file a ticket asking for the documentation improvement. Existing documentation includes the task overview main.dox in pipe_base, "how to write a task" and "how to write a command-line task". There are surely others.

            Show
            rowen Russell Owen added a comment - John Swinbank I suggest you file a ticket asking for the documentation improvement. Existing documentation includes the task overview main.dox in pipe_base, "how to write a task" and "how to write a command-line task". There are surely others.
            Hide
            swinbank John Swinbank added a comment -

            Russell Owen: I think this is a ticket asking for the documentation to be improved. Am I misunderstanding your point?

            Show
            swinbank John Swinbank added a comment - Russell Owen : I think this is a ticket asking for the documentation to be improved. Am I misunderstanding your point?
            Hide
            rowen Russell Owen added a comment -

            I misread is as part of a different ticket.

            Show
            rowen Russell Owen added a comment - I misread is as part of a different ticket.
            Hide
            tjenness Tim Jenness added a comment -

            Is this clarification still needed? Is it dev guide, pipelines.lsst.io, or pipe_task that should be updated? Is this now something for science pipelines to take on or still square?

            Show
            tjenness Tim Jenness added a comment - Is this clarification still needed? Is it dev guide, pipelines.lsst.io, or pipe_task that should be updated? Is this now something for science pipelines to take on or still square?
            Hide
            swinbank John Swinbank added a comment -

            It's clearly not a SQuaRE issue. Let's just close it.

            Show
            swinbank John Swinbank added a comment - It's clearly not a SQuaRE issue. Let's just close it.

              People

              Assignee:
              jsick Jonathan Sick
              Reporter:
              swinbank John Swinbank
              Watchers:
              John Swinbank, Russell Owen, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.