fix: only set transmission when it actually differs
CI / lint (pull_request) Successful in 37s
CI / test (3.12) (pull_request) Successful in 1m3s
CI / test (3.13) (pull_request) Successful in 1m5s
CI / test (3.14) (pull_request) Successful in 1m4s
CI / test-with-beamline-plugins (pxi_bec) (pull_request) Successful in 1m10s
CI / test-with-beamline-plugins (pxii_bec) (pull_request) Successful in 1m17s
CI / test-with-beamline-plugins (pxiii_bec) (pull_request) Successful in 1m21s
CI / test-with-coverage (pull_request) Successful in 1m31s
CI / lint (push) Successful in 32s
Docs build and publish / docker (push) Successful in 14s
CI / coverage-analysis (pull_request) Successful in 4s
CI / test (3.12) (push) Canceled after 45s
CI / test (3.13) (push) Canceled after 41s
CI / test (3.14) (push) Canceled after 40s
CI / test-with-beamline-plugins (pxi_bec) (push) Canceled after 36s
CI / test-with-beamline-plugins (pxii_bec) (push) Canceled after 35s
CI / test-with-beamline-plugins (pxiii_bec) (push) Canceled after 31s
CI / test-with-coverage (push) Canceled after 30s
CI / coverage-analysis (push) Canceled after 0s
Build and Publish / release (push) Successful in 35s
CI / lint (pull_request) Successful in 37s
CI / test (3.12) (pull_request) Successful in 1m3s
CI / test (3.13) (pull_request) Successful in 1m5s
CI / test (3.14) (pull_request) Successful in 1m4s
CI / test-with-beamline-plugins (pxi_bec) (pull_request) Successful in 1m10s
CI / test-with-beamline-plugins (pxii_bec) (pull_request) Successful in 1m17s
CI / test-with-beamline-plugins (pxiii_bec) (pull_request) Successful in 1m21s
CI / test-with-coverage (pull_request) Successful in 1m31s
CI / lint (push) Successful in 32s
Docs build and publish / docker (push) Successful in 14s
CI / coverage-analysis (pull_request) Successful in 4s
CI / test (3.12) (push) Canceled after 45s
CI / test (3.13) (push) Canceled after 41s
CI / test (3.14) (push) Canceled after 40s
CI / test-with-beamline-plugins (pxi_bec) (push) Canceled after 36s
CI / test-with-beamline-plugins (pxii_bec) (push) Canceled after 35s
CI / test-with-beamline-plugins (pxiii_bec) (push) Canceled after 31s
CI / test-with-coverage (push) Canceled after 30s
CI / coverage-analysis (push) Canceled after 0s
Build and Publish / release (push) Successful in 35s
The guard introduced with the transmission IOC workaround was inverted: `abs(request - readback) < 0.001` fires only when the beamline is already at the requested value, so any real transmission change was skipped and collections ran at whatever transmission happened to be set. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Ni1Uhb1WqzpXPwcYsvqnH1
This commit was merged in pull request #218.
This commit is contained in:
+1
-1
@@ -1962,7 +1962,7 @@ class AareDAQ:
|
||||
|
||||
if (
|
||||
request.transmission is not None
|
||||
and abs(request.transmission - self._devs.transmission) < 0.001
|
||||
and abs(request.transmission - self._devs.transmission) > 0.001
|
||||
):
|
||||
logger.info(f"requesting transmission to move to {request.transmission}")
|
||||
self._devs.transmission = request.transmission
|
||||
|
||||
Reference in New Issue
Block a user