From 12883c3c9065e8d7c0a32136e74f7d1ea94e81b3 Mon Sep 17 00:00:00 2001 From: Tyler Goodlet Date: Fri, 10 Mar 2023 09:56:55 -0500 Subject: [PATCH] Don't double send `enable_modules` and `debug_mode` in kwargs.. This broke non-disti-mode actor tree spawn / runtime, seemingly because the cli entrypoint for a `piker chart` also sends these values down through the call stack independently? Pretty sure we don't need to send the `enable_modules` from the chart actor anyway. --- piker/service/_actor_runtime.py | 2 -- piker/ui/cli.py | 3 --- 2 files changed, 5 deletions(-) diff --git a/piker/service/_actor_runtime.py b/piker/service/_actor_runtime.py index 42829990..b92ad221 100644 --- a/piker/service/_actor_runtime.py +++ b/piker/service/_actor_runtime.py @@ -174,7 +174,6 @@ async def open_pikerd( # having the root have more then permissions to # spawn other specialized daemons I think? enable_modules=_root_modules, - loglevel=loglevel, debug_mode=debug_mode, registry_addr=registry_addr, @@ -330,7 +329,6 @@ async def maybe_open_pikerd( # configured address async with open_pikerd( loglevel=loglevel, - debug_mode=kwargs.get('debug_mode', False), registry_addr=registry_addr, # ahabd (docker super) specific controls diff --git a/piker/ui/cli.py b/piker/ui/cli.py index 9b8385f2..15b3e9f6 100644 --- a/piker/ui/cli.py +++ b/piker/ui/cli.py @@ -181,9 +181,6 @@ def chart( 'debug_mode': pdb, 'loglevel': tractorloglevel, 'name': 'chart', - 'enable_modules': [ - 'piker.clearing._client' - ], 'registry_addr': config.get('registry_addr'), }, )