Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DM-2639: {{Standardize primary method names, run/runDataRef, across PipeTasks}} #125

Merged
merged 1 commit into from Aug 3, 2018

Conversation

czwa
Copy link
Contributor

@czwa czwa commented Jul 26, 2018

Rename methods according to RFC-352.
Fifteen total repositories with changes.

Copy link
Contributor

@r-owen r-owen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is another case where adding a run method would probably make sense.

Prepare Tasks for Gen3 PipelineTask conversion by:
* Renaming CmdlineTask's entry method to `runDataRef`.  The default
  CmdlineTask.TaskRunner now calls a Task's `runDataRef` method on the
  parsed command line inputs.
  `runDataRef` method can take any Gen2 Butler data products.
* Renaming CmdlineTasks previous core methods (e.g. assemble,
  characterize) to `run` when they exist.
@czwa czwa merged commit 4bafbe1 into master Aug 3, 2018
@czwa
Copy link
Contributor Author

czwa commented Aug 3, 2018

In the interest in merging the changes for this ticket that refactoring depends on, @yalsayyad has helped by opening a ticket on this refactorization. DM-15310 covers this change, and I'll begin work on it next week.

@ktlim ktlim deleted the tickets/DM-2639 branch August 25, 2018 06:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants