Upgrade create
interface to be compliant with executorlib
0.1.0
#584
Labels
create
interface to be compliant with executorlib
0.1.0
#584
#583 is going to fail because we
from executorlib import Executor
, which is no longer available since this was decomposed into a variety of more specific executors.I see three likely resolutions:
Workflow.create.ExecutorlibExecutor
, we'd have something likeWorkflow.create.executorlib.SingleNodeExecutor
orWorkflow.create.executorlib.SlurmJobExecutor
.executorlib
in the test suite, and probably the notebooks where we sell it, but then they wouldn't need to be in the core dependencies.I like both (2) and (3). Since the mandatory dependencies are so light for
executorlib
, I would probably go for (2) as a first attack.I don't mind this issue being on my plate, but I'm not likely to be able to get to it imminently and it should be a relatively straightforward fix, so I laid out the attack patterns here in case someone else wants to take a crack at it in the meantime.
The text was updated successfully, but these errors were encountered: