[Bdt] Switch to dcargs

[gbalke]

Signed-off-by: Greg Balke [email protected]

Topic: bdt_uses_dcargs
Reviewers: brent

[gbalke]

Reviews in this chain:
└#69 [Bdt] Switch to dcargs

[gbalke]

#	head	base	diff	date	summary
0	83f41825	f552a7fd	diff	Sep 25 9:13 PM	4 files changed, 94 insertions(+), 61 deletions(-)
1	0b7072fc	f552a7fd	diff	Oct 04 12:44 AM	1 file changed, 5 insertions(+), 4 deletions(-)

[brentyi]

probably a literal could be used here?

from typing import Literal

actuation: Literal["current", "phase"], etc

[brentyi]

in general, variable-length positional arguments are scary because it means you can't have add more positional args afterward

[brentyi]

is this still needed? since board_ids is annotated as List[int] now

[gbalke]

Discussed offline... I think the current behavior where betz drive tools creates a connection per tool execution is a pretty poor design for doing continuous testing. I believe dcargs will work much better if it doesn't have to deal with lists of lists of command args (which is the current hacky think I'm doing for control_motor. I'll leave this PR and hold off until I have a better architecture in place for the server system.