[Draft] Fix BackgroundCommand.send_signal() and add trace-cmd record support

ARM-software / devlib

Library for interaction with and instrumentation of remote devices.

Apache License 2.0

47 stars 78 forks source link

[Draft] Fix BackgroundCommand.send_signal() and add trace-cmd record support #689

Open douglas-raillard-arm opened 4 months ago

douglas-raillard-arm commented 4 months ago

Fixes https://github.com/ARM-software/devlib/issues/645 Supersedes https://github.com/ARM-software/devlib/pull/643

TODO:

[ ] Add/update docstrings
[x] Figure out if moving some init logic for the various subclasses of BackgroundCommand inside the class would simplify the code and avoid BackgroundCommand.from_factory() layer.

douglas-raillard-arm commented 4 months ago

Upon deeper inspection, the from_factory() solution is still better than moving the code from <subclass of Target>._background() to their associated BackgroundCommand subclass, as it typically requires using a bunch of private function/method from the connection type.

douglas-raillard-arm commented 3 months ago

PR rebased