Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix selection nearest point on init_guess to nearest edge. #4871

Merged
merged 6 commits into from
Jan 12, 2023

Conversation

satabol
Copy link
Collaborator

@satabol satabol commented Jan 11, 2023

Windows 11, Blender 3.4.1, Sverchok-master.

there is some glitches in Curves->Nearest Point on Curve:

image

image

After fix all look good:
image

Nearest point on curve 005

.blend file for tests: https://disk.yandex.ru/d/ODhSkPCp3DqYog

@satabol satabol requested a review from portnov January 11, 2023 22:52
max_axis = 1
if max_dist < dist_iz:
max_dist = dist_iz
max_axis = 2
Copy link
Collaborator

@portnov portnov Jan 12, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

max_axis = np.argmax(abs(points_i1[0] - points_i0[0]))

?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

?

appended

@satabol satabol merged commit 9f44d34 into master Jan 12, 2023
@satabol satabol deleted the nearest_point_on_curve_fix_001 branch January 12, 2023 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants