OpenSCAD: lint OpenSCADUtils.py
This commit is contained in:
@@ -176,7 +176,7 @@ def callopenscad(inputfilename,outputfilename=None, outputext='csg', keepname=Fa
|
||||
elif transferMechanism == 2: # Use pipes instead of tempfiles
|
||||
return call_openscad_with_pipes(inputfilename, outputfilename, outputext, keepname)
|
||||
else:
|
||||
raise OpenSCADError("Invalid transfer mechanism specified");
|
||||
raise OpenSCADError("Invalid transfer mechanism specified")
|
||||
|
||||
if osfilename and os.path.isfile(osfilename):
|
||||
if not outputfilename:
|
||||
@@ -231,7 +231,7 @@ def call_openscad_with_pipes(input_filename, output_filename, output_extension,
|
||||
output_filename=os.path.join(dir1,'%s.%s' % \
|
||||
(next(tempfilenamegen),output_extension))
|
||||
with open(output_filename,"w") as outfile:
|
||||
outfile.write(stdoutd);
|
||||
outfile.write(stdoutd)
|
||||
return output_filename
|
||||
return None
|
||||
|
||||
@@ -487,7 +487,7 @@ def roundrotation(rot, maxangulardistance=1e-5):
|
||||
for euler in itertools.permutations((0,0,angle)):
|
||||
eulers.append(euler)
|
||||
for euler in itertools.product((0,45,90,135,180,-45,-90,-135), repeat=3):
|
||||
eulers.append(euler)
|
||||
eulers.append(euler)
|
||||
for euler in eulers:
|
||||
r2 = FreeCAD.Rotation(*euler)
|
||||
if comparerotations(r1, r2) < maxangulardistance:
|
||||
@@ -587,9 +587,9 @@ def meshoponobjs(opname, inobjs):
|
||||
else:
|
||||
pass #neither a mesh nor a part
|
||||
if len(objs) > 0:
|
||||
return (meshoptempfile(opname,meshes),objs)
|
||||
return (meshoptempfile(opname,meshes),objs)
|
||||
else:
|
||||
return (None,[])
|
||||
return (None,[])
|
||||
|
||||
|
||||
def process2D_ObjectsViaOpenSCADShape(ObjList, Operation, doc):
|
||||
@@ -634,9 +634,9 @@ def process2D_ObjectsViaOpenSCAD(ObjList, Operation, doc=None):
|
||||
obj.Shape=face
|
||||
# Hide Children
|
||||
if FreeCAD.GuiUp:
|
||||
for index in ObjList :
|
||||
index.ViewObject.hide()
|
||||
return(obj)
|
||||
for index in ObjList :
|
||||
index.ViewObject.hide()
|
||||
return obj
|
||||
|
||||
|
||||
def process3D_ObjectsViaOpenSCADShape(ObjList, Operation, maxmeshpoints=None):
|
||||
@@ -661,7 +661,7 @@ def process3D_ObjectsViaOpenSCADShape(ObjList, Operation, maxmeshpoints=None):
|
||||
solid = Part.Solid(sh)
|
||||
solid = solid.removeSplitter()
|
||||
if solid.Volume < 0:
|
||||
solid.complement()
|
||||
solid.complement()
|
||||
return solid
|
||||
|
||||
|
||||
@@ -671,9 +671,9 @@ def process3D_ObjectsViaOpenSCAD(doc,ObjList, Operation):
|
||||
obj = doc.addObject('Part::Feature',Operation) #non-parametric object
|
||||
obj.Shape=solid#.removeSplitter()
|
||||
if FreeCAD.GuiUp:
|
||||
for index in ObjList:
|
||||
index.ViewObject.hide()
|
||||
return(obj)
|
||||
for index in ObjList:
|
||||
index.ViewObject.hide()
|
||||
return obj
|
||||
|
||||
|
||||
def process_ObjectsViaOpenSCADShape(doc, children, name, maxmeshpoints=None):
|
||||
|
||||
Reference in New Issue
Block a user