Skip to content

fix turning point check in findStraightPath #736

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Guokj-Git
Copy link

@Guokj-Git Guokj-Git commented Oct 17, 2024

This commit try to prevent inappropriate turning point by accepting the candidate left or right point as the new portalLeft or portalRight if any 2 among 3 related points are overlapped. These 3 related points can be either:
portalLeft, left, portalApex
or
portalRight, right, portalApex.

Detailed reason for this commit is discussed in issue#735.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants