core icon indicating copy to clipboard operation
core copied to clipboard

Bug in snapToInterpolate

Open JaredCrean2 opened this issue 6 years ago • 2 comments

I think there is a bug in the curved mesh snapping code here. This code updates the position of a vertex without also updating the CAD parametric coordinates. I think there should be a call to m->setParam() right after m->setPoint().

JaredCrean2 avatar Sep 02 '19 14:09 JaredCrean2

At this point in the code verts have to already be snapped to the model and have the correct parametric coordinates. In other words, I believe the portion of the code that loops over verts should not do anything.

mortezah avatar Sep 25 '19 14:09 mortezah

I have not seen any incorrect behavior from the code (although I haven't look at the parametric coordinates after mesh adaptation), I filed the issue based only on looking at the code. If you are confident the parametric coordinates are handled elsewhere, we can close the issue.

JaredCrean2 avatar Sep 25 '19 16:09 JaredCrean2