Skip to content

Commit

Permalink
Go all in on "task manager" naming
Browse files Browse the repository at this point in the history
  • Loading branch information
goodboy committed May 19, 2023
1 parent 940e65f commit b24f0d5
Showing 1 changed file with 8 additions and 11 deletions.
19 changes: 8 additions & 11 deletions tractor/trionics/_supervisor.py
Original file line number Diff line number Diff line change
Expand Up @@ -102,22 +102,22 @@ async def wait_for_result(self) -> Any:
return self.result


class ScopePerTaskNursery(Struct):
class TaskManagerNursery(Struct):
_n: Nursery
_scopes: dict[
Task,
tuple[CancelScope, Outcome]
] = {}

scope_manager: Generator[Any, Outcome, None] | None = None
task_manager: Generator[Any, Outcome, None] | None = None

async def start_soon(
self,
async_fn,
*args,

name=None,
scope_manager: ContextManager | None = None,
task_manager: Generator[Any, Outcome, None] | None = None

) -> tuple[CancelScope, Task]:

Expand All @@ -131,7 +131,7 @@ async def start_soon(

n: Nursery = self._n

sm = self.scope_manager
sm = self.task_manager
# we do default behavior of a scope-per-nursery
# if the user did not provide a task manager.
if sm is None:
Expand Down Expand Up @@ -261,14 +261,11 @@ def add_task_handle_and_crash_handling(

@acm
async def open_nursery(
scope_manager = None,
task_manager = None,
**kwargs,
):
async with trio.open_nursery(**kwargs) as nurse:
yield ScopePerTaskNursery(
nurse,
scope_manager=scope_manager,
)
yield TaskManagerNursery(nurse, task_manager=task_manager)


async def sleep_then_return_val(val: str):
Expand All @@ -293,7 +290,7 @@ async def ensure_cancelled():

async def main():
async with open_nursery(
scope_manager=add_task_handle_and_crash_handling,
task_manager=add_task_handle_and_crash_handling,
) as sn:
for _ in range(3):
outcome, _ = await sn.start_soon(trio.sleep_forever)
Expand All @@ -312,7 +309,7 @@ async def main():

await trio.sleep(0.6)
print(
'Cancelling and waiting on {err_outcome.lowlevel_task} '
f'Cancelling and waiting on {err_outcome.lowlevel_task} '
'to CRASH..'
)
cs.cancel()
Expand Down

0 comments on commit b24f0d5

Please sign in to comment.