summaryrefslogtreecommitdiff
path: root/gerbonara
diff options
context:
space:
mode:
authorjaseg <git@jaseg.de>2022-01-30 15:58:20 +0100
committerjaseg <git@jaseg.de>2022-01-30 15:58:20 +0100
commitfd9e0d00792256aed5c30d13abf894df31a357c4 (patch)
tree4ba1fad9473e881bf21cec36ceca3d9958c10ba1 /gerbonara
parentc8bf837a4b5dcc6242b7dac383f09e9390deca35 (diff)
downloadgerbonara-fd9e0d00792256aed5c30d13abf894df31a357c4.tar.gz
gerbonara-fd9e0d00792256aed5c30d13abf894df31a357c4.tar.bz2
gerbonara-fd9e0d00792256aed5c30d13abf894df31a357c4.zip
Fix aperture macro outline primitive rendering
Diffstat (limited to 'gerbonara')
-rw-r--r--gerbonara/gerber/aperture_macros/expression.py6
-rw-r--r--gerbonara/gerber/aperture_macros/parse.py5
-rw-r--r--gerbonara/gerber/aperture_macros/primitive.py14
-rw-r--r--gerbonara/gerber/apertures.py6
-rw-r--r--gerbonara/gerber/cam.py1
-rw-r--r--gerbonara/gerber/rs274x.py12
-rw-r--r--gerbonara/gerber/tests/test_rs274x.py6
7 files changed, 36 insertions, 14 deletions
diff --git a/gerbonara/gerber/aperture_macros/expression.py b/gerbonara/gerber/aperture_macros/expression.py
index f25fdd2..0cf055a 100644
--- a/gerbonara/gerber/aperture_macros/expression.py
+++ b/gerbonara/gerber/aperture_macros/expression.py
@@ -21,6 +21,9 @@ class Expression:
def __str__(self):
return f'<{self.to_gerber()}>'
+ def __repr__(self):
+ return f'<E {self.to_gerber()}>'
+
def converted(self, unit):
return self
@@ -76,6 +79,9 @@ class UnitExpression(Expression):
def __str__(self):
return f'<{self._expr.to_gerber()} {self.unit}>'
+ def __repr__(self):
+ return f'<UE {self._expr.to_gerber()} {self.unit}>'
+
def converted(self, unit):
if self.unit is None or unit is None or self.unit == unit:
return self._expr
diff --git a/gerbonara/gerber/aperture_macros/parse.py b/gerbonara/gerber/aperture_macros/parse.py
index 3a6f6e1..0fa936f 100644
--- a/gerbonara/gerber/aperture_macros/parse.py
+++ b/gerbonara/gerber/aperture_macros/parse.py
@@ -91,7 +91,10 @@ class ApertureMacro:
self._name = name
def __str__(self):
- return f'<Aperture macro, variables {str(self.variables)}, primitives {self.primitives}>'
+ return f'<Aperture macro {self.name}, variables {str(self.variables)}, primitives {self.primitives}>'
+
+ def __repr__(self):
+ return str(self)
def __eq__(self, other):
return hasattr(other, 'to_gerber') and self.to_gerber() == other.to_gerber()
diff --git a/gerbonara/gerber/aperture_macros/primitive.py b/gerbonara/gerber/aperture_macros/primitive.py
index 18aaf51..8732520 100644
--- a/gerbonara/gerber/aperture_macros/primitive.py
+++ b/gerbonara/gerber/aperture_macros/primitive.py
@@ -48,6 +48,9 @@ class Primitive:
attrs = ','.join(str(getattr(self, name)).strip('<>') for name in type(self).__annotations__)
return f'<{type(self).__name__} {attrs}>'
+ def __repr__(self):
+ return str(self)
+
class Calculator:
def __init__(self, instance, variable_binding={}, unit=None):
self.instance = instance
@@ -222,18 +225,19 @@ class Outline(Primitive):
self.coords = [(UnitExpression(x, unit), UnitExpression(y, unit)) for x, y in zip(args[0::2], args[1::2])]
+ def __str__(self):
+ return f'<Outline {len(self.coords)} points>'
+
def to_gerber(self, unit=None):
coords = ','.join(coord.to_gerber(unit) for xy in self.coords for coord in xy)
return f'{self.code},{self.exposure.to_gerber()},{len(self.coords)-1},{coords},{self.rotation.to_gerber()}'
def to_graphic_primitives(self, offset, rotation, variable_binding={}, unit=None, polarity_dark=True):
with self.Calculator(self, variable_binding, unit) as calc:
- bound_coords = [ (calc(x)+offset[0], calc(y)+offset[1]) for x, y in self.coords ]
- bound_radii = [None] * len(bound_coords)
-
rotation += deg_to_rad(calc.rotation)
- bound_coords = [ gp.rotate_point(*p, rotation, 0, 0) for p in bound_coords ]
-
+ bound_coords = [ gp.rotate_point(calc(x), calc(y), rotation, 0, 0) for x, y in self.coords ]
+ bound_coords = [ (x+offset[0], y+offset[1]) for x, y in bound_coords ]
+ bound_radii = [None] * len(bound_coords)
return [gp.ArcPoly(bound_coords, bound_radii, polarity_dark=(bool(calc.exposure) == polarity_dark))]
def dilate(self, offset, unit):
diff --git a/gerbonara/gerber/apertures.py b/gerbonara/gerber/apertures.py
index ab4a1a1..ab62077 100644
--- a/gerbonara/gerber/apertures.py
+++ b/gerbonara/gerber/apertures.py
@@ -43,6 +43,7 @@ class Aperture:
_ : KW_ONLY
unit : str = None
attrs : dict = field(default_factory=dict)
+ original_number : str = None
@property
def hole_shape(self):
@@ -329,9 +330,10 @@ class ApertureMacroInstance(Aperture):
return self.macro.name
def primitives(self, x, y, unit=None, polarity_dark=True):
- return self.macro.to_graphic_primitives(
+ out = list(self.macro.to_graphic_primitives(
offset=(x, y), rotation=self.rotation,
- parameters=self.parameters, unit=unit, polarity_dark=polarity_dark)
+ parameters=self.parameters, unit=unit, polarity_dark=polarity_dark))
+ return out
def dilated(self, offset, unit=MM):
return replace(self, macro=self.macro.dilated(offset, unit))
diff --git a/gerbonara/gerber/cam.py b/gerbonara/gerber/cam.py
index 4c8ab19..ccf1d2a 100644
--- a/gerbonara/gerber/cam.py
+++ b/gerbonara/gerber/cam.py
@@ -152,7 +152,6 @@ class FileSettings:
else: # no or trailing zero suppression
value = value + '0'*integer_digits
out = float(sign + value[:integer_digits] + '.' + value[integer_digits:])
- print(self.zeros, self.number_format, value, out)
return out
def write_gerber_value(self, value, unit=None):
diff --git a/gerbonara/gerber/rs274x.py b/gerbonara/gerber/rs274x.py
index d3719b3..8bd622b 100644
--- a/gerbonara/gerber/rs274x.py
+++ b/gerbonara/gerber/rs274x.py
@@ -369,10 +369,11 @@ class GraphicsState:
self.unit_warning = True
attrs = attrs or {}
self.update_point(x, y)
- return go.Flash(*self.map_coord(*self.point), self.aperture,
+ obj = go.Flash(*self.map_coord(*self.point), self.aperture,
polarity_dark=self.polarity_dark,
unit=self.file_settings.unit,
attrs=attrs)
+ return obj
def interpolate(self, x, y, i=None, j=None, aperture=True, multi_quadrant=False, attrs=None):
if self.point is None:
@@ -737,6 +738,7 @@ class GerberParser:
def _parse_aperture_definition(self, match):
# number, shape, modifiers
modifiers = [ float(val) for val in match['modifiers'].strip(' ,').split('X') ] if match['modifiers'] else []
+ number = int(match['number'])
aperture_classes = {
'C': apertures.CircleAperture,
@@ -752,15 +754,17 @@ class GerberParser:
if match['shape'] in 'RO' and (math.isclose(modifiers[0], 0) or math.isclose(modifiers[1], 0)):
self.warn('Definition of zero-width and/or zero-height rectangle or obround aperture. This is invalid according to spec.' )
- new_aperture = kls(*modifiers, unit=self.file_settings.unit, attrs=self.aperture_attrs.copy())
+ new_aperture = kls(*modifiers, unit=self.file_settings.unit, attrs=self.aperture_attrs.copy(),
+ original_number=number)
elif (macro := self.aperture_macros.get(match['shape'])):
- new_aperture = apertures.ApertureMacroInstance(macro, modifiers, unit=self.file_settings.unit, attrs=self.aperture_attrs.copy())
+ new_aperture = apertures.ApertureMacroInstance(macro, modifiers, unit=self.file_settings.unit,
+ attrs=self.aperture_attrs.copy(), original_number=number)
else:
raise ValueError(f'Aperture shape "{match["shape"]}" is unknown')
- self.aperture_map[int(match['number'])] = new_aperture
+ self.aperture_map[number] = new_aperture
def _parse_aperture_macro(self, match):
self.aperture_macros[match['name']] = ApertureMacro.parse_macro(
diff --git a/gerbonara/gerber/tests/test_rs274x.py b/gerbonara/gerber/tests/test_rs274x.py
index 13192a3..ba8be7b 100644
--- a/gerbonara/gerber/tests/test_rs274x.py
+++ b/gerbonara/gerber/tests/test_rs274x.py
@@ -449,7 +449,11 @@ def test_svg_export(reference, tmpfile):
svg_to_png(out_svg, out_png, dpi=72, bg='white') # make dpi match Cairo's default
mean, _max, hist = image_difference(ref_png, out_png, diff_out=tmpfile('Difference', '.png'))
- assert mean < 1.2e-3
+ if 'Minnow' in reference.name:
+ # This is a dense design with lots of traces, leading to lots of aliasing artifacts.
+ assert mean < 10e-3
+ else:
+ assert mean < 1.2e-3
assert hist[9] < 1
assert hist[3:].sum() < 1e-3*hist.size