rahix/openflexure-microscope-server
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Apply suggestions from code review of branch smart_stack_as_mixin

Browse source 
Co-authored-by: Joe Knapper <joe.knapper@glasgow.ac.uk>
This commit is contained in:
Julian Stirling 2026-03-25 12:26:32 +00:00
parent b7be12e183
commit 17feec33e2
1 changed files with 4 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

7
src/openflexure_microscope_server/things/scan_workflows.py
View file

@ -328,7 +328,7 @@ class SmartStackMixin:
) )
"""The minimum number of images to capture in a stack. """The minimum number of images to capture in a stack.
This many images are captures and tested for focus, if the focus is not central This many images are captured and tested for focus, if the focus is not central
enough more images may be captured. After new images are captured, this value sets enough more images may be captured. After new images are captured, this value sets
the number of images used for checking if focus is achieved. the number of images used for checking if focus is achieved.
@ -347,9 +347,10 @@ class SmartStackMixin:
@property @property
def as_workflow(self) -> ScanWorkflow: def as_workflow(self) -> ScanWorkflow:
"""Return self with correct a ScanWorkflow type. """Return self as a ScanWorkflow.
Check this object is ScanWorkflow with the mixin, Ensures this mixin is only used with ScanWorkflow instances,
raising TypeError otherwise.
""" """
if not isinstance(self, ScanWorkflow): if not isinstance(self, ScanWorkflow):
raise TypeError("SmartStackMixin must be mixed into a ScanWorkflow") raise TypeError("SmartStackMixin must be mixed into a ScanWorkflow")