diff --git a/CHANGELOG.md b/CHANGELOG.md
index 3c033dec..1b0aa775 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -7,6 +7,14 @@ CHANGELOG for FlatCAM beta
=================================================
+2.11.2020
+
+- fixed the Tcl Command AlignDrill
+- fixed the Tcl Command AlignDrillGrid
+- fixed the Tcl COmmand Panelize, Excellon panelization section
+- Fixed an issue in Tool Calibration export_excellon method call
+- PEP8 corrections all over the app
+
1.11.2020
- updated the French Translation (by Olivier Cornet)
@@ -20,6 +28,7 @@ CHANGELOG for FlatCAM beta
- in Join Excellon functionality made sure that the new Combo Exellon object will have copied the data from source objects and not just references, therefore will survive the delete of its parents
- updated Turkish translation (by Mehmet Kaya)
- updated all the languages except Turkish
+- in the Tool PDF fixed the creation of Excellon objects to the current Excellon object data structure
31.10.2020
diff --git a/appDatabase.py b/appDatabase.py
index e0b498ea..d9793347 100644
--- a/appDatabase.py
+++ b/appDatabase.py
@@ -1971,8 +1971,7 @@ class ToolsDB2(QtWidgets.QWidget):
else:
new_name = "new_tool_1"
- dict_elem = {}
- dict_elem['name'] = new_name
+ dict_elem = {'name': new_name}
if type(self.app.defaults["geometry_cnctooldia"]) == float:
dict_elem['tooldia'] = self.app.defaults["geometry_cnctooldia"]
else:
diff --git a/appEditors/AppExcEditor.py b/appEditors/AppExcEditor.py
index 4bd7d1e0..929d2263 100644
--- a/appEditors/AppExcEditor.py
+++ b/appEditors/AppExcEditor.py
@@ -11,7 +11,6 @@ from PyQt5.QtCore import Qt
from camlib import distance, arc, FlatCAMRTreeStorage
from appGUI.GUIElements import FCEntry, FCComboBox, FCTable, FCDoubleSpinner, RadioSet, FCSpinner, FCButton
from appEditors.AppGeoEditor import FCShapeTool, DrawTool, DrawToolShape, DrawToolUtilityShape, AppGeoEditor
-from appParsers.ParseExcellon import Excellon
from shapely.geometry import LineString, LinearRing, MultiLineString, Polygon, MultiPolygon, Point
import shapely.affinity as affinity
diff --git a/appEditors/AppGeoEditor.py b/appEditors/AppGeoEditor.py
index b9887f3e..2ba27f9c 100644
--- a/appEditors/AppGeoEditor.py
+++ b/appEditors/AppGeoEditor.py
@@ -12,7 +12,7 @@
# ######################################################### ##
from PyQt5 import QtGui, QtCore, QtWidgets
-from PyQt5.QtCore import Qt, QSettings
+from PyQt5.QtCore import Qt
from camlib import distance, arc, three_point_circle, Geometry, FlatCAMRTreeStorage
from appTool import AppTool
@@ -20,7 +20,7 @@ from appGUI.GUIElements import OptionalInputSection, FCCheckBox, FCLabel, FCComb
FCDoubleSpinner, FCButton, FCInputDialog, FCTree, NumericalEvalTupleEntry
from appParsers.ParseFont import *
-from shapely.geometry import LineString, LinearRing, MultiLineString, Polygon, MultiPolygon
+from shapely.geometry import LineString, LinearRing, MultiLineString, Polygon, MultiPolygon, Point
from shapely.ops import unary_union, linemerge
import shapely.affinity as affinity
from shapely.geometry.polygon import orient
diff --git a/appEditors/AppGerberEditor.py b/appEditors/AppGerberEditor.py
index 0bda4f0e..7d8f3bda 100644
--- a/appEditors/AppGerberEditor.py
+++ b/appEditors/AppGerberEditor.py
@@ -6,7 +6,7 @@
# ##########################################################
from PyQt5 import QtGui, QtCore, QtWidgets
-from PyQt5.QtCore import Qt, QSettings
+from PyQt5.QtCore import Qt
from shapely.geometry import LineString, LinearRing, MultiLineString, Point, Polygon, MultiPolygon, box
from shapely.ops import unary_union
diff --git a/appEditors/AppTextEditor.py b/appEditors/AppTextEditor.py
index c0b7d3b7..8de4f807 100644
--- a/appEditors/AppTextEditor.py
+++ b/appEditors/AppTextEditor.py
@@ -332,7 +332,7 @@ class AppTextEditor(QtWidgets.QWidget):
r = self.code_editor.find(str(text_to_be_found), flags)
if r is False:
self.code_editor.moveCursor(QtGui.QTextCursor.Start)
- r = self.code_editor.find(str(text_to_be_found), flags)
+ self.code_editor.find(str(text_to_be_found), flags)
def handleReplaceGCode(self):
diff --git a/appEditors/appGCodeEditor.py b/appEditors/appGCodeEditor.py
index bc8b8874..4cbae0bd 100644
--- a/appEditors/appGCodeEditor.py
+++ b/appEditors/appGCodeEditor.py
@@ -462,7 +462,6 @@ class AppGCodeEditor(QtCore.QObject):
end_sel = my_text_cursor.selectionEnd()
else:
pos_list = []
- end_sel = 0
my_text_cursor = self.edit_area.textCursor()
m6_pos = my_text_cursor.selectionEnd()
diff --git a/appGUI/GUIElements.py b/appGUI/GUIElements.py
index a76aa699..081c47eb 100644
--- a/appGUI/GUIElements.py
+++ b/appGUI/GUIElements.py
@@ -12,7 +12,7 @@
# ##########################################################
from PyQt5 import QtGui, QtCore, QtWidgets
-from PyQt5.QtCore import Qt, pyqtSlot, pyqtSignal, QSettings
+from PyQt5.QtCore import Qt, pyqtSlot, pyqtSignal
from PyQt5.QtWidgets import QTextEdit, QCompleter, QAction
from PyQt5.QtGui import QKeySequence, QTextCursor
@@ -2079,7 +2079,7 @@ class FCPlainTextAreaExtended(QtWidgets.QPlainTextEdit):
"""
pos = self.textCursor().position()
self.moveCursor(QtGui.QTextCursor.StartOfLine)
- line_text = self.textCursor().block().text()
+ self.textCursor().block().text()
if self.textCursor().block().text().startswith(" "):
# skip the white space
self.moveCursor(QtGui.QTextCursor.NextWord)
@@ -2407,7 +2407,7 @@ class FCInputDialogSpinnerButton(QtWidgets.QDialog):
class FCButton(QtWidgets.QPushButton):
def __init__(self, text=None, checkable=None, click_callback=None, parent=None):
super(FCButton, self).__init__(text, parent)
- if not checkable is None:
+ if checkable is not None:
self.setCheckable(checkable)
if not click_callback is None:
diff --git a/appGUI/MainGUI.py b/appGUI/MainGUI.py
index 95f66e74..b969493c 100644
--- a/appGUI/MainGUI.py
+++ b/appGUI/MainGUI.py
@@ -10,6 +10,8 @@
# File Modified (major mod): Marius Adrian Stanciu #
# Date: 3/10/2019 #
# ##########################################################
+from PyQt5.QtCore import QSettings
+
import platform
from appGUI.GUIElements import *
@@ -20,6 +22,7 @@ from appGUI.preferences.general.GeneralPreferencesUI import GeneralPreferencesUI
from appGUI.preferences.geometry.GeometryPreferencesUI import GeometryPreferencesUI
from appGUI.preferences.gerber.GerberPreferencesUI import GerberPreferencesUI
from appEditors.AppGeoEditor import FCShapeTool
+
from matplotlib.backend_bases import KeyEvent as mpl_key_event
import webbrowser
diff --git a/appGUI/ObjectUI.py b/appGUI/ObjectUI.py
index 6e3382c9..342bb3eb 100644
--- a/appGUI/ObjectUI.py
+++ b/appGUI/ObjectUI.py
@@ -170,8 +170,8 @@ class ObjectUI(QtWidgets.QWidget):
def confirmation_message_int(self, accepted, minval, maxval):
if accepted is False:
- self.app.inform[str, bool].emit('[WARNING_NOTCL] %s: [%d, %d]' %
- (_("Edited value is out of range"), minval, maxval), False)
+ self.app.inform[str, bool].emit(
+ '[WARNING_NOTCL] %s: [%d, %d]' % (_("Edited value is out of range"), minval, maxval), False)
else:
self.app.inform[str, bool].emit('[success] %s' % _("Edited value is within limits."), False)
diff --git a/appGUI/PlotCanvas.py b/appGUI/PlotCanvas.py
index c1b2c351..0b8872b9 100644
--- a/appGUI/PlotCanvas.py
+++ b/appGUI/PlotCanvas.py
@@ -575,13 +575,13 @@ class CursorBig(QtCore.QObject):
def set_data(self, pos, **kwargs):
"""Internal event handler to draw the cursor when the mouse moves."""
- if 'edge_color' in kwargs:
- color = kwargs['edge_color']
- else:
- if self.app.defaults['global_theme'] == 'white':
- color = '#000000FF'
- else:
- color = '#FFFFFFFF'
+ # if 'edge_color' in kwargs:
+ # color = kwargs['edge_color']
+ # else:
+ # if self.app.defaults['global_theme'] == 'white':
+ # color = '#000000FF'
+ # else:
+ # color = '#FFFFFFFF'
position = [pos[0][0], pos[0][1]]
self.mouse_position_updated.emit(position)
diff --git a/appGUI/VisPyTesselators.py b/appGUI/VisPyTesselators.py
index a3dad088..1c053c77 100644
--- a/appGUI/VisPyTesselators.py
+++ b/appGUI/VisPyTesselators.py
@@ -29,9 +29,10 @@ class GLUTess:
pass
def _on_combine(self, coords, data, weight):
- return (coords[0], coords[1], coords[2])
+ return coords[0], coords[1], coords[2]
- def _on_error(self, errno):
+ @staticmethod
+ def _on_error(errno):
print("GLUTess error:", errno)
def _on_end_primitive(self):
diff --git a/appGUI/VisPyVisuals.py b/appGUI/VisPyVisuals.py
index 0e8be7cb..8e8d24d6 100644
--- a/appGUI/VisPyVisuals.py
+++ b/appGUI/VisPyVisuals.py
@@ -339,7 +339,7 @@ class ShapeCollectionVisual(CompoundVisual):
if update:
self.__update()
- def update_visibility(self, state:bool, indexes=None) -> None:
+ def update_visibility(self, state: bool, indexes=None) -> None:
# Lock sub-visuals updates
self.update_lock.acquire(True)
if indexes is None:
@@ -540,8 +540,9 @@ class ShapeCollectionVisual(CompoundVisual):
def redraw(self, indexes=None, update_colors=None):
"""
Redraws collection
- :param indexes: list
+ :param indexes: list
Shape indexes to get from process pool
+ :param update_colors:
"""
# Only one thread can update data
self.results_lock.acquire(True)
diff --git a/appObjects/FlatCAMCNCJob.py b/appObjects/FlatCAMCNCJob.py
index caeae2bc..07304678 100644
--- a/appObjects/FlatCAMCNCJob.py
+++ b/appObjects/FlatCAMCNCJob.py
@@ -2265,7 +2265,7 @@ class CNCJobObject(FlatCAMObj, CNCjob):
processed_body_gcode += gline + '\n'
gcode = processed_body_gcode
- g = self.gc_header + '\n' + self.gc_start + '\n' + preamble + '\n' + \
+ g = self.gc_header + '\n' + self.gc_start + '\n' + preamble + '\n' + \
gcode + '\n' + postamble + end_gcode
else:
# try:
@@ -2286,6 +2286,7 @@ class CNCJobObject(FlatCAMObj, CNCjob):
# _("G-code does not have a G94 code.\n"
# "Append Code snippet will not be used.."))
# g = self.gc_header + '\n' + gcode + postamble + end_gcode
+ g = ''
if preamble != '' and postamble != '':
g = self.gc_header + self.gc_start + '\n' + preamble + '\n' + gcode + '\n' + \
postamble + '\n' + end_gcode
diff --git a/appObjects/FlatCAMExcellon.py b/appObjects/FlatCAMExcellon.py
index e6dd8065..bf0102c6 100644
--- a/appObjects/FlatCAMExcellon.py
+++ b/appObjects/FlatCAMExcellon.py
@@ -11,16 +11,13 @@
# ##########################################################
-from shapely.geometry import Point, LineString
-
-from copy import deepcopy
+from shapely.geometry import LineString
from appParsers.ParseExcellon import Excellon
from appObjects.FlatCAMObj import *
import itertools
import numpy as np
-from collections import defaultdict
import gettext
import appTranslation as fcTranslate
@@ -1016,10 +1013,10 @@ class ExcellonObject(FlatCAMObj, Excellon):
# in case that the tool used has the same diameter with the hole, and since the maximum resolution
# for FlatCAM is 6 decimals,
# we add a tenth of the minimum value, meaning 0.0000001, which from our point of view is "almost zero"
- for tool in tools:
- for slot in self.tools[tool]['slots']:
+ for m_tool in tools:
+ for slot in self.tools[m_tool]['slots']:
toolstable_tool = float('%.*f' % (self.decimals, float(tooldia)))
- file_tool = float('%.*f' % (self.decimals, float(self.tools[tool]["tooldia"])))
+ file_tool = float('%.*f' % (self.decimals, float(self.tools[m_tool]["tooldia"])))
# I add the 0.0001 value to account for the rounding error in converting from IN to MM and reverse
# for the file_tool (tooldia actually)
@@ -1164,8 +1161,8 @@ class ExcellonObject(FlatCAMObj, Excellon):
r_color[3] = 1
new_color = '#'
- for idx in range(len(r_color)):
- new_color += '%x' % int(r_color[idx] * 255)
+ for idx_c in range(len(r_color)):
+ new_color += '%x' % int(r_color[idx_c] * 255)
# do it until a valid color is generated
# a valid color has the # symbol, another 6 chars for the color and the last 2 chars for alpha
# for a total of 9 chars
diff --git a/appObjects/FlatCAMGeometry.py b/appObjects/FlatCAMGeometry.py
index 34776ea0..55228611 100644
--- a/appObjects/FlatCAMGeometry.py
+++ b/appObjects/FlatCAMGeometry.py
@@ -10,7 +10,7 @@
# File modified by: Marius Stanciu #
# ##########################################################
-from shapely.geometry import Polygon, MultiPolygon, MultiLineString, LineString, LinearRing, box
+from shapely.geometry import MultiLineString, LineString, LinearRing, box
import shapely.affinity as affinity
from camlib import Geometry, grace
@@ -2606,7 +2606,7 @@ class GeometryObject(FlatCAMObj, Geometry):
# Type(cpoly) == FlatCAMRTreeStorage | None
cpoly = None
- if paint_method == 0: # Standard
+ if paint_method == 0: # Standard
cpoly = self.clear_polygon(bbox,
tooldia=tooldia,
steps_per_circle=obj.circle_steps,
@@ -2614,7 +2614,7 @@ class GeometryObject(FlatCAMObj, Geometry):
contour=True,
connect=True,
prog_plot=False)
- elif paint_method == 1: # Seed
+ elif paint_method == 1: # Seed
cpoly = self.clear_polygon2(bbox,
tooldia=tooldia,
steps_per_circle=obj.circle_steps,
@@ -2622,7 +2622,7 @@ class GeometryObject(FlatCAMObj, Geometry):
contour=True,
connect=True,
prog_plot=False)
- elif paint_method == 2: # Lines
+ elif paint_method == 2: # Lines
cpoly = self.clear_polygon3(bbox,
tooldia=tooldia,
steps_per_circle=obj.circle_steps,
diff --git a/appObjects/FlatCAMGerber.py b/appObjects/FlatCAMGerber.py
index 9feca036..f5c3d2b1 100644
--- a/appObjects/FlatCAMGerber.py
+++ b/appObjects/FlatCAMGerber.py
@@ -11,13 +11,11 @@
# ##########################################################
-from shapely.geometry import Point, Polygon, MultiPolygon, MultiLineString, LineString, LinearRing
-from shapely.ops import unary_union
+from shapely.geometry import Point, MultiLineString, LineString, LinearRing
from appParsers.ParseGerber import Gerber
from appObjects.FlatCAMObj import *
-import math
import numpy as np
from copy import deepcopy
@@ -210,7 +208,7 @@ class GerberObject(FlatCAMObj, Gerber):
pass
self.apertures_row = 0
- aper_no = self.apertures_row + 1
+
sort = []
for k in list(self.apertures.keys()):
sort.append(int(k))
@@ -397,7 +395,7 @@ class GerberObject(FlatCAMObj, Gerber):
non_copper = bounding_box.difference(self.solid_geometry)
if non_copper is None or non_copper.is_empty:
- self.app.inform.emit("[ERROR_NOTCL] %s" % _("Operation could not be done."))
+ app_obj.inform.emit("[ERROR_NOTCL] %s" % _("Operation could not be done."))
return "fail"
geo_obj.solid_geometry = non_copper
@@ -423,7 +421,7 @@ class GerberObject(FlatCAMObj, Gerber):
bounding_box = bounding_box.envelope
if bounding_box is None or bounding_box.is_empty:
- self.app.inform.emit("[ERROR_NOTCL] %s" % _("Operation could not be done."))
+ app_obj.inform.emit("[ERROR_NOTCL] %s" % _("Operation could not be done."))
return "fail"
geo_obj.solid_geometry = bounding_box
@@ -532,17 +530,16 @@ class GerberObject(FlatCAMObj, Gerber):
"startz": self.app.defaults['geometry_startz']
})
- geo_obj.tools = {}
- geo_obj.tools['1'] = {}
+ geo_obj.tools = {'1': {}}
geo_obj.tools.update({
'1': {
- 'tooldia': dia,
- 'offset': 'Path',
- 'offset_value': 0.0,
- 'type': _('Rough'),
- 'tool_type': tool_type,
- 'data': default_data,
- 'solid_geometry': geo_obj.solid_geometry
+ 'tooldia': dia,
+ 'offset': 'Path',
+ 'offset_value': 0.0,
+ 'type': _('Rough'),
+ 'tool_type': tool_type,
+ 'data': default_data,
+ 'solid_geometry': geo_obj.solid_geometry
}
})
@@ -679,17 +676,16 @@ class GerberObject(FlatCAMObj, Gerber):
"startz": self.app.defaults['geometry_startz']
})
- geo_obj.tools = {}
- geo_obj.tools['1'] = {}
+ geo_obj.tools = {'1': {}}
geo_obj.tools.update({
'1': {
- 'tooldia': dia,
- 'offset': 'Path',
- 'offset_value': 0.0,
- 'type': _('Rough'),
- 'tool_type': tool_type,
- 'data': default_data,
- 'solid_geometry': geo_obj.solid_geometry
+ 'tooldia': dia,
+ 'offset': 'Path',
+ 'offset_value': 0.0,
+ 'type': _('Rough'),
+ 'tool_type': tool_type,
+ 'data': default_data,
+ 'solid_geometry': geo_obj.solid_geometry
}
})
@@ -776,7 +772,7 @@ class GerberObject(FlatCAMObj, Gerber):
else:
follow_name = outname
- def follow_init(follow_obj, app):
+ def follow_init(follow_obj, app_obj):
# Propagate options
follow_obj.options["cnctooldia"] = str(self.app.defaults["tools_iso_tooldia"])
follow_obj.solid_geometry = self.follow_geometry
@@ -854,7 +850,7 @@ class GerberObject(FlatCAMObj, Gerber):
log.debug("FlatCAMObj.GerberObject.convert_units()")
- factor = Gerber.convert_units(self, units)
+ Gerber.convert_units(self, units)
# self.options['isotooldia'] = float(self.options['isotooldia']) * factor
# self.options['bboxmargin'] = float(self.options['bboxmargin']) * factor
diff --git a/appObjects/FlatCAMScript.py b/appObjects/FlatCAMScript.py
index 1eb4b8ac..6cbcfaeb 100644
--- a/appObjects/FlatCAMScript.py
+++ b/appObjects/FlatCAMScript.py
@@ -16,7 +16,6 @@ from appGUI.ObjectUI import *
import tkinter as tk
import sys
-from copy import deepcopy
import gettext
import appTranslation as fcTranslate
@@ -219,7 +218,7 @@ class ScriptObject(FlatCAMObj):
# it means that the script finished with an error
result = self.app.shell.tcl.eval("set errorInfo")
log.error("Exec command Exception: %s\n" % result)
- self.app.shell.append_error('ERROR: %s\n '% result)
+ self.app.shell.append_error('ERROR: %s\n' % result)
self.app.ui.fcinfo.lock_pmaps = False
self.app.shell.close_processing()
diff --git a/appObjects/ObjectCollection.py b/appObjects/ObjectCollection.py
index 5e8f667e..8b325a96 100644
--- a/appObjects/ObjectCollection.py
+++ b/appObjects/ObjectCollection.py
@@ -1090,14 +1090,14 @@ class ObjectCollection(QtCore.QAbstractItemModel):
def add_act(o_name):
obj_for_icon = self.get_by_name(o_name)
- add_action = QtWidgets.QAction(parent=self.app.ui.menuobjects)
- add_action.setCheckable(True)
- add_action.setText(o_name)
- add_action.setIcon(QtGui.QIcon(icon_files[obj_for_icon.kind]))
- add_action.triggered.connect(
- lambda: self.set_active(o_name) if add_action.isChecked() is True else
+ menu_action = QtWidgets.QAction(parent=self.app.ui.menuobjects)
+ menu_action.setCheckable(True)
+ menu_action.setText(o_name)
+ menu_action.setIcon(QtGui.QIcon(icon_files[obj_for_icon.kind]))
+ menu_action.triggered.connect(
+ lambda: self.set_active(o_name) if menu_action.isChecked() is True else
self.set_inactive(o_name))
- self.app.ui.menuobjects.addAction(add_action)
+ self.app.ui.menuobjects.addAction(menu_action)
for name in gerber_list:
add_act(name)
diff --git a/appParsers/ParseDXF.py b/appParsers/ParseDXF.py
index ab37c077..5b257243 100644
--- a/appParsers/ParseDXF.py
+++ b/appParsers/ParseDXF.py
@@ -9,13 +9,12 @@ from shapely.geometry import LineString
from shapely.affinity import rotate
from ezdxf.math.vector import Vector as ezdxf_vector
+from appParsers.ParseFont import *
+from appParsers.ParseDXF_Spline import *
import logging
log = logging.getLogger('base2')
-from appParsers.ParseFont import *
-from appParsers.ParseDXF_Spline import *
-
def distance(pt1, pt2):
return math.sqrt((pt1[0] - pt2[0]) ** 2 + (pt1[1] - pt2[1]) ** 2)
@@ -107,12 +106,12 @@ def dxfarc2shapely(arc, n_points=100):
arc_center = ocs.to_wcs(arc.dxf.center)
start_angle = arc.dxf.start_angle + 180
end_angle = arc.dxf.end_angle + 180
- dir = 'CW'
+ direction = 'CW'
else:
arc_center = arc.dxf.center
start_angle = arc.dxf.start_angle
end_angle = arc.dxf.end_angle
- dir = 'CCW'
+ direction = 'CCW'
center_x = arc_center[0]
center_y = arc_center[1]
@@ -127,7 +126,7 @@ def dxfarc2shapely(arc, n_points=100):
step_angle = float(abs(end_angle - start_angle) / n_points)
while angle <= end_angle:
- if dir == 'CCW':
+ if direction == 'CCW':
x = center_x + radius * math.cos(math.radians(angle))
y = center_y + radius * math.sin(math.radians(angle))
else:
@@ -138,7 +137,7 @@ def dxfarc2shapely(arc, n_points=100):
# in case the number of segments do not cover everything until the end of the arc
if angle != end_angle:
- if dir == 'CCW':
+ if direction == 'CCW':
x = center_x + radius * math.cos(math.radians(end_angle))
y = center_y + radius * math.sin(math.radians(end_angle))
else:
@@ -164,12 +163,12 @@ def dxfellipse2shapely(ellipse, ellipse_segments=100):
center = ocs.to_wcs(ellipse.dxf.center)
start_angle = ocs.to_wcs(ellipse.dxf.start_param)
end_angle = ocs.to_wcs(ellipse.dxf.end_param)
- dir = 'CW'
+ direction = 'CW'
else:
center = ellipse.dxf.center
start_angle = ellipse.dxf.start_param
end_angle = ellipse.dxf.end_param
- dir = 'CCW'
+ direction = 'CCW'
# print("Dir = %s" % dir)
major_axis = ellipse.dxf.major_axis
@@ -189,7 +188,7 @@ def dxfellipse2shapely(ellipse, ellipse_segments=100):
angle = start_angle
for step in range(line_seg + 1):
- if dir == 'CW':
+ if direction == 'CW':
major_dim = normalize_2(major_axis)
minor_dim = normalize_2(Vector([ratio * k for k in major_axis]))
vx = (major_dim[0] + major_dim[1]) * math.cos(angle)
@@ -381,6 +380,7 @@ def get_geo(dxf_object, container):
return geo
+
def getdxftext(exf_object, object_type, units=None):
pass
diff --git a/appParsers/ParseDXF_Spline.py b/appParsers/ParseDXF_Spline.py
index 2212d998..57d481ba 100644
--- a/appParsers/ParseDXF_Spline.py
+++ b/appParsers/ParseDXF_Spline.py
@@ -400,7 +400,7 @@ class Vector(list):
# ----------------------------------------------------------------------
def __str__(self):
- return "[%s]" % ", ".join([("%15g" % (x)).strip() for x in self])
+ return "[%s]" % ", ".join([("%15g" % x).strip() for x in self])
# ----------------------------------------------------------------------
def eq(self, v, acc=_accuracy):
@@ -805,7 +805,6 @@ class Vector(list):
#
# #-----------------------------------------------------------------------
# def __call__(self, i, x):
-# # FIXME should interpolate to find the interval
# C = self.coefficients(i)
# return ((C[0]*x + C[1])*x + C[2])*x + C[3]
#
diff --git a/appParsers/ParseFont.py b/appParsers/ParseFont.py
index 89ba9ef4..ed0f6acb 100644
--- a/appParsers/ParseFont.py
+++ b/appParsers/ParseFont.py
@@ -10,13 +10,15 @@
# ## and made it work with Python 3 #
# ######################################################################
-import re, os, sys, glob
+import re
+import os
+import sys
+import glob
-from shapely.geometry import Point, Polygon
+from shapely.geometry import Polygon
from shapely.affinity import translate, scale
from shapely.geometry import MultiPolygon
-
import freetype as ft
from fontTools import ttLib
@@ -32,7 +34,7 @@ if '_' not in builtins.__dict__:
log = logging.getLogger('base2')
-class ParseFont():
+class ParseFont:
FONT_SPECIFIER_NAME_ID = 4
FONT_SPECIFIER_FAMILY_ID = 1
@@ -69,12 +71,12 @@ class ParseFont():
if os.path.isfile(executable):
data = os.popen(executable).readlines()
match = re.compile('\d+: (.+)')
- set = []
+ set_lst = []
for line in data:
result = match.match(line)
if result:
- set.append(result.group(1))
- return set
+ set_lst.append(result.group(1))
+ return set_lst
else:
directories = [
# what seems to be the standard installation point
@@ -91,7 +93,7 @@ class ParseFont():
dir_set = []
for directory in directories:
- directory = directory = os.path.expanduser(os.path.expandvars(directory))
+ directory = os.path.expanduser(os.path.expandvars(directory))
try:
if os.path.isdir(directory):
for path, children, files in os.walk(directory):
@@ -116,7 +118,7 @@ class ParseFont():
dir_set = []
for directory in directories:
- directory = directory = os.path.expanduser(os.path.expandvars(directory))
+ directory = os.path.expanduser(os.path.expandvars(directory))
try:
if os.path.isdir(directory):
for path, children, files in os.walk(directory):
@@ -144,7 +146,7 @@ class ParseFont():
winreg.HKEY_LOCAL_MACHINE,
keyName
)
- except OSError as err:
+ except OSError:
pass
if not k:
@@ -195,7 +197,7 @@ class ParseFont():
break
return name, family
- def __init__(self, app, parent=None):
+ def __init__(self, app):
super(ParseFont, self).__init__()
self.app = app
@@ -217,7 +219,7 @@ class ParseFont():
if paths is None:
if sys.platform == 'win32':
font_directory = ParseFont.get_win32_font_path()
- paths = [font_directory,]
+ paths = [font_directory, ]
# now get all installed fonts directly...
for f in self.get_win32_fonts(font_directory):
@@ -275,7 +277,7 @@ class ParseFont():
else:
try:
name = name.replace(" Regular", '')
- except Exception as e:
+ except Exception:
pass
self.regular_f.update({name: font})
log.debug("Font parsing is finished.")
@@ -318,7 +320,7 @@ class ParseFont():
if previous > 0 and glyph_index > 0:
delta = face.get_kerning(previous, glyph_index)
pen_x += delta.x
- except Exception as e:
+ except Exception:
pass
face.load_glyph(glyph_index)
diff --git a/appParsers/ParseGerber.py b/appParsers/ParseGerber.py
index ee51e48d..d502177f 100644
--- a/appParsers/ParseGerber.py
+++ b/appParsers/ParseGerber.py
@@ -2,16 +2,13 @@ from PyQt5 import QtWidgets
from camlib import Geometry, arc, arc_angle, ApertureMacro, grace
import numpy as np
-# import re
-# import logging
import traceback
from copy import deepcopy
-# import sys
from shapely.ops import unary_union, linemerge
-# from shapely.affinity import scale, translate
import shapely.affinity as affinity
from shapely.geometry import box as shply_box
+from shapely.geometry import Point
from lxml import etree as ET
import ezdxf
@@ -315,12 +312,12 @@ class Gerber(Geometry):
First is ``G54D11*`` and seconds is ``G36*``.
- :param filename: Gerber file to parse.
- :type filename: str
- :param follow: If true, will not create polygons, just lines
- following the gerber path.
- :type follow: bool
- :return: None
+ :param filename: Gerber file to parse.
+ :type filename: str
+ :param follow: If true, will not create polygons, just lines
+ following the gerber path.
+ :type follow: bool
+ :return: None
"""
with open(filename, 'r') as gfile:
@@ -1837,12 +1834,10 @@ class Gerber(Geometry):
geos_length = 1
if geos_length == 1:
- geo_qrcode = []
- geo_qrcode.append(Polygon(geos[0].exterior))
+ geo_qrcode = [Polygon(geos[0].exterior)]
for i_el in geos[0].interiors:
geo_qrcode.append(Polygon(i_el).buffer(0, resolution=res))
- for poly in geo_qrcode:
- geos.append(poly)
+ geos = [poly for poly in geo_qrcode]
if type(self.solid_geometry) == list:
self.solid_geometry += geos
@@ -1864,15 +1859,14 @@ class Gerber(Geometry):
self.solid_geometry = [self.solid_geometry]
if '0' not in self.apertures:
- self.apertures['0'] = {}
- self.apertures['0']['type'] = 'REG'
- self.apertures['0']['size'] = 0.0
- self.apertures['0']['geometry'] = []
+ self.apertures['0'] = {
+ 'type': 'REG',
+ 'size': 0.0,
+ 'geometry': []
+ }
for pol in self.solid_geometry:
- new_el = {}
- new_el['solid'] = pol
- new_el['follow'] = pol.exterior
+ new_el = {'solid': pol, 'follow': pol.exterior}
self.apertures['0']['geometry'].append(new_el)
def import_dxf_as_gerber(self, filename, units='MM'):
@@ -1914,15 +1908,14 @@ class Gerber(Geometry):
# create the self.apertures data structure
if '0' not in self.apertures:
- self.apertures['0'] = {}
- self.apertures['0']['type'] = 'REG'
- self.apertures['0']['size'] = 0.0
- self.apertures['0']['geometry'] = []
+ self.apertures['0'] = {
+ 'type': 'REG',
+ 'size': 0.0,
+ 'geometry': []
+ }
for pol in flat_geo:
- new_el = {}
- new_el['solid'] = pol
- new_el['follow'] = pol
+ new_el = {'solid': pol, 'follow': pol}
self.apertures['0']['geometry'].append(deepcopy(new_el))
def scale(self, xfactor, yfactor=None, point=None):
diff --git a/appParsers/ParsePDF.py b/appParsers/ParsePDF.py
index 384c8803..947c78f4 100644
--- a/appParsers/ParsePDF.py
+++ b/appParsers/ParsePDF.py
@@ -82,12 +82,10 @@ class PdfParser(QtCore.QObject):
self.restore_gs_re = re.compile(r'^.*Q.*$')
# graphic stack where we save parameters like transformation, line_width
- self.gs = {}
# each element is a list composed of sublist elements
# (each sublist has 2 lists each having 2 elements: first is offset like:
# offset_geo = [off_x, off_y], second element is scale list with 2 elements, like: scale_geo = [sc_x, sc_yy])
- self.gs['transform'] = []
- self.gs['line_width'] = [] # each element is a float
+ self.gs = {'transform': [], 'line_width': []}
# conversion factor to INCH
self.point_to_unit_factor = 0.01388888888
@@ -102,15 +100,17 @@ class PdfParser(QtCore.QObject):
# 1 inch = 72 points => 1 point = 1 / 72 = 0.01388888888 inch
self.point_to_unit_factor = 1 / 72
- path = {}
- path['lines'] = [] # it's a list of lines subpaths
- path['bezier'] = [] # it's a list of bezier arcs subpaths
- path['rectangle'] = [] # it's a list of rectangle subpaths
+ path = {
+ 'lines': [], # it's a list of lines subpaths
+ 'bezier': [], # it's a list of bezier arcs subpaths
+ 'rectangle': [] # it's a list of rectangle subpaths
+ }
- subpath = {}
- subpath['lines'] = [] # it's a list of points
- subpath['bezier'] = [] # it's a list of sublists each like this [start, c1, c2, stop]
- subpath['rectangle'] = [] # it's a list of sublists of points
+ subpath = {
+ 'lines': [], # it's a list of points
+ 'bezier': [], # it's a list of sublists each like this [start, c1, c2, stop]
+ 'rectangle': [] # it's a list of sublists of points
+ }
# store the start point (when 'm' command is encountered)
current_subpath = None
@@ -141,12 +141,14 @@ class PdfParser(QtCore.QObject):
# store the apertures with clear geometry here
# we are interested only in the circular geometry (drill holes) therefore we target only Bezier subpaths
- clear_apertures_dict = {}
# everything will be stored in the '0' aperture since we are dealing with clear polygons not strokes
- clear_apertures_dict['0'] = {}
- clear_apertures_dict['0']['size'] = 0.0
- clear_apertures_dict['0']['type'] = 'C'
- clear_apertures_dict['0']['geometry'] = []
+ clear_apertures_dict = {
+ '0': {
+ 'size': 0.0,
+ 'type': 'C',
+ 'geometry': []
+ }
+ }
# on stroke color change we create a new apertures dictionary and store the old one in a storage from where
# it will be transformed into Gerber object
@@ -527,14 +529,10 @@ class PdfParser(QtCore.QObject):
for pdf_geo in path_geo:
if isinstance(pdf_geo, MultiPolygon):
for poly in pdf_geo:
- new_el = {}
- new_el['solid'] = poly
- new_el['follow'] = poly.exterior
+ new_el = {'solid': poly, 'follow': poly.exterior}
apertures_dict[copy(found_aperture)]['geometry'].append(deepcopy(new_el))
else:
- new_el = {}
- new_el['solid'] = pdf_geo
- new_el['follow'] = pdf_geo.exterior
+ new_el = {'solid': pdf_geo, 'follow': pdf_geo.exterior}
apertures_dict[copy(found_aperture)]['geometry'].append(deepcopy(new_el))
else:
if str(aperture) in apertures_dict.keys():
@@ -546,14 +544,10 @@ class PdfParser(QtCore.QObject):
for pdf_geo in path_geo:
if isinstance(pdf_geo, MultiPolygon):
for poly in pdf_geo:
- new_el = {}
- new_el['solid'] = poly
- new_el['follow'] = poly.exterior
+ new_el = {'solid': poly, 'follow': poly.exterior}
apertures_dict[str(aperture)]['geometry'].append(deepcopy(new_el))
else:
- new_el = {}
- new_el['solid'] = pdf_geo
- new_el['follow'] = pdf_geo.exterior
+ new_el = {'solid': pdf_geo, 'follow': pdf_geo.exterior}
apertures_dict[str(aperture)]['geometry'].append(deepcopy(new_el))
else:
apertures_dict[str(aperture)] = {}
@@ -563,14 +557,10 @@ class PdfParser(QtCore.QObject):
for pdf_geo in path_geo:
if isinstance(pdf_geo, MultiPolygon):
for poly in pdf_geo:
- new_el = {}
- new_el['solid'] = poly
- new_el['follow'] = poly.exterior
+ new_el = {'solid': poly, 'follow': poly.exterior}
apertures_dict[str(aperture)]['geometry'].append(deepcopy(new_el))
else:
- new_el = {}
- new_el['solid'] = pdf_geo
- new_el['follow'] = pdf_geo.exterior
+ new_el = {'solid': pdf_geo, 'follow': pdf_geo.exterior}
apertures_dict[str(aperture)]['geometry'].append(deepcopy(new_el))
continue
@@ -675,12 +665,10 @@ class PdfParser(QtCore.QObject):
if path_geo:
try:
for g in path_geo:
- new_el = {}
- new_el['clear'] = g
+ new_el = {'clear': g}
clear_apertures_dict['0']['geometry'].append(new_el)
except TypeError:
- new_el = {}
- new_el['clear'] = path_geo
+ new_el = {'clear': path_geo}
clear_apertures_dict['0']['geometry'].append(new_el)
# now that we finished searching for drill holes (this is not very precise because holes in the
@@ -690,12 +678,10 @@ class PdfParser(QtCore.QObject):
for pdf_geo in path_geo:
if isinstance(pdf_geo, MultiPolygon):
for poly in pdf_geo:
- new_el = {}
- new_el['clear'] = poly
+ new_el = {'clear': poly}
apertures_dict['0']['geometry'].append(deepcopy(new_el))
else:
- new_el = {}
- new_el['clear'] = pdf_geo
+ new_el = {'clear': pdf_geo}
apertures_dict['0']['geometry'].append(deepcopy(new_el))
except KeyError:
# in case there is no stroke width yet therefore no aperture
@@ -706,12 +692,10 @@ class PdfParser(QtCore.QObject):
for pdf_geo in path_geo:
if isinstance(pdf_geo, MultiPolygon):
for poly in pdf_geo:
- new_el = {}
- new_el['clear'] = poly
+ new_el = {'clear': poly}
apertures_dict['0']['geometry'].append(deepcopy(new_el))
else:
- new_el = {}
- new_el['clear'] = pdf_geo
+ new_el = {'clear': pdf_geo}
apertures_dict['0']['geometry'].append(deepcopy(new_el))
else:
# else, add the geometry as usual
@@ -719,14 +703,10 @@ class PdfParser(QtCore.QObject):
for pdf_geo in path_geo:
if isinstance(pdf_geo, MultiPolygon):
for poly in pdf_geo:
- new_el = {}
- new_el['solid'] = poly
- new_el['follow'] = poly.exterior
+ new_el = {'solid': poly, 'follow': poly.exterior}
apertures_dict['0']['geometry'].append(deepcopy(new_el))
else:
- new_el = {}
- new_el['solid'] = pdf_geo
- new_el['follow'] = pdf_geo.exterior
+ new_el = {'solid': pdf_geo, 'follow': pdf_geo.exterior}
apertures_dict['0']['geometry'].append(deepcopy(new_el))
except KeyError:
# in case there is no stroke width yet therefore no aperture
@@ -737,14 +717,10 @@ class PdfParser(QtCore.QObject):
for pdf_geo in path_geo:
if isinstance(pdf_geo, MultiPolygon):
for poly in pdf_geo:
- new_el = {}
- new_el['solid'] = poly
- new_el['follow'] = poly.exterior
+ new_el = {'solid': poly, 'follow': poly.exterior}
apertures_dict['0']['geometry'].append(deepcopy(new_el))
else:
- new_el = {}
- new_el['solid'] = pdf_geo
- new_el['follow'] = pdf_geo.exterior
+ new_el = {'solid': pdf_geo, 'follow': pdf_geo.exterior}
apertures_dict['0']['geometry'].append(deepcopy(new_el))
continue
@@ -890,50 +866,41 @@ class PdfParser(QtCore.QObject):
for pdf_geo in path_geo:
if isinstance(pdf_geo, MultiPolygon):
for poly in pdf_geo:
- new_el = {}
- new_el['solid'] = poly
- new_el['follow'] = poly.exterior
+ new_el = {'solid': poly, 'follow': poly.exterior}
apertures_dict[copy(found_aperture)]['geometry'].append(deepcopy(new_el))
else:
- new_el = {}
- new_el['solid'] = pdf_geo
- new_el['follow'] = pdf_geo.exterior
+ new_el = {'solid': pdf_geo, 'follow': pdf_geo.exterior}
apertures_dict[copy(found_aperture)]['geometry'].append(deepcopy(new_el))
else:
if str(aperture) in apertures_dict.keys():
aperture += 1
- apertures_dict[str(aperture)] = {}
- apertures_dict[str(aperture)]['size'] = round(applied_size, 5)
- apertures_dict[str(aperture)]['type'] = 'C'
- apertures_dict[str(aperture)]['geometry'] = []
+ apertures_dict[str(aperture)] = {
+ 'size': round(applied_size, 5),
+ 'type': 'C',
+ 'geometry': []
+ }
for pdf_geo in path_geo:
if isinstance(pdf_geo, MultiPolygon):
for poly in pdf_geo:
- new_el = {}
- new_el['solid'] = poly
- new_el['follow'] = poly.exterior
+ new_el = {'solid': poly, 'follow': poly.exterior}
apertures_dict[str(aperture)]['geometry'].append(deepcopy(new_el))
else:
- new_el = {}
- new_el['solid'] = pdf_geo
- new_el['follow'] = pdf_geo.exterior
+ new_el = {'solid': pdf_geo, 'follow': pdf_geo.exterior}
apertures_dict[str(aperture)]['geometry'].append(deepcopy(new_el))
else:
- apertures_dict[str(aperture)] = {}
- apertures_dict[str(aperture)]['size'] = round(applied_size, 5)
- apertures_dict[str(aperture)]['type'] = 'C'
- apertures_dict[str(aperture)]['geometry'] = []
+ apertures_dict[str(aperture)] = {
+ 'size': round(applied_size, 5),
+ 'type': 'C',
+ 'geometry': []
+ }
+
for pdf_geo in path_geo:
if isinstance(pdf_geo, MultiPolygon):
for poly in pdf_geo:
- new_el = {}
- new_el['solid'] = poly
- new_el['follow'] = poly.exterior
+ new_el = {'solid': poly, 'follow': poly.exterior}
apertures_dict[str(aperture)]['geometry'].append(deepcopy(new_el))
else:
- new_el = {}
- new_el['solid'] = pdf_geo
- new_el['follow'] = pdf_geo.exterior
+ new_el = {'solid': pdf_geo, 'follow': pdf_geo.exterior}
apertures_dict[str(aperture)]['geometry'].append(deepcopy(new_el))
# ############################################# ##
@@ -946,60 +913,52 @@ class PdfParser(QtCore.QObject):
for pdf_geo in path_geo:
if isinstance(pdf_geo, MultiPolygon):
for poly in fill_geo:
- new_el = {}
- new_el['clear'] = poly
+ new_el = {'clear': poly}
apertures_dict['0']['geometry'].append(deepcopy(new_el))
else:
- new_el = {}
- new_el['clear'] = pdf_geo
+ new_el = {'clear': pdf_geo}
apertures_dict['0']['geometry'].append(deepcopy(new_el))
except KeyError:
# in case there is no stroke width yet therefore no aperture
- apertures_dict['0'] = {}
- apertures_dict['0']['size'] = round(applied_size, 5)
- apertures_dict['0']['type'] = 'C'
- apertures_dict['0']['geometry'] = []
+ apertures_dict['0'] = {
+ 'size': round(applied_size, 5),
+ 'type': 'C',
+ 'geometry': []
+ }
+
for pdf_geo in fill_geo:
if isinstance(pdf_geo, MultiPolygon):
for poly in pdf_geo:
- new_el = {}
- new_el['clear'] = poly
+ new_el = {'clear': poly}
apertures_dict['0']['geometry'].append(deepcopy(new_el))
else:
- new_el = {}
- new_el['clear'] = pdf_geo
+ new_el = {'clear': pdf_geo}
apertures_dict['0']['geometry'].append(deepcopy(new_el))
else:
try:
for pdf_geo in path_geo:
if isinstance(pdf_geo, MultiPolygon):
for poly in fill_geo:
- new_el = {}
- new_el['solid'] = poly
- new_el['follow'] = poly.exterior
+ new_el = {'solid': poly, 'follow': poly.exterior}
apertures_dict['0']['geometry'].append(deepcopy(new_el))
else:
- new_el = {}
- new_el['solid'] = pdf_geo
- new_el['follow'] = pdf_geo.exterior
+ new_el = {'solid': pdf_geo, 'follow': pdf_geo.exterior}
apertures_dict['0']['geometry'].append(deepcopy(new_el))
except KeyError:
# in case there is no stroke width yet therefore no aperture
- apertures_dict['0'] = {}
- apertures_dict['0']['size'] = round(applied_size, 5)
- apertures_dict['0']['type'] = 'C'
- apertures_dict['0']['geometry'] = []
+ apertures_dict['0'] = {
+ 'size': round(applied_size, 5),
+ 'type': 'C',
+ 'geometry': []
+ }
+
for pdf_geo in fill_geo:
if isinstance(pdf_geo, MultiPolygon):
for poly in pdf_geo:
- new_el = {}
- new_el['solid'] = poly
- new_el['follow'] = poly.exterior
+ new_el = {'solid': poly, 'follow': poly.exterior}
apertures_dict['0']['geometry'].append(deepcopy(new_el))
else:
- new_el = {}
- new_el['solid'] = pdf_geo
- new_el['follow'] = pdf_geo.exterior
+ new_el = {'solid': pdf_geo, 'follow': pdf_geo.exterior}
apertures_dict['0']['geometry'].append(deepcopy(new_el))
continue
diff --git a/appParsers/ParseSVG.py b/appParsers/ParseSVG.py
index 5818d5f2..48879708 100644
--- a/appParsers/ParseSVG.py
+++ b/appParsers/ParseSVG.py
@@ -24,7 +24,7 @@ from svg.path import Line, Arc, CubicBezier, QuadraticBezier, parse_path
# from svg.path.path import Move
# from svg.path.path import Close
import svg.path
-from shapely.geometry import LineString, MultiLineString
+from shapely.geometry import LineString, MultiLineString, Point
from shapely.affinity import skew, affine_transform, rotate
import numpy as np
@@ -286,6 +286,8 @@ def svgcircle2shapely(circle, n_points=64, factor=1.0):
:type circle: xml.etree.ElementTree.Element
:param n_points: circle resolution; nr of points to b e used to approximate a circle
:type n_points: int
+ :param factor:
+ :type factor: float
:return: Shapely representation of the circle.
:rtype: shapely.geometry.polygon.LinearRing
"""
@@ -310,6 +312,8 @@ def svgellipse2shapely(ellipse, n_points=64, factor=1.0):
:type ellipse: xml.etree.ElementTree.Element
:param n_points: Number of discrete points in output.
:type n_points: int
+ :param factor:
+ :type factor: float
:return: Shapely representation of the ellipse.
:rtype: shapely.geometry.polygon.LinearRing
"""
diff --git a/appPreProcessor.py b/appPreProcessor.py
index 4ae297f7..2453a3cf 100644
--- a/appPreProcessor.py
+++ b/appPreProcessor.py
@@ -147,8 +147,10 @@ class AppPreProcTools(object, metaclass=ABCPreProcRegister):
def load_preprocessors(app):
- preprocessors_path_search = [os.path.join(app.data_path, 'preprocessors', '*.py'),
- os.path.join('preprocessors', '*.py')]
+ preprocessors_path_search = [
+ os.path.join(app.data_path, 'preprocessors', '*.py'),
+ os.path.join('preprocessors', '*.py')
+ ]
import glob
for path_search in preprocessors_path_search:
for file in glob.glob(path_search):
diff --git a/appTools/ToolAlignObjects.py b/appTools/ToolAlignObjects.py
index 99eeb6f7..b849888a 100644
--- a/appTools/ToolAlignObjects.py
+++ b/appTools/ToolAlignObjects.py
@@ -383,7 +383,7 @@ class AlignUI:
grid0.setColumnStretch(1, 1)
self.layout.addLayout(grid0)
- self.aligned_label =FCLabel('%s:' % _("MOVING object"))
+ self.aligned_label = FCLabel('%s:' % _("MOVING object"))
grid0.addWidget(self.aligned_label, 0, 0, 1, 2)
self.aligned_label.setToolTip(
@@ -478,7 +478,7 @@ class AlignUI:
grid0.addWidget(separator_line, 14, 0, 1, 2)
# Buttons
- self.align_object_button =FCButton(_("Align Object"))
+ self.align_object_button = FCButton(_("Align Object"))
self.align_object_button.setIcon(QtGui.QIcon(self.app.resource_location + '/align16.png'))
self.align_object_button.setToolTip(
_("Align the specified object to the aligner object.\n"
diff --git a/appTools/ToolCalibration.py b/appTools/ToolCalibration.py
index 9235cc93..9a19c577 100644
--- a/appTools/ToolCalibration.py
+++ b/appTools/ToolCalibration.py
@@ -639,7 +639,7 @@ class ToolCalibration(AppTool):
if obj.tools:
obj_init.tools = deepcopy(obj.tools)
except Exception as ee:
- log.debug("ToolCalibration.new_calibrated_object.initialize_geometry() --> %s" % str(ee))
+ app.log.debug("ToolCalibration.new_calibrated_object.initialize_geometry() --> %s" % str(ee))
obj_init.scale(xfactor=scalex, yfactor=scaley, point=(origin_x, origin_y))
obj_init.skew(angle_x=skewx, angle_y=skewy, point=(origin_x, origin_y))
@@ -649,7 +649,7 @@ class ToolCalibration(AppTool):
except (AttributeError, TypeError):
pass
- def initialize_gerber(obj_init, app):
+ def initialize_gerber(obj_init, app_obj):
obj_init.solid_geometry = deepcopy(obj.solid_geometry)
try:
obj_init.follow_geometry = deepcopy(obj.follow_geometry)
@@ -671,12 +671,12 @@ class ToolCalibration(AppTool):
obj_init.skew(angle_x=skewx, angle_y=skewy, point=(origin_x, origin_y))
try:
- obj_init.source_file = self.app.f_handlers.export_gerber(obj_name=obj_name, filename=None,
- local_use=obj_init, use_thread=False)
+ obj_init.source_file = app_obj.f_handlers.export_gerber(obj_name=obj_name, filename=None,
+ local_use=obj_init, use_thread=False)
except (AttributeError, TypeError):
pass
- def initialize_excellon(obj_init, app):
+ def initialize_excellon(obj_init, app_obj):
obj_init.tools = deepcopy(obj.tools)
# drills are offset, so they need to be deep copied
@@ -689,8 +689,8 @@ class ToolCalibration(AppTool):
obj_init.create_geometry()
- obj_init.source_file = self.app.export.export_excellon(obj_name=obj_name, local_use=obj, filename=None,
- use_thread=False)
+ obj_init.source_file = app_obj.f_handlers.export_excellon(obj_name=obj_name, local_use=obj, filename=None,
+ use_thread=False)
obj = self.cal_object
obj_name = obj_name
diff --git a/appTools/ToolCopperThieving.py b/appTools/ToolCopperThieving.py
index 3de8537f..ce5392c9 100644
--- a/appTools/ToolCopperThieving.py
+++ b/appTools/ToolCopperThieving.py
@@ -245,9 +245,7 @@ class ToolCopperThieving(AppTool):
break
if aperture_found:
- geo_elem = {}
- geo_elem['solid'] = self.robber_geo
- geo_elem['follow'] = self.robber_line
+ geo_elem = {'solid': self.robber_geo, 'follow': self.robber_line}
self.grb_object.apertures[aperture_found]['geometry'].append(deepcopy(geo_elem))
else:
ap_keys = list(self.grb_object.apertures.keys())
@@ -261,9 +259,7 @@ class ToolCopperThieving(AppTool):
self.grb_object.apertures[new_apid]['size'] = self.rb_thickness
self.grb_object.apertures[new_apid]['geometry'] = []
- geo_elem = {}
- geo_elem['solid'] = self.robber_geo
- geo_elem['follow'] = self.robber_line
+ geo_elem = {'solid': self.robber_geo, 'follow': self.robber_line}
self.grb_object.apertures[new_apid]['geometry'].append(deepcopy(geo_elem))
geo_obj = self.grb_object.solid_geometry
@@ -857,18 +853,14 @@ class ToolCopperThieving(AppTool):
geo_list.append(poly)
# append into the '0' aperture
- geo_elem = {}
- geo_elem['solid'] = poly
- geo_elem['follow'] = poly.exterior
+ geo_elem = {'solid': poly, 'follow': poly.exterior}
app_obj.grb_object.apertures['0']['geometry'].append(deepcopy(geo_elem))
except TypeError:
# append to the new solid geometry
geo_list.append(app_obj.new_solid_geometry)
# append into the '0' aperture
- geo_elem = {}
- geo_elem['solid'] = app_obj.new_solid_geometry
- geo_elem['follow'] = app_obj.new_solid_geometry.exterior
+ geo_elem = {'solid': app_obj.new_solid_geometry, 'follow': app_obj.new_solid_geometry.exterior}
app_obj.grb_object.apertures['0']['geometry'].append(deepcopy(geo_elem))
app_obj.grb_object.solid_geometry = MultiPolygon(geo_list).buffer(0.0000001).buffer(-0.0000001)
@@ -993,9 +985,7 @@ class ToolCopperThieving(AppTool):
break
if aperture_found:
- geo_elem = {}
- geo_elem['solid'] = robber_solid_geo
- geo_elem['follow'] = robber_line
+ geo_elem = {'solid': robber_solid_geo, 'follow': robber_line}
grb_obj.apertures[aperture_found]['geometry'].append(deepcopy(geo_elem))
else:
ap_keys = list(grb_obj.apertures.keys())
diff --git a/appTools/ToolCorners.py b/appTools/ToolCorners.py
index d21f602b..6eb0b37e 100644
--- a/appTools/ToolCorners.py
+++ b/appTools/ToolCorners.py
@@ -264,9 +264,7 @@ class ToolCorners(AppTool):
geo_buff = geo.buffer(line_thickness / 2.0, resolution=self.grb_steps_per_circle, join_style=2)
geo_buff_list.append(geo_buff)
- dict_el = {}
- dict_el['follow'] = geo
- dict_el['solid'] = geo_buff
+ dict_el = {'follow': geo, 'solid': geo_buff}
new_apertures[aperture_found]['geometry'].append(deepcopy(dict_el))
else:
ap_keys = list(new_apertures.keys())
@@ -674,4 +672,4 @@ class CornersUI:
self.app.inform[str, bool].emit('[WARNING_NOTCL] %s: [%d, %d]' %
(_("Edited value is out of range"), minval, maxval), False)
else:
- self.app.inform[str, bool].emit('[success] %s' % _("Edited value is within limits."), False)
\ No newline at end of file
+ self.app.inform[str, bool].emit('[success] %s' % _("Edited value is within limits."), False)
diff --git a/appTools/ToolCutOut.py b/appTools/ToolCutOut.py
index d31227a9..cb34fabe 100644
--- a/appTools/ToolCutOut.py
+++ b/appTools/ToolCutOut.py
@@ -389,7 +389,7 @@ class CutOut(AppTool):
def on_tool_default_add(self, dia=None, muted=None):
- dia = dia
+ dia = dia if dia else str(self.app.defaults["tools_cutout_tooldia"])
self.default_data.update({
"plot": True,
@@ -442,7 +442,7 @@ class CutOut(AppTool):
})
self.cut_tool_dict.update({
- 'tooldia': str(self.app.defaults["tools_cutout_tooldia"]),
+ 'tooldia': dia,
'offset': 'Path',
'offset_value': 0.0,
'type': _('Rough'),
@@ -874,16 +874,18 @@ class CutOut(AppTool):
if not holes:
return 'fail'
- tools = {}
- tools[1] = {}
- tools[1]["tooldia"] = mb_dia
- tools[1]['drills'] = holes
- tools[1]['solid_geometry'] = []
+ tools = {
+ 1: {
+ "tooldia": mb_dia,
+ "drills": holes,
+ "solid_geometry": []
+ }
+ }
exc_obj.tools = tools
exc_obj.create_geometry()
exc_obj.source_file = app_o.f_handlers.export_excellon(obj_name=exc_obj.options['name'],
- local_use=exc_obj,filename=None,
+ local_use=exc_obj, filename=None,
use_thread=False)
# calculate the bounds
xmin, ymin, xmax, ymax = CutOut.recursive_bounds(exc_obj.solid_geometry)
@@ -1244,11 +1246,13 @@ class CutOut(AppTool):
if not holes:
return 'fail'
- tools = {}
- tools[1] = {}
- tools[1]["tooldia"] = mb_dia
- tools[1]['drills'] = holes
- tools[1]['solid_geometry'] = []
+ tools = {
+ 1: {
+ "tooldia": mb_dia,
+ "drills": holes,
+ "solid_geometry": []
+ }
+ }
exc_obj.tools = tools
exc_obj.create_geometry()
@@ -1602,11 +1606,13 @@ class CutOut(AppTool):
if not holes:
return 'fail'
- tools = {}
- tools[1] = {}
- tools[1]["tooldia"] = mb_dia
- tools[1]['drills'] = holes
- tools[1]['solid_geometry'] = []
+ tools = {
+ 1: {
+ "tooldia": mb_dia,
+ "drills": holes,
+ "solid_geometry": []
+ }
+ }
exc_obj.tools = tools
exc_obj.create_geometry()
diff --git a/appTools/ToolEtchCompensation.py b/appTools/ToolEtchCompensation.py
index dae10b05..d515eeb7 100644
--- a/appTools/ToolEtchCompensation.py
+++ b/appTools/ToolEtchCompensation.py
@@ -190,9 +190,9 @@ class ToolEtchCompensation(AppTool):
# update the apertures attributes (keys in the apertures dict)
for ap in new_apertures:
- type = new_apertures[ap]['type']
+ ap_type = new_apertures[ap]['type']
for k in new_apertures[ap]:
- if type == 'R' or type == 'O':
+ if ap_type == 'R' or ap_type == 'O':
if k == 'width' or k == 'height':
new_apertures[ap][k] += offset
else:
@@ -207,9 +207,9 @@ class ToolEtchCompensation(AppTool):
# in case of 'R' or 'O' aperture type we need to update the aperture 'size' after
# the 'width' and 'height' keys were updated
for ap in new_apertures:
- type = new_apertures[ap]['type']
+ ap_type = new_apertures[ap]['type']
for k in new_apertures[ap]:
- if type == 'R' or type == 'O':
+ if ap_type == 'R' or ap_type == 'O':
if k == 'size':
new_apertures[ap][k] = math.sqrt(
new_apertures[ap]['width'] ** 2 + new_apertures[ap]['height'] ** 2)
@@ -233,8 +233,8 @@ class ToolEtchCompensation(AppTool):
new_obj.apertures = deepcopy(new_apertures)
new_obj.solid_geometry = deepcopy(new_solid_geometry)
- new_obj.source_file = self.app.f_handlers.export_gerber(obj_name=outname, filename=None, local_use=new_obj,
- use_thread=False)
+ new_obj.source_file = app_obj.f_handlers.export_gerber(obj_name=outname, filename=None, local_use=new_obj,
+ use_thread=False)
self.app.app_obj.new_object('gerber', outname, init_func)
diff --git a/appTools/ToolFiducials.py b/appTools/ToolFiducials.py
index 1a763223..7110824c 100644
--- a/appTools/ToolFiducials.py
+++ b/appTools/ToolFiducials.py
@@ -270,9 +270,7 @@ class ToolFiducials(AppTool):
if aperture_found:
for geo in geo_list:
- dict_el = {}
- dict_el['follow'] = geo.centroid
- dict_el['solid'] = geo
+ dict_el = {'follow': geo.centroid, 'solid': geo}
g_obj.apertures[aperture_found]['geometry'].append(deepcopy(dict_el))
else:
ap_keys = list(g_obj.apertures.keys())
@@ -287,9 +285,7 @@ class ToolFiducials(AppTool):
g_obj.apertures[new_apid]['geometry'] = []
for geo in geo_list:
- dict_el = {}
- dict_el['follow'] = geo.centroid
- dict_el['solid'] = geo
+ dict_el = {'follow': geo.centroid, 'solid': geo}
g_obj.apertures[new_apid]['geometry'].append(deepcopy(dict_el))
s_list = []
@@ -330,9 +326,7 @@ class ToolFiducials(AppTool):
geo_buff_list.append(geo_buff_h)
geo_buff_list.append(geo_buff_v)
- dict_el = {}
- dict_el['follow'] = geo_buff_h.centroid
- dict_el['solid'] = geo_buff_h
+ dict_el = {'follow': geo_buff_h.centroid, 'solid': geo_buff_h}
g_obj.apertures[aperture_found]['geometry'].append(deepcopy(dict_el))
dict_el['follow'] = geo_buff_v.centroid
dict_el['solid'] = geo_buff_v
@@ -344,10 +338,11 @@ class ToolFiducials(AppTool):
else:
new_apid = '10'
- g_obj.apertures[new_apid] = {}
- g_obj.apertures[new_apid]['type'] = 'C'
- g_obj.apertures[new_apid]['size'] = line_thickness
- g_obj.apertures[new_apid]['geometry'] = []
+ g_obj.apertures[new_apid] = {
+ 'type': 'C',
+ 'size': line_thickness,
+ 'geometry': []
+ }
for geo in geo_list:
geo_buff_h = geo[0].buffer(line_thickness / 2.0, self.grb_steps_per_circle)
@@ -355,9 +350,7 @@ class ToolFiducials(AppTool):
geo_buff_list.append(geo_buff_h)
geo_buff_list.append(geo_buff_v)
- dict_el = {}
- dict_el['follow'] = geo_buff_h.centroid
- dict_el['solid'] = geo_buff_h
+ dict_el = {'follow': geo_buff_h.centroid, 'solid': geo_buff_h}
g_obj.apertures[new_apid]['geometry'].append(deepcopy(dict_el))
dict_el['follow'] = geo_buff_v.centroid
dict_el['solid'] = geo_buff_v
@@ -412,9 +405,7 @@ class ToolFiducials(AppTool):
for geo in geo_list:
geo_buff_list.append(geo)
- dict_el = {}
- dict_el['follow'] = geo.centroid
- dict_el['solid'] = geo
+ dict_el = {'follow': geo.centroid, 'solid': geo}
g_obj.apertures[aperture_found]['geometry'].append(deepcopy(dict_el))
else:
ap_keys = list(g_obj.apertures.keys())
@@ -423,19 +414,18 @@ class ToolFiducials(AppTool):
else:
new_apid = '10'
- g_obj.apertures[new_apid] = {}
- g_obj.apertures[new_apid]['type'] = 'R'
- g_obj.apertures[new_apid]['size'] = new_ap_size
- g_obj.apertures[new_apid]['width'] = fid_size
- g_obj.apertures[new_apid]['height'] = fid_size
- g_obj.apertures[new_apid]['geometry'] = []
+ g_obj.apertures[new_apid] = {
+ 'type': 'R',
+ 'size': new_ap_size,
+ 'width': fid_size,
+ 'height': fid_size,
+ 'geometry': []
+ }
for geo in geo_list:
geo_buff_list.append(geo)
- dict_el = {}
- dict_el['follow'] = geo.centroid
- dict_el['solid'] = geo
+ dict_el = {'follow': geo.centroid, 'solid': geo}
g_obj.apertures[new_apid]['geometry'].append(deepcopy(dict_el))
s_list = []
diff --git a/appTools/ToolFilm.py b/appTools/ToolFilm.py
index a1207129..4a474374 100644
--- a/appTools/ToolFilm.py
+++ b/appTools/ToolFilm.py
@@ -613,7 +613,8 @@ class Film(AppTool):
drawing = svg2rlg(doc_final)
p_size = self.ui.pagesize_combo.get_value()
- if p_size == 'Bounds': renderPDF.drawToFile(drawing, filename)
+ if p_size == 'Bounds':
+ renderPDF.drawToFile(drawing, filename)
else:
if self.ui.orientation_radio.get_value() == 'p':
page_size = portrait(self.ui.pagesize[p_size])
diff --git a/appTools/ToolImage.py b/appTools/ToolImage.py
index 12728ff8..e1013ba1 100644
--- a/appTools/ToolImage.py
+++ b/appTools/ToolImage.py
@@ -79,11 +79,10 @@ class ToolImage(AppTool):
def on_file_importimage(self):
"""
Callback for menu item File->Import IMAGE.
- :param type_of_obj: to import the IMAGE as Geometry or as Gerber
- :type type_of_obj: str
+
:return: None
"""
- mask = []
+
self.app.log.debug("on_file_importimage()")
_filter = "Image Files(*.BMP *.PNG *.JPG *.JPEG);;" \
@@ -147,7 +146,7 @@ class ToolImage(AppTool):
geo_obj.import_image(filename, units=units, dpi=dpi, mode=mode, mask=mask)
geo_obj.multigeo = False
- with self.app.proc_container.new(_("Importing Image")) as proc:
+ with self.app.proc_container.new(_("Importing Image")):
# Object name
name = outname or filename.split('/')[-1].split('\\')[-1]
diff --git a/appTools/ToolInvertGerber.py b/appTools/ToolInvertGerber.py
index 3f6f6ec0..51185de6 100644
--- a/appTools/ToolInvertGerber.py
+++ b/appTools/ToolInvertGerber.py
@@ -131,21 +131,18 @@ class ToolInvertGerber(AppTool):
new_apertures = {}
if '0' not in new_apertures:
- new_apertures['0'] = {}
- new_apertures['0']['type'] = 'C'
- new_apertures['0']['size'] = 0.0
- new_apertures['0']['geometry'] = []
+ new_apertures['0'] = {
+ 'type': 'C',
+ 'size': 0.0,
+ 'geometry': []
+ }
try:
for poly in new_solid_geometry:
- new_el = {}
- new_el['solid'] = poly
- new_el['follow'] = poly.exterior
+ new_el = {'solid': poly, 'follow': poly.exterior}
new_apertures['0']['geometry'].append(new_el)
except TypeError:
- new_el = {}
- new_el['solid'] = new_solid_geometry
- new_el['follow'] = new_solid_geometry.exterior
+ new_el = {'solid': new_solid_geometry, 'follow': new_solid_geometry.exterior}
new_apertures['0']['geometry'].append(new_el)
def init_func(new_obj, app_obj):
@@ -157,8 +154,8 @@ class ToolInvertGerber(AppTool):
new_obj.apertures = deepcopy(new_apertures)
new_obj.solid_geometry = deepcopy(new_solid_geometry)
- new_obj.source_file = self.app.f_handlers.export_gerber(obj_name=outname, filename=None,
- local_use=new_obj, use_thread=False)
+ new_obj.source_file = app_obj.f_handlers.export_gerber(obj_name=outname, filename=None,
+ local_use=new_obj, use_thread=False)
self.app.app_obj.new_object('gerber', outname, init_func)
@@ -315,4 +312,4 @@ class InvertUI:
self.app.inform[str, bool].emit('[WARNING_NOTCL] %s: [%d, %d]' %
(_("Edited value is out of range"), minval, maxval), False)
else:
- self.app.inform[str, bool].emit('[success] %s' % _("Edited value is within limits."), False)
\ No newline at end of file
+ self.app.inform[str, bool].emit('[success] %s' % _("Edited value is within limits."), False)
diff --git a/appTools/ToolIsolation.py b/appTools/ToolIsolation.py
index 701c3a10..b5b376cf 100644
--- a/appTools/ToolIsolation.py
+++ b/appTools/ToolIsolation.py
@@ -793,8 +793,8 @@ class ToolIsolation(AppTool, Gerber):
parent=self.app.ui)
tool_add_popup.setWindowIcon(QtGui.QIcon(self.app.resource_location + '/letter_t_32.png'))
- def find_optimal(val):
- tool_add_popup.set_value(float(val))
+ def find_optimal(valor):
+ tool_add_popup.set_value(float(valor))
self.optimal_found_sig.connect(find_optimal)
@@ -1620,8 +1620,7 @@ class ToolIsolation(AppTool, Gerber):
self.app.proc_container.update_view_text(' %s' % _("Subtracting Geo"))
geo_obj.solid_geometry = self.area_subtraction(geo_obj.solid_geometry)
- geo_obj.tools = {}
- geo_obj.tools['1'] = {}
+ geo_obj.tools = {'1': {}}
geo_obj.tools.update({
'1': {
'tooldia': float(tool_dia),
diff --git a/appTools/ToolMilling.py b/appTools/ToolMilling.py
index fd1e569d..02e7ac46 100644
--- a/appTools/ToolMilling.py
+++ b/appTools/ToolMilling.py
@@ -635,7 +635,7 @@ class ToolMilling(AppTool, Excellon):
# Get source object.
try:
self.excellon_obj = self.app.collection.get_by_name(self.obj_name)
- except Exception as e:
+ except Exception:
self.app.inform.emit('[ERROR_NOTCL] %s: %s' % (_("Could not retrieve object"), str(self.obj_name)))
return
diff --git a/appTools/ToolMove.py b/appTools/ToolMove.py
index f0f4d9cc..13166e0d 100644
--- a/appTools/ToolMove.py
+++ b/appTools/ToolMove.py
@@ -160,7 +160,7 @@ class ToolMove(AppTool):
if obj.options['plot'] and obj.visible is True]
def job_move(app_obj):
- with self.app.proc_container.new(_("Moving...")) as proc:
+ with self.app.proc_container.new(_("Moving...")):
if not obj_list:
app_obj.app.inform.emit('[WARNING_NOTCL] %s' % _("No object(s) selected."))
diff --git a/appTools/ToolPDF.py b/appTools/ToolPDF.py
index b0100d17..ad125a72 100644
--- a/appTools/ToolPDF.py
+++ b/appTools/ToolPDF.py
@@ -109,9 +109,10 @@ class ToolPDF(AppTool):
short_name = filename.split('/')[-1].split('\\')[-1]
self.parsing_promises.append(short_name)
- self.pdf_parsed[short_name] = {}
- self.pdf_parsed[short_name]['pdf'] = {}
- self.pdf_parsed[short_name]['filename'] = filename
+ self.pdf_parsed[short_name] = {
+ 'pdf': {},
+ 'filename': filename
+ }
self.pdf_decompressed[short_name] = ''
@@ -181,22 +182,26 @@ class ToolPDF(AppTool):
name_tool = 0
for dia in sorted_dia:
name_tool += 1
+ tool = str(name_tool)
- # create tools dictionary
- spec = {"C": dia, 'solid_geometry': []}
- exc_obj.tools[str(name_tool)] = spec
+ exc_obj.tools[tool] = {
+ 'tooldia': dia,
+ 'drills': [],
+ 'solid_geometry': []
+ }
- # create drill list of dictionaries
+ # update the drill list
for dia_points in points:
if dia == dia_points:
for pt in points[dia_points]:
- exc_obj.drills.append({'point': Point(pt), 'tool': str(name_tool)})
+ exc_obj.tools[tool]['drills'].append(Point(pt))
break
ret = exc_obj.create_geometry()
if ret == 'fail':
log.debug("Could not create geometry for Excellon object.")
return "fail"
+
for tool in exc_obj.tools:
if exc_obj.tools[tool]['solid_geometry']:
return
diff --git a/appTools/ToolPunchGerber.py b/appTools/ToolPunchGerber.py
index 6ffcd408..cd4fb3b8 100644
--- a/appTools/ToolPunchGerber.py
+++ b/appTools/ToolPunchGerber.py
@@ -472,14 +472,14 @@ class ToolPunchGerber(AppTool):
# since there may be drills that do not drill into a pad we test only for
# drills in a pad
if drill_pt.within(elem['solid']):
- geo_elem = {}
- geo_elem['clear'] = drill_pt
+ geo_elem = {'clear': drill_pt}
if clear_apid_size not in holes_apertures:
- holes_apertures[clear_apid_size] = {}
- holes_apertures[clear_apid_size]['type'] = 'C'
- holes_apertures[clear_apid_size]['size'] = clear_apid_size
- holes_apertures[clear_apid_size]['geometry'] = []
+ holes_apertures[clear_apid_size] = {
+ 'type': 'C',
+ 'size': clear_apid_size,
+ 'geometry': []
+ }
holes_apertures[clear_apid_size]['geometry'].append(deepcopy(geo_elem))
@@ -498,8 +498,8 @@ class ToolPunchGerber(AppTool):
new_obj.apertures = deepcopy(new_apertures)
new_obj.solid_geometry = deepcopy(punched_solid_geometry)
- new_obj.source_file = self.app.f_handlers.export_gerber(obj_name=outname, filename=None,
- local_use=new_obj, use_thread=False)
+ new_obj.source_file = app_obj.f_handlers.export_gerber(obj_name=outname, filename=None,
+ local_use=new_obj, use_thread=False)
self.app.app_obj.new_object('gerber', outname, init_func)
@@ -605,14 +605,14 @@ class ToolPunchGerber(AppTool):
# since there may be drills that do not drill into a pad we test only for drills in a pad
if geo.within(elem['solid']):
- geo_elem = {}
- geo_elem['clear'] = geo.centroid
+ geo_elem = {'clear': geo.centroid}
if clear_apid_size not in holes_apertures:
- holes_apertures[clear_apid_size] = {}
- holes_apertures[clear_apid_size]['type'] = 'C'
- holes_apertures[clear_apid_size]['size'] = clear_apid_size
- holes_apertures[clear_apid_size]['geometry'] = []
+ holes_apertures[clear_apid_size] = {
+ 'type': 'C',
+ 'size': clear_apid_size,
+ 'geometry': []
+ }
holes_apertures[clear_apid_size]['geometry'].append(deepcopy(geo_elem))
@@ -631,8 +631,8 @@ class ToolPunchGerber(AppTool):
new_obj.apertures = deepcopy(new_apertures)
new_obj.solid_geometry = deepcopy(punched_solid_geometry)
- new_obj.source_file = self.app.f_handlers.export_gerber(obj_name=outname, filename=None,
- local_use=new_obj, use_thread=False)
+ new_obj.source_file = app_obj.f_handlers.export_gerber(obj_name=outname, filename=None,
+ local_use=new_obj, use_thread=False)
self.app.app_obj.new_object('gerber', outname, init_func)
@@ -754,14 +754,14 @@ class ToolPunchGerber(AppTool):
# since there may be drills that do not drill into a pad we test only for geos in a pad
if geo.within(elem['solid']):
- geo_elem = {}
- geo_elem['clear'] = geo.centroid
+ geo_elem = {'clear': geo.centroid}
if clear_apid_size not in holes_apertures:
- holes_apertures[clear_apid_size] = {}
- holes_apertures[clear_apid_size]['type'] = 'C'
- holes_apertures[clear_apid_size]['size'] = clear_apid_size
- holes_apertures[clear_apid_size]['geometry'] = []
+ holes_apertures[clear_apid_size] = {
+ 'type': 'C',
+ 'size': clear_apid_size,
+ 'geometry': []
+ }
holes_apertures[clear_apid_size]['geometry'].append(deepcopy(geo_elem))
@@ -780,8 +780,8 @@ class ToolPunchGerber(AppTool):
new_obj.apertures = deepcopy(new_apertures)
new_obj.solid_geometry = deepcopy(punched_solid_geometry)
- new_obj.source_file = self.app.f_handlers.export_gerber(obj_name=outname, filename=None,
- local_use=new_obj, use_thread=False)
+ new_obj.source_file = app_obj.f_handlers.export_gerber(obj_name=outname, filename=None,
+ local_use=new_obj, use_thread=False)
self.app.app_obj.new_object('gerber', outname, init_func)
@@ -898,14 +898,14 @@ class ToolPunchGerber(AppTool):
# since there may be drills that do not drill into a pad we test only for geos in a pad
if geo.within(elem['solid']):
- geo_elem = {}
- geo_elem['clear'] = geo.centroid
+ geo_elem = {'clear': geo.centroid}
if clear_apid_size not in holes_apertures:
- holes_apertures[clear_apid_size] = {}
- holes_apertures[clear_apid_size]['type'] = 'C'
- holes_apertures[clear_apid_size]['size'] = clear_apid_size
- holes_apertures[clear_apid_size]['geometry'] = []
+ holes_apertures[clear_apid_size] = {
+ 'type': 'C',
+ 'size': clear_apid_size,
+ 'geometry': []
+ }
holes_apertures[clear_apid_size]['geometry'].append(deepcopy(geo_elem))
@@ -924,8 +924,8 @@ class ToolPunchGerber(AppTool):
new_obj.apertures = deepcopy(new_apertures)
new_obj.solid_geometry = deepcopy(punched_solid_geometry)
- new_obj.source_file = self.app.f_handlers.export_gerber(obj_name=outname, filename=None,
- local_use=new_obj, use_thread=False)
+ new_obj.source_file = app_obj.f_handlers.export_gerber(obj_name=outname, filename=None,
+ local_use=new_obj, use_thread=False)
self.app.app_obj.new_object('gerber', outname, init_func)
diff --git a/appTools/ToolQRCode.py b/appTools/ToolQRCode.py
index 29398b01..79275300 100644
--- a/appTools/ToolQRCode.py
+++ b/appTools/ToolQRCode.py
@@ -270,9 +270,11 @@ class QRCode(AppTool):
# don't know if the condition is required since I already made sure above that the new_apid is a new one
if new_apid not in self.grb_object.apertures:
- self.grb_object.apertures[new_apid] = {}
- self.grb_object.apertures[new_apid]['geometry'] = []
- self.grb_object.apertures[new_apid]['type'] = 'R'
+ self.grb_object.apertures[new_apid] = {
+ 'type': 'R',
+ 'geometry': []
+ }
+
# TODO: HACK
# I've artificially added 1% to the height and width because otherwise after loading the
# exported file, it will not be correctly reconstructed (it will be made from multiple shapes instead of
@@ -282,15 +284,15 @@ class QRCode(AppTool):
self.grb_object.apertures[new_apid]['size'] = deepcopy(math.sqrt(box_size ** 2 + box_size ** 2))
if '0' not in self.grb_object.apertures:
- self.grb_object.apertures['0'] = {}
- self.grb_object.apertures['0']['geometry'] = []
- self.grb_object.apertures['0']['type'] = 'REG'
- self.grb_object.apertures['0']['size'] = 0.0
+ self.grb_object.apertures['0'] = {
+ 'type': 'REG',
+ 'size': 0.0,
+ 'geometry': []
+ }
# in case that the QRCode geometry is dropped onto a copper region (found in the '0' aperture)
# make sure that I place a cutout there
- zero_elem = {}
- zero_elem['clear'] = offset_mask_geo
+ zero_elem = {'clear': offset_mask_geo}
self.grb_object.apertures['0']['geometry'].append(deepcopy(zero_elem))
try:
@@ -304,13 +306,13 @@ class QRCode(AppTool):
try:
for geo in self.qrcode_geometry:
- geo_elem = {}
- geo_elem['solid'] = translate(geo, xoff=pos[0], yoff=pos[1])
- geo_elem['follow'] = translate(geo.centroid, xoff=pos[0], yoff=pos[1])
+ geo_elem = {
+ 'solid': translate(geo, xoff=pos[0], yoff=pos[1]),
+ 'follow': translate(geo.centroid, xoff=pos[0], yoff=pos[1])
+ }
self.grb_object.apertures[new_apid]['geometry'].append(deepcopy(geo_elem))
except TypeError:
- geo_elem = {}
- geo_elem['solid'] = self.qrcode_geometry
+ geo_elem = {'solid': self.qrcode_geometry}
self.grb_object.apertures[new_apid]['geometry'].append(deepcopy(geo_elem))
# update the source file with the new geometry:
diff --git a/appTools/ToolRulesCheck.py b/appTools/ToolRulesCheck.py
index b48882c1..15bf43a2 100644
--- a/appTools/ToolRulesCheck.py
+++ b/appTools/ToolRulesCheck.py
@@ -631,16 +631,18 @@ class RulesCheck(AppTool):
copper_list = []
copper_name_1 = self.ui.copper_t_object.currentText()
if copper_name_1 != '' and self.ui.copper_t_cb.get_value():
- elem_dict = {}
- elem_dict['name'] = deepcopy(copper_name_1)
- elem_dict['apertures'] = deepcopy(self.app.collection.get_by_name(copper_name_1).apertures)
+ elem_dict = {
+ 'name': deepcopy(copper_name_1),
+ 'apertures': deepcopy(app_obj.collection.get_by_name(copper_name_1).apertures)
+ }
copper_list.append(elem_dict)
copper_name_2 = self.ui.copper_b_object.currentText()
if copper_name_2 != '' and self.ui.copper_b_cb.get_value():
- elem_dict = {}
- elem_dict['name'] = deepcopy(copper_name_2)
- elem_dict['apertures'] = deepcopy(self.app.collection.get_by_name(copper_name_2).apertures)
+ elem_dict = {
+ 'name': deepcopy(copper_name_2),
+ 'apertures': deepcopy(app_obj.collection.get_by_name(copper_name_2).apertures)
+ }
copper_list.append(elem_dict)
trace_size = float(self.ui.trace_size_entry.get_value())
@@ -664,7 +666,7 @@ class RulesCheck(AppTool):
if copper_t_obj != '':
copper_t_dict['name'] = deepcopy(copper_t_obj)
- copper_t_dict['apertures'] = deepcopy(self.app.collection.get_by_name(copper_t_obj).apertures)
+ copper_t_dict['apertures'] = deepcopy(app_obj.collection.get_by_name(copper_t_obj).apertures)
self.results.append(self.pool.apply_async(self.check_inside_gerber_clearance,
args=(copper_t_dict,
@@ -675,7 +677,7 @@ class RulesCheck(AppTool):
copper_b_dict = {}
if copper_b_obj != '':
copper_b_dict['name'] = deepcopy(copper_b_obj)
- copper_b_dict['apertures'] = deepcopy(self.app.collection.get_by_name(copper_b_obj).apertures)
+ copper_b_dict['apertures'] = deepcopy(app_obj.collection.get_by_name(copper_b_obj).apertures)
self.results.append(self.pool.apply_async(self.check_inside_gerber_clearance,
args=(copper_b_dict,
@@ -683,7 +685,7 @@ class RulesCheck(AppTool):
_("BOTTOM -> Copper to Copper clearance"))))
if self.ui.copper_t_cb.get_value() is False and self.ui.copper_b_cb.get_value() is False:
- self.app.inform.emit('[ERROR_NOTCL] %s. %s' % (
+ app_obj.inform.emit('[ERROR_NOTCL] %s. %s' % (
_("Copper to Copper clearance"),
_("At least one Gerber object has to be selected for this rule but none is selected.")))
return
@@ -697,29 +699,29 @@ class RulesCheck(AppTool):
copper_top = self.ui.copper_t_object.currentText()
if copper_top != '' and self.ui.copper_t_cb.get_value():
top_dict['name'] = deepcopy(copper_top)
- top_dict['apertures'] = deepcopy(self.app.collection.get_by_name(copper_top).apertures)
+ top_dict['apertures'] = deepcopy(app_obj.collection.get_by_name(copper_top).apertures)
copper_bottom = self.ui.copper_b_object.currentText()
if copper_bottom != '' and self.ui.copper_b_cb.get_value():
bottom_dict['name'] = deepcopy(copper_bottom)
- bottom_dict['apertures'] = deepcopy(self.app.collection.get_by_name(copper_bottom).apertures)
+ bottom_dict['apertures'] = deepcopy(app_obj.collection.get_by_name(copper_bottom).apertures)
copper_outline = self.ui.outline_object.currentText()
if copper_outline != '' and self.ui.out_cb.get_value():
outline_dict['name'] = deepcopy(copper_outline)
- outline_dict['apertures'] = deepcopy(self.app.collection.get_by_name(copper_outline).apertures)
+ outline_dict['apertures'] = deepcopy(app_obj.collection.get_by_name(copper_outline).apertures)
try:
copper_outline_clearance = float(self.ui.clearance_copper2ol_entry.get_value())
except Exception as e:
log.debug("RulesCheck.execute.worker_job() --> %s" % str(e))
- self.app.inform.emit('[ERROR_NOTCL] %s. %s' % (
+ app_obj.inform.emit('[ERROR_NOTCL] %s. %s' % (
_("Copper to Outline clearance"),
_("Value is not valid.")))
return
if not top_dict and not bottom_dict or not outline_dict:
- self.app.inform.emit('[ERROR_NOTCL] %s. %s' % (
+ app_obj.inform.emit('[ERROR_NOTCL] %s. %s' % (
_("Copper to Outline clearance"),
_("One of the copper Gerber objects or the Outline Gerber object is not valid.")))
return
@@ -732,7 +734,7 @@ class RulesCheck(AppTool):
if outline_dict:
objs.append(outline_dict)
else:
- self.app.inform.emit('[ERROR_NOTCL] %s. %s' % (
+ app_obj.inform.emit('[ERROR_NOTCL] %s. %s' % (
_("Copper to Outline clearance"),
_("Outline Gerber object presence is mandatory for this rule but it is not selected.")))
return
@@ -750,7 +752,7 @@ class RulesCheck(AppTool):
silk_silk_clearance = float(self.ui.clearance_silk2silk_entry.get_value())
except Exception as e:
log.debug("RulesCheck.execute.worker_job() --> %s" % str(e))
- self.app.inform.emit('[ERROR_NOTCL] %s. %s' % (
+ app_obj.inform.emit('[ERROR_NOTCL] %s. %s' % (
_("Silk to Silk clearance"),
_("Value is not valid.")))
return
@@ -759,7 +761,7 @@ class RulesCheck(AppTool):
silk_obj = self.ui.ss_t_object.currentText()
if silk_obj != '':
silk_dict['name'] = deepcopy(silk_obj)
- silk_dict['apertures'] = deepcopy(self.app.collection.get_by_name(silk_obj).apertures)
+ silk_dict['apertures'] = deepcopy(app_obj.collection.get_by_name(silk_obj).apertures)
self.results.append(self.pool.apply_async(self.check_inside_gerber_clearance,
args=(silk_dict,
@@ -769,7 +771,7 @@ class RulesCheck(AppTool):
silk_obj = self.ui.ss_b_object.currentText()
if silk_obj != '':
silk_dict['name'] = deepcopy(silk_obj)
- silk_dict['apertures'] = deepcopy(self.app.collection.get_by_name(silk_obj).apertures)
+ silk_dict['apertures'] = deepcopy(app_obj.collection.get_by_name(silk_obj).apertures)
self.results.append(self.pool.apply_async(self.check_inside_gerber_clearance,
args=(silk_dict,
@@ -777,7 +779,7 @@ class RulesCheck(AppTool):
_("BOTTOM -> Silk to Silk clearance"))))
if self.ui.ss_t_cb.get_value() is False and self.ui.ss_b_cb.get_value() is False:
- self.app.inform.emit('[ERROR_NOTCL] %s. %s' % (
+ app_obj.inform.emit('[ERROR_NOTCL] %s. %s' % (
_("Silk to Silk clearance"),
_("At least one Gerber object has to be selected for this rule but none is selected.")))
return
@@ -797,38 +799,38 @@ class RulesCheck(AppTool):
silk_top = self.ui.ss_t_object.currentText()
if silk_top != '' and self.ui.ss_t_cb.get_value():
silk_t_dict['name'] = deepcopy(silk_top)
- silk_t_dict['apertures'] = deepcopy(self.app.collection.get_by_name(silk_top).apertures)
+ silk_t_dict['apertures'] = deepcopy(app_obj.collection.get_by_name(silk_top).apertures)
top_ss = True
silk_bottom = self.ui.ss_b_object.currentText()
if silk_bottom != '' and self.ui.ss_b_cb.get_value():
silk_b_dict['name'] = deepcopy(silk_bottom)
- silk_b_dict['apertures'] = deepcopy(self.app.collection.get_by_name(silk_bottom).apertures)
+ silk_b_dict['apertures'] = deepcopy(app_obj.collection.get_by_name(silk_bottom).apertures)
bottom_ss = True
sm_top = self.ui.sm_t_object.currentText()
if sm_top != '' and self.ui.sm_t_cb.get_value():
sm_t_dict['name'] = deepcopy(sm_top)
- sm_t_dict['apertures'] = deepcopy(self.app.collection.get_by_name(sm_top).apertures)
+ sm_t_dict['apertures'] = deepcopy(app_obj.collection.get_by_name(sm_top).apertures)
top_sm = True
sm_bottom = self.ui.sm_b_object.currentText()
if sm_bottom != '' and self.ui.sm_b_cb.get_value():
sm_b_dict['name'] = deepcopy(sm_bottom)
- sm_b_dict['apertures'] = deepcopy(self.app.collection.get_by_name(sm_bottom).apertures)
+ sm_b_dict['apertures'] = deepcopy(app_obj.collection.get_by_name(sm_bottom).apertures)
bottom_sm = True
try:
silk_sm_clearance = float(self.ui.clearance_silk2sm_entry.get_value())
except Exception as e:
log.debug("RulesCheck.execute.worker_job() --> %s" % str(e))
- self.app.inform.emit('[ERROR_NOTCL] %s. %s' % (
+ app_obj.inform.emit('[ERROR_NOTCL] %s. %s' % (
_("Silk to Solder Mask Clearance"),
_("Value is not valid.")))
return
if (not silk_t_dict and not silk_b_dict) or (not sm_t_dict and not sm_b_dict):
- self.app.inform.emit('[ERROR_NOTCL] %s. %s' % (
+ app_obj.inform.emit('[ERROR_NOTCL] %s. %s' % (
_("Silk to Solder Mask Clearance"),
_("One or more of the Gerber objects is not valid.")))
return
@@ -846,7 +848,7 @@ class RulesCheck(AppTool):
silk_sm_clearance,
_("BOTTOM -> Silk to Solder Mask Clearance"))))
else:
- self.app.inform.emit('[ERROR_NOTCL] %s. %s' % (
+ app_obj.inform.emit('[ERROR_NOTCL] %s. %s' % (
_("Silk to Solder Mask Clearance"),
_("Both Silk and Solder Mask Gerber objects has to be either both Top or both Bottom.")))
return
@@ -860,29 +862,29 @@ class RulesCheck(AppTool):
silk_top = self.ui.ss_t_object.currentText()
if silk_top != '' and self.ui.ss_t_cb.get_value():
top_dict['name'] = deepcopy(silk_top)
- top_dict['apertures'] = deepcopy(self.app.collection.get_by_name(silk_top).apertures)
+ top_dict['apertures'] = deepcopy(app_obj.collection.get_by_name(silk_top).apertures)
silk_bottom = self.ui.ss_b_object.currentText()
if silk_bottom != '' and self.ui.ss_b_cb.get_value():
bottom_dict['name'] = deepcopy(silk_bottom)
- bottom_dict['apertures'] = deepcopy(self.app.collection.get_by_name(silk_bottom).apertures)
+ bottom_dict['apertures'] = deepcopy(app_obj.collection.get_by_name(silk_bottom).apertures)
copper_outline = self.ui.outline_object.currentText()
if copper_outline != '' and self.ui.out_cb.get_value():
outline_dict['name'] = deepcopy(copper_outline)
- outline_dict['apertures'] = deepcopy(self.app.collection.get_by_name(copper_outline).apertures)
+ outline_dict['apertures'] = deepcopy(app_obj.collection.get_by_name(copper_outline).apertures)
try:
copper_outline_clearance = float(self.ui.clearance_copper2ol_entry.get_value())
except Exception as e:
log.debug("RulesCheck.execute.worker_job() --> %s" % str(e))
- self.app.inform.emit('[ERROR_NOTCL] %s. %s' % (
+ app_obj.inform.emit('[ERROR_NOTCL] %s. %s' % (
_("Silk to Outline Clearance"),
_("Value is not valid.")))
return
if not top_dict and not bottom_dict or not outline_dict:
- self.app.inform.emit('[ERROR_NOTCL] %s. %s' % (
+ app_obj.inform.emit('[ERROR_NOTCL] %s. %s' % (
_("Silk to Outline Clearance"),
_("One of the Silk Gerber objects or the Outline Gerber object is not valid.")))
return
@@ -896,7 +898,7 @@ class RulesCheck(AppTool):
if outline_dict:
objs.append(outline_dict)
else:
- self.app.inform.emit('[ERROR_NOTCL] %s. %s' % (
+ app_obj.inform.emit('[ERROR_NOTCL] %s. %s' % (
_("Silk to Outline Clearance"),
_("Outline Gerber object presence is mandatory for this rule but it is not selected.")))
return
@@ -914,7 +916,7 @@ class RulesCheck(AppTool):
sm_sm_clearance = float(self.ui.clearance_sm2sm_entry.get_value())
except Exception as e:
log.debug("RulesCheck.execute.worker_job() --> %s" % str(e))
- self.app.inform.emit('[ERROR_NOTCL] %s. %s' % (
+ app_obj.inform.emit('[ERROR_NOTCL] %s. %s' % (
_("Minimum Solder Mask Sliver"),
_("Value is not valid.")))
return
@@ -923,7 +925,7 @@ class RulesCheck(AppTool):
solder_obj = self.ui.sm_t_object.currentText()
if solder_obj != '':
sm_dict['name'] = deepcopy(solder_obj)
- sm_dict['apertures'] = deepcopy(self.app.collection.get_by_name(solder_obj).apertures)
+ sm_dict['apertures'] = deepcopy(app_obj.collection.get_by_name(solder_obj).apertures)
self.results.append(self.pool.apply_async(self.check_inside_gerber_clearance,
args=(sm_dict,
@@ -933,7 +935,7 @@ class RulesCheck(AppTool):
solder_obj = self.ui.sm_b_object.currentText()
if solder_obj != '':
sm_dict['name'] = deepcopy(solder_obj)
- sm_dict['apertures'] = deepcopy(self.app.collection.get_by_name(solder_obj).apertures)
+ sm_dict['apertures'] = deepcopy(app_obj.collection.get_by_name(solder_obj).apertures)
self.results.append(self.pool.apply_async(self.check_inside_gerber_clearance,
args=(sm_dict,
@@ -941,7 +943,7 @@ class RulesCheck(AppTool):
_("BOTTOM -> Minimum Solder Mask Sliver"))))
if self.ui.sm_t_cb.get_value() is False and self.ui.sm_b_cb.get_value() is False:
- self.app.inform.emit('[ERROR_NOTCL] %s. %s' % (
+ app_obj.inform.emit('[ERROR_NOTCL] %s. %s' % (
_("Minimum Solder Mask Sliver"),
_("At least one Gerber object has to be selected for this rule but none is selected.")))
return
@@ -956,36 +958,36 @@ class RulesCheck(AppTool):
copper_top = self.ui.copper_t_object.currentText()
if copper_top != '' and self.ui.copper_t_cb.get_value():
top_dict['name'] = deepcopy(copper_top)
- top_dict['apertures'] = deepcopy(self.app.collection.get_by_name(copper_top).apertures)
+ top_dict['apertures'] = deepcopy(app_obj.collection.get_by_name(copper_top).apertures)
copper_bottom = self.ui.copper_b_object.currentText()
if copper_bottom != '' and self.ui.copper_b_cb.get_value():
bottom_dict['name'] = deepcopy(copper_bottom)
- bottom_dict['apertures'] = deepcopy(self.app.collection.get_by_name(copper_bottom).apertures)
+ bottom_dict['apertures'] = deepcopy(app_obj.collection.get_by_name(copper_bottom).apertures)
excellon_1 = self.ui.e1_object.currentText()
if excellon_1 != '' and self.ui.e1_cb.get_value():
exc_1_dict['name'] = deepcopy(excellon_1)
exc_1_dict['tools'] = deepcopy(
- self.app.collection.get_by_name(excellon_1).tools)
+ app_obj.collection.get_by_name(excellon_1).tools)
excellon_2 = self.ui.e2_object.currentText()
if excellon_2 != '' and self.ui.e2_cb.get_value():
exc_2_dict['name'] = deepcopy(excellon_2)
exc_2_dict['tools'] = deepcopy(
- self.app.collection.get_by_name(excellon_2).tools)
+ app_obj.collection.get_by_name(excellon_2).tools)
try:
ring_val = float(self.ui.ring_integrity_entry.get_value())
except Exception as e:
log.debug("RulesCheck.execute.worker_job() --> %s" % str(e))
- self.app.inform.emit('[ERROR_NOTCL] %s. %s' % (
+ app_obj.inform.emit('[ERROR_NOTCL] %s. %s' % (
_("Minimum Annular Ring"),
_("Value is not valid.")))
return
if (not top_dict and not bottom_dict) or (not exc_1_dict and not exc_2_dict):
- self.app.inform.emit('[ERROR_NOTCL] %s. %s' % (
+ app_obj.inform.emit('[ERROR_NOTCL] %s. %s' % (
_("Minimum Annular Ring"),
_("One of the Copper Gerber objects or the Excellon objects is not valid.")))
return
@@ -1001,7 +1003,7 @@ class RulesCheck(AppTool):
elif exc_2_dict:
objs.append(exc_2_dict)
else:
- self.app.inform.emit('[ERROR_NOTCL] %s. %s' % (
+ app_obj.inform.emit('[ERROR_NOTCL] %s. %s' % (
_("Minimum Annular Ring"),
_("Excellon object presence is mandatory for this rule but none is selected.")))
return
@@ -1016,16 +1018,18 @@ class RulesCheck(AppTool):
exc_list = []
exc_name_1 = self.ui.e1_object.currentText()
if exc_name_1 != '' and self.ui.e1_cb.get_value():
- elem_dict = {}
- elem_dict['name'] = deepcopy(exc_name_1)
- elem_dict['tools'] = deepcopy(self.app.collection.get_by_name(exc_name_1).tools)
+ elem_dict = {
+ 'name': deepcopy(exc_name_1),
+ 'tools': deepcopy(app_obj.collection.get_by_name(exc_name_1).tools)
+ }
exc_list.append(elem_dict)
exc_name_2 = self.ui.e2_object.currentText()
if exc_name_2 != '' and self.ui.e2_cb.get_value():
- elem_dict = {}
- elem_dict['name'] = deepcopy(exc_name_2)
- elem_dict['tools'] = deepcopy(self.app.collection.get_by_name(exc_name_2).tools)
+ elem_dict = {
+ 'name': deepcopy(exc_name_2),
+ 'tools': deepcopy(app_obj.collection.get_by_name(exc_name_2).tools)
+ }
exc_list.append(elem_dict)
hole_clearance = float(self.ui.clearance_d2d_entry.get_value())
@@ -1036,16 +1040,18 @@ class RulesCheck(AppTool):
exc_list = []
exc_name_1 = self.ui.e1_object.currentText()
if exc_name_1 != '' and self.ui.e1_cb.get_value():
- elem_dict = {}
- elem_dict['name'] = deepcopy(exc_name_1)
- elem_dict['tools'] = deepcopy(self.app.collection.get_by_name(exc_name_1).tools)
+ elem_dict = {
+ 'name': deepcopy(exc_name_1),
+ 'tools': deepcopy(app_obj.collection.get_by_name(exc_name_1).tools)
+ }
exc_list.append(elem_dict)
exc_name_2 = self.ui.e2_object.currentText()
if exc_name_2 != '' and self.ui.e2_cb.get_value():
- elem_dict = {}
- elem_dict['name'] = deepcopy(exc_name_2)
- elem_dict['tools'] = deepcopy(self.app.collection.get_by_name(exc_name_2).tools)
+ elem_dict = {
+ 'name': deepcopy(exc_name_2),
+ 'tools': deepcopy(app_obj.collection.get_by_name(exc_name_2).tools)
+ }
exc_list.append(elem_dict)
drill_size = float(self.ui.drill_size_entry.get_value())
@@ -1056,7 +1062,7 @@ class RulesCheck(AppTool):
output.append(p.get())
self.tool_finished.emit(output)
- self.app.proc_container.view.set_idle()
+ app_obj.proc_container.view.set_idle()
log.debug("RuleCheck() finished")
diff --git a/appTools/ToolSub.py b/appTools/ToolSub.py
index 8a94e34d..46a2f4b5 100644
--- a/appTools/ToolSub.py
+++ b/appTools/ToolSub.py
@@ -195,9 +195,7 @@ class ToolSub(AppTool):
def worker_job(app_obj):
# SUBTRACTOR geometry (always the same)
- sub_geometry = {}
- sub_geometry['solid'] = []
- sub_geometry['clear'] = []
+ sub_geometry = {'solid': [], 'clear': []}
# iterate over SUBTRACTOR geometry and load it in the sub_geometry dict
for s_apid in app_obj.sub_grb_obj.apertures:
for s_el in app_obj.sub_grb_obj.apertures[s_apid]['geometry']:
@@ -206,13 +204,13 @@ class ToolSub(AppTool):
if "clear" in s_el:
sub_geometry['clear'].append(s_el["clear"])
- for apid in app_obj.target_grb_obj.apertures:
+ for ap_id in app_obj.target_grb_obj.apertures:
# TARGET geometry
- target_geo = [geo for geo in app_obj.target_grb_obj.apertures[apid]['geometry']]
+ target_geo = [geo for geo in app_obj.target_grb_obj.apertures[ap_id]['geometry']]
# send the job to the multiprocessing JOB
app_obj.results.append(
- app_obj.pool.apply_async(app_obj.aperture_intersection, args=(apid, target_geo, sub_geometry))
+ app_obj.pool.apply_async(app_obj.aperture_intersection, args=(ap_id, target_geo, sub_geometry))
)
output = []
@@ -348,8 +346,8 @@ class ToolSub(AppTool):
grb_obj.solid_geometry = deepcopy(poly_buff)
grb_obj.follow_geometry = deepcopy(follow_buff)
- grb_obj.source_file = self.app.f_handlers.export_gerber(obj_name=outname, filename=None,
- local_use=grb_obj, use_thread=False)
+ grb_obj.source_file = app_obj.f_handlers.export_gerber(obj_name=outname, filename=None,
+ local_use=grb_obj, use_thread=False)
with self.app.proc_container.new(_("Generating new object ...")):
ret = self.app.app_obj.new_object('gerber', outname, obj_init, autoselected=False)
@@ -542,7 +540,7 @@ class ToolSub(AppTool):
for tool in geo_obj.tools:
geo_obj.tools[tool]['solid_geometry'] = deepcopy(self.new_solid_geometry)
except Exception as e:
- log.debug("ToolSub.new_geo_object() --> %s" % str(e))
+ app_obj.log.debug("ToolSub.new_geo_object() --> %s" % str(e))
geo_obj.multigeo = False
with self.app.proc_container.new(_("Generating new object ...")):
diff --git a/camlib.py b/camlib.py
index 0ee202dc..4fd0cc0c 100644
--- a/camlib.py
+++ b/camlib.py
@@ -3172,12 +3172,12 @@ class CNCjob(Geometry):
# Depth parameters
self.z_cut = tool_dict['tools_drill_cutz']
- old_zcut = deepcopy(tool_dict["tools_drill_cutz"]) # multidepth use this
+ old_zcut = deepcopy(tool_dict["tools_drill_cutz"]) # multidepth use this
self.multidepth = tool_dict['tools_drill_multidepth']
self.z_depthpercut = tool_dict['tools_drill_depthperpass']
self.z_move = tool_dict['tools_drill_travelz']
- self.f_plunge = tool_dict["tools_drill_f_plunge"] # used directly in the preprocessor Toolchange method
- self.f_retract = tool_dict["tools_drill_f_retract"] # used in the current method
+ self.f_plunge = tool_dict["tools_drill_f_plunge"] # used directly in the preprocessor Toolchange method
+ self.f_retract = tool_dict["tools_drill_f_retract"] # used in the current method
# Feedrate parameters
self.z_feedrate = tool_dict['tools_drill_feedrate_z']
@@ -5283,8 +5283,8 @@ class CNCjob(Geometry):
self.z_move = float(z_move) if z_move is not None else None
self.feedrate = float(feedrate) if feedrate else self.app.defaults["geometry_feedrate"]
- self.z_feedrate = float(feedrate_z) if feedrate_z is not None else self.app.defaults["geometry_feedrate_z"]
- self.feedrate_rapid = float(feedrate_rapid) if feedrate_rapid else self.app.defaults["geometry_feedrate_rapid"]
+ self.z_feedrate = float(feedrate_z) if feedrate_z is not None else self.app.defaults["geometry_feedrate_z"]
+ self.feedrate_rapid = float(feedrate_rapid) if feedrate_rapid else self.app.defaults["geometry_feedrate_rapid"]
self.spindlespeed = int(spindlespeed) if spindlespeed != 0 else None
self.spindledir = spindledir
diff --git a/descartes/patch.py b/descartes/patch.py
index ea0c22b5..556798a2 100644
--- a/descartes/patch.py
+++ b/descartes/patch.py
@@ -36,6 +36,7 @@ def PolygonPath(polygon):
geometric object"""
this = Polygon(polygon)
assert this.geom_type == 'Polygon'
+
def coding(ob):
# The codes will be all "LINETO" commands, except for "MOVETO"s at the
# beginning of each subpath
@@ -43,12 +44,9 @@ def PolygonPath(polygon):
vals = ones(n, dtype=Path.code_type) * Path.LINETO
vals[0] = Path.MOVETO
return vals
- vertices = concatenate(
- [asarray(this.exterior)]
- + [asarray(r) for r in this.interiors])
- codes = concatenate(
- [coding(this.exterior)]
- + [coding(r) for r in this.interiors])
+
+ vertices = concatenate([asarray(this.exterior)] + [asarray(r) for r in this.interiors])
+ codes = concatenate([coding(this.exterior)] + [coding(r) for r in this.interiors])
return Path(vertices, codes)
diff --git a/tclCommands/TclCommand.py b/tclCommands/TclCommand.py
index 2c927ebd..80f19dd9 100644
--- a/tclCommands/TclCommand.py
+++ b/tclCommands/TclCommand.py
@@ -79,8 +79,7 @@ class TclCommand(object):
:return: current command
"""
- command_string = []
- command_string.append(self.aliases[0])
+ command_string = [self.aliases[0]]
if self.original_args is not None:
for arg in self.original_args:
@@ -417,7 +416,7 @@ class TclCommandSignaled(TclCommand):
# set detail for processing, it will be there until next open or close
self.app.shell.open_processing(self.get_current_command())
- def handle_finished(obj):
+ def handle_finished():
self.app.shell_command_finished.disconnect(handle_finished)
if self.error is not None:
self.raise_tcl_unknown_error(self.error)
diff --git a/tclCommands/TclCommandAlignDrill.py b/tclCommands/TclCommandAlignDrill.py
index 250d82be..c467d6bd 100644
--- a/tclCommands/TclCommandAlignDrill.py
+++ b/tclCommands/TclCommandAlignDrill.py
@@ -108,8 +108,10 @@ class TclCommandAlignDrill(TclCommandSignaled):
xscale, yscale = {"X": (1.0, -1.0), "Y": (-1.0, 1.0)}[axis]
+ tooldia = args['dia']
+
# Tools
- tools = {"1": {"C": args['dia']}}
+ # tools = {"1": {"C": args['dia']}}
def alligndrill_init_me(init_obj, app_obj):
"""
@@ -126,8 +128,8 @@ class TclCommandAlignDrill(TclCommandSignaled):
for hole in holes:
point = Point(hole)
point_mirror = affinity.scale(point, xscale, yscale, origin=(px, py))
- drills.append({"point": point, "tool": "1"})
- drills.append({"point": point_mirror, "tool": "1"})
+ drills.append(point)
+ drills.append(point_mirror)
else:
if 'box' not in args:
return "ERROR: -grid can be used only for -box"
@@ -167,11 +169,17 @@ class TclCommandAlignDrill(TclCommandSignaled):
for hole in localholes:
point = Point(hole)
point_mirror = affinity.scale(point, xscale, yscale, origin=(px, py))
- drills.append({"point": point, "tool": "1"})
- drills.append({"point": point_mirror, "tool": "1"})
+ drills.append(point)
+ drills.append(point_mirror)
+
+ init_obj.tools = {
+ '1': {
+ 'tooldia': tooldia,
+ 'drills': drills,
+ 'solid_geometry': []
+ }
+ }
- init_obj.tools = tools
- init_obj.drills = drills
init_obj.create_geometry()
# Box
diff --git a/tclCommands/TclCommandAlignDrillGrid.py b/tclCommands/TclCommandAlignDrillGrid.py
index e560290d..8dcc2c2a 100644
--- a/tclCommands/TclCommandAlignDrillGrid.py
+++ b/tclCommands/TclCommandAlignDrillGrid.py
@@ -80,8 +80,9 @@ class TclCommandAlignDrillGrid(TclCommandSignaled):
else:
gridoffsety = args['gridoffsety']
+ tooldia = args['dia']
# Tools
- tools = {"1": {"C": args['dia']}}
+ # tools = {"1": {"C": args['dia']}}
def aligndrillgrid_init_me(init_obj, app_obj):
"""
@@ -101,13 +102,18 @@ class TclCommandAlignDrillGrid(TclCommandSignaled):
for col in range(args['columns']):
point = Point(currentx + gridoffsetx, currenty + gridoffsety)
- drills.append({"point": point, "tool": "1"})
+ drills.append(point)
currentx = currentx + args['gridx']
currenty = currenty + args['gridy']
- init_obj.tools = tools
- init_obj.drills = drills
+ init_obj.tools = {
+ '1': {
+ 'tooldia': tooldia,
+ 'drills': drills,
+ 'solid_geometry': []
+ }
+ }
init_obj.create_geometry()
# Create the new object
diff --git a/tclCommands/TclCommandCncjob.py b/tclCommands/TclCommandCncjob.py
index 0c145063..a7102b93 100644
--- a/tclCommands/TclCommandCncjob.py
+++ b/tclCommands/TclCommandCncjob.py
@@ -154,7 +154,7 @@ class TclCommandCncjob(TclCommandSignaled):
if "dpp" in args:
args["multidepth"] = True
if args["dpp"] is None:
- args["dpp"] =self.app.defaults["geometry_depthperpass"]
+ args["dpp"] = self.app.defaults["geometry_depthperpass"]
else:
args["dpp"] = float(args["dpp"])
else:
@@ -188,7 +188,7 @@ class TclCommandCncjob(TclCommandSignaled):
args["dwell"] = self.app.defaults["geometry_dwell"]
args["dwelltime"] = self.app.defaults["geometry_dwelltime"]
- args["pp"] = args["pp"] if "pp" in args and args["pp"] else self.app.defaults["geometry_ppname_g"]
+ args["pp"] = args["pp"] if "pp" in args and args["pp"] else self.app.defaults["geometry_ppname_g"]
if "toolchangez" in args:
args["toolchange"] = True
diff --git a/tclCommands/TclCommandCopperClear.py b/tclCommands/TclCommandCopperClear.py
index 9f23b0ed..b8facd23 100644
--- a/tclCommands/TclCommandCopperClear.py
+++ b/tclCommands/TclCommandCopperClear.py
@@ -252,7 +252,7 @@ class TclCommandCopperClear(TclCommand):
# Non-Copper clear all polygons in the non-copper clear object
if 'all' in args:
self.app.ncclear_tool.clear_copper_tcl(ncc_obj=obj,
- select_method=0, # ITSELF
+ select_method=0, # ITSELF
ncctooldia=tooldia,
overlap=overlap,
order=order,
@@ -283,7 +283,7 @@ class TclCommandCopperClear(TclCommand):
self.app.ncclear_tool.clear_copper_tcl(ncc_obj=obj,
sel_obj=box_obj,
- select_method=2, # REFERENCE OBJECT
+ select_method=2, # REFERENCE OBJECT
ncctooldia=tooldia,
overlap=overlap,
order=order,
diff --git a/tclCommands/TclCommandExportGcode.py b/tclCommands/TclCommandExportGcode.py
index 5572a285..d983ca99 100644
--- a/tclCommands/TclCommandExportGcode.py
+++ b/tclCommands/TclCommandExportGcode.py
@@ -80,5 +80,5 @@ class TclCommandExportGcode(TclCommandSignaled):
self.raise_tcl_error('!!!Promises exists, but should not here!!!')
del args['name']
- modified_gcode = obj.get_gcode(**args)
+ obj.get_gcode(**args)
return
diff --git a/tclCommands/TclCommandExportGerber.py b/tclCommands/TclCommandExportGerber.py
index e495048e..64a9c046 100644
--- a/tclCommands/TclCommandExportGerber.py
+++ b/tclCommands/TclCommandExportGerber.py
@@ -49,4 +49,4 @@ class TclCommandExportGerber(TclCommand):
"""
if 'filename' not in args:
args['filename'] = self.app.defaults["global_last_save_folder"] + '/' + args['name']
- self.app.f_handlers.export_gerber(use_thread=False,**args)
+ self.app.f_handlers.export_gerber(use_thread=False, **args)
diff --git a/tclCommands/TclCommandExportSVG.py b/tclCommands/TclCommandExportSVG.py
index 10999641..e231472e 100644
--- a/tclCommands/TclCommandExportSVG.py
+++ b/tclCommands/TclCommandExportSVG.py
@@ -1,7 +1,6 @@
from tclCommands.TclCommand import TclCommand
import collections
-from copy import copy
class TclCommandExportSVG(TclCommand):
diff --git a/tclCommands/TclCommandFollow.py b/tclCommands/TclCommandFollow.py
index f0f95904..9ef27800 100644
--- a/tclCommands/TclCommandFollow.py
+++ b/tclCommands/TclCommandFollow.py
@@ -65,4 +65,4 @@ class TclCommandFollow(TclCommandSignaled):
return "Operation failed: %s" % str(e)
# in the end toggle the visibility of the origin object so we can see the generated Geometry
- # self.app.collection.get_by_name(name).ui.plot_cb.toggle()
\ No newline at end of file
+ # self.app.collection.get_by_name(name).ui.plot_cb.toggle()
diff --git a/tclCommands/TclCommandGetPath.py b/tclCommands/TclCommandGetPath.py
index 91c1276f..527b6c2b 100644
--- a/tclCommands/TclCommandGetPath.py
+++ b/tclCommands/TclCommandGetPath.py
@@ -8,7 +8,6 @@
from tclCommands.TclCommand import TclCommand
import collections
-import os
import logging
import gettext
import appTranslation as fcTranslate
diff --git a/tclCommands/TclCommandMillDrills.py b/tclCommands/TclCommandMillDrills.py
index 9e45d7a0..832738e6 100644
--- a/tclCommands/TclCommandMillDrills.py
+++ b/tclCommands/TclCommandMillDrills.py
@@ -77,7 +77,7 @@ class TclCommandMillDrills(TclCommandSignaled):
try:
obj = self.app.collection.get_by_name(str(name))
- except Exception as e:
+ except Exception:
obj = None
self.raise_tcl_error("Could not retrieve object: %s" % name)
diff --git a/tclCommands/TclCommandPanelize.py b/tclCommands/TclCommandPanelize.py
index af6f1a0f..50c6bada 100644
--- a/tclCommands/TclCommandPanelize.py
+++ b/tclCommands/TclCommandPanelize.py
@@ -167,7 +167,8 @@ class TclCommandPanelize(TclCommand):
# for col in range(columns):
# local_outname = outname + ".tmp." + str(col) + "." + str(row)
# if isinstance(obj, ExcellonObject):
- # self.app.app_obj.new_object("excellon", local_outname, initialize_local_excellon, plot=False,
+ # self.app.app_obj.new_object("excellon", local_outname, initialize_local_excellon,
+ # plot=False,
# autoselected=False)
# else:
# self.app.app_obj.new_object("geometry", local_outname, initialize_local, plot=False,
@@ -199,41 +200,36 @@ class TclCommandPanelize(TclCommand):
def job_init_excellon(obj_fin, app_obj):
currenty = 0.0
+
obj_fin.tools = obj.tools.copy()
- obj_fin.drills = []
- obj_fin.slots = []
- obj_fin.solid_geometry = []
+ if 'drills' not in obj_fin.tools:
+ obj_fin.tools['drills'] = []
+ if 'slots' not in obj_fin.tools:
+ obj_fin.tools['slots'] = []
+ if 'solid_geometry' not in obj_fin.tools:
+ obj_fin.tools['solid_geometry'] = []
for option in obj.options:
if option != 'name':
try:
obj_fin.options[option] = obj.options[option]
except Exception as e:
- log.warning("Failed to copy option: %s" % str(option))
- log.debug("TclCommandPanelize.execute().panelize2() --> %s" % str(e))
+ app_obj.log.warning("Failed to copy option: %s" % str(option))
+ app_obj.log.debug("TclCommandPanelize.execute().panelize2() --> %s" % str(e))
for row in range(rows):
currentx = 0.0
for col in range(columns):
- if obj.drills:
- for tool_dict in obj.drills:
- point_offseted = affinity.translate(tool_dict['point'], currentx, currenty)
- obj_fin.drills.append(
- {
- "point": point_offseted,
- "tool": tool_dict['tool']
- }
- )
- if obj.slots:
- for tool_dict in obj.slots:
- start_offseted = affinity.translate(tool_dict['start'], currentx, currenty)
- stop_offseted = affinity.translate(tool_dict['stop'], currentx, currenty)
- obj_fin.slots.append(
- {
- "start": start_offseted,
- "stop": stop_offseted,
- "tool": tool_dict['tool']
- }
+ if 'drills' in obj.tools:
+ for drill_pt in obj.tools['drills']:
+ point_offseted = affinity.translate(drill_pt, currentx, currenty)
+ obj_fin.tools['drills'].append(point_offseted)
+ if 'slots' in obj.tools:
+ for slot_tuple in obj.tools['slots']:
+ start_offseted = affinity.translate(slot_tuple[0], currentx, currenty)
+ stop_offseted = affinity.translate(slot_tuple[1], currentx, currenty)
+ obj_fin.tools['slots'].append(
+ (start_offseted, stop_offseted)
)
currentx += lenghtx
currenty += lenghty
@@ -292,17 +288,15 @@ class TclCommandPanelize(TclCommand):
self.app.app_obj.new_object("geometry", outname, job_init_geometry, plot=False, autoselected=True)
if threaded is True:
- proc = self.app.proc_container.new("Generating panel ... Please wait.")
+ self.app.proc_container.new("Generating panel ... Please wait.")
def job_thread(app_obj):
try:
panelize_2()
- self.app.inform.emit("[success] Panel created successfully.")
+ app_obj.inform.emit("[success] Panel created successfully.")
except Exception as ee:
- proc.done()
log.debug(str(ee))
return
- proc.done()
self.app.collection.promise(outname)
self.app.worker_task.emit({'fcn': job_thread, 'params': [self.app]})
diff --git a/tclCommands/TclCommandPlotObjects.py b/tclCommands/TclCommandPlotObjects.py
index 1a230b3b..bcf9cc14 100644
--- a/tclCommands/TclCommandPlotObjects.py
+++ b/tclCommands/TclCommandPlotObjects.py
@@ -67,7 +67,7 @@ class TclCommandPlotObjects(TclCommand):
names = [x.strip() for x in args['names'].split(",") if x != '']
objs = []
for name in names:
- obj= self.app.collection.get_by_name(name)
+ obj = self.app.collection.get_by_name(name)
obj.options["plot"] = True if plot_status is True else False
objs.append(obj)
diff --git a/tclCommands/TclCommandSaveSys.py b/tclCommands/TclCommandSaveSys.py
index 3c163158..06fc5d29 100644
--- a/tclCommands/TclCommandSaveSys.py
+++ b/tclCommands/TclCommandSaveSys.py
@@ -50,4 +50,4 @@ class TclCommandSaveSys(TclCommandSignaled):
:return: None or exception
"""
- self.app.preferencesUiManager.save_defaults(args)
\ No newline at end of file
+ self.app.preferencesUiManager.save_defaults(args)
diff --git a/tclCommands/TclCommandScale.py b/tclCommands/TclCommandScale.py
index 4c7e4491..8b32a380 100644
--- a/tclCommands/TclCommandScale.py
+++ b/tclCommands/TclCommandScale.py
@@ -104,10 +104,9 @@ class TclCommandScale(TclCommand):
raise Exception
except Exception as e:
self.raise_tcl_error('%s\n%s' % (_("Expected -origin or "
- "-origin or "
- "-origin or "
- "- origin 3.0,4.2."), str(e))
- )
+ "-origin or "
+ "-origin or "
+ "- origin 3.0,4.2."), str(e)))
return 'fail'
if 'factor' in args:
diff --git a/tclCommands/TclCommandSetPath.py b/tclCommands/TclCommandSetPath.py
index 46a52fc4..4050c4a9 100644
--- a/tclCommands/TclCommandSetPath.py
+++ b/tclCommands/TclCommandSetPath.py
@@ -52,7 +52,7 @@ class TclCommandSetPath(TclCommand):
"By using this command there is no need for usage of the absolute path to the files.",
'args': collections.OrderedDict([
('path', 'A folder path to where the user is supposed to have the file that he will work with.\n'
- 'WARNING: No spaces allowed. Use quotes around the path if it contains spaces.'),
+ 'WARNING: No spaces allowed. Use quotes around the path if it contains spaces.'),
]),
'examples': ['set_path D:\\Project_storage_path']
}
diff --git a/tclCommands/TclCommandSetSys.py b/tclCommands/TclCommandSetSys.py
index c773ba36..df211817 100644
--- a/tclCommands/TclCommandSetSys.py
+++ b/tclCommands/TclCommandSetSys.py
@@ -87,4 +87,3 @@ class TclCommandSetSys(TclCommand):
else:
self.raise_tcl_error("No such system parameter \"{}\".".format(param))
-