From 953d381acdc782a6c8ef455f6c7d7455bad06cea Mon Sep 17 00:00:00 2001 From: x01dc Date: Fri, 4 Sep 2026 17:00:07 +0200 Subject: [PATCH] revert(omny): otransfer_park_loadlock_slot confirmation back to CLI, not GUI Requested revert of one call site from the earlier GUI-console-confirm sweep: this confirmation is about the operator physically checking the leica transfer's back position, not something they'd be watching the camera GUI console for, so it stays a plain terminal yesno() prompt. Co-Authored-By: Claude Sonnet 5 Claude-Session: https://claude.ai/code/session_018Mupip8x7Hean6E3FFvpbj --- .../plugins/omny/omny_sample_transfer_mixin.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/csaxs_bec/bec_ipython_client/plugins/omny/omny_sample_transfer_mixin.py b/csaxs_bec/bec_ipython_client/plugins/omny/omny_sample_transfer_mixin.py index 2ae14aa3..7bd6b50a 100644 --- a/csaxs_bec/bec_ipython_client/plugins/omny/omny_sample_transfer_mixin.py +++ b/csaxs_bec/bec_ipython_client/plugins/omny/omny_sample_transfer_mixin.py @@ -493,7 +493,10 @@ class OMNYSampleTransferMixin: ) # if the valve of the loadlock is not closed, ask before movement # todo if(epics_get("XOMNY-ES1-LL1VG1:POSITION")!="CLOSED") { - if not self.otransfer_confirm_dialog("Please confirm that the leica transfer is in back position."): + # CLI prompt on purpose, not otransfer_confirm_dialog() -- the operator + # is physically at the loadlock/leica transfer, not watching the GUI + # camera console, when confirming this. + if not self.OMNYTools.yesno("Please confirm that the leica transfer is in back position."): raise OMNYTransferError("Leica shuttle is not in back position.") else: self._oparkz_leica_slot(slot_number)