diff options
| author | Christian Kolset <ckolset@colostate.edu> | 2026-07-29 12:59:10 -0600 |
|---|---|---|
| committer | Christian Kolset <ckolset@colostate.edu> | 2026-07-29 12:59:10 -0600 |
| commit | 9360dea6e6327004b905eb6d7c782cc7c0d3ba13 (patch) | |
| tree | 54366e05a55f96cd4c676d71cfb2136d72392984 /devices/base_device.py | |
| parent | b407a079000f6a4b04ecf38eca17c57719e7a7ec (diff) | |
Hide non-writable channels from Controls picker, gate writes on enabled+writable
Adds ChannelConfig.writable so the Controls editor can only bind to
channels with a real write mapping (VS/MA/ME/MD/ST for CML, do*/DO for
digital I/O, write_cmd-configured SCPI channels, holding-register Modbus
channels) instead of read-only telemetry — this is exactly the class of
mistake hit earlier (binding a motor-speed control to M1_TV instead of
M1_VS). Also carries forward the enabled-channel gating for controls
(hide disabled channels from the picker, block writes to them at
_write() time) done alongside this investigation.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Diffstat (limited to 'devices/base_device.py')
| -rw-r--r-- | devices/base_device.py | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/devices/base_device.py b/devices/base_device.py index 32fe787..83b8536 100644 --- a/devices/base_device.py +++ b/devices/base_device.py @@ -29,6 +29,7 @@ class ChannelConfig: max_value: float = 100.0 enabled: bool = True color: str = "#00d4ff" + writable: bool = False # True if this channel accepts write_channel() calls extra: Dict[str, Any] = field(default_factory=dict) |