From 4ce05f77da98bb1a6dffca5fb584df956f0c167e Mon Sep 17 00:00:00 2001 From: Tim Jenness Date: Wed, 27 Sep 2023 10:52:54 -0700 Subject: [PATCH] Remove writing of expanded pipeline That option does not exist. --- python/lsst/pipe/base/pipeline_doc_builder.py | 19 +------------------ 1 file changed, 1 insertion(+), 18 deletions(-) diff --git a/python/lsst/pipe/base/pipeline_doc_builder.py b/python/lsst/pipe/base/pipeline_doc_builder.py index ebf076cb4..a6e8e2836 100644 --- a/python/lsst/pipe/base/pipeline_doc_builder.py +++ b/python/lsst/pipe/base/pipeline_doc_builder.py @@ -376,7 +376,7 @@ def from_dirs( def __call__(self, task_defs: Sequence[TaskDef] | None = None) -> None: if task_defs is None: task_defs = list(self.pipeline) - self.write_expanded_pipeline(task_defs) + self.pipeline.write_to_uri(self.yaml_path.parent) self.write_dot(task_defs) self.write_rst(task_defs) @@ -406,23 +406,6 @@ def iter_graph_dot_paths(self) -> Iterator[tuple[Path, Path]]: for task_paths in self.tasks.values(): yield (task_paths.graph_path, task_paths.dot_path) - def write_expanded_pipeline(self, task_defs: Sequence[TaskDef] | None = None) -> None: - """Write the expanded pipeline. - - This just calls `Pipeline.write_to_uri` with ``expand=True``. - - Parameters - ---------- - task_defs : `Sequence` [ `TaskDef` ], optional - The result of a call to `Pipeline.toExpandedPipeline`, captured in - a sequence. May be `None` (default) to expand internally; provided - as a way for calling code to only expand the pipeline once. - """ - if task_defs is None: - task_defs = list(self.pipeline) - self.yaml_path.parent.mkdir(parents=True, exist_ok=True) - self.pipeline.write_to_uri(self.yaml_path.parent, expand=True, task_defs=task_defs) - def write_dot(self, task_defs: Sequence[TaskDef] | None = None) -> None: """Write the GraphViz DOT representations of the pipeline and its tasks.