docs: add comment to timeout adjustment
Docs build and publish / docker (push) Successful in 20s
CI / lint (push) Successful in 42s
CI / test (3.12) (push) Canceled after 46s
CI / test (3.13) (push) Canceled after 44s
CI / test (3.14) (push) Canceled after 41s
CI / test-with-beamline-plugins (pxi_bec) (push) Canceled after 39s
CI / test-with-beamline-plugins (pxii_bec) (push) Canceled after 36s
CI / test-with-beamline-plugins (pxiii_bec) (push) Canceled after 34s
CI / test-with-coverage (push) Canceled after 31s
CI / coverage-analysis (push) Canceled after 0s
Build and Publish / release (push) Successful in 32s

This commit was merged in pull request #234.
This commit is contained in:
David Perl
2026-09-17 14:31:08 +02:00
committed by perl_d
parent 5d770bddc0
commit b385559842
2 changed files with 5 additions and 2 deletions
+2 -2
View File
@@ -3099,9 +3099,9 @@ class AareDAQ:
sample = self._cfg.current_sample
try:
self._devs.tell.get_state()
tell_connected, tell_error = True, None
tell_connected, tell_error = True, None
except TellCommunicationError as e:
tell_connected, tell_error = False, f"TELL unavailable: {e}"
tell_connected, tell_error = False, f"TELL unavailable: {e}"
return sample, tell_connected, tell_error
def _aerotech_status(self) -> tuple[bool, str | None]:
@@ -152,6 +152,9 @@ class MountingService:
self.ctx.deps.devs.tell.set_in_mount_position(True)
def _unmount_current_sample(self, timeout: float = 60.0):
# Timeout here is reduced from the default 360s because an error should be caught by
# the retry handler later. If errors from this function are not propagated correctly
# (e.g. through _handle_operation_error) it may cause weird problems that appear as timeouts!
self._prepare_mount_hardware()
previous_sample = self.ctx.deps.cfg.current_sample
if previous_sample is not None: