Apply suggestions from code review of branch sample-size-sim

Co-authored-by: Julian Stirling <julian@julianstirling.co.uk>
This commit is contained in:
Joe Knapper 2025-10-06 10:16:22 +00:00
parent 2fc92fc1ff
commit a34a9f99c5

View file

@ -92,9 +92,7 @@ class SimulatedCamera(BaseCamera):
Currently only tests that the sample size is not greater than the canvas size in any dimension. Currently only tests that the sample size is not greater than the canvas size in any dimension.
""" """
# Iterate through elements in both tuples. As strict is False, will use the shorter of the two tuples # Iterate through elements in both tuples. As strict is False, will use the shorter of the two tuples
for _i, (a, b) in enumerate( for a, b in zip(self.canvas_shape, self.sample_limits, strict=False):
zip(self.canvas_shape, self.sample_limits, strict=False)
):
if a < b: if a < b:
raise ValueError( raise ValueError(
"Canvas size must be bigger than or equal to canvas size" "Canvas size must be bigger than or equal to canvas size"