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

Change name of processFakes to match the task name processCcdWithFakes

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    • Story Points:
      1
    • Sprint:
      DRP F19-1, DRP F19-2
    • Team:
      Data Release Production

      Description

      The name of the file should match the name of the task it contains.

        Attachments

          Issue Links

            Activity

            Hide
            sophiereed Sophie Reed added a comment -

            pipe_task PR not showing up: https://github.com/lsst/pipe_tasks/pull/301

            Show
            sophiereed Sophie Reed added a comment - pipe_task PR not showing up:  https://github.com/lsst/pipe_tasks/pull/301
            Hide
            sophiereed Sophie Reed added a comment -

            If you have a chance to review this that would be great. Thank you.

            Show
            sophiereed Sophie Reed added a comment - If you have a chance to review this that would be great. Thank you.
            Hide
            Parejkoj John Parejko added a comment -

            Thanks for doing this rename. This is an improvement.

            I think it's probably worth doing the same for `insertFakes`; wouldn't that be `insertFakesIntoCoadd` or somesuch?

            Show
            Parejkoj John Parejko added a comment - Thanks for doing this rename. This is an improvement. I think it's probably worth doing the same for `insertFakes`; wouldn't that be `insertFakesIntoCoadd` or somesuch?

              People

              Assignee:
              sophiereed Sophie Reed
              Reporter:
              sophiereed Sophie Reed
              Reviewers:
              John Parejko
              Watchers:
              John Parejko, Sophie Reed, Yusra AlSayyad
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins Builds

                  No builds found.