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

Add option to deblender to turn off smoothing.

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: meas_deblender
    • Labels:
    • Story Points:
      4
    • Epic Link:
    • Team:
      Data Release Production

      Description

      As shown in DM-8059, the deblender can be improved by turning off the smoothing of the templates. This ticket is to add this option as a config parameter to the deblend task.

        Attachments

          Issue Links

          blocks

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-9870 Release hscPipe 5.0-beta1

          • Done

            Activity

            Ascending order - Click to sort in descending order
            No builds found.
            rearmstr Bob Armstrong created issue -
            swinbank John Swinbank made changes -
            Field Original Value New Value
            Epic Link DM-5789 [ 23855 ]
            swinbank John Swinbank made changes -
            Story Points 4
            Team Data Release Production [ 10301 ]
            rearmstr Bob Armstrong made changes -
            Link This issue blocks DM-9870 [ DM-9870 ]
            rearmstr Bob Armstrong made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            rearmstr Bob Armstrong made changes -
            Assignee Bob Armstrong [ rearmstr ]
            Hide
            Permalink
            rearmstr Bob Armstrong added a comment -

            Thanks for reviewing this.

            Show
            rearmstr Bob Armstrong added a comment - Thanks for reviewing this.
            rearmstr Bob Armstrong made changes -
            Reviewers Paul Price [ price ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            Hide
            Permalink
            price Paul Price added a comment -

            This changes the default behaviour. Was that intended? If so, please add a note in the commit message.

            Don't need the parens around the doc part of the Field.

            Show
            price Paul Price added a comment - This changes the default behaviour. Was that intended? If so, please add a note in the commit message. Don't need the parens around the doc part of the Field .
            price Paul Price made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            Hide
            Permalink
            rearmstr Bob Armstrong added a comment -

            No, this was not intended. We want to be able to turn it on for some of the RC testing.

            Show
            rearmstr Bob Armstrong added a comment - No, this was not intended. We want to be able to turn it on for some of the RC testing.
            Hide
            Permalink
            rearmstr Bob Armstrong added a comment -

            I also removed the extra parentheses. It looks like a number of config options have this from which I copied and pasted.

            Show
            rearmstr Bob Armstrong added a comment - I also removed the extra parentheses. It looks like a number of config options have this from which I copied and pasted.
            rearmstr Bob Armstrong made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]
            price Paul Price made changes -
            Component/s meas_deblender [ 10741 ]

              People

              Assignee:
              rearmstr Bob Armstrong
              Reporter:
              rearmstr Bob Armstrong
              Reviewers:
              Paul Price
              Watchers:
              Bob Armstrong, Paul Price
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.