I used the “Resample” menu in the Markups module to resample a curve and constrain it to a surface model. Unlike in previous versions, the curve was not constrained to the surface, as seen below:
If I set “Constrain to model” under the “Curve settings” menu first, then I get the resampled curve snapped to my surface model, shown below. Is this change from previous behavior of curve resampling intended? If so, should resample curves be updated to remove the surface constraint?
@lassoan @Connor-Bowley could you comment on whether setting the curve type before resampling is intended as part of the addition of the curve settings menu, or is a bug?
I am using the windows rev 30663.
Since constraining to model is now available in “Curve settings” it is not needed anymore in “Resample curve”, so it should be removed from there. If the projection settings (in “Advanced” section) are all available in “Curve settings” then those too can be removed from “Resample curve” section.
I’ll remove these redundant settings from the curve resampling section today or tomorrow.
I just got back to this and was about to submit a pull request.
Have you already started working on it?
Yes, I just finished testing and was rebasing my branch. But it’s a pretty simple fix if it’s faster for you to submit.
Please submit it then, I’ll review and consolidate it with my WIP changes.