From ddc4c0c2dda6b44ef3d496b1dee5be4fded1392c Mon Sep 17 00:00:00 2001 From: x01dc Date: Fri, 24 Jul 2026 12:42:30 +0200 Subject: [PATCH] fix(LamNI): unambiguous confirmation wording, freeze frame before click - Reword the extended-loop accept/iterate prompt: the old either/or phrasing made a "yes" answer ambiguous about which half it affirmed. - Freeze a single frame (and close the shutter/live-view per keep_shutter_open) before requesting the rotation-centre click, same as every other click site in this file -- previously the shutter and 5 Hz live view stayed on for the whole time the operator was deciding where to click, exposing the sample for no benefit. Co-Authored-By: Claude Sonnet 5 --- csaxs_bec/bec_ipython_client/plugins/LamNI/x_ray_eye_align.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/csaxs_bec/bec_ipython_client/plugins/LamNI/x_ray_eye_align.py b/csaxs_bec/bec_ipython_client/plugins/LamNI/x_ray_eye_align.py index d283ab5..ff189de 100644 --- a/csaxs_bec/bec_ipython_client/plugins/LamNI/x_ray_eye_align.py +++ b/csaxs_bec/bec_ipython_client/plugins/LamNI/x_ray_eye_align.py @@ -848,6 +848,7 @@ class XrayEyeAlign: "Watch the sample rotate and locate the stationary rotation centre..." ) self._live_sweep([180, 0]) + self.update_frame(keep_shutter_open) self.movement_buttons_enabled(True, True) center_x, center_y = self._collect_click( 1, @@ -868,8 +869,7 @@ class XrayEyeAlign: self.update_frame(keep_shutter_open) answer = ( input( - "Are you happy with this alignment, or shall we do another iteration? " - "[Y/n]: " + "Alignment acceptable -- stop here? [Y/n] (n = run another iteration): " ) .strip() .lower()