You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
"I think this could be improved. As you know, the topo_parent is set to whatever Shape the selector was working with so Face.edges() is going to assigned a Face as the parent (which is correct but a bit incomplete). fillet uses this parent to decide if this is a 1, 2 or 3D fillet operation which in this case isn't really what is desired. A more advanced type of fillet identification algorithm should be used here with takes into consideration the object passed to fillet, the listed topo_parent and even the higher level topological parents of the listed topo_parent. This needs to be carefully done though to avoid any negative impacts (like loosing the ability to fillet a Face that was extracted from a Solid)."
The text was updated successfully, but these errors were encountered:
One of the selection patterns I use is first to select the right face and the edges.
If I try this, I get
ValueError: 2D fillet operation takes only Vertices
.Kind of clear, because the
topo_parent
of the edges are faces.Changing it to
makes it work.
Roger's comment on Discord:
"I think this could be improved. As you know, the
topo_parent
is set to whatever Shape the selector was working with soFace.edges()
is going to assigned a Face as the parent (which is correct but a bit incomplete).fillet
uses this parent to decide if this is a 1, 2 or 3D fillet operation which in this case isn't really what is desired. A more advanced type of fillet identification algorithm should be used here with takes into consideration the object passed to fillet, the listedtopo_parent
and even the higher level topological parents of the listedtopo_parent
. This needs to be carefully done though to avoid any negative impacts (like loosing the ability to fillet a Face that was extracted from a Solid)."The text was updated successfully, but these errors were encountered: