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

Fix example of IsrTask to be callable with data on disk

    XMLWordPrintable

    Details

    • Story Points:
      2
    • Sprint:
      Science Pipelines DM-W15-4, Science Pipelines DM-S15-1, Science Pipelines DM-S15-2
    • Team:
      Alert Production

      Description

      Currently the example of the IsrTask takes a fake dataref. This is hard to use with real data. In DM-1113 we will update IsrTask to not take a dataRef. This will make it easy to update the example script to work with real data.

      This ticket will also include removing from the unit tests any fake dataRefs that have become unnecessary as a result of DM-1299.

        Attachments

          Issue Links

            Activity

            Hide
            rhl Robert Lupton added a comment -

            I don't understand the comment about structuring the docs. My page on how to do this (https://confluence.lsstcorp.org/display/DM/How+to+document+a+Task) requests the order:

            1. The Task's purpose
            2. How to initialise the Task object
            3. How to invoke a Task
            4. The config parameters
            5. Debugging options enabled with "import debug"
            6. A complete example.

            and that's pretty much what you propose in your example, although you omitted the sections on Initialisation and Invoking. If you don't like my proposal file an RFC, but please don't a) fail to follow the instructions, or b) tell other people to do the same.

            For a compliant Task, see
            https://lsst-web.ncsa.illinois.edu/doxygen/x_masterDoxyDoc/classlsst_1_1meas_1_1algorithms_1_1detection_1_1_source_detection_task.html#SourceDetectionTask_

            Show
            rhl Robert Lupton added a comment - I don't understand the comment about structuring the docs. My page on how to do this ( https://confluence.lsstcorp.org/display/DM/How+to+document+a+Task ) requests the order: The Task's purpose How to initialise the Task object How to invoke a Task The config parameters Debugging options enabled with "import debug" A complete example. and that's pretty much what you propose in your example, although you omitted the sections on Initialisation and Invoking. If you don't like my proposal file an RFC, but please don't a) fail to follow the instructions, or b) tell other people to do the same. For a compliant Task, see https://lsst-web.ncsa.illinois.edu/doxygen/x_masterDoxyDoc/classlsst_1_1meas_1_1algorithms_1_1detection_1_1_source_detection_task.html#SourceDetectionTask_
            Hide
            rowen Russell Owen added a comment -

            I was not recommending anything against the rules. Robert Lupton If you had looked at the served documentation you would have seen the issue. Many of the early tasks had the debugging information separated into two subsections with the example separating them. The second section was boilerplate that we moved elsewhere. Your example task does it the way I recommended: omitting the second debugging boilerplate and providing a link, instead.

            Show
            rowen Russell Owen added a comment - I was not recommending anything against the rules. Robert Lupton If you had looked at the served documentation you would have seen the issue. Many of the early tasks had the debugging information separated into two subsections with the example separating them. The second section was boilerplate that we moved elsewhere. Your example task does it the way I recommended: omitting the second debugging boilerplate and providing a link, instead.
            Hide
            rhl Robert Lupton added a comment -

            Your comment was:

            Yes Robert Lupton's original template separated them, with the example in the middle, but that makes the debug info very hard to use. This also centralizes boilerplate.

            If this refers to old email it's unhelpful; if it refers to the confluence page I didn't understand what you meant and asked for clarification.

            The rule you didn't follow was to include sections on Initialisation and Invoking in your example.

            Show
            rhl Robert Lupton added a comment - Your comment was: Yes Robert Lupton's original template separated them, with the example in the middle, but that makes the debug info very hard to use. This also centralizes boilerplate. If this refers to old email it's unhelpful; if it refers to the confluence page I didn't understand what you meant and asked for clarification. The rule you didn't follow was to include sections on Initialisation and Invoking in your example.
            Hide
            yusra Yusra AlSayyad added a comment -

            Robert, there are sections on Initialization and Invoking in this example. They are called "Task initialization" and "Inputs/Outputs to the run method." Is that the issue?

            Please look at the doc as linked to my working directory and please let me know what is missing. I'm going to merge this today:

            http://lsst-web.ncsa.illinois.edu/~yusra/ip_isr_doc/classlsst_1_1ip_1_1isr_1_1isr_task_1_1_isr_task.html

            Show
            yusra Yusra AlSayyad added a comment - Robert, there are sections on Initialization and Invoking in this example. They are called "Task initialization" and "Inputs/Outputs to the run method." Is that the issue? Please look at the doc as linked to my working directory and please let me know what is missing. I'm going to merge this today: http://lsst-web.ncsa.illinois.edu/~yusra/ip_isr_doc/classlsst_1_1ip_1_1isr_1_1isr_task_1_1_isr_task.html
            Hide
            yusra Yusra AlSayyad added a comment -

            Russell, thank you for super-fast turnaround and helpful comments.
            I addressed all of them. Re: the display flag, I looked at other examples and saw that --ds9 was the most commonly used flag used to display the results in example tasks. I changed --displayResult to --ds9, and instead of suggesting it in the example call, I showed the three optional flags available on runIsrTask.py explicitly.
            Merged to master

            Show
            yusra Yusra AlSayyad added a comment - Russell, thank you for super-fast turnaround and helpful comments. I addressed all of them. Re: the display flag, I looked at other examples and saw that --ds9 was the most commonly used flag used to display the results in example tasks. I changed --displayResult to --ds9, and instead of suggesting it in the example call, I showed the three optional flags available on runIsrTask.py explicitly. Merged to master

              People

              Assignee:
              yusra Yusra AlSayyad
              Reporter:
              krughoff Simon Krughoff
              Reviewers:
              Russell Owen
              Watchers:
              Robert Lupton, Russell Owen, Simon Krughoff, Yusra AlSayyad
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.