From 96a05c46aec77dea853f08752323b39ce560b4fc Mon Sep 17 00:00:00 2001 From: OwenKephart Date: Sat, 16 Nov 2024 12:31:12 -0800 Subject: [PATCH] [DA] Fix issue with label for dep operators (#25966) ## Summary & Motivation See changelog ## How I Tested These Changes ## Changelog Fixed an issue that would cause the label for `AutomationCondition.any_deps_match()` and `AutomationCondition.all_deps_match()` to render incorrectly when `allow_selection` or `ignore_selection` were set. --- .../declarative_automation/operators/dep_operators.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/python_modules/dagster/dagster/_core/definitions/declarative_automation/operators/dep_operators.py b/python_modules/dagster/dagster/_core/definitions/declarative_automation/operators/dep_operators.py index d217f6e765ef3..78682a944d9a2 100644 --- a/python_modules/dagster/dagster/_core/definitions/declarative_automation/operators/dep_operators.py +++ b/python_modules/dagster/dagster/_core/definitions/declarative_automation/operators/dep_operators.py @@ -75,9 +75,9 @@ def name(self) -> str: name = self.base_name props = [] if self.allow_selection is not None: - props.append("allow_selection={self.allow_selection}") + props.append(f"allow_selection={self.allow_selection}") if self.ignore_selection is not None: - props.append("ignore_selection={self.ignore_selection}") + props.append(f"ignore_selection={self.ignore_selection}") if props: name += f"({','.join(props)})"