Skip to content

Commit 8f62329

Browse files
committed
Move dotTools module and test to pipe_base.
1 parent 43e2950 commit 8f62329

4 files changed

Lines changed: 9 additions & 428 deletions

File tree

python/lsst/ctrl/mpexec/__init__.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@
2020
# along with this program. If not, see <https://www.gnu.org/licenses/>.
2121

2222
from .cmdLineFwk import *
23-
from .dotTools import *
2423
from .executionGraphFixup import *
2524
from .mpGraphExecutor import *
2625
from .preExecInit import *

python/lsst/ctrl/mpexec/cmdLineFwk.py

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -53,11 +53,12 @@
5353
QuantumGraph,
5454
TaskDef,
5555
buildExecutionButler,
56+
graph2dot,
57+
pipeline2dot,
5658
)
5759
from lsst.utils import doImport
5860

5961
from . import util
60-
from .dotTools import graph2dot, pipeline2dot
6162
from .executionGraphFixup import ExecutionGraphFixup
6263
from .mpGraphExecutor import MPGraphExecutor
6364
from .preExecInit import PreExecInit
@@ -534,10 +535,15 @@ def makePipeline(self, args):
534535
pipeline.write_to_uri(args.save_pipeline)
535536

536537
if args.expand_pipeline:
537-
pipeline.write_to_uri(args.expand_pipeline, expand=True)
538+
task_defs = list(pipeline)
539+
pipeline.write_to_uri(args.expand_pipeline, expand=True, task_defs=task_defs)
540+
else:
541+
task_defs = None
538542

539543
if args.pipeline_dot:
540-
pipeline2dot(pipeline, args.pipeline_dot)
544+
# Pass in existing task_defs if we have them to avoid re-importing
545+
# and expanding everything.
546+
pipeline2dot(pipeline if task_defs is None else task_defs, args.pipeline_dot)
541547

542548
return pipeline
543549

python/lsst/ctrl/mpexec/dotTools.py

Lines changed: 0 additions & 266 deletions
This file was deleted.

0 commit comments

Comments
 (0)