3D-printing/Cura
1
0
Fork 0
mirror of https://github.com/Ultimaker/Cura.git synced 2025-07-08 07:27:29 -06:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Simplify code flow a bit

Browse source 
Use if branches for the unhappy flow, so that it's clear that this behaves as an early-out. This also prevents the need for declaring variables out of scope, which is not an issue with Python anyway.
This commit is contained in:
Ghostkeeper 2018-01-04 17:23:43 +01:00
parent 564a97d5a8
commit 2c45efb70d
No known key found for this signature in database
GPG key ID: 5252B696FB5E7C7A
1 changed files with 33 additions and 34 deletions
Download patch file Download diff file Expand all files Collapse all files

11
cura/Scene/ConvexHullDecorator.py
View file

@ -160,10 +160,10 @@ class ConvexHullDecorator(SceneNodeDecorator):
return offset_hull
else:
if not self._node.getMeshData(): #Node has no mesh data, so just return an empty Polygon.
return Polygon([])
offset_hull = None
mesh = None
world_transform = None
if self._node.getMeshData():
mesh = self._node.getMeshData()
world_transform = self._node.getWorldTransformation()
@ -177,7 +177,8 @@ class ConvexHullDecorator(SceneNodeDecorator):
# Do not throw away vertices: the convex hull may be too small and objects can collide.
# vertex_data = vertex_data[vertex_data[:,1] >= -0.01]
if vertex_data and len(vertex_data) >= 4:
if not vertex_data or len(vertex_data) < 4:
return Polygon([])
# Round the vertex data to 1/10th of a mm, then remove all duplicate vertices
# This is done to greatly speed up further convex hull calculations as the convex hull
# becomes much less complex when dealing with highly detailed models.
@ -200,8 +201,6 @@ class ConvexHullDecorator(SceneNodeDecorator):
if len(vertex_data) >= 3:
convex_hull = hull.getConvexHull()
offset_hull = self._offsetHull(convex_hull)
else:
return Polygon([]) # Node has no mesh data, so just return an empty Polygon.
# Store the result in the cache
self._2d_convex_hull_mesh = mesh