Skip to content

Commit e52e32b

Browse files
authored
Merge pull request #739 from ghisvail/bug/workflow-fixes
FIX: Add environment argument to TaskBase._run_task() and Workflow subclass
2 parents 8bdb994 + 14d1e19 commit e52e32b

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

pydra/engine/core.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -380,7 +380,7 @@ def can_resume(self):
380380
return self._can_resume
381381

382382
@abc.abstractmethod
383-
def _run_task(self):
383+
def _run_task(self, environment=None):
384384
pass
385385

386386
@property
@@ -1329,7 +1329,7 @@ async def _run(self, submitter=None, rerun=False, **kwargs):
13291329
self._check_for_hash_changes()
13301330
return result
13311331

1332-
async def _run_task(self, submitter, rerun=False):
1332+
async def _run_task(self, submitter, rerun=False, environment=None):
13331333
if not submitter:
13341334
raise Exception("Submitter should already be set.")
13351335
for nd in self.graph.nodes:

pydra/engine/submitter.py

-1
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,6 @@ async def submit_from_call(self, runnable, rerun, environment):
9090
# 2
9191
if runnable.state is None:
9292
# run_el should always return a coroutine
93-
print("in SUBM", environment)
9493
await self.worker.run_el(runnable, rerun=rerun, environment=environment)
9594
# 3
9695
else:

0 commit comments

Comments
 (0)