Skip to content

Commit

Permalink
Replace use of toExpandedPipeline in dotTools
Browse files Browse the repository at this point in the history
This package will be deprecated shortly.
  • Loading branch information
timj committed Aug 9, 2024
1 parent 73293e7 commit 5ba267e
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions python/lsst/ctrl/mpexec/dotTools.py
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,6 @@
import html
import io
import re
import warnings
from collections.abc import Iterable
from typing import TYPE_CHECKING, Any

Expand Down Expand Up @@ -282,10 +281,8 @@ def expand_dimensions(connection: connectionTypes.BaseConnection) -> list[str]:
allDatasets: set[str | tuple[str, str]] = set()
if isinstance(pipeline, Pipeline):
# TODO: DM-40639 will rewrite this code and finish off the deprecation
# of toExpandedPipeline.
with warnings.catch_warnings():
warnings.simplefilter("ignore", category=FutureWarning)
pipeline = pipeline.toExpandedPipeline()
# of toExpandedPipeline but for now use the compatibility API.
pipeline = pipeline.to_graph()._iter_task_defs()

Check warning on line 285 in python/lsst/ctrl/mpexec/dotTools.py

View check run for this annotation

Codecov / codecov/patch

python/lsst/ctrl/mpexec/dotTools.py#L285

Added line #L285 was not covered by tests

# The next two lines are a workaround until DM-29658 at which time metadata
# connections should start working with the above code
Expand Down

0 comments on commit 5ba267e

Please sign in to comment.