From deb2bb2bbfc13e6dce8adf493221a4fe4929a344 Mon Sep 17 00:00:00 2001
From: jaseg <git@jaseg.de>
Date: Sun, 23 Jan 2022 01:19:30 +0100
Subject: Squash some more bugs

---
 gerbonara/gerber/graphic_objects.py | 35 +++++++++++++++++++++--------------
 1 file changed, 21 insertions(+), 14 deletions(-)

(limited to 'gerbonara/gerber/graphic_objects.py')

diff --git a/gerbonara/gerber/graphic_objects.py b/gerbonara/gerber/graphic_objects.py
index 29f0d38..4d9a1f8 100644
--- a/gerbonara/gerber/graphic_objects.py
+++ b/gerbonara/gerber/graphic_objects.py
@@ -143,8 +143,7 @@ class Region(GerberObject):
             yield self.poly
         else:
             to = lambda value: self.unit.convert_to(unit, value)
-            conv_outline = [ (to(x), to(y))
-                    for x, y in self.poly.outline ]
+            conv_outline = [ (to(x), to(y)) for x, y in self.poly.outline ]
             convert_entry = lambda entry: (entry[0], (to(entry[1][0]), to(entry[1][1])))
             conv_arc = [ None if entry is None else convert_entry(entry) for entry in self.poly.arc_centers ]
 
@@ -182,7 +181,6 @@ class Region(GerberObject):
 
         yield 'G37*'
 
-
 @dataclass
 class Line(GerberObject):
     # Line with *round* end caps.
@@ -269,6 +267,25 @@ class Arc(GerberObject):
     def _with_offset(self, dx, dy):
         return replace(self, x1=self.x1+dx, y1=self.y1+dy, x2=self.x2+dx, y2=self.y2+dy)
 
+    def numeric_error(self, unit=None):
+        conv = self.converted(unit)
+        cx, cy = conv.cx + conv.x1, conv.cy + conv.y1
+        r1 = math.dist((cx, cy), conv.p1)
+        r2 = math.dist((cx, cy), conv.p2)
+        return abs(r1 - r2)
+
+    def sweep_angle(self):
+        f = math.atan2(self.x2, self.y2) - math.atan2(self.x1, self.y1)
+        f = (f + math.pi) % (2*math.pi) - math.pi
+
+        if self.clockwise:
+            f = -f
+
+        if f > math.pi:
+            f = 2*math.pi - f
+
+        return f
+
     @property
     def p1(self):
         return self.x1, self.y1
@@ -346,16 +363,6 @@ class Arc(GerberObject):
         ctx.set_current_point(self.unit, self.x2, self.y2)
 
     def curve_length(self, unit=MM):
-        r = math.hypot(self.cx, self.cy)
-        f = math.atan2(self.x2, self.y2) - math.atan2(self.x1, self.y1)
-        f = (f + math.pi) % (2*math.pi) - math.pi
-
-        if self.clockwise:
-            f = -f
-
-        if f > math.pi:
-            f = 2*math.pi - f
-
-        return self.unit.convert_to(unit, 2*math.pi*r * (f/math.pi))
+        return self.unit.convert_to(unit, math.hypot(self.cx, self.cy) * self.sweep_angle)
 
 
-- 
cgit