Skip to content

Commit a2cc367

Browse files
authored
Merge pull request #9 from djarecka/mgxd-rf/submitter
adding one more case to submit_from_call
2 parents 85f8630 + 6e46b8f commit a2cc367

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

.pre-commit-config.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ repos:
99
- id: check-yaml
1010
- id: check-added-large-files
1111
- repo: https://github.com/psf/black
12-
rev: 22.1.0
12+
rev: 22.3.0
1313
hooks:
1414
- id: black
1515
- repo: https://github.com/codespell-project/codespell

pydra/engine/submitter.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,8 @@ async def submit_from_call(self, runnable, rerun):
4646
This coroutine should only be called once per Submitter call,
4747
and serves as the bridge between sync/async lands.
4848
49-
There are 3 potential paths based on the type of runnable:
50-
49+
There are 4 potential paths based on the type of runnable:
50+
0) Workflow has a different plugin than a submitter
5151
1) Workflow without State
5252
2) Task without State
5353
3) (Workflow or Task) with State
@@ -58,6 +58,10 @@ async def submit_from_call(self, runnable, rerun):
5858
if is_workflow(runnable):
5959
# connect and calculate the checksum of the graph before running
6060
runnable._connect_and_propagate_to_tasks(override_task_caches=True)
61+
# 0
62+
if runnable.plugin and runnable.plugin != self.plugin:
63+
# if workflow has a different plugin it's treated as a single element
64+
await self.worker.run_el(runnable, rerun=rerun)
6165
# 1
6266
if runnable.state is None:
6367
await runnable._run(self, rerun=rerun)

0 commit comments

Comments
 (0)