Monkey-patch https://github.com/pyiron/executorlib/pull/943

[liamhuber]

No description provided.

[github-actions]

👈 Launch a binder notebook on branch pyiron/pyiron_workflow/internalize-sed

[liamhuber]

I'll first let it roll to see if it causes any new problems, and whether it solves the simple integration test problem. If it does, I'll roll back the sed clause in the slurm tests and see if it still works. If all those are green lights, then I guess pyiron/executorlib#943 will resolve everything here 🤞

[liamhuber]

Well, what I did re-introduced the failure on the slurm test, which is sad

[liamhuber]

Aha, because in my short-circuit I left out an intermediate step

[liamhuber]

Ah, sadly this attack is misguided. The patch targets a task scheduler, but I am mixing in a new class inheriting from BaseExecutor with the NodeSlurmExecutor. While these also have a shutdown method for me to override, they don't have the same properties as the task scheduler in the base patch and I hit: AttributeError: 'NodeSlurmExecutor' object has no attribute '_process'