Merge pull request #30 from lsst/tickets/DM-43721

DM-43721: Fix compute_site keyword error in submit.
lsst · Apr 8, 2024 · ab80713 · ab80713
2 parents 145af14 + f57de89
commit ab80713
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/doc/changes/DM-43721.bugfix.rst b/doc/changes/DM-43721.bugfix.rst
@@ -0,0 +1 @@
+Fix compute_site keyword error in submit introduced by DM-38138.
diff --git a/python/lsst/ctrl/bps/parsl/service.py b/python/lsst/ctrl/bps/parsl/service.py
@@ -25,6 +25,8 @@
 # You should have received a copy of the GNU General Public License
 # along with this program.  If not, see <https://www.gnu.org/licenses/>.
 
+from typing import Any
+
 from lsst.ctrl.bps import BaseWmsService, BaseWmsWorkflow, BpsConfig, GenericWorkflow
 
 from .workflow import ParslWorkflow
@@ -61,13 +63,15 @@ def prepare(
         workflow.write(out_prefix)
         return workflow
 
-    def submit(self, workflow: BaseWmsWorkflow):
+    def submit(self, workflow: BaseWmsWorkflow, **kwargs: Any):
         """Submit a single WMS workflow.
 
         Parameters
         ----------
         workflow : `lsst.ctrl.bps.BaseWmsWorkflow`
             Prepared WMS Workflow to submit for execution.
+        **kwargs : `~typing.Any`
+            Additional modifiers to the configuration.
         """
         workflow.start()
         workflow.run()