From 61b5fe40836c2d298d1b6d1aa9671affd014f8c6 Mon Sep 17 00:00:00 2001 From: Jeffrey Blum Date: Mon, 15 Dec 2025 17:31:12 -0500 Subject: [PATCH 1/2] add cohenSutherland break for case that was causing infinite loop --- handlers/multistage-diagram-tactile-svg/tactile_svg.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/handlers/multistage-diagram-tactile-svg/tactile_svg.py b/handlers/multistage-diagram-tactile-svg/tactile_svg.py index 82b5f94b3..0d5ba2d79 100644 --- a/handlers/multistage-diagram-tactile-svg/tactile_svg.py +++ b/handlers/multistage-diagram-tactile-svg/tactile_svg.py @@ -307,6 +307,11 @@ def cohenSutherlandClip(src, tgt, lims): # If both endpoints lie within rectangle if code1 == 0 and code2 == 0: break + + # If neither endpoint lies within rectangle + elif code1 > 0 and code2 > 0: + logging.debug("cohenSutherlandClip: no endpoint in rectangle!") + break # Some segment lies within the rectangle else: From cea58436819a622077c981b886d77fb19f57cc7a Mon Sep 17 00:00:00 2001 From: Jeffrey Blum Date: Tue, 13 Jan 2026 13:08:24 -0500 Subject: [PATCH 2/2] pep8 --- handlers/multistage-diagram-tactile-svg/tactile_svg.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/handlers/multistage-diagram-tactile-svg/tactile_svg.py b/handlers/multistage-diagram-tactile-svg/tactile_svg.py index 0d5ba2d79..37eb84ad9 100644 --- a/handlers/multistage-diagram-tactile-svg/tactile_svg.py +++ b/handlers/multistage-diagram-tactile-svg/tactile_svg.py @@ -307,7 +307,7 @@ def cohenSutherlandClip(src, tgt, lims): # If both endpoints lie within rectangle if code1 == 0 and code2 == 0: break - + # If neither endpoint lies within rectangle elif code1 > 0 and code2 > 0: logging.debug("cohenSutherlandClip: no endpoint in rectangle!")